Remove ellipsis on 'Create getter and setter for' label

... since the action doesn't lead to a selection dialog. Signed-off-by: Fred Bricon <[email protected]>
eclipse-jdtls · Apr 30, 2019 · 5a2756f · 5a2756f
1 parent 2d41763
commit 5a2756f
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/...dt.ls.core/src/org/eclipse/jdt/ls/core/internal/corrections/CorrectionMessages.properties b/...dt.ls.core/src/org/eclipse/jdt/ls/core/internal/corrections/CorrectionMessages.properties
@@ -443,7 +443,7 @@ FixCorrectionProposal_WarningAdditionalProposalInfo=Warning:
 SuppressWarningsSubProcessor_fix_suppress_token_label=Change to ''{0}''
 SuppressWarningsSubProcessor_remove_annotation_label=Remove ''{0}'' token 
 ModifierCorrectionSubProcessor_changefieldmodifiertononstatic_description=Remove ''static'' modifier of ''{0}''
-GetterSetterCorrectionSubProcessor_creategetterunsingencapsulatefield_description=Create getter and setter for ''{0}''...
+GetterSetterCorrectionSubProcessor_creategetterunsingencapsulatefield_description=Create getter and setter for ''{0}''
 VarargsWarningsSubProcessor_add_safevarargs_label=Add @SafeVarargs
 VarargsWarningsSubProcessor_add_safevarargs_to_method_label=Add @SafeVarargs to ''{0}(..)''
 VarargsWarningsSubProcessor_remove_safevarargs_label=Remove @SafeVarargs

diff --git a/...dt.ls.tests/src/org/eclipse/jdt/ls/core/internal/correction/GetterSetterQuickFixTest.java b/...dt.ls.tests/src/org/eclipse/jdt/ls/core/internal/correction/GetterSetterQuickFixTest.java
@@ -86,7 +86,7 @@ public void testInvisibleFieldToGetterSetter() throws Exception {
 		buf.append("        c.setTest(c.getTest() + 1);\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 'test'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 'test'", buf.toString());
 		assertCodeActions(cu, e1, e2);
 	}
 
@@ -144,7 +144,7 @@ public void testInvisibleFieldToGetterSetter_2() throws Exception {
 		buf.append("        c.setTest(c.getTest() + (1 + 2));\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 'test'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 'test'", buf.toString());
 
 		assertCodeActions(cu, e1, e2);
 	}
@@ -203,7 +203,7 @@ public void testInvisibleFieldToGetterSetter_3() throws Exception {
 		buf.append("        c.setTest(c.getTest() - (1 + 2));\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 'test'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 'test'", buf.toString());
 
 		assertCodeActions(cu, e1, e2);
 	}
@@ -262,7 +262,7 @@ public void testInvisibleFieldToGetterSetter_4() throws Exception {
 		buf.append("        c.setTest(c.getTest() * (1 + 2));\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 'test'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 'test'", buf.toString());
 
 		assertCodeActions(cu, e1, e2);
 	}
@@ -413,7 +413,7 @@ public void testCreateFieldUsingSef() throws Exception {
 		buf.append("        new A().setT(5);\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 't'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 't'", buf.toString());
 
 		assertCodeActions(cu, e1, e2);
 	}

diff --git a/...ls.tests/src/org/eclipse/jdt/ls/core/internal/correction/LocalCorrectionQuickFixTest.java b/...ls.tests/src/org/eclipse/jdt/ls/core/internal/correction/LocalCorrectionQuickFixTest.java
@@ -140,7 +140,7 @@ public void testUnusedPrivateField() throws Exception {
 		buf.append("        this.count = count;\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 'count'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 'count'", buf.toString());
 
 		assertCodeActions(cu, e1, e2);
 	}
@@ -188,7 +188,7 @@ public void testUnusedPrivateField1() throws Exception {
 		buf.append("        this.color = color;\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 'color'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 'color'", buf.toString());
 
 		assertCodeActions(cu, e1, e2);
 	}
@@ -234,7 +234,7 @@ public void testUnusedPrivateField2() throws Exception {
 		buf.append("        this.count = count;\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 'count'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 'count'", buf.toString());
 
 		assertCodeActions(cu, e1, e2);
 	}

diff --git a/...ests/src/org/eclipse/jdt/ls/core/internal/correction/ModifierCorrectionsQuickFixTest.java b/...ests/src/org/eclipse/jdt/ls/core/internal/correction/ModifierCorrectionsQuickFixTest.java
@@ -582,7 +582,7 @@ public void testInvisibleFieldRequestedInSamePackage1() throws Exception {
 		buf.append("         c.setTest(1);\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e2 = new Expected("Create getter and setter for 'test'...", buf.toString());
+		Expected e2 = new Expected("Create getter and setter for 'test'", buf.toString());
 
 		assertCodeActions(cu, e1, e2);
 	}
@@ -687,7 +687,7 @@ public void testInvisibleFieldRequestedInSamePackage2() throws Exception {
 		buf.append("         setTest(1);\n");
 		buf.append("    }\n");
 		buf.append("}\n");
-		Expected e6 = new Expected("Create getter and setter for 'test'...", buf.toString());
+		Expected e6 = new Expected("Create getter and setter for 'test'", buf.toString());
 
 		assertCodeActions(cu, e1, e2, e3, e4, e5, e6);
 	}