Compare commits
	
		
			No commits in common. "5a7f6515477f59885b5418d36ae1fa562014c288" and "09aeb31cb351e1fdffc44094674b450e3f3a2d9a" have entirely different histories.
		
	
	
		
			5a7f651547
			...
			09aeb31cb3
		
	
		
@ -1,6 +1,4 @@
 | 
				
			|||||||
defmodule BasicSignupWhitelist do
 | 
					defmodule BasicSignupWhitelist do
 | 
				
			||||||
  import FunCore.BasicSignupWhitelist
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
  @moduledoc """
 | 
					  @moduledoc """
 | 
				
			||||||
  Checks if an email address is allowed based on the SIGNUP_ALLOWED_EMAILS env variable.
 | 
					  Checks if an email address is allowed based on the SIGNUP_ALLOWED_EMAILS env variable.
 | 
				
			||||||
  """
 | 
					  """
 | 
				
			||||||
@ -22,8 +20,7 @@ defmodule BasicSignupWhitelist do
 | 
				
			|||||||
      false
 | 
					      false
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  """
 | 
					  """
 | 
				
			||||||
  def mail_whitelisted(email) do
 | 
					  def mail_whitelisted(_email) do
 | 
				
			||||||
    env_value = System.get_env("SIGNUP_ALLOWED_EMAILS")
 | 
					    false
 | 
				
			||||||
    mail_whitelisted_fun(env_value, email)
 | 
					 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
end
 | 
					end
 | 
				
			||||||
 | 
				
			|||||||
@ -2,32 +2,7 @@ defmodule BasicSignupWhitelistTest do
 | 
				
			|||||||
  use ExUnit.Case
 | 
					  use ExUnit.Case
 | 
				
			||||||
  doctest BasicSignupWhitelist
 | 
					  doctest BasicSignupWhitelist
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  defp allow_signups_for(whitelist) do
 | 
					 | 
				
			||||||
    System.put_env("SIGNUP_ALLOWED_EMAILS", whitelist)
 | 
					 | 
				
			||||||
  end
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
  setup do
 | 
					 | 
				
			||||||
    # Save original env var
 | 
					 | 
				
			||||||
    original_env = System.get_env("SIGNUP_ALLOWED_EMAILS")
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
    on_exit(fn ->
 | 
					 | 
				
			||||||
      # Restore original env var after each test
 | 
					 | 
				
			||||||
      case original_env do
 | 
					 | 
				
			||||||
        nil -> System.delete_env("SIGNUP_ALLOWED_EMAILS")
 | 
					 | 
				
			||||||
        value -> System.put_env("SIGNUP_ALLOWED_EMAILS", value)
 | 
					 | 
				
			||||||
      end
 | 
					 | 
				
			||||||
    end)
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
    :ok
 | 
					 | 
				
			||||||
  end
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
  test "When not set, not whitelisted" do
 | 
					  test "When not set, not whitelisted" do
 | 
				
			||||||
    System.delete_env("SIGNUP_ALLOWED_EMAILS")
 | 
					    assert BasicSignupWhitelist.mail_whitelisted("joe@example.com") == false
 | 
				
			||||||
    refute BasicSignupWhitelist.mail_whitelisted("joe@example.com")
 | 
					 | 
				
			||||||
  end
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
  test "When set to star, whitelisted" do
 | 
					 | 
				
			||||||
    allow_signups_for("*")
 | 
					 | 
				
			||||||
    assert BasicSignupWhitelist.mail_whitelisted("joe@example.com")
 | 
					 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
end
 | 
					end
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user