apify-test-tools 0.10.2-beta.15 → 0.10.2-beta.17

package/docs/cli.md

@@ -296,9 +296,10 @@ The input is a JSON array of Actors, each given by its full name, its raw ID or
 with it (`buildId` and `buildNumber`, as `build` and `build-from-local` print them), or against its default build
 without one. Other fields are ignored, so the output of [`get-actor-configs`](#listing-actors-get-actor-configs) works
 as it is. `test` looks up the ID an Actor is listed without with `TESTER_APIFY_TOKEN`, so its tests run whichever ID
-they name it by. It fails if the token can't see the Actor. A dry-run plan, empty or invalid input is an
-error, as with `build`. So is a missing `TESTER_APIFY_TOKEN`: without it the lib skips every platform test, and the
-run would pass having tested nothing. `test` passes the Actors to vitest as `ACTOR_BUILDS`, in place of any
+they name it by. An Actor the token can't see keeps only the ID it was listed with, and a warning names it, so an
+Actor no test uses doesn't stop the run. A dry-run plan, empty or invalid input is an error, as with `build`. So is a
+missing `TESTER_APIFY_TOKEN`: without it the lib skips every platform test, and the run would pass having tested
+nothing. `test` passes the Actors to vitest as `ACTOR_BUILDS`, in place of any
 `ACTOR_BUILDS` in its own environment.
 
 It runs vitest with `--maxConcurrency=20 --fileParallelism --maxWorkers=100`, since platform tests mostly wait on

package/docs/workflows.md

@@ -107,9 +107,9 @@ Releases the Actors a push changed. Skipped when the head commit message contain
 4. `delete-old-builds` deletes old builds of every Actor in the config (see
    [`delete-old-builds`](cli.md#deleting-old-builds-delete-old-builds)).
 
-| Input                  | Type   | Default | Description                                                   |
-| ---------------------- | ------ | ------- | ------------------------------------------------------------- |
-| `report-slack-channel` | string | `''`    | Slack channel for the report. Empty uses `#notif-<repo-name>` |
+| Input                  | Type   | Default | Description                                     |
+| ---------------------- | ------ | ------- | ----------------------------------------------- |
+| `report-slack-channel` | string | `''`    | Slack channel for the report. Empty skips Slack |
 
 Secrets: `SLACK_BOT_TOKEN`, the Actor tokens in `CONFIG_SECRETS`, and optionally `NPM_TOKEN`.
 
@@ -126,6 +126,8 @@ on:
 jobs:
     buildLatest:
         uses: apify/apify-test-tools/.github/workflows/public_push-build-latest.yaml@workflows-v0
+        with:
+            report-slack-channel: '#notif-my-repo'
         secrets:
             SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
             # One entry per secret apify-test-tools.config.json names, today each `tokenEnvVar`
@@ -148,7 +150,7 @@ vitest's own JSON report are uploaded as the `vitest-results` artifact. Runs onl
 | ---------------------- | ------ | ------- | ---------------------------------------------------------------------------- |
 | `test-files-glob`      | string | `''`    | Which tests to run, relative to `test/platform`. Empty runs everything in it |
 | `subtest`              | string | `''`    | Deprecated, use `test-files-glob`                                            |
-| `report-slack-channel` | string | `''`    | Slack channel for the results. Empty uses `#notif-<repo-name>`               |
+| `report-slack-channel` | string | `''`    | Slack channel for the results. Empty skips Slack                             |
 
 Secrets: `TESTER_APIFY_TOKEN` (required), `SLACK_BOT_TOKEN`, and optionally `NPM_TOKEN`.
 
@@ -166,6 +168,8 @@ on:
 jobs:
     platformTestsDaily:
         uses: apify/apify-test-tools/.github/workflows/public_platform-tests.yaml@workflows-v0
+        with:
+            report-slack-channel: '#notif-my-repo'
         secrets:
             TESTER_APIFY_TOKEN: ${{ secrets.TESTER_APIFY_TOKEN }}
             SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
@@ -187,6 +191,7 @@ jobs:
         uses: apify/apify-test-tools/.github/workflows/public_platform-tests.yaml@workflows-v0
         with:
             test-files-glob: hourly
+            report-slack-channel: '#notif-my-repo'
         secrets:
             TESTER_APIFY_TOKEN: ${{ secrets.TESTER_APIFY_TOKEN }}
             SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
@@ -286,6 +291,7 @@ jobs:
         uses: apify/apify-test-tools/.github/workflows/public_platform-tests.yaml@workflows-v0
         with:
             test-files-glob: hourly
+            report-slack-channel: '#notif-my-repo'
         secrets:
             TESTER_APIFY_TOKEN: ${{ secrets.TESTER_APIFY_TOKEN }}
             SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "apify-test-tools",
-  "version": "0.10.2-beta.15",
+  "version": "0.10.2-beta.17",
   "type": "module",
   "description": "TBD",
   "repository": {

package/test/unit/bin/actor-ids.test.ts

@@ -32,12 +32,16 @@ describe('withActorIds', () => {
         expect(client.actor).not.toHaveBeenCalled();
     });
 
-    it('fails for an Actor the token cannot see, instead of skipping its tests', async () => {
-        await expect(withActorIds([{ actorFullName: 'owner/missing' }], fakeClient())).rejects.toThrow(
-            'Actor owner/missing not found',
-        );
-        await expect(withActorIds([{ actorRawId: 'rawMissing' }], fakeClient())).rejects.toThrow(
-            'Actor rawMissing not found',
-        );
+    it('keeps an Actor the token cannot see as listed, instead of failing the run', async () => {
+        const targets = [
+            { actorFullName: 'owner/missing' },
+            { actorRawId: 'rawMissing' },
+            { actorFullName: 'owner/a' },
+        ];
+        expect(await withActorIds(targets, fakeClient())).toEqual([
+            { actorFullName: 'owner/missing' },
+            { actorRawId: 'rawMissing' },
+            { actorFullName: 'owner/a', actorRawId: 'rawA' },
+        ]);
     });
 });