Convert remaining address page tests to page module calls

Decouple design concerns from test concerns by changing test selectors
to use `data-test` attribute selection instead of overloading css
classes for both purposes.

Co-authored-by: jimmay5469 <jimmay5469@gmail.com>
Co-authored-by: stamates <stamates@hotmail.com>

apps/explorer/test/support/factories/chain/transaction_factory.ex

@@ -23,14 +23,19 @@ defmodule Explorer.Chain.TransactionFactory do
         }
       end
 
+      def list_with_block(transactions, block \\ nil) do
+        Enum.map(transactions, fn transaction -> with_block(transaction, block) end)
+      end
+
       def with_block(transaction, block \\ nil) do
         block = block || insert(:block)
-        insert(:block_transaction, %{block_id: block.id, transaction_id: transaction.id})
+        insert(:block_transaction, block_id: block.id, transaction_id: transaction.id)
         transaction
       end
 
-      def list_with_block(transactions, block \\ nil) do
-        Enum.map(transactions, fn transaction -> with_block(transaction, block) end)
+      def with_receipt(transaction) do
+        insert(:receipt, transaction_id: transaction.id)
+        transaction
       end
     end
   end

apps/explorer_web/lib/explorer_web/templates/address/overview.html.eex

@@ -7,7 +7,7 @@
     <dl>
       <div class="address__item">
         <dt class="address__item-key"><%= gettext "Balance" %></dt>
-        <dd class="address__item-value address__balance" title="<%= @address.hash %>">
+        <dd class="address__item-value address__balance" title="<%= @address.hash %>" data-test="address_balance">
           <%= balance(@address) %> <%= gettext "Ether" %>
         </dd>
       </div>

apps/explorer_web/lib/explorer_web/templates/address_internal_transaction/index.html.eex

@@ -27,30 +27,33 @@
     <div class="dropdown-menu dropdown-menu-right filter" aria-labelledby="dropdownMenu2">
       <%= link(
         gettext("All"),
+        to: address_internal_transaction_path(@conn, :index, @conn.assigns.locale, @conn.params["address_id"]),
         class: "address__link address__link--active dropdown-item",
-        to: address_internal_transaction_path(@conn, :index, @conn.assigns.locale, @conn.params["address_id"])
+        "data-test": "filter_option"
       ) %>
       <%= link(
         gettext("To"),
-        class: "address__link address__link--active dropdown-item",
         to: address_internal_transaction_path(
           @conn,
           :index,
           @conn.assigns.locale,
           @conn.params["address_id"],
           filter: "to"
-        )
+        ),
+        class: "address__link address__link--active dropdown-item",
+        "data-test": "filter_option"
       ) %>
       <%= link(
         gettext("From"),
-        class: "address__link address__link--active dropdown-item",
         to: address_internal_transaction_path(
           @conn,
           :index,
           @conn.assigns.locale,
           @conn.params["address_id"],
           filter: "from"
-        )
+        ),
+        class: "address__link address__link--active dropdown-item",
+        "data-test": "filter_option"
       ) %>
     </div>
   </div>

apps/explorer_web/lib/explorer_web/templates/address_transaction/index.html.eex

@@ -27,30 +27,33 @@
       <div class="dropdown-menu dropdown-menu-right filter" aria-labelledby="dropdownMenu2">
         <%= link(
               gettext("All"),
+              to: address_transaction_path(@conn, :index, @conn.assigns.locale, @conn.params["address_id"]),
               class: "address__link address__link--active dropdown-item",
-              to: address_transaction_path(@conn, :index, @conn.assigns.locale, @conn.params["address_id"])
+              "data-test": "filter_option"
             ) %>
         <%= link(
               gettext("To"),
-              class: "address__link address__link--active dropdown-item",
               to: address_transaction_path(
                 @conn,
                 :index,
                 @conn.assigns.locale,
                 @conn.params["address_id"],
                 filter: "to"
-              )
+              ),
+              class: "address__link address__link--active dropdown-item",
+              "data-test": "filter_option"
             ) %>
         <%= link(
               gettext("From"),
-              class: "address__link address__link--active dropdown-item",
               to: address_transaction_path(
                 @conn,
                 :index,
                 @conn.assigns.locale,
                 @conn.params["address_id"],
                 filter: "from"
-              )
+              ),
+              class: "address__link address__link--active dropdown-item",
+              "data-test": "filter_option"
             ) %>
       </div>
     </div>
