do not call save credentials if the validation failed

SwissDataScienceCenter · Aug 27, 2024 · 65dd221 · 65dd221
1 parent 756da48
commit 65dd221
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/client/src/features/project/components/cloudStorage/CloudStorageModal.tsx b/client/src/features/project/components/cloudStorage/CloudStorageModal.tsx
@@ -391,7 +391,8 @@ export default function CloudStorageModal({
     const shouldSaveCredentials = !!(
       isV2 &&
       storageDetails.options &&
-      state.saveCredentials
+      state.saveCredentials &&
+      validationSucceeded
     );
     if (!shouldSaveCredentials) {
       return;
@@ -423,6 +424,7 @@ export default function CloudStorageModal({
     schema,
     storageDetails.options,
     storageDetails.schema,
+    validationSucceeded,
   ]);
 
   // Visual elements

diff --git a/tests/cypress/e2e/projectV2DataSourceCredentials.spec.ts b/tests/cypress/e2e/projectV2DataSourceCredentials.spec.ts
@@ -55,7 +55,12 @@ describe("Set up data sources with credentials", () => {
         fixture: "cloudStorage/cloud-storage-with-secrets-values-empty.json",
         name: "getCloudStorageV2",
       })
-      .testCloudStorage({ success: false });
+      .testCloudStorage({ success: false })
+      .postCloudStorageSecrets({
+        content: [],
+        // No call to postCloudStorageSecrets is expected
+        shouldNotBeCalled: true,
+      });
     cy.visit("/v2/projects/user1-uuid/test-2-v2-project");
     cy.wait("@readProjectV2");
     // add data source

diff --git a/tests/cypress/support/renkulab-fixtures/cloudStorage.ts b/tests/cypress/support/renkulab-fixtures/cloudStorage.ts
@@ -36,6 +36,7 @@ interface PostCloudStorageSecretsArgs extends CloudStorageSecretsArgs {
     name: string;
     value: string;
   }[];
+  shouldNotBeCalled?: boolean;
 }
 
 interface TestCloudStorageArgs extends SimpleFixture {
@@ -112,6 +113,7 @@ export function CloudStorage<T extends FixturesConstructor>(Parent: T) {
         content,
         fixture = "cloudStorage/cloud-storage-secrets.json",
         name = "postCloudStorageSecrets",
+        shouldNotBeCalled = false,
         storageId = 2,
       } = args ?? {};
       cy.fixture(fixture).then((secrets) => {
@@ -120,6 +122,8 @@ export function CloudStorage<T extends FixturesConstructor>(Parent: T) {
           "POST",
           `/ui-server/api/data/storages_v2/${storageId}/secrets`,
           (req) => {
+            if (shouldNotBeCalled)
+              throw new Error("No call to post secrets expected");
             const newSecrets = req.body;
             expect(newSecrets.length).equal(content.length);
             newSecrets.forEach((secret, index) => {