fix: address billing menu review feedback

Amp-Thread-ID: https://ampcode.com/threads/T-01a001d5-47e3-720d-8e82-82c45995251e
Co-authored-by: Amp <amp@ampcode.com>

Author: dante01yoon

src/platform/workspace/components/CurrentUserPopoverWorkspace.test.ts

@@ -252,7 +252,7 @@ describe('CurrentUserPopoverWorkspace', () => {
     ).not.toBeInTheDocument()
   })
 
-  it.each(['payment_failed', 'paused'])(
+  it.for(['payment_failed', 'paused'])(
     'keeps Manage plan available instead of Subscribe when billing is %s',
     (billingStatus) => {
       state.billingStatus = billingStatus
@@ -354,10 +354,13 @@ describe('CurrentUserPopoverWorkspace', () => {
       state.canManageSubscription = true
       const { emitted } = renderComponent(workspaceType)
 
-      const menuItem = screen.getByTestId('manage-plan-menu-item')
+      const menuItem = screen.getByRole('button', {
+        name: enMessages.subscription.managePlan
+      })
       expect(menuItem).toHaveTextContent(enMessages.subscription.managePlan)
 
-      await user.click(menuItem)
+      menuItem.focus()
+      await user.keyboard('{Enter}')
 
       expect(state.showSettingsDialog).toHaveBeenCalledWith('workspace')
       expect(emitted('close')).toHaveLength(1)

src/platform/workspace/components/CurrentUserPopoverWorkspace.vue

@@ -143,8 +143,9 @@
       }}</span>
     </div>
 
-    <div
+    <button
       v-if="!accountActionsOnly && showManagePlan"
+      type="button"
       class="flex cursor-pointer items-center gap-2 px-4 py-2 hover:bg-secondary-background-hover"
       data-testid="manage-plan-menu-item"
       @click="handleOpenPlanAndCreditsSettings"
@@ -153,7 +154,7 @@
       <span class="flex-1 text-sm text-base-foreground">{{
         $t('subscription.managePlan')
       }}</span>
-    </div>
+    </button>
 
     <!-- Partner Nodes Pricing (always shown) -->
     <div

src/platform/workspace/composables/useWorkspaceMenuItems.test.ts

@@ -116,7 +116,7 @@ describe('useWorkspaceMenuItems', () => {
     )
   })
 
-  it.each(['payment_failed', 'paused'])(
+  it.for(['payment_failed', 'paused'])(
     'allows cancellation while a %s plan needs payment recovery',
     (billingStatus) => {
       state.billingStatus = billingStatus