From 4b49ba26b1463a6cc265c9427aaa6438b1d272f9 Mon Sep 17 00:00:00 2001
From: Mo Mesgin <mmesgin@Mos-M2-MacBook-Pro.local>
Date: Tue, 5 Mar 2024 17:49:55 -0800
Subject: [PATCH 1/8] fix creating repository with secret

---
 .../po/components/select-or-create-auth.po.ts |  2 +-
 cypress/e2e/po/edit/chart-repositories.po.ts  |  9 +++
 .../tests/pages/manager/repositories.spec.ts  | 64 +++++++++++++++++++
 .../Form/TextArea/TextAreaAutoGrow.vue        |  2 +-
 shell/edit/catalog.cattle.io.clusterrepo.vue  |  7 +-
 5 files changed, 81 insertions(+), 3 deletions(-)

diff --git a/cypress/e2e/po/components/select-or-create-auth.po.ts b/cypress/e2e/po/components/select-or-create-auth.po.ts
index edfb65dc0b6..8a9532abd53 100644
--- a/cypress/e2e/po/components/select-or-create-auth.po.ts
+++ b/cypress/e2e/po/components/select-or-create-auth.po.ts
@@ -23,7 +23,7 @@ export default class SelectOrCreateAuthPo extends ComponentPo {
 
   createSSHAuth(privateKey: string, publicKey: string) {
     this.authSelect().toggle();
-    this.authSelect().clickOptionWithLabel('Create a SSH Auth Secret');
+    this.authSelect().clickOptionWithLabel('Create a SSH Key Secret');
     this.setAuthSecret('ssh', privateKey, publicKey);
   }
 }
diff --git a/cypress/e2e/po/edit/chart-repositories.po.ts b/cypress/e2e/po/edit/chart-repositories.po.ts
index 02acb43339b..4d2584a70ba 100644
--- a/cypress/e2e/po/edit/chart-repositories.po.ts
+++ b/cypress/e2e/po/edit/chart-repositories.po.ts
@@ -4,6 +4,7 @@ import AsyncButtonPo from '@/cypress/e2e/po/components/async-button.po';
 import LabeledSelectPo from '@/cypress/e2e/po/components/labeled-select.po';
 import RadioGroupInputPo from '@/cypress/e2e/po/components/radio-group-input.po';
 import NameNsDescription from '@/cypress/e2e/po/components/name-ns-description.po';
