apify-test-tools 0.10.2-beta.2 → 0.10.2-beta.4

package/test/unit/bin/stdin-input.test.ts

@@ -0,0 +1,61 @@
+import { describe, expect, it } from 'vitest';
+
+import { parseBuildInput, parseReleaseChanges } from '../../../bin/stdin-input.js';
+
+describe('parseBuildInput', () => {
+    it('reads the Actors', () => {
+        expect(parseBuildInput('{"actors":[{"actorFullName":"owner/a"}]}')).toStrictEqual({
+            actors: [{ actorFullName: 'owner/a' }],
+        });
+    });
+
+    it('ignores the other fields, so whole get-affected-actors fields can be passed along', () => {
+        const input = { head: 'f'.repeat(40), ref: 'feature', actors: [{ actorFullName: 'owner/a', extra: 1 }] };
+        expect(parseBuildInput(JSON.stringify(input))).toStrictEqual({ actors: [{ actorFullName: 'owner/a' }] });
+    });
+
+    it('accepts an empty list, which builds nothing', () => {
+        expect(parseBuildInput('{"actors":[]}')).toStrictEqual({ actors: [] });
+    });
+
+    it('rejects empty input, which is what a failed command earlier in a pipe leaves behind', () => {
+        expect(() => parseBuildInput('  \n')).toThrow('Expected the Actors to build as JSON on stdin, got nothing');
+    });
+
+    it('rejects input that is not JSON', () => {
+        expect(() => parseBuildInput('owner/a')).toThrow('not valid JSON');
+    });
+
+    it('rejects JSON that is not a list of Actors, e.g. a whole get-affected-actors document', () => {
+        expect(() => parseBuildInput('{"functionallyChangedActors":[{"actorFullName":"owner/a"}]}')).toThrow(
+            'is not the Actors to build',
+        );
+    });
+});
+
+describe('parseReleaseChanges', () => {
+    const commit = { sha: '1'.repeat(40), author: 'Dev<dev@example.com>', date: 'Date', message: 'feat: foo' };
+
+    it('reads what changed from a get-affected-actors document, ignoring the rest', () => {
+        const document = {
+            version: 1,
+            ref: 'master',
+            head: '1'.repeat(40),
+            commits: [commit],
+            changedFiles: ['src/main.ts'],
+            changelog: '- Added foo',
+            functionallyChangedActors: [{ actorFullName: 'owner/a' }],
+            cosmeticallyChangedActors: [],
+        };
+        expect(parseReleaseChanges(JSON.stringify(document))).toStrictEqual({
+            commits: [commit],
+            changedFiles: ['src/main.ts'],
+            changelog: '- Added foo',
+        });
+    });
+
+    it('rejects empty input and input that is not a get-affected-actors document', () => {
+        expect(() => parseReleaseChanges('')).toThrow('got nothing');
+        expect(() => parseReleaseChanges('{"actors":[]}')).toThrow('not the output of get-affected-actors');
+    });
+});

package/test/unit/should-built-and-test.test.ts

@@ -41,6 +41,16 @@ const actorConfig = (
     };
 };
 
