fix sourceVersion handling for the source override use case

Author: taoyong-ty

code-build.js

@@ -306,7 +306,10 @@ function inputs2Parameters(inputs) {
 
   const sourceOverride = !disableSourceOverride
     ? {
-        sourceVersion: sourceVersion,
+        // sourceVersion should not be set when using sourceTypeOverride or sourceLocationOverride
+        ...(sourceTypeOverride || sourceLocationOverride
+          ? {}
+          : {sourceVersion}),
         sourceTypeOverride: sourceTypeOverride || "GITHUB",
         sourceLocationOverride: sourceLocationOverride || `https://github.com/${owner}/${repo}.git`,
       }

dist/index.js

@@ -312,7 +312,10 @@ function inputs2Parameters(inputs) {
 
   const sourceOverride = !disableSourceOverride
     ? {
-        sourceVersion: sourceVersion,
+        // sourceVersion should not be set when using sourceTypeOverride or sourceLocationOverride
+        ...(sourceTypeOverride || sourceLocationOverride
+          ? {}
+          : {sourceVersion}),
         sourceTypeOverride: sourceTypeOverride || "GITHUB",
         sourceLocationOverride: sourceLocationOverride || `https://github.com/${owner}/${repo}.git`,
       }

test/code-build-test.js

@@ -431,6 +431,7 @@ describe("inputs2Parameters", () => {
     });
     expect(test).to.haveOwnProperty("sourceTypeOverride").and.to.equal("S3");
     expect(test).to.haveOwnProperty("sourceLocationOverride").and.to.equal("bucket-name/object-name");
+    expect(test).to.not.haveOwnProperty("sourceVersion");
   });
 
   it("can process disable-github-env-vars", () => {