Affects Version/s: None
Fix Version/s: 3.2.0
Noticed this when testing the patch for
Looking through the code, I'm noticing a couple of issues.
1. testDiffCommand() is in ITestS3GuardToolLocal, but it should really be running for all MetadataStore implementations. Seems like it should live in AbstractS3GuardToolTestBase.
2. AbstractS3GuardToolTestBase#createFile() seems wrong. When onMetadataStore is false, it does a ContractTestUtils.touch(file), but the fs is initialized with a MetadataStore present, so seem like the fs will still put the file in the MetadataStore?
There are other tests which explicitly go around the MetadataStore by using fs.setMetadataStore(nullMS), e.g. ITestS3AInconsistency. We should do something similar in AbstractS3GuardToolTestBase#createFile(), minding any issues with parallel test runs.