+// What a PR (functional changes only) or a release (isLatest, all changes) builds, in config order
+const changedActorsFor = async ({
+    isLatest = false,
+    ...options
+}: Parameters<typeof getChangedActors>[0] & { isLatest?: boolean }): Promise<ActorConfig[]> => {
+    const { functional, cosmetic } = await getChangedActors(options);
+    const changed = isLatest ? [...functional, ...cosmetic] : functional;
+    return options.actorConfigs.filter((config) => changed.includes(config));
+};
+
 describe('Should build and test parser', () => {
     // From https://github.com/apify-store/testing-repo-for-github-actions
     const ACTOR_CONFIGS: ActorConfig[] = [
@@ -70,7 +80,7 @@ describe('Should build and test parser', () => {
     test('Ignores dev-only readme', async () => {
         const FILES = ['README.md', 'code/README.md', 'shared/README.md'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -83,7 +93,7 @@ describe('Should build and test parser', () => {
     test('Ignores other ignored files and folders', async () => {
         const FILES = ['.vscode/settings.json', '.gitignore', '.husky/pre-commit', '.eslintrc', '.editorconfig'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -96,7 +106,7 @@ describe('Should build and test parser', () => {
     test('.actor/ changes trigger builds for broad-context actors', async () => {
         const FILES = ['.actor/actor.json'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -109,7 +119,7 @@ describe('Should build and test parser', () => {
     test('Configured changelog is cosmetic for every actor, only on latest', async () => {
         const FILES = ['CHANGELOG.md'];
 
-        const actorsChangedNotLatest = await getChangedActors({
+        const actorsChangedNotLatest = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -117,7 +127,7 @@ describe('Should build and test parser', () => {
         });
         expect(actorsChangedNotLatest).toEqual([]);
 
-        const actorsChangedLatest = await getChangedActors({
+        const actorsChangedLatest = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: true,
             filepathsChanged: FILES,
@@ -129,7 +139,7 @@ describe('Should build and test parser', () => {
     test('A changelog nested inside one actor own folder is excluded for sibling actors', async () => {
         const FILES = ['actors/lukaskrivka_testing-github-integration-1/CHANGELOG.md'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: true,
             filepathsChanged: FILES,
@@ -142,7 +152,7 @@ describe('Should build and test parser', () => {
     test('Code updated, tests broad-context actors', async () => {
         const FILES = ['code/src/main.ts', 'package.json'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -158,7 +168,7 @@ describe('Should build and test parser', () => {
             'standalone-actors/lukaskrivka_test-standalone/Dockerfile',
         ];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -171,7 +181,7 @@ describe('Should build and test parser', () => {
     test('src/main.ts updated', async () => {
         const FILES = ['src/main.ts'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: true,
             filepathsChanged: FILES,
@@ -188,7 +198,7 @@ describe('Should build and test parser', () => {
             'standalone-actors/lukaskrivka_test-standalone/Dockerfile',
         ];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -204,7 +214,7 @@ describe('Should build and test parser', () => {
             'standalone-actors/lukaskrivka_test-standalone/README.md',
         ];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -217,7 +227,7 @@ describe('Should build and test parser', () => {
     test('Actor JSON changes in PR context trigger tests', async () => {
         const FILES = ['actors/lukaskrivka_testing-github-integration-1/.actor/actor.json'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -230,7 +240,7 @@ describe('Should build and test parser', () => {
     test('Actor JSON changes in latest context trigger builds', async () => {
         const FILES = ['actors/lukaskrivka_testing-github-integration-1/.actor/actor.json'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: true,
             filepathsChanged: FILES,
@@ -243,7 +253,7 @@ describe('Should build and test parser', () => {
     test('JSON file with functional changes triggers tests', async () => {
         const FILES = ['actors/lukaskrivka_testing-github-integration-1/.actor/actor.json'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -259,7 +269,7 @@ describe('Should build and test parser', () => {
             'actors/lukaskrivka_testing-github-integration-2/.actor/input_schema.json',
         ];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -272,7 +282,7 @@ describe('Should build and test parser', () => {
     test('Narrow-context actor JSON change in PR context triggers tests', async () => {
         const FILES = ['standalone-actors/lukaskrivka_test-standalone/.actor/actor.json'];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS,
             isLatest: false,
             filepathsChanged: FILES,
@@ -331,7 +341,7 @@ describe('Should build and test parser', () => {
             ),
         ];
 
-        const actorsChanged = await getChangedActors({
+        const actorsChanged = await changedActorsFor({
             actorConfigs: ACTOR_CONFIGS_GOOGLE_MAPS,
             isLatest: false,
             filepathsChanged: FILES,