Fix test (#142)

* fix test

* comment out block test

* remove commented block

* async: false test comment

* comment out block test

* test trace

* validate sdk integration test turned off

* remove trace from github CI

* block test is back

* - block test + test trace

* print Registry entry

* add registry items to warning message

* message fix

* test finished log

* allow IO.puts

* print registry

* clear registry before all test

* add extra print_registry_contents

* cleanup

* move block test into integration test

* fix module in integration_test

* revert clear_registry + inline using_block_test

* run CI tests with max-cases 1

* run CI integration tests with max-cases 1

* CI: merge coverall reports

* remove max-cases 1

* Revert "remove max-cases 1"

This reverts commit 877ed28.

.github/workflows/elixir-ci.yml

@@ -58,7 +58,9 @@ jobs:
       run: mix dialyzer
 
     - name: Execute tests
-      run: mix coveralls.json --warnings-as-errors
+      run: |
+        mix test --only integration --warnings-as-errors --max-cases 1 --cover --export-coverage integration-coverage
+        mix coveralls --exclude integration --warnings-as-errors --import-cover cover
 
     - name: Upload coverage report
       run: bash <(curl -s https://codecov.io/bash)

test/config_cat_test.exs

@@ -1,5 +1,8 @@
 defmodule ConfigCatTest do
-  use ConfigCat.ClientCase, async: true
+  # Must be async: false to avoid a collision with other tests.
+  # Now that we only allow a single ConfigCat instance to use the same SDK key,
+  # one of the async tests would fail due to the existing running instance.
+  use ConfigCat.ClientCase, async: false
 
   import ExUnit.CaptureLog
   import Jason.Sigil

test/integration_test.exs

@@ -9,6 +9,8 @@ defmodule ConfigCat.IntegrationTest do
   alias ConfigCat.InMemoryCache
   alias ConfigCat.LocalMapDataSource
 
+  @moduletag :integration
+
   @sdk_key "configcat-sdk-1/PKDVCLf-Hq-h-kCzMp-L7Q/1cGEJXUwYUGZCBOL-E2sOw"
 
   describe "SDK key validation" do