Improve tests (#446)

* Rename TestHelpers to DateHelpers

* Refactor ServerHelpers

* Refactor telemetry testing to use messages instead of waiting

* Refactor log testing to use messages instead of capture_log

* Make tests run sync now that we've made it (mostly) safe

Still a few loose ends to chase down

* Avoid collisions between finch pools for similarly named tests

* Tag off a few intrinsically slow tests to not run locally

* Lower default timeout to 100ms

* Increase wait time for log/telemetry messages to 500ms

* Tweak a few explicitly required sleeps

* Fix 'out of process' tests to work around weird Mint behaviour

* Update SimpleH2Client to provide more info about frame types

* Improve h2 test that is inherently racy

* Add sleeps since it seems as if log handler addition is a bit racy

* Increase Websocket timeouts

(these will go away once we refactor websocket logging to support receive-based testing)

* Increase read timeout on TLS warnings test

.github/workflows/autobahn.yml

@@ -21,7 +21,7 @@ jobs:
       matrix:
         elixir: ${{ fromJSON(inputs.elixirs) }}
         otp: ${{ fromJSON(inputs.erlangs) }}
-        case: 
+        case:
           - "1.*,2.*,3.*,4.*,5.*,6.*,7.*,8.*,10.*"
           - "9.1.*"
           - "9.2.*"
@@ -69,4 +69,4 @@ jobs:
       run: |
         mix deps.get
     - name: Run Autobahn test
-      run: mix test --only external_conformance test/bandit/websocket/autobahn_test.exs
+      run: mix test --only slow test/bandit/websocket/autobahn_test.exs

.github/workflows/h2spec.yml

@@ -50,4 +50,4 @@ jobs:
       run: |
         mix deps.get
     - name: Run h2spec test
-      run: mix test --only external_conformance test/bandit/http2/h2spec_test.exs
+      run: mix test --only slow test/bandit/http2/h2spec_test.exs

test/bandit/clock_test.exs

@@ -9,7 +9,7 @@ defmodule ClockTest do
     warnings =
       capture_log(fn ->
         {"date", date} = Bandit.Clock.date_header()
-        assert TestHelpers.valid_date_header?(date)
+        assert DateHelpers.valid_date_header?(date)
       end)
 
     assert warnings =~ "Header timestamp couldn't be fetched from ETS cache"