Merge pull request #2224 from nulib/1885-flappy-action-states-test

Don't expect file set to be role A in action states test
nulib · May 18, 2021 · faeb456 · faeb456
2 parents 010cbd9 + ca9b592
commit faeb456
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/test/meadow/error_test.exs b/test/meadow/error_test.exs
@@ -34,7 +34,7 @@ defmodule Meadow.ErrorTest do
       end
 
       bad_function.(0)
-      assert_receive {:api_request, report}, 1000
+      assert_receive {:api_request, report}, 2500
 
       with request_context <- report |> get_in(["request", "context"]) do
         assert request_context |> Map.get("extra_data") == "foo"

diff --git a/test/meadow_web/schema/query/action_states_test.exs b/test/meadow_web/schema/query/action_states_test.exs
@@ -17,7 +17,14 @@ defmodule MeadowWeb.Schema.Query.ActionStatesTest do
     {:ok, result} = query_gql(variables: %{"objectId" => file_set.id}, context: gql_context())
 
     with trail <- get_in(result, [:data, "actionStates"]) do
-      assert(length(trail) == 7)
+      case file_set.role.id do
+        "A" ->
+          assert(length(trail) == 7)
+
+        "P" ->
+          assert(length(trail) == 6)
+      end
+
       assert_all_have_value(trail, "outcome", "WAITING")
     end
   end