Skip to content

Commit

Permalink
Removed commented lines and added Tests header
Browse files Browse the repository at this point in the history
  • Loading branch information
Aravind-psiog committed Jan 5, 2024
1 parent b4e90f7 commit 1ad8460
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 9 deletions.
5 changes: 4 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Added

- Added feature to use custom python files as modules to be used in the electron function
- Added test coverage for executor resource deployment.

### Changed

Expand All @@ -37,6 +36,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Allow `cloudpickle` >= 3.0.0
- Remove `boto3` dependency from `tests/requirements.txt`

### Tests

- Added test coverage for executor resource deployment.

## [0.232.0-rc.0] - 2023-12-01

### Authors
Expand Down
8 changes: 0 additions & 8 deletions tests/covalent_tests/cloud_resource_manager/core_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -385,9 +385,6 @@ def test_docker(mocker, crm, caplog):
["mock-terraform init", "mock-terraform destroy"],
)
def test_log_error_msg(mocker, crm, cmd):
# mocker.patch(
# "covalent.cloud_resource_manager.core.CloudResourceManager._log_error_msg",
# )
mocker.patch(
"covalent.cloud_resource_manager.core.open",
new=mock.mock_open(read_data="test data"),
Expand Down Expand Up @@ -650,11 +647,6 @@ def test_down(mocker, crm):
return_value=0,
)

# mock_path_unlink = mocker.patch(
# "covalent.cloud_resource_manager.core.get_plugin_settings",
# # return_value={"test": "default"},
# )

crm.down(print_callback=None)

mock_get_tf_path.assert_called_once()
Expand Down

0 comments on commit 1ad8460

Please sign in to comment.