Fix unit test and move it to TypeCheckNoTranspileTests.java so it run…

…s without class transpilation and remove debugging code so the test passes.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=208519103

src/com/google/javascript/jscomp/TypeCheck.java

@@ -1324,8 +1324,7 @@ private void visitObjectOrClassLiteralKey(
       // Allow the definition of "constructor" as an unquoted key in class bodies.  They will
       // never be renamed so quoting is unimportant for the ALL_UNQUOTED renaming policy
       // and the property is not accessed directly.
-      boolean temp = true;
-      if (temp || !key.isMemberFunctionDef()
+      if (!key.isMemberFunctionDef()
           || !key.getParent().isClassMembers()
           || !key.getString().equals("constructor")) {
         report(t, key, ILLEGAL_OBJLIT_KEY, "dict");

test/com/google/javascript/jscomp/TypeCheckNoTranspileTest.java

@@ -4024,5 +4024,9 @@ public void testDefaultValueForNestedArrayPatternMustBeIterable() {
             "required: Iterable"));
   }
 
+  public void testDictClass1() {
+    testTypes("/** @dict */ var C = class { constructor() {} 'x'(){} };");
+  }
+
   // TODO(b/77597706): add tests for missing property warnings on object destructuring string keys
 }

test/com/google/javascript/jscomp/TypeCheckTest.java

@@ -1199,10 +1199,6 @@ public void testObjLitDef7() {
         "Illegal key, the object literal is a dict");
   }
 
-  public void testDictClass1() {
-    testTypes("var C = /** @dict */ class { constructor() {} 'x'(){} };");
-  }
-
   public void testInstanceOfReduction1() {
     testTypes("/** @constructor */ var T = function() {};\n" +
         "/** @param {T|string} x\n@return {T} */\n" +