Skip to content

Commit

Permalink
add logs inside wait-for and before,after logout (#19362)
Browse files Browse the repository at this point in the history
## Summary

PR made to add logs to help determine why integration test hangs on logout.

## Testing notes

No need for testing since we only ever add logs in test code here.

related to: #19256
  • Loading branch information
siddarthkay authored Apr 2, 2024
1 parent dd7795a commit 68e4cf3
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion src/test_helpers/integration.cljs
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@

(defn logout
[]
(log/info (str "==== before dispatch logout ===="))
(rf/dispatch [:logout]))

(defn log-headline
Expand Down Expand Up @@ -123,6 +124,7 @@
:timeout-ms timeout-ms}
::timeout)))
timeout-ms)]
(log/info (str "==== inside wait-for after let block ===="))
(rf/add-post-event-callback
cb-id
(fn [[event-id & _]]
Expand Down Expand Up @@ -256,7 +258,9 @@
(setup-app)
(setup-account)
(logout)
(wait-for [::logout/logout-method])))))
(log/info (str "==== before wait-for logout ===="))
(wait-for [::logout/logout-method])
(log/info (str "==== after wait-for logout ===="))))))

;;;; Fixtures

Expand All @@ -282,7 +286,9 @@
:after (fn []
(test/async done
(promesa/do (logout)
(log/info (str "==== before wait-for logout ===="))
(wait-for [::logout/logout-method])
(log/info (str "==== after wait-for logout ===="))
(done))))})
([] (fixture-session [:new-account])))

Expand Down

0 comments on commit 68e4cf3

Please sign in to comment.