Skip to content

Conversation

@phillxnet
Copy link
Member

Imports changed to reflect now build-in unittest.mock.

Fixes #2575

Includes copywrite changes:

  • Added missing copyright.
  • Majority in affected directories updated re date https.

Testing

Post the proposed changes we no longer have any test failing due to the issue/pr title:

...
Ran 252 tests in 26.009s

FAILED (failures=5, errors=3)

Imports changed to reflect now build-in unittest.mock.

# Includes copywrite changes:
- Added missing copyright.
- Majority in affected directories updated re date https.
@phillxnet
Copy link
Member Author

@FroggyFlox I have partially addressed your prior issue re style of import on the last pr.
@Hooverdan96 I have updated almost all tests re date and http to https.

Given this pr re-enabled around 200 tests and we were down to 58 actually running I'm going to merge without review as we then have almost all of our test suite back with only likely stylistic issues on the import. Plus there are not issues of the issued error remaining.

If we end up using mock/patch elsewhere (via old now removed import) we can approach that issue in it's own pull request.

@phillxnet phillxnet merged commit c2bfcb5 into rockstor:testing Jun 6, 2023
@phillxnet phillxnet deleted the 2575_Py3.6_ModuleNotFoundError_No_module_named_'mock' branch June 6, 2023 20:34
@FroggyFlox
Copy link
Member

Thanks a lot for taking care of such a tedious work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants