fix: revert "fix(toolbar): allow running Django tests runners..."
This reverts commit 22b83193.
While well intentioned, the original fix had some issues:
- The fix was to allow running
poe manage testbut the documented way to run tests is via one ofpoe test,poe tox:localorpoe pytest:localwhich are all tested in CI. All of those three work because they use pytest as the test runner andIS_RUNNING_TESTS's default value detects this case. - By hard-coding
IS_RUNNING_TESTStoTrue, we are in fact lying for the other cases wheresettings.developeris used. Most notably those settings are used when running the application viapoe upwhich is not when tests are being run.
The fact that the fix is not quite right has been highlighted in the original issue.
#129 has been opened to address the issue with running poe manage test.
Edited by Dr Rich Wareham