Additional fix for IDEABKL-4743 Automatic placement of ";" characters

-- after review improvement 4: added test cases for string/char literals

java/java-tests/testData/codeInsight/typing/semicolonBeforeRightParenInLiterals_after.java

@@ -0,0 +1,26 @@
+class Foo {
+
+  int takesCharSequence(CharSequence str) {
+    return 0;
+  }
+
+  int takesChar(char chr) {
+    return 0;
+  }
+
+  void testOutside() {
+    // should be moved
+    takesCharSequence(""/*typehere*/);
+    takesChar(''/*typehere*/);
+  }
+
+  void testInside() {
+    // should not be moved
+    takesCharSequence("/*typehere*/;")
+    takesChar('/*typehere*/;')
+  }
+
+  void testInComment() {
+    // some comment blabla(/*typehere*/;)
+  }
+}

java/java-tests/testData/codeInsight/typing/semicolonBeforeRightParenInLiterals_before.java

@@ -0,0 +1,26 @@
+class Foo {
+
+  int takesCharSequence(CharSequence str) {
+    return 0;
+  }
+
+  int takesChar(char chr) {
+    return 0;
+  }
+
+  void testOutside() {
+    // should be moved
+    takesCharSequence(""/*typehere*/)
+    takesChar(''/*typehere*/)
+  }
+
+  void testInside() {
+    // should not be moved
+    takesCharSequence("/*typehere*/")
+    takesChar('/*typehere*/')
+  }
+
+  void testInComment() {
+    // some comment blabla(/*typehere*/)
+  }
+}

java/java-tests/testSrc/com/intellij/java/codeInsight/JavaTypingTest.java

@@ -124,6 +124,10 @@ public class JavaTypingTest extends LightPlatformCodeInsightFixtureTestCase {
     doMultiTypeTest(';');
   }
 
+  public void testSemicolonBeforeRightParenInLiterals() {
+    doMultiTypeTest(';');
+  }
+
   public void testCommaAfterDefaultAnnotationArgumentWhenArrayIsExpected() {
     doTest(',');
   }