@@ -80,8 +83,9 @@
                 <div class="transactions__hash">
                   <%= link(
                         transaction.hash,
+                        to: transaction_path(@conn, :show, @conn.assigns.locale, transaction.hash),
                         class: "transactions__link transactions__link--truncated transactions__link--long-hash",
-                        to: transaction_path(@conn, :show, @conn.assigns.locale, transaction.hash)
+                        "data-test": "transaction_hash"
                       ) %>
                 </div>
               </td>

apps/explorer_web/test/explorer_web/features/address_page_test.exs

@@ -1,109 +1,90 @@
 defmodule ExplorerWeb.AddressPageTest do
   use ExplorerWeb.FeatureCase, async: true
 
-  import Wallaby.Query, only: [css: 2]
-
-  alias Explorer.Chain.{Credit, Debit}
   alias ExplorerWeb.AddressPage
 
   setup do
-    block =
-      insert(:block, %{
-        number: 555,
-        timestamp: Timex.now() |> Timex.shift(hours: -2),
-        gas_used: 123_987
-      })
-
-    for _ <- 0..3, do: insert(:transaction) |> with_block(block)
-    insert(:transaction, hash: "0xC001", gas: 5891) |> with_block
-
-    lincoln = insert(:address, hash: "0xlincoln")
-    taft = insert(:address, hash: "0xhowardtaft")
-
-    transaction =
-      insert(
-        :transaction,
-        hash: "0xSk8",
-        value: Explorer.Chain.Wei.from(Decimal.new(5656), :ether),
-        gas: Decimal.new(1_230_000_000_000_123_123),
-        gas_price: Decimal.new(7_890_000_000_898_912_300_045),
-        input: "0x00012",
-        nonce: 99045,
-        inserted_at: Timex.parse!("1970-01-01T00:00:18-00:00", "{ISO:Extended}"),
-        updated_at: Timex.parse!("1980-01-01T00:00:18-00:00", "{ISO:Extended}"),
-        from_address_id: taft.id,
-        to_address_id: lincoln.id
-      )
-
-    insert(:block_transaction, block: block, transaction: transaction)
-
-    receipt = insert(:receipt, transaction: transaction, status: 1)
-    insert(:log, address_id: lincoln.id, receipt: receipt)
-
-    # From Lincoln to Taft.
-    txn_from_lincoln =
-      insert(
-        :transaction,
-        hash: "0xrazerscooter",
-        from_address_id: lincoln.id,
-        to_address_id: taft.id
-      )
-
-    insert(:block_transaction, block: block, transaction: txn_from_lincoln)
-
-    insert(:receipt, transaction: txn_from_lincoln)
-
-    Credit.refresh()
-    Debit.refresh()
-
-    {:ok, %{address: lincoln, transaction: transaction}}
+    block = insert(:block)
+
+    lincoln = insert(:address)
+    taft = insert(:address)
+
+    from_taft =
+      :transaction
+      |> insert(from_address_id: taft.id, to_address_id: lincoln.id)
+      |> with_block(block)
+      |> with_receipt()
+
+    from_lincoln =
+      :transaction
+      |> insert(from_address_id: lincoln.id, to_address_id: taft.id)
+      |> with_block(block)
+      |> with_receipt()
+
+    {:ok,
+     %{
+       transactions: %{from_lincoln: from_lincoln, from_taft: from_taft},
+       addresses: %{lincoln: lincoln, taft: taft}
+     }}
   end
 
   test "viewing address overview information", %{session: session} do
-    insert(:address, hash: "0xthinmints", balance: 500)
+    address = insert(:address, balance: 500)
 
     session
-    |> visit("/en/addresses/0xthinmints")
-    |> assert_has(css(".address__balance", text: "0.000,000,000,000,000,500 POA"))
+    |> AddressPage.visit_page(address)
+    |> assert_text(AddressPage.balance(), "0.000,000,000,000,000,500 POA")
   end
 
   describe "viewing transactions" do
-    test "sees all addresses transactions by default", %{address: address, session: session} do
+    test "sees all addresses transactions by default", %{
+      addresses: addresses,
+      session: session,
+      transactions: transactions
+    } do
       session
-      |> AddressPage.visit_page(address.hash)
-      |> assert_has(AddressPage.transaction("0xSk8"))
-      |> assert_has(AddressPage.transaction("0xrazerscooter"))
+      |> AddressPage.visit_page(addresses.lincoln)
+      |> assert_has(AddressPage.transaction(transactions.from_taft))
+      |> assert_has(AddressPage.transaction(transactions.from_lincoln))
     end
 
