From a3f26af4c549b2831bdf4ea9e53685a8755b84ae Mon Sep 17 00:00:00 2001
From: Jordan Reimer <zofskeez@gmail.com>
Date: Thu, 23 Mar 2023 10:25:56 -0600
Subject: [PATCH] Secret Metadata Breadcrumb Bug (#19703)

* fixes issue navigating back a level using the breadcrumbs from kvv2 metadata view

* adds changelog entry

* deletes kv mount after breadcrumb test -- attempt to fix unrelated failing secrets tests
---
 changelog/19703.txt                           |  3 ++
 .../cluster/secrets/backend/metadata.hbs      |  2 +-
 .../secrets/backend/kv/breadcrumbs-test.js    | 29 +++++++++++++++++++
 3 files changed, 33 insertions(+), 1 deletion(-)
 create mode 100644 changelog/19703.txt
 create mode 100644 ui/tests/acceptance/secrets/backend/kv/breadcrumbs-test.js

diff --git a/changelog/19703.txt b/changelog/19703.txt
new file mode 100644
index 000000000..6bf8e5c18
--- /dev/null
+++ b/changelog/19703.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+ui: fixes issue navigating back a level using the breadcrumb from secret metadata view
+```
\ No newline at end of file
diff --git a/ui/app/templates/vault/cluster/secrets/backend/metadata.hbs b/ui/app/templates/vault/cluster/secrets/backend/metadata.hbs
index ba923f2e7..ca96d8289 100644
--- a/ui/app/templates/vault/cluster/secrets/backend/metadata.hbs
+++ b/ui/app/templates/vault/cluster/secrets/backend/metadata.hbs
@@ -2,7 +2,7 @@
   <p.top>
     <KeyValueHeader
       @baseKey={{hash id=this.id}}
-      @path="vault.cluster.secrets.backend.show"
+      @path="vault.cluster.secrets.backend.list"
       @mode="show"
       @showCurrent={{true}}
       @root={{this.backendCrumb}}
diff --git a/ui/tests/acceptance/secrets/backend/kv/breadcrumbs-test.js b/ui/tests/acceptance/secrets/backend/kv/breadcrumbs-test.js
new file mode 100644
index 000000000..02e3b4a7d
--- /dev/null
+++ b/ui/tests/acceptance/secrets/backend/kv/breadcrumbs-test.js
@@ -0,0 +1,29 @@
+import { create } from 'ember-cli-page-object';
+import { module, test } from 'qunit';
+import { setupApplicationTest } from 'ember-qunit';
+import { click, currentURL, fillIn, visit } from '@ember/test-helpers';
+import authPage from 'vault/tests/pages/auth';
+import consoleClass from 'vault/tests/pages/components/console/ui-panel';
+
+const consolePanel = create(consoleClass);
+
+module('Acceptance | kv | breadcrumbs', function (hooks) {
+  setupApplicationTest(hooks);
+
+  test('it should route back to parent path from metadata tab', async function (assert) {
+    await authPage.login();
+    await consolePanel.runCommands(['delete sys/mounts/kv', 'write sys/mounts/kv type=kv-v2']);
+    await visit('/vault/secrets/kv/list');
+    await click('[data-test-secret-create]');
+    await fillIn('[data-test-secret-path]', 'foo/bar');
+    await click('[data-test-secret-save]');
+    await click('[data-test-secret-metadata-tab]');
+    await click('[data-test-secret-breadcrumb="foo"]');
+    assert.strictEqual(
+      currentURL(),
+      '/vault/secrets/kv/list/foo/',
+      'Routes back to list view on breadcrumb click'
+    );
+    await consolePanel.runCommands(['delete sys/mounts/kv']);
+  });
+});