Feat: skip building actors if only tests changed

[kazadoiyul]

Closes #1

[oklinov]

sorry for the delay 🙁

[oklinov]

I think this isn't needed...you can extend getDefaultVersionAndTag to return buildId:

    const defaultBuildId = actorInfo.taggedBuilds![defaultBuildTag].buildId!;

[oklinov]

Suggested change

	import {getAllDefaultBuilds, runBuilds} from './build.js';
	import { getAllDefaultBuilds, runBuilds } from './build.js';

[oklinov]

this should check if the lowercaseFilePath is a platform test, you can do it like this:

        return lowercaseFilePath.match(/^(code\/)?test\/platform\//);

I'd also rename it to isPlatformTestFile

[oklinov]

this will run always the tests for the actors that we're not building, no? My idea was to run them only if there is a changed in platform tests (in test/platfrom).

What I would do is to have getChangedActors return actorsToTest if there is such change. So, if we detect that there is a change in platform tests, we put all the actors that are not being built to actorsToTest.

I prepared some unit tests that hopefully explain the logic:

    test('only core tests changed', async () => {
        const FILES = [
            'test/platform/core/core.test.ts',
        ];

        const { actorsChanged, actorsToTest, codeChanged } = await getChangedActors({ actorConfigs: ACTOR_CONFIGS, isLatest: false, filepathsChanged: FILES });

        expect(actorsChanged).toEqual([]);
        expect(actorsToTest).toEqual(ACTOR_CONFIGS);
        expect(codeChanged).toBe(false);
    });

    test('core tests and src (excluding standalone actors) changed', async () => {
        const FILES = [
            'src/main.ts',
            'test/platform/core/core.test.ts',
        ];

        const { actorsChanged, actorsToTest, codeChanged } = await getChangedActors({ actorConfigs: ACTOR_CONFIGS, isLatest: false, filepathsChanged: FILES });

        expect(actorsChanged).toEqual(ACTOR_CONFIGS.filter((actor) => actor.isStandalone));
        expect(actorsToTest).toEqual(ACTOR_CONFIGS.filter((actor) => !actor.isStandalone));
        expect(codeChanged).toBe(false);
    });

    test('core tests and one input_schema changed', async () => {
        const FILES = [
            'test/platform/core/core.test.ts',
            'actors/lukaskrivka_testing-github-integration-1/.actor/INPUT_SCHEMA.json'
        ];

        const { actorsChanged, actorsToTest, codeChanged } = await getChangedActors({ actorConfigs: ACTOR_CONFIGS, isLatest: false, filepathsChanged: FILES });

        expect(actorsChanged).toEqual([ACTOR_CONFIGS[0]]);
        expect(actorsToTest).toEqual(ACTOR_CONFIGS.slice(1));
        expect(codeChanged).toBe(false);
    });

let me know if something's not clear

[metalwarrior665]

This is a good feature to have but currently it is a bit hard to implement since we don't know what builds to use. Let's look at it after we finish the current round of refactors, most likely just to write it from scratch.