Removed ping due to the const method being empty and therefore being optimizd out

Author: Huddie

test/Interop/Cxx/class/method/Inputs/ambiguous_methods.h

@@ -3,10 +3,6 @@
 
 struct HasAmbiguousMethods {
 
-  // No input
-  void ping() { ++mutableMethodsCalledCount; }
-  void ping() const {}
-
   // One input (const first)
   int increment(int a) const {
     return a + 1;

test/Interop/Cxx/class/method/ambiguous-methods.swift

@@ -11,23 +11,7 @@ var CxxAmbiguousMethodTestSuite = TestSuite("CxxAmbiguousMethods")
 // It's important to check that both calling the const version first
 // and the mutable version first pass. This helps confirm that the lookup
 // table is being properly seeded.
-CxxAmbiguousMethodTestSuite.test("ping const first") {
-  var instance = HasAmbiguousMethods()
-
-  instance.ping()
-  instance.pingMutable()
-  expectEqual(1, instance.numberOfMutableMethodsCalled())
-}
-
-CxxAmbiguousMethodTestSuite.test("ping mutable first") {
-  var instance = HasAmbiguousMethods()
-
-  instance.pingMutable()
-  instance.ping()
-  expectEqual(1, instance.numberOfMutableMethodsCalled())
-}
-
-CxxAmbiguousMethodTestSuite.test("numberOfMutableMethodsCalled: () -> Int") {
+CxxAmbiguousMethodTestSuite.test("[Const First] numberOfMutableMethodsCalled: () -> Int") {
   var instance = HasAmbiguousMethods()
 
   // Sanity check. Make sure we start at 0
@@ -41,6 +25,15 @@ CxxAmbiguousMethodTestSuite.test("numberOfMutableMethodsCalled: () -> Int") {
   expectEqual(1, instance.numberOfMutableMethodsCalledMutating())
 }
 
+CxxAmbiguousMethodTestSuite.test("[Mutable First] numberOfMutableMethodsCalled: () -> Int") {
+  var instance = HasAmbiguousMethods()
+
+  // Call mutable first
+  expectEqual(1, instance.numberOfMutableMethodsCalledMutating())
+  expectEqual(1, instance.numberOfMutableMethodsCalled())
+
+}
+
 CxxAmbiguousMethodTestSuite.test("Basic Increment: (Int) -> Int") {
   var instance = HasAmbiguousMethods()
   var a: Int32 = 0