Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Swap order

Co-authored-by: Sutou Kouhei <[email protected]>
  • Loading branch information
Tom-Newton and kou authored Oct 4, 2024
1 parent d368050 commit 39e671d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions cpp/src/arrow/filesystem/azurefs_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -406,10 +406,10 @@ class TestAzuriteGeneric : public TestGeneric {
protected:
// Azurite doesn't block writing files over directories.
bool allow_write_file_over_dir() const override { return true; }
// Azurite doesn't support moving files.
bool allow_move_file() const override { return false; }
// Azurite doesn't support moving directories.
bool allow_move_dir() const override { return false; }
// Azurite doesn't support moving files.
bool allow_move_file() const override { return false; }
// Azurite doesn't support directory mtime.
bool have_directory_mtimes() const override { return false; }
// DeleteDir() doesn't work with Azurite on macOS
Expand All @@ -432,10 +432,10 @@ class TestAzureFlatNSGeneric : public TestGeneric {
protected:
// Flat namespace account doesn't block writing files over directories.
bool allow_write_file_over_dir() const override { return true; }
// Flat namespace account doesn't support moving files.
bool allow_move_file() const override { return false; }
// Flat namespace account doesn't support moving directories.
bool allow_move_dir() const override { return false; }
// Flat namespace account doesn't support moving files.
bool allow_move_file() const override { return false; }
// Flat namespace account doesn't support directory mtime.
bool have_directory_mtimes() const override { return false; }
};
Expand Down

0 comments on commit 39e671d

Please sign in to comment.