Make completedRepositoryCount non-optional

We're always passing in a non-undefined number anyway, so this optional value isn't giving us any benefit.

Author: robertbrignull

extensions/ql-vscode/src/view/variant-analysis/VariantAnalysisRepositoriesStats.tsx

@@ -12,15 +12,15 @@ type Props = {
   variantAnalysisStatus: VariantAnalysisStatus;
 
   totalRepositoryCount: number;
-  completedRepositoryCount?: number | undefined;
+  completedRepositoryCount: number;
 
   showWarning?: boolean;
 };
 
 export const VariantAnalysisRepositoriesStats = ({
   variantAnalysisStatus,
   totalRepositoryCount,
-  completedRepositoryCount = 0,
+  completedRepositoryCount,
   showWarning,
 }: Props) => {
   if (variantAnalysisStatus === VariantAnalysisStatus.Failed) {

extensions/ql-vscode/src/view/variant-analysis/VariantAnalysisStats.tsx

@@ -12,7 +12,7 @@ export type VariantAnalysisStatsProps = {
   variantAnalysisStatus: VariantAnalysisStatus;
 
   totalRepositoryCount: number;
-  completedRepositoryCount?: number | undefined;
+  completedRepositoryCount: number;
 
   hasWarnings?: boolean;
 
@@ -32,7 +32,7 @@ const Row = styled.div`
 export const VariantAnalysisStats = ({
   variantAnalysisStatus,
   totalRepositoryCount,
-  completedRepositoryCount = 0,
+  completedRepositoryCount,
   hasWarnings,
   resultCount,
   createdAt,

extensions/ql-vscode/src/view/variant-analysis/__tests__/VariantAnalysisStats.spec.tsx

@@ -19,6 +19,7 @@ describe(VariantAnalysisStats.name, () => {
       <VariantAnalysisStats
         variantAnalysisStatus={VariantAnalysisStatus.InProgress}
         totalRepositoryCount={10}
+        completedRepositoryCount={0}
         onViewLogsClick={onViewLogsClick}
         createdAt={new Date()}
         {...props}