fix: don't send empty repositories_mode

The server expects it to be either `replace` or `append`, not `""`.

Signed-off-by: Paul Spooren <mail@aparcar.org>

Author: aparcar

tests/js/asu.test.js

@@ -173,7 +173,7 @@ describe("createAsuRequestBuilder", () => {
     assert.equal(capturedBody.repositories_mode, "append");
   });
 
-  it("sends empty repositories_mode for unsupported mode values", async () => {
+  it("omits repositories_mode for unsupported mode values", async () => {
     let capturedBody;
     globalThis.fetch = fetchWithLangStub((_url, opts) => {
       capturedBody = JSON.parse(opts.body);
@@ -196,7 +196,7 @@ describe("createAsuRequestBuilder", () => {
     build();
     await new Promise((r) => setTimeout(r, 50));
 
-    assert.equal(capturedBody.repositories_mode, "");
+    assert.ok(!("repositories_mode" in capturedBody));
   });
 
   it("sends GET with hash appended when requestHash is provided", async () => {

www/js/asu.js

@@ -77,8 +77,6 @@ export function createAsuRequestBuilder(context) {
 
     const selectedVersion = $("#versions").value;
     const reposMode = config.asu_repositories_mode;
-    const repositoriesMode =
-      reposMode === "replace" || reposMode === "append" ? reposMode : "";
     const buildBody = {
       profile: currentDevice.id,
       target: currentDevice.target,
@@ -94,8 +92,10 @@ export function createAsuRequestBuilder(context) {
         selectedVersion
       ),
       repository_keys: config.asu_repository_keys || [],
-      repositories_mode: repositoriesMode,
     };
+    if (reposMode === "replace" || reposMode === "append") {
+      buildBody.repositories_mode = reposMode;
+    }
     const requestUrl =
       `${config.asu_url}/api/v1/build` + (requestHash ? `/${requestHash}` : "");