RyanSkraba commented on code in PR #20267:
URL: https://github.com/apache/flink/pull/20267#discussion_r1003338793
##########
flink-filesystems/flink-azure-fs-hadoop/src/test/java/org/apache/flink/fs/azurefs/AzureFileSystemBehaviorITCase.java:
##########
@@ -85,9 +93,9 @@ private static boolean isHttpsTrafficOnly() throws
IOException {
return true;
}
- Assume.assumeTrue(
- "Azure storage account not configured, skipping test...",
- !StringUtils.isNullOrWhitespaceOnly(ACCOUNT));
+ assumeThat(ACCOUNT)
+ .describedAs("Azure storage account not configured, skipping
test...")
+ .isNotBlank();
Review Comment:
I rewrote the logic here so that all of the `assumeThat` are more explicit.
I'm pretty sure it's equivalent, but I'd appreciate a bit of extra review on
this change!
--
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]
