Support non-local ancestor builds with uncommittedHash

node-src/git/findAncestorBuildWithCommit.test.ts

@@ -15,6 +15,7 @@ const makeBuild = (build: Partial<Build> = {}): Build => ({
   number: 1,
   commit: 'missing',
   uncommittedHash: '',
+  isLocalBuild: false,
   ...build,
 });
 const makeResult = (ancestorBuilds: Build[]): AncestorBuildsQueryResult => ({
@@ -36,14 +37,24 @@ describe('findAncestorBuildWithCommit', () => {
     expect(client.runQuery.mock.calls[0][1]).toMatchObject({ buildNumber: 1 });
   });
 
-  it('does not return build with uncommitted changes', async () => {
+  it('does not return a local build with uncommitted changes', async () => {
     client.runQuery.mockReturnValue(
-      makeResult([makeBuild({ commit: 'exists', uncommittedHash: 'abc123' })])
+      makeResult([makeBuild({ commit: 'exists', uncommittedHash: 'abc123', isLocalBuild: true })])
     );
 
     expect(await findAncestorBuildWithCommit({ client }, 1, { page: 1, limit: 1 })).toBeNull();
   });
 
+  it('DOES return a CI build with uncommitted changes', async () => {
+    client.runQuery.mockReturnValue(
+      makeResult([makeBuild({ commit: 'exists', uncommittedHash: 'abc123' })])
+    );
+
+    expect(await findAncestorBuildWithCommit({ client }, 1, { page: 1, limit: 1 })).toMatchObject({
+      commit: 'exists',
+    });
+  });
+
   it('passes skip and limit and recurse', async () => {
     const toFind = makeBuild({ number: 3, commit: 'exists' });
     client.runQuery

node-src/git/findAncestorBuildWithCommit.ts

@@ -70,7 +70,9 @@ export async function findAncestorBuildWithCommit(
         return [build, exists] as const;
       })
     );
-    const result = results.find(([build, exists]) => !build.uncommittedHash && exists);
+    const result = results.find(
+      ([build, exists]) => !(build.isLocalBuild && build.uncommittedHash) && exists
+    );
 
     if (result) return result[0];