From d6087e56cff2cfa4a6e274dcea44fdffa674c841 Mon Sep 17 00:00:00 2001
From: davelopez <46503462+davelopez@users.noreply.github.com>
Date: Tue, 21 May 2024 12:32:13 +0200
Subject: [PATCH 1/2] Do not show the Edit UI when error accessing the dataset

---
 .../components/DatasetInformation/DatasetAttributes.vue    | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/client/src/components/DatasetInformation/DatasetAttributes.vue b/client/src/components/DatasetInformation/DatasetAttributes.vue
index e197e126b6a9..eef869f1e405 100644
--- a/client/src/components/DatasetInformation/DatasetAttributes.vue
+++ b/client/src/components/DatasetInformation/DatasetAttributes.vue
@@ -5,7 +5,7 @@
             {{ messageText | l }}
         </b-alert>
         <DatasetAttributesProvider :id="datasetId" v-slot="{ result, loading }" @error="onError">
-            <div v-if="!loading" class="mt-3">
+            <div v-if="!loading && !hasError" class="mt-3">
                 <b-tabs>
                     <b-tab v-if="!result['attribute_disable']">
                         <template v-slot:title>
@@ -121,6 +121,11 @@ export default {
             formData: {},
         };
     },
+    computed: {
+        hasError() {
+            return this.messageVariant === "danger";
+        },
+    },
     methods: {
         onAttribute(data) {
             this.formData["attribute"] = data;

From 5bd9fed08e368daaa309ee54fc5ee1b5c8e8e4a0 Mon Sep 17 00:00:00 2001
From: davelopez <46503462+davelopez@users.noreply.github.com>
Date: Tue, 21 May 2024 13:14:40 +0200
Subject: [PATCH 2/2] Add client unit test for edit dataset error

---
 .../DatasetAttributes.test.js                 | 36 +++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/client/src/components/DatasetInformation/DatasetAttributes.test.js b/client/src/components/DatasetInformation/DatasetAttributes.test.js
index 7862c91da571..4822815f3805 100644
--- a/client/src/components/DatasetInformation/DatasetAttributes.test.js
+++ b/client/src/components/DatasetInformation/DatasetAttributes.test.js
@@ -33,6 +33,31 @@ async function buildWrapper(conversion_disable = false) {
     return wrapper;
 }
 
+async function buildWrapperWithError(error) {
+    const wrapper = mount(DatasetAttributes, {
+        propsData: {
+            datasetId: "dataset_id",
+            messageText: error,
+            messageVariant: "danger",
+        },
+        computed: {
+            hasError() {
+                return true;
+            },
+        },
+        localVue,
+        stubs: {
+            DatasetAttributesProvider: MockProvider({
+                result: null,
+            }),
+            FontAwesomeIcon: false,
+            FormElement: false,
+        },
+    });
+    await flushPromises();
+    return wrapper;
+}
+
 describe("DatasetAttributes", () => {
     it("check rendering", async () => {
         const axiosMock = new MockAdapter(axios);
@@ -61,4 +86,15 @@ describe("DatasetAttributes", () => {
         expect(wrapper.findAll(".tab-pane").length).toBe(3);
         expect(wrapper.findAll(".ui-portlet-section").length).toBe(1);
     });
+
+    it("doesn't render edit controls with error", async () => {
+        const wrapper = await buildWrapperWithError("error");
+        expect(wrapper.findAll("button").length).toBe(0);
+        expect(wrapper.findAll("#attribute_text").length).toBe(0);
+        expect(wrapper.findAll("#conversion_text").length).toBe(0);
+        expect(wrapper.findAll("#datatype_text").length).toBe(0);
+        expect(wrapper.findAll("#permission_text").length).toBe(0);
+        expect(wrapper.findAll(".tab-pane").length).toBe(0);
+        expect(wrapper.findAll(".ui-portlet-section").length).toBe(0);
+    });
 });