[FIX] column no longer styled if selectedTool is null

neurobagel · Oct 17, 2023 · d3ebcf8 · d3ebcf8
1 parent ea0dbc8
commit d3ebcf8
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/components/category-toolgroup.vue b/components/category-toolgroup.vue
@@ -104,7 +104,10 @@
                 this.alterColumnToToolMapping({columnName: row.column, toolIdentifier: this.selectedTool.identifier});
             },
             styleRow(p_row) {
-                if (this.columnToToolMap[p_row.column] === this.selectedTool.identifier) {
+                if (
+                    (this.columnToToolMap[p_row.column] !== null) &&
+                    (this.columnToToolMap[p_row.column] === this.selectedTool.identifier)
+                ) {
                     return "selected-tool";
                 } else {
                     return "";
@@ -119,4 +122,4 @@
     background-color: red !important;
 }
 
-</style>
+</style>
diff --git a/cypress/component/category-toolgroup.cy.js b/cypress/component/category-toolgroup.cy.js
@@ -99,6 +99,25 @@ describe("Tool Group component", () => {
         cy.get("[data-cy='assessment-column-table']").contains("column3");
     });
 
+    it("before a tool is selected, all columns are unstyled", () => {
+        store.state.columnToToolMap = {
+                column1: null,
+                column2: null,
+                column3: null
+        };
+
+        cy.mount(categoryToolGroup, {
+            mocks: {
+
+                $store: store
+            }
+        });
+
+        cy.get("[data-cy='assessment-column-table']").find("tr:contains('column1')").should('not.have.class', 'selected-tool');
+        cy.get("[data-cy='assessment-column-table']").find("tr:contains('column3')").should('not.have.class', 'selected-tool');
+
+    });
+
     it("gets assessment tool names from the store and shows them in a dropdown", () => {
         cy.mount(categoryToolGroup, {
             mocks: {