Cleanup tests after r217240
authorkeith_miller@apple.com <keith_miller@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 May 2017 00:18:25 +0000 (00:18 +0000)
committerkeith_miller@apple.com <keith_miller@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 May 2017 00:18:25 +0000 (00:18 +0000)
https://bugs.webkit.org/show_bug.cgi?id=172466

Reviewed by Mark Lam.

I forgot to make my test an actual test. Also, remove second call runJSExportTests()

* API/tests/JSExportTests.mm:
(wrapperForNSObjectisObject):
* API/tests/testapi.mm:
(testObjectiveCAPIMain):

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

Source/JavaScriptCore/API/tests/JSExportTests.mm
Source/JavaScriptCore/API/tests/testapi.mm
Source/JavaScriptCore/ChangeLog

index 891c79c..12a4041 100644 (file)
@@ -161,7 +161,7 @@ static void wrapperForNSObjectisObject()
         context.exception = nil;
 
         context[@"A"] = NSObject.class;
-        NSLog(@"here: %@", [context exception]);
+        checkResult(@"Should not throw an exception when wrapping NSObject and Object has been changed", [context exception]);
     }
 }
 
index 5a66817..e98d6cf 100644 (file)
@@ -512,8 +512,6 @@ static void* multiVMThreadMain(void* okPtr)
 
 static void testObjectiveCAPIMain()
 {
-    runJSExportTests();
-
     @autoreleasepool {
         JSVirtualMachine* vm = [[JSVirtualMachine alloc] init];
         JSContext* context = [[JSContext alloc] initWithVirtualMachine:vm];
index a7d8383..42b5f8c 100644 (file)
@@ -1,3 +1,17 @@
+2017-05-25  Keith Miller  <keith_miller@apple.com>
+
+        Cleanup tests after r217240
+        https://bugs.webkit.org/show_bug.cgi?id=172466
+
+        Reviewed by Mark Lam.
+
+        I forgot to make my test an actual test. Also, remove second call runJSExportTests()
+
+        * API/tests/JSExportTests.mm:
+        (wrapperForNSObjectisObject):
+        * API/tests/testapi.mm:
+        (testObjectiveCAPIMain):
+
 2017-05-25  Michael Saboff  <msaboff@apple.com>
 
         The default setting of Option::criticalGCMemoryThreshold is too high for iOS