+import SelectOrCreateAuthPo from '@/cypress/e2e/po/components/select-or-create-auth.po';
 
 export default class ChartRepositoriesCreateEditPo extends PagePo {
   private static createPath(clusterId: string, product: 'apps' | 'manager', repoName?: string ) {
@@ -44,6 +45,14 @@ export default class ChartRepositoriesCreateEditPo extends PagePo {
     return new AsyncButtonPo('[data-testid="action-button-async-button"]', this.self());
   }
 
+  authSelectOrCreate(selector: string) {
+    return new SelectOrCreateAuthPo(selector);
+  }
+
+  clusterrepoAuthSelectOrCreate() {
+    return this.authSelectOrCreate('[data-testid="clusterrepo-auth-secret"]');
+  }
+
   saveAndWaitForRequests(method: string, url: string) {
     cy.intercept(method, url).as('request');
     this.saveCreateForm().click();
diff --git a/cypress/e2e/tests/pages/manager/repositories.spec.ts b/cypress/e2e/tests/pages/manager/repositories.spec.ts
index 8a7f4fa4e14..079fcd1c064 100644
--- a/cypress/e2e/tests/pages/manager/repositories.spec.ts
+++ b/cypress/e2e/tests/pages/manager/repositories.spec.ts
@@ -126,4 +126,68 @@ describe('Cluster Management Helm Repositories', { testIsolation: 'off', tags: [
     // check list details
     cy.contains(this.repoName).should('not.exist');
   });
+
+  it('can create a repository with basic auth', function() {
+    ChartRepositoriesPagePo.navTo();
+    repositoriesPage.waitForPage();
+    repositoriesPage.waitForGoTo('/v1/catalog.cattle.io.clusterrepos?exclude=metadata.managedFields');
+    repositoriesPage.create();
+    repositoriesPage.createEditRepositories().waitForPage();
+    repositoriesPage.createEditRepositories().nameNsDescription().name().set(this.repoName);
+    repositoriesPage.createEditRepositories().nameNsDescription().description().set(`${ this.repoName }-description`);
+    repositoriesPage.createEditRepositories().repoRadioBtn().set(1);
+    repositoriesPage.createEditRepositories().gitRepoUrl().set('https://git.rancher.io/charts');
+    repositoriesPage.createEditRepositories().gitBranch().set('release-v2.8');
+    repositoriesPage.createEditRepositories().clusterrepoAuthSelectOrCreate().createBasicAuth('test', 'test');
+    repositoriesPage.createEditRepositories().saveAndWaitForRequests('POST', '/v1/catalog.cattle.io.clusterrepos');
+    repositoriesPage.waitForPage();
+
+    // check list details
+    repositoriesPage.list().details(this.repoName, 2).should('be.visible');
+    repositoriesPage.list().details(this.repoName, 1).contains('In Progress').should('be.visible');
+    repositoriesPage.list().actionMenu(`${ this.repoName }`).getMenuItem('Delete').click();
+
+    const promptRemove = new PromptRemove();
+
+    cy.intercept('DELETE', `v1/catalog.cattle.io.clusterrepos/${ this.repoName }`).as('deleteRepository');
+
+    promptRemove.remove();
+    cy.wait('@deleteRepository');
+    repositoriesPage.waitForPage();
+
+    // check list details
+    cy.contains(`${ this.repoName }`).should('not.exist');
+  });
+
+  it('can create a repository with SSH key', function() {
+    ChartRepositoriesPagePo.navTo();
+    repositoriesPage.waitForPage();
+    repositoriesPage.waitForGoTo('/v1/catalog.cattle.io.clusterrepos?exclude=metadata.managedFields');
+    repositoriesPage.create();
+    repositoriesPage.createEditRepositories().waitForPage();
+    repositoriesPage.createEditRepositories().nameNsDescription().name().set(this.repoName);
+    repositoriesPage.createEditRepositories().nameNsDescription().description().set(`${ this.repoName }-description`);
+    repositoriesPage.createEditRepositories().repoRadioBtn().set(1);
+    repositoriesPage.createEditRepositories().gitRepoUrl().set('https://git.rancher.io/charts');
+    repositoriesPage.createEditRepositories().gitBranch().set('release-v2.8');
+    repositoriesPage.createEditRepositories().clusterrepoAuthSelectOrCreate().createSSHAuth('privateKey', 'publicKey');
+    repositoriesPage.createEditRepositories().saveAndWaitForRequests('POST', '/v1/catalog.cattle.io.clusterrepos');
+    repositoriesPage.waitForPage();
+
+    // check list details
+    repositoriesPage.list().details(this.repoName, 2).should('be.visible');
+
+    repositoriesPage.list().actionMenu(`${ this.repoName }`).getMenuItem('Delete').click();
+
+    const promptRemove = new PromptRemove();
+
+    cy.intercept('DELETE', `v1/catalog.cattle.io.clusterrepos/${ this.repoName }`).as('deleteRepository');
+
+    promptRemove.remove();
+    cy.wait('@deleteRepository');
+    repositoriesPage.waitForPage();
+
+    // check list details
+    cy.contains(`${ this.repoName }`).should('not.exist');
+  });
 });
diff --git a/pkg/rancher-components/src/components/Form/TextArea/TextAreaAutoGrow.vue b/pkg/rancher-components/src/components/Form/TextArea/TextAreaAutoGrow.vue
index e69287393f4..67e7dd6a048 100644
--- a/pkg/rancher-components/src/components/Form/TextArea/TextAreaAutoGrow.vue
+++ b/pkg/rancher-components/src/components/Form/TextArea/TextAreaAutoGrow.vue
@@ -155,7 +155,7 @@ export default Vue.extend({
 <template>
   <textarea
     ref="ta"
-    data-testid="text-area-auto-grow"
+    :data-testid="$attrs['data-testid'] ? $attrs['data-testid'] : 'text-area-auto-grow'"
     :disabled="isDisabled"
     :style="style"
     :placeholder="placeholder"
diff --git a/shell/edit/catalog.cattle.io.clusterrepo.vue b/shell/edit/catalog.cattle.io.clusterrepo.vue
index 7d02fbb644d..4809fc89804 100644
--- a/shell/edit/catalog.cattle.io.clusterrepo.vue
+++ b/shell/edit/catalog.cattle.io.clusterrepo.vue
@@ -28,6 +28,9 @@ export default {
   },
 
   computed: {
+    inStore() {
+      return this.$store.getters['currentProduct'].inStore;
+    },
     secretNamespace() {
       const tryNames = ['cattle-system', 'default'];
 
@@ -37,7 +40,7 @@ export default {
         }
       }
 
-      return this.$store.getters['cluster/all'](NAMESPACE)[0]?.id;
+      return this.$store.getters[`${ this.inStore }/all`](NAMESPACE)[0]?.id;
     }
   },
 
@@ -118,10 +121,12 @@ export default {
     />
 
     <SelectOrCreateAuthSecret
+      data-testid="clusterrepo-auth-secret"
       v-model="value.spec.clientSecret"
       :register-before-hook="registerBeforeHook"
       :namespace="secretNamespace"
       :limit-to-namespace="false"
+      :in-store="inStore"
       generate-name="clusterrepo-auth-"
     />
 

From c216c976ebfdf9569ac53b8d02fd465cbabc17e5 Mon Sep 17 00:00:00 2001
From: Mo Mesgin <mmesgin@Mos-M2-MacBook-Pro.local>
Date: Tue, 5 Mar 2024 18:08:31 -0800
Subject: [PATCH 2/8] fix unit test

---
 shell/components/form/__tests__/KeyValue.test.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/shell/components/form/__tests__/KeyValue.test.ts b/shell/components/form/__tests__/KeyValue.test.ts
index 38ed8ae7791..8aa8d421ae9 100644
--- a/shell/components/form/__tests__/KeyValue.test.ts
+++ b/shell/components/form/__tests__/KeyValue.test.ts
@@ -47,7 +47,7 @@ describe('component: KeyValue', () => {
       stubs:      { CodeMirror: true }
     });
 
-    const inputFieldTextArea = wrapper.find('[data-testid="text-area-auto-grow"]');
+    const inputFieldTextArea = wrapper.find('[data-testid="value-multiline"]');
 
     inputFieldTextArea.setValue('bar\n');
 

From f01f7dc310c3612e3b3ec0000ec99cf7fc34b5d9 Mon Sep 17 00:00:00 2001
From: Mo Mesgin <mmesgin@Mos-M2-MacBook-Pro.local>
Date: Tue, 5 Mar 2024 18:22:44 -0800
Subject: [PATCH 3/8] lint

---
 shell/edit/catalog.cattle.io.clusterrepo.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/shell/edit/catalog.cattle.io.clusterrepo.vue b/shell/edit/catalog.cattle.io.clusterrepo.vue
index 4809fc89804..12edc2fff36 100644
--- a/shell/edit/catalog.cattle.io.clusterrepo.vue
+++ b/shell/edit/catalog.cattle.io.clusterrepo.vue
@@ -121,8 +121,8 @@ export default {
     />
 
     <SelectOrCreateAuthSecret
-      data-testid="clusterrepo-auth-secret"
       v-model="value.spec.clientSecret"
+      data-testid="clusterrepo-auth-secret"
       :register-before-hook="registerBeforeHook"
       :namespace="secretNamespace"
       :limit-to-namespace="false"

From b2be9811f140a2689d7bd254b12959a3a1ea1009 Mon Sep 17 00:00:00 2001
From: Mo Mesgin <mmesgin@Mos-M2-MacBook-Pro.local>
Date: Wed, 6 Mar 2024 11:26:25 -0800
Subject: [PATCH 4/8] fix unit test + minor po refactor

---
 .../po/components/select-or-create-auth.po.ts | 22 +++++++++++++------
 .../po/pages/global-settings/home-links.po.ts |  2 +-
 .../tests/pages/manager/repositories.spec.ts  |  8 +++----
 .../form/SelectOrCreateAuthSecret.vue         |  4 ++--
 4 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/cypress/e2e/po/components/select-or-create-auth.po.ts b/cypress/e2e/po/components/select-or-create-auth.po.ts
index 8a9532abd53..3e93b88f3b1 100644
--- a/cypress/e2e/po/components/select-or-create-auth.po.ts
+++ b/cypress/e2e/po/components/select-or-create-auth.po.ts
@@ -7,23 +7,31 @@ export default class SelectOrCreateAuthPo extends ComponentPo {
     return new LabeledSelectPo(`${ this.selector } [data-testid="auth-secret-select"]`, this.self());
   }
 
-  setAuthSecret(secretType: string, username: string, password: string) {
-    const privateKey = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-${ secretType }-private-key"]`, this.self());
-    const publicKey = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-${ secretType }-public-key"]`, this.self());
+  setBasicAuthSecret(username: string, password: string) {
+    const usernameInput = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-basic-username"]`, this.self());
+    const passwordInput = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-basic-ppassword"]`, this.self());
 
-    privateKey.set(username);
-    publicKey.set(password);
+    usernameInput.set(username);
+    passwordInput.set(password);
+  }
+
+  setSSHSecret(privateKey: string, publicKey: string) {
+    const privateKeyInput = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-ssh-private-key"]`, this.self());
+    const publicKeyInput = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-ssh-public-key"]`, this.self());
+
+    privateKeyInput.set(privateKey);
+    publicKeyInput.set(publicKey);
   }
 
   createBasicAuth(username = 'auth-test-user', password = 'auth-test-password') {
     this.authSelect().toggle();
     this.authSelect().clickOptionWithLabel('Create a HTTP Basic Auth Secret');
-    this.setAuthSecret('basic', username, password);
+    this.setBasicAuthSecret(username, password);
   }
 
   createSSHAuth(privateKey: string, publicKey: string) {
     this.authSelect().toggle();
     this.authSelect().clickOptionWithLabel('Create a SSH Key Secret');
-    this.setAuthSecret('ssh', privateKey, publicKey);
+    this.setSSHSecret(privateKey, publicKey);
   }
 }
diff --git a/cypress/e2e/po/pages/global-settings/home-links.po.ts b/cypress/e2e/po/pages/global-settings/home-links.po.ts
index ccb434fe0e6..c186ec02db0 100644
--- a/cypress/e2e/po/pages/global-settings/home-links.po.ts
+++ b/cypress/e2e/po/pages/global-settings/home-links.po.ts
@@ -44,7 +44,7 @@ export class HomeLinksPagePo extends RootClusterPage {
   }
 
   urlInput(): LabeledInputPo {
-    return new LabeledInputPo('[data-testid="text-area-auto-grow"]');
+    return new LabeledInputPo('[data-testid="value-multiline"]');
   }
 
   applyButton() {
diff --git a/cypress/e2e/tests/pages/manager/repositories.spec.ts b/cypress/e2e/tests/pages/manager/repositories.spec.ts
index 079fcd1c064..a480e6bb425 100644
--- a/cypress/e2e/tests/pages/manager/repositories.spec.ts
+++ b/cypress/e2e/tests/pages/manager/repositories.spec.ts
@@ -145,7 +145,7 @@ describe('Cluster Management Helm Repositories', { testIsolation: 'off', tags: [
     // check list details
     repositoriesPage.list().details(this.repoName, 2).should('be.visible');
     repositoriesPage.list().details(this.repoName, 1).contains('In Progress').should('be.visible');
-    repositoriesPage.list().actionMenu(`${ this.repoName }`).getMenuItem('Delete').click();
+    repositoriesPage.list().actionMenu(this.repoName).getMenuItem('Delete').click();
 
     const promptRemove = new PromptRemove();
 
@@ -156,7 +156,7 @@ describe('Cluster Management Helm Repositories', { testIsolation: 'off', tags: [
     repositoriesPage.waitForPage();
 
     // check list details
-    cy.contains(`${ this.repoName }`).should('not.exist');
+    cy.contains(this.repoName).should('not.exist');
   });
 
   it('can create a repository with SSH key', function() {
@@ -177,7 +177,7 @@ describe('Cluster Management Helm Repositories', { testIsolation: 'off', tags: [
     // check list details
     repositoriesPage.list().details(this.repoName, 2).should('be.visible');
 
-    repositoriesPage.list().actionMenu(`${ this.repoName }`).getMenuItem('Delete').click();
+    repositoriesPage.list().actionMenu(this.repoName).getMenuItem('Delete').click();
 
     const promptRemove = new PromptRemove();
 
@@ -188,6 +188,6 @@ describe('Cluster Management Helm Repositories', { testIsolation: 'off', tags: [
     repositoriesPage.waitForPage();
 
     // check list details
-    cy.contains(`${ this.repoName }`).should('not.exist');
+    cy.contains(this.repoName).should('not.exist');
   });
 });
diff --git a/shell/components/form/SelectOrCreateAuthSecret.vue b/shell/components/form/SelectOrCreateAuthSecret.vue
index fc5d056f999..c9b6ed0b7d4 100644
--- a/shell/components/form/SelectOrCreateAuthSecret.vue
+++ b/shell/components/form/SelectOrCreateAuthSecret.vue
@@ -509,7 +509,7 @@ export default {
         <div :class="moreCols">
           <LabeledInput
             v-model="publicKey"
-            data-testid="auth-secret-basic-public-key"
+            data-testid="auth-secret-basic-username"
             :mode="mode"
             label-key="selectOrCreateAuthSecret.basic.username"
           />
@@ -517,7 +517,7 @@ export default {
         <div :class="moreCols">
           <LabeledInput
             v-model="privateKey"
-            data-testid="auth-secret-basic-private-key"
+            data-testid="auth-secret-basic-password"
             :mode="mode"
             type="password"
             label-key="selectOrCreateAuthSecret.basic.password"

From fc8c542aa5274cda701251d7770e72f4e7704fdd Mon Sep 17 00:00:00 2001
From: Mo Mesgin <mmesgin@Mos-M2-MacBook-Pro.local>
Date: Wed, 6 Mar 2024 11:50:53 -0800
Subject: [PATCH 5/8] fix typo

---
 cypress/e2e/po/components/select-or-create-auth.po.ts | 2 +-
 package.json                                          | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/cypress/e2e/po/components/select-or-create-auth.po.ts b/cypress/e2e/po/components/select-or-create-auth.po.ts
index 3e93b88f3b1..88a0a5e2715 100644
--- a/cypress/e2e/po/components/select-or-create-auth.po.ts
+++ b/cypress/e2e/po/components/select-or-create-auth.po.ts
@@ -9,7 +9,7 @@ export default class SelectOrCreateAuthPo extends ComponentPo {
 
   setBasicAuthSecret(username: string, password: string) {
     const usernameInput = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-basic-username"]`, this.self());
-    const passwordInput = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-basic-ppassword"]`, this.self());
+    const passwordInput = new LabeledInputPo(`${ this.selector } [data-testid="auth-secret-basic-password"]`, this.self());
 
     usernameInput.set(username);
     passwordInput.set(password);
diff --git a/package.json b/package.json
index 1996feff346..37cc570efe1 100644
--- a/package.json
+++ b/package.json
@@ -37,7 +37,7 @@
     "generate": "yarn build",
     "dev-debug": "node --inspect ./node_modules/.bin/vue-cli-service serve",
     "cy:e2e": "cypress open --e2e --browser chrome",
-    "cy:open": "cypress open",
+    "cy:open": "CATTLE_BOOTSTRAP_PASSWORD=suse123!@#12 cypress open",
     "cy:run": "cypress run --browser chrome",
     "cy:run:sorry": "./scripts/e2e",
     "e2e:pre-dev": "yarn docker:local:stop && yarn docker:local:start && NODE_ENV=dev TEST_INSTRUMENT=true yarn build",

From baac60a2060ff7525a4a66965daaab20fff00145 Mon Sep 17 00:00:00 2001
From: Mo Mesgin <mmesgin@Mos-M2-MacBook-Pro.local>
Date: Wed, 6 Mar 2024 12:39:28 -0800
Subject: [PATCH 6/8] cleanup

---
 package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package.json b/package.json
index 37cc570efe1..1996feff346 100644
--- a/package.json
+++ b/package.json
@@ -37,7 +37,7 @@
     "generate": "yarn build",
     "dev-debug": "node --inspect ./node_modules/.bin/vue-cli-service serve",
     "cy:e2e": "cypress open --e2e --browser chrome",
-    "cy:open": "CATTLE_BOOTSTRAP_PASSWORD=suse123!@#12 cypress open",
+    "cy:open": "cypress open",
     "cy:run": "cypress run --browser chrome",
     "cy:run:sorry": "./scripts/e2e",
     "e2e:pre-dev": "yarn docker:local:stop && yarn docker:local:start && NODE_ENV=dev TEST_INSTRUMENT=true yarn build",

From 74dbdc3ab605588fc67049ed74860265c7478073 Mon Sep 17 00:00:00 2001
From: Mo Mesgin <mmesgin@Mos-M2-MacBook-Pro.local>
Date: Thu, 7 Mar 2024 13:07:06 -0800
Subject: [PATCH 7/8] fix sortBy usage

---
 shell/components/form/SelectOrCreateAuthSecret.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/shell/components/form/SelectOrCreateAuthSecret.vue b/shell/components/form/SelectOrCreateAuthSecret.vue
index c9b6ed0b7d4..e630d0a5ac1 100644
--- a/shell/components/form/SelectOrCreateAuthSecret.vue
+++ b/shell/components/form/SelectOrCreateAuthSecret.vue
@@ -207,7 +207,7 @@ export default {
         types.push(SECRET_TYPES.RKE_AUTH_CONFIG);
       }
 
-      const out = this.allSecrets
+      let out = this.allSecrets
         .filter((x) => this.namespace && this.limitToNamespace ? x.metadata.namespace === this.namespace : true)
         .filter((x) => {
           // Must match one of the types if given
@@ -254,7 +254,7 @@ export default {
       }
 
       if ( !this.limitToNamespace ) {
-        sortBy(out, 'group');
+        out = sortBy(out, 'group');
         if ( out.length ) {
           let lastGroup = '';
 

From 66993893a8e82a80265d868dcf9f016022afafc1 Mon Sep 17 00:00:00 2001
From: Mo Mesgin <mmesgin@Mos-M2-MacBook-Pro.local>
Date: Mon, 11 Mar 2024 09:09:57 -0700
Subject: [PATCH 8/8] fallback for inStore

---
 shell/edit/catalog.cattle.io.clusterrepo.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/shell/edit/catalog.cattle.io.clusterrepo.vue b/shell/edit/catalog.cattle.io.clusterrepo.vue
index 12edc2fff36..6031fea60de 100644
--- a/shell/edit/catalog.cattle.io.clusterrepo.vue
+++ b/shell/edit/catalog.cattle.io.clusterrepo.vue
@@ -7,7 +7,7 @@ import { RadioGroup } from '@components/Form/Radio';
 import NameNsDescription from '@shell/components/form/NameNsDescription';
 import Labels from '@shell/components/form/Labels';
 import SelectOrCreateAuthSecret from '@shell/components/form/SelectOrCreateAuthSecret';
-import { NAMESPACE } from '@shell/config/types';
+import { MANAGEMENT, NAMESPACE } from '@shell/config/types';
 
 export default {
   name: 'CruCatalogRepo',
@@ -29,7 +29,7 @@ export default {
 
   computed: {
     inStore() {
-      return this.$store.getters['currentProduct'].inStore;
+      return this.$store.getters['currentProduct']?.inStore || MANAGEMENT;
     },
     secretNamespace() {
       const tryNames = ['cattle-system', 'default'];