WIP: mock maintnotif tests

[ndyakov]

No description provided.

[jit-ci]

❌ Jit has detected 3 important findings in this PR that you should review.
The findings are detailed below as separate comments.
It’s highly recommended that you fix these security issues before merge.

Repository Risks:

• Database Integration: Connects to a database, often involving sensitive data that must be securely managed.
• Internally Accessible: Accessible only within the internal network, reducing exposure to external threats but still requiring proper controls.

Repository Context:

graph LR
    GitHub$Repository_U23_redis/go_U2D_redis["GitHub Repository<br/>redis/go-redis"]:::GitHub$Repository
    Team_U23_client_U2D_developers["Team<br/>client-developers"]:::Team
    Team_U23_client_U2D_docs["Team<br/>client-docs"]:::Team
    DBIntegration_U23_redis["DBIntegration<br/>redis"]:::DBIntegration
    Team_U23_client_U2D_developers -- "Owns" --> GitHub$Repository_U23_redis/go_U2D_redis
    Team_U23_client_U2D_docs -- "Owns" --> GitHub$Repository_U23_redis/go_U2D_redis
    GitHub$Repository_U23_redis/go_U2D_redis -- "Is accessible to" --> DBIntegration_U23_redis

Loading

[ofekshenawa]

Maybe use t.Cleanup instead of defer?
This ensures the restoration logic runs even if the test panics.
Something like this:

t.Cleanup(func() {
    if origConfigPath != "" {
			os.Setenv("REDIS_ENDPOINTS_CONFIG_PATH", origConfigPath)
		}
		if origFIURL != "" {
			os.Setenv("FAULT_INJECTION_API_URL", origFIURL)
		}
    })