Fix app configuration in create_app for testing. #5808

Aadesh-Baral · 2023-05-19T08:58:24Z

When running tests for the function that involves creating a thread and an associated app context, we encountered an issue with the app configuration. The create_app function, responsible for creating the app context, was using the default EnvironmentConfig class even during testing.

To address this issue, we made a modification in the create_app function. We added a condition that checks if the TM_ENVIRONMENT environment variable is set to "test". If so, the configuration is set to TestEnvironmentConfig instead of the default EnvironmentConfig.

By introducing this conditional logic, it ensures that the create_app function correctly sets the TestEnvironmentConfig for the app created across the project when the test environment is detected.

closes #5691

eternaltyro · 2023-05-30T11:05:42Z

@Aadesh-Baral can you move the integration test out of this PR and into its own PR?

-------------------------------------------------------------- Modified the create_app function to include a condition that sets the configuration to TestEnvironmentConfig when the TM_ENVIRONMENT environment variable is set to "test". This change ensures that the app uses the appropriate configuration, even during testing.

…esting ------------------------------------------------------------------- Since this function runs on a seperate thread it causes issues while performing database cleanup as soon as the test is complete as the function will still be executing on different thread.