PM-9001 - Item with MP Reprompt does not prompt for MP when downloadi…

…ng attachment (#764)
bitwarden · Jul 26, 2024 · 5d78b16 · 5d78b16
1 parent 4712b00
commit 5d78b16
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/BitwardenShared/UI/Vault/VaultItem/ViewItem/ViewItemAction.swift b/BitwardenShared/UI/Vault/VaultItem/ViewItem/ViewItemAction.swift
@@ -46,14 +46,14 @@ enum ViewItemAction: Equatable {
         switch self {
         case .cardItemAction,
              .customFieldVisibilityPressed,
+             .downloadAttachment,
              .editPressed,
              .morePressed,
              .passwordVisibilityPressed:
             true
         case let .copyPressed(_, field):
             field.requiresMasterPasswordReprompt
         case .dismissPressed,
-             .downloadAttachment,
              .passwordHistoryPressed,
              .toastShown:
             false

diff --git a/BitwardenShared/UI/Vault/VaultItem/ViewItem/ViewItemActionTests.swift b/BitwardenShared/UI/Vault/VaultItem/ViewItem/ViewItemActionTests.swift
@@ -38,7 +38,7 @@ class ViewItemActionTests: BitwardenTestCase {
 
         XCTAssertFalse(ViewItemAction.dismissPressed.requiresMasterPasswordReprompt)
 
-        XCTAssertFalse(ViewItemAction.downloadAttachment(.fixture()).requiresMasterPasswordReprompt)
+        XCTAssertTrue(ViewItemAction.downloadAttachment(.fixture()).requiresMasterPasswordReprompt)
 
         XCTAssertTrue(ViewItemAction.editPressed.requiresMasterPasswordReprompt)