Tom-Newton commented on code in PR #38269:
URL: https://github.com/apache/arrow/pull/38269#discussion_r1364204599
##########
cpp/src/arrow/filesystem/azurefs_test.cc:
##########
@@ -113,33 +123,6 @@ AzuriteEnv* GetAzuriteEnv() {
// Placeholder tests
// TODO: GH-18014 Remove once a proper test is added
-TEST(AzureFileSystem, UploadThenDownload) {
Review Comment:
I kept it because I consider `InitializeCredentials` to also be a
placeholder test. That was created just to make sure the Azure SDK build was
working and I expect it to be deleted once we add support for initialising the
`AzureFileSystem` with all the different types of auth.
--
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]
