HADOOP-16941. ITestS3GuardOutOfBandOperations.testListingDelete failing on versioned bucket (#1919)
Contributed by Steve Loughran. Removed the failing probe and replacing with two probes which will fail on both versioned and unversioned buckets.
This commit is contained in:
parent
b397a3a875
commit
68a9562848
@ -76,6 +76,7 @@
|
||||
import static org.apache.hadoop.test.LambdaTestUtils.eventually;
|
||||
import static org.apache.hadoop.test.LambdaTestUtils.intercept;
|
||||
|
||||
import static org.apache.hadoop.test.LambdaTestUtils.interceptFuture;
|
||||
import static org.junit.Assume.assumeTrue;
|
||||
import static org.mockito.Mockito.mock;
|
||||
import static org.mockito.Mockito.when;
|
||||
@ -964,6 +965,14 @@ public void testListingDelete() throws Exception {
|
||||
// Delete the file without S3Guard (raw)
|
||||
deleteFile(rawFS, testFilePath);
|
||||
|
||||
// now, versioned FS or not, it will not be readable from the
|
||||
// raw FS, and this will fail in both open APIs during the open
|
||||
// phase, rather than when a read is attempted.
|
||||
interceptFuture(FileNotFoundException.class, "",
|
||||
rawFS.openFile(testFilePath).build());
|
||||
intercept(FileNotFoundException.class, () ->
|
||||
rawFS.open(testFilePath).close());
|
||||
|
||||
// File status will be still readable from s3guard
|
||||
S3AFileStatus status = (S3AFileStatus)
|
||||
guardedFs.getFileStatus(testFilePath);
|
||||
@ -985,8 +994,6 @@ public void testListingDelete() throws Exception {
|
||||
Assertions.assertThat(toChar(bytes))
|
||||
.describedAs("open(%s)", testFilePath)
|
||||
.isEqualTo(text);
|
||||
expectExceptionWhenReadingOpenFileAPI(rawFS, testFilePath, text,
|
||||
null);
|
||||
} else {
|
||||
// unversioned sequence
|
||||
expectExceptionWhenReading(testFilePath, text);
|
||||
|
Loading…
Reference in New Issue
Block a user