Skip to content

Stub Arkose::Settings#enabled_for_signup? in tests

What does this MR do and why?

The test currently relies on the env variables being actually set (in the project' settings). This is wrong and we should stub the variables instead.

This was discovered in #389523 (comment 1255230233) so be sure to read the explanation (and the thread).

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #389523 (closed).

Edited by Rémy Coutable

Merge request reports