Flaky API Test TestWebKitAPI.WebKit.UploadDirectory
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Nov 2019 17:50:36 +0000 (17:50 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Nov 2019 17:50:36 +0000 (17:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=203652

Patch by Alex Christensen <achristensen@webkit.org> on 2019-11-04
Reviewed by Alexey Proskuryakov.

It is not important that the directory to upload does not exist before the test starts.
Make the test more robust to state left behind by previous iterations of this test by removing the check.
The important check in this test is the uploaded body size.

* TestWebKitAPI/Tests/WebKitCocoa/UploadDirectory.mm:
(TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@251996 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/TestWebKitAPI/Tests/WebKitCocoa/UploadDirectory.mm

index 5d4a06e..224d401 100644 (file)
@@ -1,3 +1,17 @@
+2019-11-04  Alex Christensen  <achristensen@webkit.org>
+
+        Flaky API Test TestWebKitAPI.WebKit.UploadDirectory
+        https://bugs.webkit.org/show_bug.cgi?id=203652
+
+        Reviewed by Alexey Proskuryakov.
+
+        It is not important that the directory to upload does not exist before the test starts.
+        Make the test more robust to state left behind by previous iterations of this test by removing the check.
+        The important check in this test is the uploaded body size.
+
+        * TestWebKitAPI/Tests/WebKitCocoa/UploadDirectory.mm:
+        (TEST):
+
 2019-11-04  Kate Cheney  <katherine_cheney@apple.com>
 
         updated email in contributors.json to match bugzilla.
index 849278b..02a2529 100644 (file)
@@ -72,7 +72,7 @@ TEST(WebKit, UploadDirectory)
     NSFileManager *fileManager = [NSFileManager defaultManager];
     NSError *error = nil;
     NSURL *directory = [NSURL fileURLWithPath:[NSTemporaryDirectory() stringByAppendingPathComponent:@"UploadDirectory"] isDirectory:YES];
-    EXPECT_FALSE([fileManager fileExistsAtPath:directory.path]);
+    [fileManager removeItemAtPath:directory.path error:nil];
     EXPECT_TRUE([fileManager createDirectoryAtURL:directory withIntermediateDirectories:YES attributes:nil error:&error]);
     EXPECT_FALSE(error);
     NSData *testData = [@"testdata" dataUsingEncoding:NSUTF8StringEncoding];