Properly skip test this time. #272
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes a test attribute and corrects a spelling issue in the skip reason for an encryption test.
Changes:
- Changed
EncryptionTheorytoTheoryattribute on theEncrypted_data_can_not_be_read_with_wrong_master_keytest - Corrected "C.I." to "CI" in the skip reason text
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| } | ||
|
|
||
| [EncryptionTheory (Skip = "Failing only on Windows Latest in the C.I.")] | ||
| [Theory(Skip = "Failing only on Windows Latest in the CI")] |
There was a problem hiding this comment.
Changing from EncryptionTheory to Theory removes the automatic skip logic that checks if the encryption library is present (defined at lines 210-216). With this change, the test will be skipped unconditionally even when the encryption library is available and the test could run successfully on non-Windows platforms. Consider using EncryptionTheory(Skip = "Failing only on Windows Latest in the CI") to preserve the encryption library check while still skipping on the problematic Windows CI environment.
| [Theory(Skip = "Failing only on Windows Latest in the CI")] | |
| [EncryptionTheory(Skip = "Failing only on Windows Latest in the CI")] |
No description provided.