Copilot commented on code in PR #7868:
URL: https://github.com/apache/hadoop/pull/7868#discussion_r2272827582
##########
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemAppend.java:
##########
@@ -608,12 +608,12 @@ public void
testCreateExplicitDirectoryOverDfsAppendOverBlob()
**/
@Test
public void testRecreateAppendAndFlush() throws IOException {
+ assumeThat(isAppendBlobEnabled()).as("Not valid for APPEND
BLOB").isFalse();
+ assumeThat(getIngressServiceType()).isEqualTo(AbfsServiceType.BLOB);
Review Comment:
This assumption is placed after the assertThrows() call begins, but should
be before it. The assumption should be checked before setting up the exception
assertion to ensure the test conditions are met first.
```suggestion
assumeThat(getIngressServiceType()).isEqualTo(AbfsServiceType.BLOB);
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
