felipecrv commented on code in PR #39009:
URL: https://github.com/apache/arrow/pull/39009#discussion_r1414706522
##########
cpp/src/arrow/filesystem/azurefs_test.cc:
##########
@@ -213,7 +213,7 @@ class AzureFileSystemTest : public ::testing::Test {
suite_skipped_ = true;
GTEST_SKIP() << options.status().message();
}
- container_name_ = RandomChars(32);
+ container_name_ = "z" + RandomChars(31);
Review Comment:
The `Prexisting*` paths are added automatically by `SetUp`. I want it to
start with `z` to make all paths sort deterministically when other paths are
added which is guaranteed by my paths starting with letters before `z`.
The right solution here is to have every test populate the filesystem
explicitly and make `SetUp` simpler, but I don't feel like doing it now when so
many people are adding and changing the tests in the same file.
--
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]