-    test "can filter to only see transactions to an address", %{address: address, session: session} do
+    test "can filter to only see transactions from an address", %{
+      addresses: addresses,
+      session: session,
+      transactions: transactions
+    } do
       session
-      |> visit("/en/addresses/#{address.hash}")
-      |> click(css("[data-test='filter_dropdown']", text: "Filter: All"))
-      |> click(css(".address__link", text: "To"))
-      |> assert_has(css(".transactions__link--long-hash", text: "0xSk8"))
-      |> refute_has(css(".transactions__link--long-hash", text: "0xrazerscooter"))
+      |> AddressPage.visit_page(addresses.lincoln)
+      |> AddressPage.apply_filter("From")
+      |> assert_has(AddressPage.transaction(transactions.from_lincoln))
+      |> refute_has(AddressPage.transaction(transactions.from_taft))
     end
 
-    test "can filter to only see transactions from an address", %{address: address, session: session} do
+    test "can filter to only see transactions to an address", %{
+      addresses: addresses,
+      session: session,
+      transactions: transactions
+    } do
       session
-      |> visit("/en/addresses/#{address.hash}")
-      |> click(css("[data-test='filter_dropdown']", text: "Filter: All"))
-      |> click(css(".address__link", text: "From"))
-      |> assert_has(css(".transactions__link--long-hash", text: "0xrazerscooter"))
-      |> refute_has(css(".transactions__link--long-hash", text: "0xSk8"))
+      |> AddressPage.visit_page(addresses.lincoln)
+      |> AddressPage.apply_filter("To")
+      |> refute_has(AddressPage.transaction(transactions.from_lincoln))
+      |> assert_has(AddressPage.transaction(transactions.from_taft))
     end
   end
 
   describe "viewing internal transactions" do
-    setup %{address: address, transaction: transaction} do
-      insert(:internal_transaction, transaction_id: transaction.id, to_address_id: address.id)
-      insert(:internal_transaction, transaction_id: transaction.id, from_address_id: address.id)
+    setup %{addresses: addresses, transactions: transactions} do
+      address_id = addresses.lincoln.id
+      transaction_id = transactions.from_lincoln.id
+      insert(:internal_transaction, transaction_id: transaction_id, to_address_id: address_id, index: 0)
+      insert(:internal_transaction, transaction_id: transaction_id, from_address_id: address_id, index: 1)
       :ok
     end
 
-    test "can see internal transactions for an address", %{address: address, session: session} do
+    test "can see internal transactions for an address", %{addresses: addresses, session: session} do
       session
-      |> AddressPage.visit_page(address.hash)
+      |> AddressPage.visit_page(addresses.lincoln)
       |> AddressPage.click_internal_transactions()
       |> assert_has(AddressPage.internal_transactions(count: 2))
     end
@@ -118,7 +99,7 @@ defmodule ExplorerWeb.AddressPageTest do
 
     test "can filter to only see internal transactions to an address", %{addresses: addresses, session: session} do
       session
-      |> AddressPage.visit_page(address.hash)
+      |> AddressPage.visit_page(addresses.lincoln)
       |> AddressPage.click_internal_transactions()
       |> AddressPage.apply_filter("To")
       |> assert_has(AddressPage.internal_transactions(count: 1))

apps/explorer_web/test/explorer_web/features/pages/address_page.ex

@@ -3,6 +3,9 @@ defmodule ExplorerWeb.AddressPage do
 
   use Wallaby.DSL
   import Wallaby.Query, only: [css: 1, css: 2]
+  alias Explorer.Chain.{Address, Transaction}
+
+  def visit_page(session, %Address{hash: address_hash}), do: visit_page(session, address_hash)
 
   def visit_page(session, address_hash) do
     visit(session, "/en/addresses/#{address_hash}")
@@ -13,7 +16,9 @@ defmodule ExplorerWeb.AddressPage do
     click(session, css(@internal_transactions_link_selector))
   end
 
-  @transaction_selector ".transactions__link--long-hash"
+  @transaction_selector "[data-test='transaction_hash']"
+  def transaction(%Transaction{hash: transaction_hash}), do: transaction(transaction_hash)
+
   def transaction(transaction_hash) do
     css(@transaction_selector, text: transaction_hash)
   end
@@ -26,6 +31,10 @@ defmodule ExplorerWeb.AddressPage do
   def apply_filter(session, direction) do
     session
     |> click(css("[data-test='filter_dropdown']", text: "Filter: All"))
-    |> click(css(".address__link", text: direction))
+    |> click(css("[data-test='filter_option']", text: direction))
+  end
+
+  def balance do
+    css("[data-test='address_balance']")
   end
 end