Skip to content

Commit

Permalink
Update SasCredential test to cover extra query parmaters
Browse files Browse the repository at this point in the history
Tom-Newton committed Dec 12, 2024
1 parent a2b826c commit 9c6ec0e
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion cpp/src/arrow/filesystem/azurefs_test.cc
Original file line number Diff line number Diff line change
@@ -1658,7 +1658,11 @@ class TestAzureFileSystem : public ::testing::Test {
ASSERT_OK_AND_ASSIGN(auto env, GetAzureEnv());
ASSERT_OK_AND_ASSIGN(auto options, MakeOptions(env));
ASSERT_OK_AND_ASSIGN(auto sas_token, GetContainerSasToken(data.container_name));
ARROW_EXPECT_OK(options.ConfigureSasCredential(sas_token));
// AzureOptions::FromUri will not cut off extra query parameters that it consumes, so
// make sure these don't cause problems.
ARROW_EXPECT_OK(options.ConfigureSasCredential(
"?blob_storage_authority=dummy_value0&" + sas_token.substr(1) +
"&credential_kind=dummy-value1"));
EXPECT_OK_AND_ASSIGN(auto fs, AzureFileSystem::Make(options));

AssertFileInfo(fs.get(), data.ObjectPath(), FileType::File);

0 comments on commit 9c6ec0e

Please sign in to comment.