Gave up trying to reset env var - set to "*" after test

And after doctest. it stayed at "*" after the last test anyway, letting
the doctest fail.

doctests are not recommended for side effectful tests anyway https://hexdocs.pm/ex_unit/main/ExUnit.DocTest.html

lib/basic_signup_whitelist.ex

@@ -21,6 +21,9 @@ defmodule BasicSignupWhitelist do
       iex> System.delete_env("SIGNUP_ALLOWED_EMAILS")
       iex> BasicSignupWhitelist.mail_whitelisted("joe@example.com")
       false
+      iex> System.put_env("SIGNUP_ALLOWED_EMAILS","*")
+      iex> BasicSignupWhitelist.mail_whitelisted("joe@example.com")
+      true
 
   """
   def mail_whitelisted(email) do

test/basic_signup_whitelist_test.exs

@@ -1,21 +1,16 @@
 defmodule BasicSignupWhitelistTest do
-  use ExUnit.Case
+  use ExUnit.Case, async: false
   doctest BasicSignupWhitelist
 
   defp allow_signups_for(whitelist) do
     System.put_env("SIGNUP_ALLOWED_EMAILS", whitelist)
   end
 
+  describe "setup in describe block" do
     setup do
-    # Save original env var
-    original_env = System.get_env("SIGNUP_ALLOWED_EMAILS")
-
       on_exit(fn ->
-      # Restore original env var after each test
+        # resetting values did not work, so 'just' set it to the effect without this module configured.
-      case original_env do
+        System.put_env("SIGNUP_ALLOWED_EMAILS", "*")
-        nil -> System.delete_env("SIGNUP_ALLOWED_EMAILS")
-        value -> System.put_env("SIGNUP_ALLOWED_EMAILS", value)
-      end
       end)
 
       :ok
@@ -30,4 +25,5 @@ defmodule BasicSignupWhitelistTest do
       allow_signups_for("*")
       assert BasicSignupWhitelist.mail_whitelisted("joe@example.com")
     end
+  end
 end