Skip to content

[Storage] Live Tests - self.fail does not exist#45003

Merged
weirongw23-msft merged 1 commit intoAzure:mainfrom
weirongw23-msft:weirongw23/fix
Feb 5, 2026
Merged

[Storage] Live Tests - self.fail does not exist#45003
weirongw23-msft merged 1 commit intoAzure:mainfrom
weirongw23-msft:weirongw23/fix

Conversation

@weirongw23-msft
Copy link
Member

No description provided.

@weirongw23-msft weirongw23-msft marked this pull request as ready for review February 4, 2026 20:11
Copilot AI review requested due to automatic review settings February 4, 2026 20:11
@github-actions github-actions bot added the Storage Storage Service (Queues, Blobs, Files) label Feb 4, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes failing live tests by replacing self.fail(...) (not available on these pytest-style test classes) with pytest.fail(...).

Changes:

  • Update async file copy polling helper to use pytest.fail on timeout.
  • Update sync file copy polling helper to use pytest.fail on timeout.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
sdk/storage/azure-storage-file-share/tests/test_file_async.py Use pytest.fail(...) instead of self.fail(...) in async copy wait helper.
sdk/storage/azure-storage-file-share/tests/test_file.py Use pytest.fail(...) instead of self.fail(...) in sync copy wait helper.

@weirongw23-msft
Copy link
Member Author

/check-enforcer override

@weirongw23-msft weirongw23-msft merged commit e72c26f into Azure:main Feb 5, 2026
30 of 31 checks passed
@weirongw23-msft weirongw23-msft deleted the weirongw23/fix branch February 5, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants