Skip to content

Commit

Permalink
fix(unit_tests/test_config): clear env variables
Browse files Browse the repository at this point in the history
those test are building heavily on `SCT_*` env variables
and seems like other unit tests might be leaving behind
some of those variables in a way it's breaks
`ConfigurationTests.test_02_verify_config` in the following way:

```
>           raise ValueError("not all multi region values are equal: \n\t{}".format(region_count))
E           ValueError: not all multi region values are equal:
E           	{'region_name': 1, 'n_db_nodes': 2, 'ami_id_db_scylla': 1, 'ami_id_loader': 1}
```
  • Loading branch information
fruch committed Jan 23, 2025
1 parent 318fe95 commit e86e8d7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion unit_tests/test_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ def setUpClass(cls):
logging.getLogger('botocore').setLevel(logging.CRITICAL)
logging.getLogger('boto3').setLevel(logging.CRITICAL)
logging.getLogger('anyconfig').setLevel(logging.ERROR)

cls.clear_sct_env_variables()
cls.setup_default_env()

os.environ['SCT_CLUSTER_BACKEND'] = 'aws'
Expand Down

0 comments on commit e86e8d7

Please sign in to comment.