Fix test specificity

maybe-finance · Nov 4, 2024 · 8d11ee9 · 8d11ee9
1 parent f46dc0b
commit 8d11ee9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/app/views/account/entries/index.html.erb b/app/views/account/entries/index.html.erb
@@ -2,7 +2,7 @@
   <div class="bg-white p-5 border border-alpha-black-25 rounded-xl shadow-xs">
     <div class="flex items-center justify-between mb-4">
       <%= tag.h2 t(".title"), class: "font-medium text-lg" %>
-      <div data-controller="menu">
+      <div data-controller="menu" data-testid="activity-menu">
         <button class="btn btn--secondary flex items-center gap-2" data-menu-target="button">
           <%= lucide_icon("plus", class: "w-4 h-4") %>
           <%= tag.span t(".new") %>

diff --git a/test/system/trades_test.rb b/test/system/trades_test.rb
@@ -66,8 +66,10 @@ class TradesTest < ApplicationSystemTestCase
   private
 
     def open_new_trade_modal
-      click_on "New"
-      click_on "New transaction"
+      within "[data-testid='activity-menu']" do
+        click_on "New"
+        click_on "New transaction"
+      end
     end
 
     def within_trades(&block)