Merge pull request #2470 from github/dependabot/npm_and_yarn/extensions/ql-vscode/faker-js/faker-8.0.2

Bump @faker-js/faker from 7.5.0 to 8.0.2 in /extensions/ql-vscode

Author: koesie10

extensions/ql-vscode/package-lock.json

@@ -1694,7 +1694,7 @@
   "devDependencies": {
     "@babel/core": "^7.18.13",
     "@babel/plugin-transform-modules-commonjs": "^7.18.6",
-    "@faker-js/faker": "^7.5.0",
+    "@faker-js/faker": "^8.0.2",
     "@github/markdownlint-github": "^0.3.0",
     "@octokit/plugin-throttling": "^5.0.1",
     "@storybook/addon-actions": "^6.5.17-alpha.0",

extensions/ql-vscode/package.json

@@ -135,14 +135,14 @@ const manyScannedRepos = Array.from({ length: 1000 }, (_, i) => {
   return {
     ...mockedScannedRepo,
     analysisStatus: VariantAnalysisRepoStatus.Succeeded,
-    resultCount: faker.datatype.number({ min: 0, max: 1000 }),
+    resultCount: faker.number.int({ min: 0, max: 1000 }),
     repository: {
       ...mockedScannedRepo.repository,
       // We need to ensure the ID is unique for React keys
-      id: faker.helpers.unique(faker.datatype.number, [], {
+      id: faker.helpers.unique(faker.number.int, [], {
         store: uniqueStore,
       }),
-      fullName: `octodemo/${faker.helpers.unique(faker.random.word, [], {
+      fullName: `octodemo/${faker.helpers.unique(faker.word.sample, [], {
         store: uniqueStore,
       })}`,
     },

extensions/ql-vscode/src/stories/variant-analysis/VariantAnalysisAnalyzedRepos.stories.tsx

@@ -41,10 +41,10 @@ export function createDbConfig({
 }
 
 export function createLocalDbConfigItem({
-  name = `database${faker.datatype.number()}`,
+  name = `database${faker.number.int()}`,
   dateAdded = faker.date.past().getTime(),
-  language = `language${faker.datatype.number()}`,
-  storagePath = `storagePath${faker.datatype.number()}`,
+  language = `language${faker.number.int()}`,
+  storagePath = `storagePath${faker.number.int()}`,
 }: {
   name?: string;
   dateAdded?: number;

extensions/ql-vscode/test/factories/db-config-factories.ts

@@ -29,7 +29,7 @@ export function createRootRemoteDbItem({
 }
 
 export function createRemoteOwnerDbItem({
-  ownerName = `owner${faker.datatype.number()}`,
+  ownerName = `owner${faker.number.int()}`,
   selected = false,
 }: {
   ownerName?: string;
@@ -43,7 +43,7 @@ export function createRemoteOwnerDbItem({
 }
 
 export function createRemoteRepoDbItem({
-  repoFullName = `owner${faker.datatype.number()}/repo${faker.datatype.number}`,
+  repoFullName = `owner${faker.number.int()}/repo${faker.number.int()}`,
   selected = false,
   parentListName = undefined,
 }: {
@@ -60,7 +60,7 @@ export function createRemoteRepoDbItem({
 }
 
 export function createRemoteSystemDefinedListDbItem({
-  listName = `top_${faker.datatype.number()}`,
+  listName = `top_${faker.number.int()}`,
   listDisplayName = `Display Name`,
   listDescription = `Description`,
   selected = false,
@@ -82,7 +82,7 @@ export function createRemoteSystemDefinedListDbItem({
 export function createRemoteUserDefinedListDbItem({
   expanded = false,
   selected = false,
-  listName = `list${faker.datatype.number()}`,
+  listName = `list${faker.number.int()}`,
   repos = [
     createRemoteRepoDbItem({
       parentListName: listName,
@@ -119,10 +119,10 @@ export function createRootLocalDbItem({
 }
 
 export function createLocalDatabaseDbItem({
-  databaseName = `database${faker.datatype.number()}`,
+  databaseName = `database${faker.number.int()}`,
   dateAdded = faker.date.past().getTime(),
-  language = `language${faker.datatype.number()}`,
-  storagePath = `storagePath${faker.datatype.number()}`,
+  language = `language${faker.number.int()}`,
+  storagePath = `storagePath${faker.number.int()}`,
   selected = false,
 }: {
   databaseName?: string;
@@ -142,7 +142,7 @@ export function createLocalDatabaseDbItem({
 }
 
 export function createLocalListDbItem({
-  listName = `top_${faker.datatype.number()}`,
+  listName = `top_${faker.number.int()}`,
   selected = false,
   expanded = false,
   databases = [],

extensions/ql-vscode/test/factories/db-item-factories.ts

@@ -42,7 +42,7 @@ export function createMockLocalQueryInfo({
       name: dbName,
     },
     start: startTime,
-    id: faker.datatype.number().toString(),
+    id: faker.number.int().toString(),
     userSpecifiedLabel,
   } as InitialQueryInfo;
 

extensions/ql-vscode/test/factories/query-history/local-query-history-item.ts

@@ -4,9 +4,9 @@ import {
   RepositoryWithMetadata,
 } from "../../../../src/variant-analysis/gh-api/repository";
 
-export function createMockRepository(name = faker.random.word()): Repository {
+export function createMockRepository(name = faker.word.sample()): Repository {
   return {
-    id: faker.datatype.number(),
+    id: faker.number.int(),
     name,
     full_name: `github/${name}`,
     private: faker.datatype.boolean(),
@@ -16,7 +16,7 @@ export function createMockRepository(name = faker.random.word()): Repository {
 export function createMockRepositoryWithMetadata(): RepositoryWithMetadata {
   return {
     ...createMockRepository(),
-    stargazers_count: faker.datatype.number(),
+    stargazers_count: faker.number.int(),
     updated_at: faker.date.past().toISOString(),
   };
 }

extensions/ql-vscode/test/factories/variant-analysis/gh-api/repository.ts

@@ -18,8 +18,8 @@ export function createMockScannedRepo(
       private: isPrivate,
     },
     analysis_status: analysisStatus,
-    result_count: faker.datatype.number(),
-    artifact_size_in_bytes: faker.datatype.number(),
+    result_count: faker.number.int(),
+    artifact_size_in_bytes: faker.number.int(),
   };
 }
 

extensions/ql-vscode/test/factories/variant-analysis/gh-api/scanned-repositories.ts

@@ -26,8 +26,8 @@ export function createMockSkippedRepoGroup(): VariantAnalysisSkippedRepositoryGr
 }
 
 export function createMockNotFoundSkippedRepoGroup(): VariantAnalysisNotFoundRepositoryGroup {
-  const repoName1 = `github/${faker.random.word()}`;
-  const repoName2 = `github/${faker.random.word()}`;
+  const repoName1 = `github/${faker.word.sample()}`;
+  const repoName2 = `github/${faker.word.sample()}`;
 
   return {
     repository_count: 2,

extensions/ql-vscode/test/factories/variant-analysis/gh-api/skipped-repositories.ts

@@ -16,7 +16,7 @@ export function createMockApiResponse(
   skippedRepos: VariantAnalysisSkippedRepositories = createMockSkippedRepos(),
 ): VariantAnalysisApiResponse {
   const variantAnalysis: VariantAnalysisApiResponse = {
-    id: faker.datatype.number(),
+    id: faker.number.int(),
     controller_repo: {
       ...createMockRepository(),
       name: "pickles",
@@ -28,7 +28,7 @@ export function createMockApiResponse(
     created_at: faker.date.recent().toISOString(),
     updated_at: faker.date.recent().toISOString(),
     status,
-    actions_workflow_run_id: faker.datatype.number(),
+    actions_workflow_run_id: faker.number.int(),
     scanned_repositories: scannedRepos,
     skipped_repositories: skippedRepos,
   };