From e0abaab4d9f7cd83a718866e8d01e02d66bb5983 Mon Sep 17 00:00:00 2001
From: Roman Shevchenko <roman.shevchenko@jetbrains.com>
Date: Wed, 5 Jul 2017 17:17:33 +0200
Subject: [PATCH] [java] unneeded semicolon on completion in module-info files
 (IDEA-164119)

---
 .../codeInsight/completion/JavaModuleCompletionContributor.java | 2 +-
 .../java/codeInsight/completion/ModuleCompletionTest.kt         | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/java/java-impl/src/com/intellij/codeInsight/completion/JavaModuleCompletionContributor.java b/java/java-impl/src/com/intellij/codeInsight/completion/JavaModuleCompletionContributor.java
index c4ec7e941d2f..4b1e9a175c05 100644
--- a/java/java-impl/src/com/intellij/codeInsight/completion/JavaModuleCompletionContributor.java
+++ b/java/java-impl/src/com/intellij/codeInsight/completion/JavaModuleCompletionContributor.java
@@ -157,7 +157,7 @@ class JavaModuleCompletionContributor {
   private static void processPackage(PsiPackage pkg, GlobalSearchScope scope, Consumer<LookupElement> result) {
     String packageName = pkg.getQualifiedName();
     if (isQualified(packageName) && !PsiUtil.isPackageEmpty(pkg.getDirectories(scope), packageName)) {
-      result.consume(new OverrideableSpace(lookupElement(pkg), TailType.SEMICOLON));
+      result.consume(new OverrideableSpace(lookupElement(pkg), TailType.NONE));
     }
     for (PsiPackage subPackage : pkg.getSubPackages(scope)) {
       processPackage(subPackage, scope, result);
diff --git a/java/java-tests/testSrc/com/intellij/java/codeInsight/completion/ModuleCompletionTest.kt b/java/java-tests/testSrc/com/intellij/java/codeInsight/completion/ModuleCompletionTest.kt
index 9691bf133323..ccc6ba500f2b 100644
--- a/java/java-tests/testSrc/com/intellij/java/codeInsight/completion/ModuleCompletionTest.kt
+++ b/java/java-tests/testSrc/com/intellij/java/codeInsight/completion/ModuleCompletionTest.kt
@@ -42,7 +42,7 @@ class ModuleCompletionTest : LightJava9ModulesCodeInsightFixtureTestCase() {
     addFile("pkg/main/C.java", "package pkg.main;\nclass C { }")
     addFile("pkg/other/C.groovy", "package pkg.other\nclass C { }")
     variants("module M { exports pkg.<caret> }", "main", "other")
-    complete("module M { exports pkg.o<caret> }", "module M { exports pkg.other;<caret> }")
+    complete("module M { exports pkg.o<caret> }", "module M { exports pkg.other<caret> }")
   }
 
   fun testUses() {