apify-test-tools 0.5.7-beta.4 → 0.5.7-beta.6

package/bin/main.ts

@@ -3,6 +3,7 @@
 import process from 'node:process';
 
 import yargs, { type Argv } from 'yargs';
+// eslint-disable-next-line import/extensions --- With .js, it cannot find types
 import { hideBin } from 'yargs/helpers';
 
 import { runBuilds } from './build.js';
@@ -11,7 +12,7 @@ import { getChangedFiles, getCommits } from './git.js';
 import { getPushData } from './github.js';
 import { notifyToSlack } from './slack.js';
 import { reportTestResults } from './test-report.js';
-import { getRepoActors, setCwd,spawnCommandInGhWorkspace } from './utils.js';
+import { getRepoActors, setCwd, spawnCommandInGhWorkspace } from './utils.js';
 
 /**
  * Middlewares to be run before every command execution
@@ -65,13 +66,18 @@ await yargs()
         async () => {
             const actorConfigs = await getRepoActors();
             console.log(JSON.stringify(actorConfigs));
-        }
+        },
     )
     .command('get-affected-actors', '', buildOptions, async (args) => {
         const commits = getCommits(args);
         const changedFiles = getChangedFiles(commits);
         const actorConfigs = await getRepoActors();
-        const actorsChanged = getChangedActors({ filepathsChanged: changedFiles, actorConfigs, isLatest: false, commits });
+        const actorsChanged = getChangedActors({
+            filepathsChanged: changedFiles,
+            actorConfigs,
+            isLatest: false,
+            commits,
+        });
         console.log(JSON.stringify(actorsChanged));
     })
     .command(
@@ -85,7 +91,7 @@ await yargs()
                 .option('workflow-name', { type: 'string' }),
         async (args) => {
             await reportTestResults(args);
-        }
+        },
     )
     .command(
         'build',
@@ -104,7 +110,7 @@ await yargs()
             // git@github.com:apify-store/google-maps#:actors/lukaskrivka_google-maps-with-contact-details
             const repoUrl = spawnCommandInGhWorkspace(`git remote get-url origin`).replace(
                 /^https:\/\/github\.com\//,
-                'git@github.com:'
+                'git@github.com:',
             );
 
             const builds = await runBuilds({
@@ -114,7 +120,7 @@ await yargs()
                 dryRun: args.dryRun,
             });
             console.log(JSON.stringify(builds));
-        }
+        },
     )
     .command(
         'release',
@@ -127,7 +133,7 @@ await yargs()
                 .option('release-slack-channel', { type: 'string' }),
         async (args) => {
             const { branch, changedFiles, repoUrl, commits, changelog, repository, author } = await getPushData(
-                args.pushEventPath
+                args.pushEventPath,
             );
             const isLatest = true;
             const actorConfigs = await getRepoActors();
@@ -158,7 +164,7 @@ await yargs()
                 reportSlackChannel,
                 releaseSlackChannel,
             });
-        }
+        },
     )
     .strictCommands()
     .demandCommand(1, 'Command is required')

package/bin/slack.ts

@@ -1,5 +1,6 @@
 import { WebClient } from '@slack/web-api';
-import { Commit } from './types.js';
+
+import type { Commit } from './types.js';
 import { getEnvVar } from './utils.js';
 
 type NotifyToSlackOptions = {
@@ -46,7 +47,10 @@ export const notifyToSlack = async ({
     }
 
     const commitsMessage = `${commits
-        .map(({ author, message }, index) => `${index + 1}. Commit message: ${message}\n\tAuthor: ${author}.`)
+        .map(
+            ({ author: commitAuthor, message }, index) =>
+                `${index + 1}. Commit message: ${message}\n\tAuthor: ${commitAuthor}.`,
+        )
         .join('\n')}`;
     const changedFilesMessage = `**Files changed**: ${changedFiles.join(', ')}`;
     const longMessage = `${shortMessage}\n**Commit list**:\n${commitsMessage}\n\n${changedFilesMessage}`;

package/bin/test-report.ts

@@ -1,13 +1,14 @@
-import fs from 'fs/promises';
+import fs from 'node:fs/promises';
+
 import { sendSlackMessage } from './slack.js';
 import { getEnvVar } from './utils.js';
 
 interface ReportTestResultsOptions {
-    reportFile: string
-    dryRun: boolean
-    reportSlackChannel?: string
-    jobUrl?: string
-    workflowName?: string
+    reportFile: string;
+    dryRun: boolean;
+    reportSlackChannel?: string;
+    jobUrl?: string;
+    workflowName?: string;
 }
 
 export const reportTestResults = async ({
@@ -35,7 +36,7 @@ export const reportTestResults = async ({
         }
     }
 
-    const failedAssertions: { message: string; runLink: string, actorName: string }[] = [];
+    const failedAssertions: { message: string; runLink: string; actorName: string }[] = [];
 
     console.error();
     console.error(`PASSED: ${passed.length}, FAILED: ${failed.length}`);
@@ -57,7 +58,13 @@ export const reportTestResults = async ({
     for (const [i, aResult] of failed.entries()) {
         const { failureMessages, fullName, meta } = aResult;
         if (failureMessages) {
-            failedAssertions.push(...failureMessages.map(message => ({ message: message.split('\n')?.[0], runLink: meta.runLink, actorName: meta.actorName })));
+            failedAssertions.push(
+                ...failureMessages.map((message) => ({
+                    message: message.split('\n')?.[0],
+                    runLink: meta.runLink,
+                    actorName: meta.actorName,
+                })),
+            );
         }
         console.error(`${i + 1}) ${fullName} ... ${meta.runLink}`);
         console.error();
@@ -65,9 +72,11 @@ export const reportTestResults = async ({
     console.error();
     console.error(`PASSED: ${passed.length}, FAILED: ${failed.length}`);
     console.error();
-    
+
     if (!reportSlackChannel) {
-        console.error(`Skipping slack notification. If you want to enable it, add --report-slack-channel flag and make sure SLACK_TOKEN_TESTS_BOT env variable is set.`);
+        console.error(
+            `Skipping slack notification. If you want to enable it, add --report-slack-channel flag and make sure SLACK_TOKEN_TESTS_BOT env variable is set.`,
+        );
         return;
     }
 
@@ -81,7 +90,9 @@ export const reportTestResults = async ({
     let slackMessage = `\`${workflowName ?? '-'}\``;
     slackMessage += `: has ${failedAssertions.length} failed assertions. Failing test suites: ${failed.length}/${total}.${jobLink}`;
     slackMessage += `\n\n${failedAssertions[0].message} --- <${failedAssertions[0].runLink}|${failedAssertions[0].actorName}>`;
-    const blocks = failedAssertions.slice(1).map(({ message, runLink, actorName }) => `• ${message} --- <${runLink}|${actorName}>`);
+    const blocks = failedAssertions
+        .slice(1)
+        .map(({ message, runLink, actorName }) => `• ${message} --- <${runLink}|${actorName}>`);
 
     console.error('SLACK:', slackMessage);
     console.error('\tblocks:', blocks.join('\n\t\t'));
@@ -96,52 +107,52 @@ export const reportTestResults = async ({
     }
 };
 
-type Status = 'passed' | 'failed' | 'skipped' | 'pending' | 'todo' | 'disabled'
-type Milliseconds = number
+type Status = 'passed' | 'failed' | 'skipped' | 'pending' | 'todo' | 'disabled';
+type Milliseconds = number;
 interface Callsite {
-    line: number
-    column: number
+    line: number;
+    column: number;
 }
 
 interface JsonAssertionResult {
-    ancestorTitles: Array<string>
-    fullName: string
-    status: Status
-    title: string
+    ancestorTitles: string[];
+    fullName: string;
+    status: Status;
+    title: string;
     meta: {
-        runId: string
-        runLink: string
-        actorName: string
-    }
-    duration?: Milliseconds | null
-    failureMessages: Array<string> | null
-    location?: Callsite | null
+        runId: string;
+        runLink: string;
+        actorName: string;
+    };
+    duration?: Milliseconds | null;
+    failureMessages: string[] | null;
+    location?: Callsite | null;
 }
 
 interface JsonTestResult {
-    message: string
-    name: string
-    status: 'failed' | 'passed'
-    startTime: number
-    endTime: number
-    assertionResults: Array<JsonAssertionResult>
+    message: string;
+    name: string;
+    status: 'failed' | 'passed';
+    startTime: number;
+    endTime: number;
+    assertionResults: JsonAssertionResult[];
     // summary: string
     // coverage: unknown
 }
 
 interface JsonTestResults {
-    numFailedTests: number
-    numFailedTestSuites: number
-    numPassedTests: number
-    numPassedTestSuites: number
-    numPendingTests: number
-    numPendingTestSuites: number
-    numTodoTests: number
-    numTotalTests: number
-    numTotalTestSuites: number
-    startTime: number
-    success: boolean
-    testResults: Array<JsonTestResult>
+    numFailedTests: number;
+    numFailedTestSuites: number;
+    numPassedTests: number;
+    numPassedTestSuites: number;
+    numPendingTests: number;
+    numPendingTestSuites: number;
+    numTodoTests: number;
+    numTotalTests: number;
+    numTotalTestSuites: number;
+    startTime: number;
+    success: boolean;
+    testResults: JsonTestResult[];
     // snapshot: SnapshotSummary
     // coverageMap?: CoverageMap | null | undefined
     // numRuntimeErrorTestSuites: number

package/bin/types.ts

@@ -1,91 +1,88 @@
 export interface Config {
-    targetBranch: string
-    sourceBranch: string
-    baseCommit?: string
-    workspace?: string
+    targetBranch: string;
+    sourceBranch: string;
+    baseCommit?: string;
+    workspace?: string;
 }
 
 export type Commit = {
-    sha: string
-    author: string
-    date: string
-    message: string
-}
+    sha: string;
+    author: string;
+    date: string;
+    message: string;
+};
 
 // NOTE: The GitHub types are incomplete, feel free to add fields/complete (not sure how stable GitHub API is)
 export interface GitHubHeadCommit {
-    added: string[]
-    removed: string[]
-    modified: string[],
+    added: string[];
+    removed: string[];
+    modified: string[];
     author: {
-        name: string
-    },
-    message: string,
-    id: string,
+        name: string;
+    };
+    message: string;
+    id: string;
 }
 
 export interface Repository {
-    full_name: string
-    name: string
-    ssh_url: string
+    full_name: string;
+    name: string;
+    ssh_url: string;
     owner: {
-        login: string
-    }
+        login: string;
+    };
 }
 
 export type GitHubEvent = GitHubEventPullRequest | GitHubEventPush;
 
-// All optional type so we check that we have all required fields for each event type
-export type RawGitHubEvent = Partial<Omit<GitHubEventPullRequest, 'type'> & Omit<GitHubEventPush, 'type'>>;
-
 export interface GitHubEventPullRequest {
     // Type is our own variable we inject to have discriminated union
     // The rest of the properties are raw from GitHub API
-    type: 'pull_request',
+    type: 'pull_request';
     pull_request: {
-        number: number
+        number: number;
         base: {
-            ref: string,
-            sha: string
-        }
+            ref: string;
+            sha: string;
+        };
         head: {
-            ref: string
-            sha: string
-        }
-    }
-    repository: Repository,
+            ref: string;
+            sha: string;
+        };
+    };
+    repository: Repository;
 }
 
 export interface GitHubEventPush {
-    type: 'push',
-    head_commit: GitHubHeadCommit,
-    repository: Repository,
-    ref: string
-    commits: GithubCommit[]
-    before: string
-    after: string
+    type: 'push';
+    head_commit: GitHubHeadCommit;
+    repository: Repository;
+    ref: string;
+    commits: GithubCommit[];
+    before: string;
+    after: string;
 }
 
 export interface GithubCommit {
-    id: string
-    tree_id: string
-    distinct: boolean
-    message: string
-    timestamp: string
-    url: string
+    id: string;
+    tree_id: string;
+    distinct: boolean;
+    message: string;
+    timestamp: string;
+    url: string;
     author: {
-        name: string
-        email: string
-        username: string
-    }
+        name: string;
+        email: string;
+        username: string;
+    };
     committer: {
-        name: string
-        email: string
-        username: string
-    }
-    added: string[]
-    removed: string[]
-    modified: string[]
+        name: string;
+        email: string;
+        username: string;
+    };
+    added: string[];
+    removed: string[];
+    modified: string[];
 }
 
 export interface BuildData {
@@ -99,5 +96,5 @@ export interface BuildData {
 export interface ActorConfig {
     actorName: string;
     folder: string;
-    isStandalone: boolean
+    isStandalone: boolean;
 }

package/bin/utils.ts

@@ -1,10 +1,7 @@
 import { spawnSync } from 'node:child_process';
 import fs from 'node:fs/promises';
 
-import type {
-    ActorConfig,
-    GitHubEvent,
-} from './types.js';
+import type { ActorConfig, GitHubEvent } from './types.js';
 
 export const spawnCommandInGhWorkspace = (command: string, args: string[] = []) => {
     console.error(command, args.join(' '));
@@ -49,14 +46,14 @@ export const getRepoActors = async (): Promise<ActorConfig[]> => {
     let actorDirs: string[];
     try {
         actorDirs = (await fs.readdir(`./actors`)).map((dir) => `actors/${dir}`);
-    } catch (err) {
+    } catch {
         console.warn(`No /actors directory found in repo`);
         actorDirs = [];
     }
     let standaloneActorDirs: string[];
     try {
         standaloneActorDirs = (await fs.readdir(`./standalone-actors`)).map((dir) => `standalone-actors/${dir}`);
-    } catch (err) {
+    } catch {
         console.warn(`No /standalone-actors directory found in repo`);
         standaloneActorDirs = [];
     }
@@ -73,8 +70,18 @@ export const getRepoActors = async (): Promise<ActorConfig[]> => {
             isStandalone: folderType === 'standalone-actors',
         });
     }
-    console.error(`Actors in repo: ${actorConfigs.filter(({ isStandalone }) => !isStandalone).map(({ actorName }) => actorName).join(', ')}`);
-    console.error(`Standalone actors in repo: ${actorConfigs.filter(({ isStandalone }) => !!isStandalone).map(({ actorName }) => actorName).join(', ')}`);
+    console.error(
+        `Actors in repo: ${actorConfigs
+            .filter(({ isStandalone }) => !isStandalone)
+            .map(({ actorName }) => actorName)
+            .join(', ')}`,
+    );
+    console.error(
+        `Standalone actors in repo: ${actorConfigs
+            .filter(({ isStandalone }) => !!isStandalone)
+            .map(({ actorName }) => actorName)
+            .join(', ')}`,
+    );
     return actorConfigs;
 };
 
@@ -85,10 +92,8 @@ export const setCwd = ({ workspace }: { workspace: string | undefined }) => {
     }
     const ghWorkspace = getEnvVar('GITHUB_WORKSPACE', process.cwd());
     process.chdir(ghWorkspace);
-}
+};
 
 export const getHeadCommitSha = (githubEvent: GitHubEvent) => {
-    return githubEvent.type === 'pull_request'
-        ? githubEvent.pull_request.head.sha
-        : githubEvent.head_commit.id;
+    return githubEvent.type === 'pull_request' ? githubEvent.pull_request.head.sha : githubEvent.head_commit.id;
 };

package/dist/bin/build.d.ts

@@ -12,7 +12,7 @@ type RunBuildsOptions = {
     branch: string;
     dryRun: boolean;
 };
-export declare const runBuilds: ({ repoUrl, branch, actorConfigs, isLatest, dryRun, }: RunBuildsOptions) => Promise<BuildPrActorOptions[] | BuildData[]>;
+export declare const runBuilds: ({ repoUrl, branch, actorConfigs, isLatest, dryRun }: RunBuildsOptions) => Promise<BuildPrActorOptions[] | BuildData[]>;
 export declare const deleteOldBuilds: (actorConfigs: ActorConfig[]) => Promise<void>;
 export {};
 //# sourceMappingURL=build.d.ts.map

package/dist/bin/build.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"build.d.ts","sourceRoot":"","sources":["../../bin/build.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,WAAW,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AAGzD,KAAK,mBAAmB,GAAG;IACvB,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,aAAa,EAAE,MAAM,CAAA;IACrB,UAAU,EAAE,MAAM,CAAA;IAClB,SAAS,EAAE,MAAM,CAAA;CACpB,CAAA;AAuKD,KAAK,gBAAgB,GAAG;IACpB,YAAY,EAAE,WAAW,EAAE,CAAA;IAC3B,QAAQ,CAAC,EAAE,OAAO,CAAA;IAClB,OAAO,EAAE,MAAM,CAAA;IACf,MAAM,EAAE,MAAM,CAAA;IACd,MAAM,EAAE,OAAO,CAAA;CAClB,CAAA;AAED,eAAO,MAAM,SAAS,GAAU,sDAM7B,gBAAgB,iDAiDlB,CAAC;AAEF,eAAO,MAAM,eAAe,GAAU,cAAc,WAAW,EAAE,kBAIhE,CAAC"}
+{"version":3,"file":"build.d.ts","sourceRoot":"","sources":["../../bin/build.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,WAAW,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AAGzD,KAAK,mBAAmB,GAAG;IACvB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;IACtB,UAAU,EAAE,MAAM,CAAC;IACnB,SAAS,EAAE,MAAM,CAAC;CACrB,CAAC;AA+MF,KAAK,gBAAgB,GAAG;IACpB,YAAY,EAAE,WAAW,EAAE,CAAC;IAC5B,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,OAAO,CAAC;CACnB,CAAC;AAEF,eAAO,MAAM,SAAS,GAAU,qDAA6D,gBAAgB,iDAsD5G,CAAC;AAEF,eAAO,MAAM,eAAe,GAAU,cAAc,WAAW,EAAE,kBAIhE,CAAC"}