[server] Add more tests around "prebuild selection" (#20358)

* [server] Extend and fix context parser tests around prebuild discovery

* [server] Add more "prebuild selection" tests to ContextService

* review comments

* [server] Fix order of commits in MockRepositoryProvider to be aligned with actual implementation

Co-authored-by: Filip Troníček <[email protected]>

---------

Co-authored-by: Filip Troníček <[email protected]>

Author: geropl

components/server/src/prebuilds/incremental-workspace-service.ts

@@ -66,7 +66,7 @@ export class IncrementalWorkspaceService {
         return history;
     }
 
-    public async findGoodBaseForIncrementalBuild(
+    public async findBaseForIncrementalWorkspace(
         context: CommitContext,
         config: WorkspaceConfig,
         history: WithCommitHistory,

components/server/src/prebuilds/prebuild-manager.ts

@@ -381,7 +381,11 @@ export class PrebuildManager {
                         JSON.stringify(filterPrebuildTasks(config?.tasks));
                     // If there is an existing prebuild that isn't failed and it's based on the current config, we return it here instead of triggering a new prebuild.
                     if (isSameConfig) {
-                        return { prebuildId: existingPB.id, wsid: existingPB.buildWorkspaceId, done: true };
+                        return {
+                            prebuildId: existingPB.id,
+                            wsid: existingPB.buildWorkspaceId,
+                            done: existingPB.state === "available",
+                        };
                     }
                 }
             }
@@ -407,7 +411,7 @@ export class PrebuildManager {
                         commitHistory: repoHist.commitHistory.slice(0, prebuildInterval),
                     })),
                 };
-                const prebuild = await this.incrementalPrebuildsService.findGoodBaseForIncrementalBuild(
+                const prebuild = await this.incrementalPrebuildsService.findBaseForIncrementalWorkspace(
                     context,
                     config,
                     history,
@@ -416,7 +420,11 @@ export class PrebuildManager {
                     true,
                 );
                 if (prebuild) {
-                    return { prebuildId: prebuild.id, wsid: prebuild.buildWorkspaceId, done: true };
+                    return {
+                        prebuildId: prebuild.id,
+                        wsid: prebuild.buildWorkspaceId,
+                        done: prebuild.state === "available",
+                    };
                 }
             }