Only spawn a browser when using the feature macro, not normal tests

.github/workflows/ci.yml

@@ -163,7 +163,7 @@ jobs:
     - name: Run Tests
       run: mix test || mix test --failed || mix test --failed
 
-    - uses: actions/upload-artifact@v2
+    - uses: actions/upload-artifact@v4
       if: always()
       with:
         name: Selenium Logs
@@ -210,7 +210,7 @@ jobs:
     - name: Run Tests
       run: mix test || mix test --failed || mix test --failed
 
-    - uses: actions/upload-artifact@v2
+    - uses: actions/upload-artifact@v4
       if: always()
       with:
         name: Selenium Logs
@@ -250,7 +250,7 @@ jobs:
     - name: Run Tests
       run: mix test || mix test --failed || mix test --failed
 
-    - uses: actions/upload-artifact@v2
+    - uses: actions/upload-artifact@v4
       if: always()
       with:
         name: Selenium Logs

integration_test/cases/feature/use_feature_test.exs

@@ -3,7 +3,7 @@
   use Wallaby.Feature
 
   @sessions 2
   feature "multi session", %{sessions: [session_1, session_2]} do
     session_1
     |> visit("/page_1.html")
     |> find(Query.css("body > h1"), fn el ->
@@ -17,7 +17,7 @@
     end)
   end
 
   feature "single session", %{session: only_session} do
     only_session
     |> visit("/page_1.html")
     |> find(Query.css("body > h1"), fn el ->
@@ -31,7 +31,11 @@
                            "I'm a capability"
                          )
   @sessions [[capabilities: @expected_capabilities]]
   feature "reads capabilities from session attribute", %{session: %{capabilities: capabilities}} do
     assert capabilities.test == @expected_capabilities.test
   end
+
+  test "does not set up a session for non-feature tests", context do
+    refute is_map_key(context, :session)
+  end
 end

lib/wallaby/feature.ex

@@ -23,22 +23,26 @@ defmodule Wallaby.Feature do
       import Wallaby.Feature
 
       setup context do
-        metadata = unquote(__MODULE__).Utils.maybe_checkout_repos(context[:async])
-
-        start_session_opts =
-          [metadata: metadata]
-          |> unquote(__MODULE__).Utils.put_create_session_fn(context[:create_session_fn])
-
-        get_in(context, [:registered, :sessions])
-        |> unquote(__MODULE__).Utils.sessions_iterable()
-        |> Enum.map(fn
-          opts when is_list(opts) ->
-            unquote(__MODULE__).Utils.start_session(opts, start_session_opts)
-
-          i when is_number(i) ->
-            unquote(__MODULE__).Utils.start_session([], start_session_opts)
-        end)
-        |> unquote(__MODULE__).Utils.build_setup_return()
+        if context[:test_type] == :feature do
+          metadata = unquote(__MODULE__).Utils.maybe_checkout_repos(context[:async])
+
+          start_session_opts =
+            [metadata: metadata]
+            |> unquote(__MODULE__).Utils.put_create_session_fn(context[:create_session_fn])
+
+          get_in(context, [:registered, :sessions])
+          |> unquote(__MODULE__).Utils.sessions_iterable()
+          |> Enum.map(fn
+            opts when is_list(opts) ->
+              unquote(__MODULE__).Utils.start_session(opts, start_session_opts)
+
+            i when is_number(i) ->
+              unquote(__MODULE__).Utils.start_session([], start_session_opts)
+          end)
+          |> unquote(__MODULE__).Utils.build_setup_return()
+        else
+          :ok
+        end
       end
     end
   end