diff --git a/ui/app/templates/components/secret-delete-menu.hbs b/ui/app/templates/components/secret-delete-menu.hbs
index a5d7a7781623..fb6c6dbad19b 100644
--- a/ui/app/templates/components/secret-delete-menu.hbs
+++ b/ui/app/templates/components/secret-delete-menu.hbs
@@ -27,7 +27,7 @@
       <button
         type="button"
         class="toolbar-link"
-        {{on "click" (fn (mut this.showDeleteModal) false)}}
+        {{on "click" (fn (mut this.showDeleteModal) true)}}
         data-test-delete-open-modal
       >
         {{if (and (not @modelForData.deleted) (not @modelForData.destroyed)) "Delete" "Destroy"}}
diff --git a/ui/tests/acceptance/secrets/backend/kv/secret-test.js b/ui/tests/acceptance/secrets/backend/kv/secret-test.js
index be9c9b890a9b..0869ec757918 100644
--- a/ui/tests/acceptance/secrets/backend/kv/secret-test.js
+++ b/ui/tests/acceptance/secrets/backend/kv/secret-test.js
@@ -641,6 +641,7 @@ module('Acceptance | secrets/secret/create', function(hooks) {
     await settled();
     await click('[data-test-delete-open-modal]');
     await settled();
+    assert.dom('.modal.is-active').exists('Modal appears');
     assert.dom('[data-test-delete-modal="destroy-all-versions"]').exists(); // we have a if Ember.testing catch in the delete action because it breaks things in testing
     // we can however destroy the versions
     await click('#destroy-all-versions');