Skip to content

Commit

Permalink
remove assert
Browse files Browse the repository at this point in the history
  • Loading branch information
AdrienWehrle committed Dec 12, 2024
1 parent 7ec7f3d commit 2dd7d8a
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,15 +29,13 @@ def pytest_addoption(parser):
def SH_CLIENT_ID() -> None:
"""Create local client id from environment variable"""
# check if variable in environment variables
assert "SH_CLIENT_ID" in os.environ
SH_CLIENT_ID = os.environ["SH_CLIENT_ID"]
return SH_CLIENT_ID

@pytest.fixture(scope="session")
def SH_CLIENT_SECRET() -> None:
"""Create local client secret from environment variable"""
# check if variable in environment variables
assert "SH_CLIENT_SECRET" in os.environ
SH_CLIENT_SECRET = os.environ["SH_CLIENT_SECRET"]

Check failure

Code scanning / CodeQL

Clear-text storage of sensitive information High test

This expression stores
sensitive data (secret)
as clear text.
return SH_CLIENT_SECRET

Expand Down

0 comments on commit 2dd7d8a

Please sign in to comment.