Only permitted certain emails to continue through authorization.
Context for reviewers
This was tricky to test. Because SiteConfig are loaded on application launch, rspec doesn't reload everything when stubbing the environment variable for the emails. The only way to get this to test properly was to define my own site_config or sites in the omniauth controller, which is reloaded every for every test (and therefore picks up my stub).
Ticket
Resolves 1036
Changes
Only permitted certain emails to continue through authorization.
Context for reviewers
This was tricky to test. Because SiteConfig are loaded on application launch, rspec doesn't reload everything when stubbing the environment variable for the emails. The only way to get this to test properly was to define my own
site_configorsitesin the omniauth controller, which is reloaded every for every test (and therefore picks up my stub).Testing
Automated and manual testing