diff --git a/compiler/tests/org/jetbrains/jet/InTextDirectivesUtils.java b/compiler/tests/org/jetbrains/jet/InTextDirectivesUtils.java index e29904fb0c6..d91eb0cec1e 100644 --- a/compiler/tests/org/jetbrains/jet/InTextDirectivesUtils.java +++ b/compiler/tests/org/jetbrains/jet/InTextDirectivesUtils.java @@ -36,7 +36,7 @@ public final class InTextDirectivesUtils { @Nullable public static Integer getPrefixedInt(String fileText, String prefix) { - String[] numberStrings = findArrayWithPrefix(fileText, prefix); + String[] numberStrings = findArrayWithPrefixes(fileText, prefix); if (numberStrings.length > 0) { return Integer.parseInt(numberStrings[0]); } @@ -45,15 +45,15 @@ public final class InTextDirectivesUtils { } @NotNull - public static String[] findArrayWithPrefix(String fileText, String... prefixes) { - return ArrayUtil.toStringArray(findListWithPrefix(fileText, prefixes)); + public static String[] findArrayWithPrefixes(String fileText, String... prefixes) { + return ArrayUtil.toStringArray(findListWithPrefixes(fileText, prefixes)); } @NotNull - public static List findListWithPrefix(String fileText, String... prefixes) { + public static List findListWithPrefixes(String fileText, String... prefixes) { List result = new ArrayList(); - for (String line : findLinesWithPrefixRemoved(fileText, prefixes)) { + for (String line : findLinesWithPrefixesRemoved(fileText, prefixes)) { String[] variants = line.split(","); for (String variant : variants) { @@ -65,18 +65,18 @@ public final class InTextDirectivesUtils { } @Nullable - public static String findStringWithPrefix(String fileText, String... prefixes) { - List strings = findListWithPrefix(fileText, prefixes); + public static String findStringWithPrefixes(String fileText, String... prefixes) { + List strings = findListWithPrefixes(fileText, prefixes); if (strings.isEmpty()) { return null; } assert strings.size() == 1 : "There is more than one string with given prefixes " + - Arrays.toString(prefixes) + ". Use findListWithPrefix() instead."; + Arrays.toString(prefixes) + ". Use findListWithPrefixes() instead."; return strings.get(0); } @NotNull - public static List findLinesWithPrefixRemoved(String fileText, String... prefixes) { + public static List findLinesWithPrefixesRemoved(String fileText, String... prefixes) { List result = new ArrayList(); List cleanedPrefixes = cleanDirectivesFromComments(Arrays.asList(prefixes)); diff --git a/compiler/tests/org/jetbrains/jet/codegen/defaultConstructor/AbstractDefaultConstructorCodegenTest.java b/compiler/tests/org/jetbrains/jet/codegen/defaultConstructor/AbstractDefaultConstructorCodegenTest.java index dc603fe516c..4773e9a81f3 100644 --- a/compiler/tests/org/jetbrains/jet/codegen/defaultConstructor/AbstractDefaultConstructorCodegenTest.java +++ b/compiler/tests/org/jetbrains/jet/codegen/defaultConstructor/AbstractDefaultConstructorCodegenTest.java @@ -25,7 +25,7 @@ import java.io.IOException; import java.lang.reflect.Constructor; import java.util.List; -import static org.jetbrains.jet.InTextDirectivesUtils.findListWithPrefix; +import static org.jetbrains.jet.InTextDirectivesUtils.findListWithPrefixes; public abstract class AbstractDefaultConstructorCodegenTest extends CodegenTestCase { @@ -64,7 +64,7 @@ public abstract class AbstractDefaultConstructorCodegenTest extends CodegenTestC } private String loadInstructionValue(String fileContent, String instructionName) { - List testedClass = findListWithPrefix(fileContent, "// " + instructionName + ": "); + List testedClass = findListWithPrefixes(fileContent, "// " + instructionName + ": "); assertTrue("Cannot find " + instructionName + " instruction", !testedClass.isEmpty()); assertTrue(instructionName + " instruction must have only one element", testedClass.size() == 1); return testedClass.get(0); diff --git a/compiler/tests/org/jetbrains/jet/codegen/flags/AbstractWriteFlagsTest.java b/compiler/tests/org/jetbrains/jet/codegen/flags/AbstractWriteFlagsTest.java index 54e3d7a27ad..f18d9ca666b 100644 --- a/compiler/tests/org/jetbrains/jet/codegen/flags/AbstractWriteFlagsTest.java +++ b/compiler/tests/org/jetbrains/jet/codegen/flags/AbstractWriteFlagsTest.java @@ -31,8 +31,8 @@ import java.io.IOException; import java.lang.reflect.Field; import java.util.List; -import static org.jetbrains.jet.InTextDirectivesUtils.findListWithPrefix; -import static org.jetbrains.jet.InTextDirectivesUtils.findStringWithPrefix; +import static org.jetbrains.jet.InTextDirectivesUtils.findListWithPrefixes; +import static org.jetbrains.jet.InTextDirectivesUtils.findStringWithPrefixes; /* * Test correctness of written flags in class file @@ -94,7 +94,7 @@ public abstract class AbstractWriteFlagsTest extends UsefulTestCase { private static TestedObject parseExpectedTestedObject(String fileText) { TestedObject result = new TestedObject(); - List testedObjects = findListWithPrefix(fileText, "// TESTED_OBJECTS: "); + List testedObjects = findListWithPrefixes(fileText, "// TESTED_OBJECTS: "); assertTrue("Cannot find TESTED_OBJECTS instruction", !testedObjects.isEmpty()); result.containingClass = testedObjects.get(0); if (testedObjects.size() == 1) { @@ -108,8 +108,8 @@ public abstract class AbstractWriteFlagsTest extends UsefulTestCase { "TESTED_OBJECTS instruction must contains one (for class) or two (for function and property) values"); } - result.kind = findStringWithPrefix(fileText, "// TESTED_OBJECT_KIND: "); - List isFullName = findListWithPrefix(fileText, "// IS_FULL_CONTAINING_CLASS_NAME: "); + result.kind = findStringWithPrefixes(fileText, "// TESTED_OBJECT_KIND: "); + List isFullName = findListWithPrefixes(fileText, "// IS_FULL_CONTAINING_CLASS_NAME: "); if (isFullName.size() == 1) { result.isFullContainingClassName = Boolean.parseBoolean(isFullName.get(0)); } @@ -150,7 +150,7 @@ public abstract class AbstractWriteFlagsTest extends UsefulTestCase { private static int getExpectedFlags(String text) { int expectedAccess = 0; Class klass = Opcodes.class; - List flags = findListWithPrefix(text, "// FLAGS: "); + List flags = findListWithPrefixes(text, "// FLAGS: "); for (String flag : flags) { try { Field field = klass.getDeclaredField(flag); diff --git a/idea/tests/org/jetbrains/jet/completion/ExpectedCompletionUtils.java b/idea/tests/org/jetbrains/jet/completion/ExpectedCompletionUtils.java index 263dcc98e05..e8cc18c94af 100644 --- a/idea/tests/org/jetbrains/jet/completion/ExpectedCompletionUtils.java +++ b/idea/tests/org/jetbrains/jet/completion/ExpectedCompletionUtils.java @@ -167,7 +167,7 @@ public class ExpectedCompletionUtils { public static CompletionProposal[] processProposalAssertions(String fileText, String... prefixes) { Collection proposals = new ArrayList(); - for (String proposalStr : InTextDirectivesUtils.findListWithPrefix(fileText, prefixes)) { + for (String proposalStr : InTextDirectivesUtils.findListWithPrefixes(fileText, prefixes)) { Matcher matcher = CompletionProposal.PATTERN.matcher(proposalStr); matcher.find(); proposals.add(new CompletionProposal(matcher.group(CompletionProposal.LOOKUP_STRING_GROUP_INDEX), diff --git a/idea/tests/org/jetbrains/jet/completion/confidence/JetConfidenceTest.java b/idea/tests/org/jetbrains/jet/completion/confidence/JetConfidenceTest.java index 48105ac2044..bed31014e2f 100644 --- a/idea/tests/org/jetbrains/jet/completion/confidence/JetConfidenceTest.java +++ b/idea/tests/org/jetbrains/jet/completion/confidence/JetConfidenceTest.java @@ -79,7 +79,7 @@ public class JetConfidenceTest extends LightCompletionTestCase { protected static String getTypeTextFromFile() { String text = getEditor().getDocument().getText(); - String[] directives = InTextDirectivesUtils.findArrayWithPrefix(text, TYPE_DIRECTIVE_PREFIX); + String[] directives = InTextDirectivesUtils.findArrayWithPrefixes(text, TYPE_DIRECTIVE_PREFIX); assertEquals("One directive with \"" + TYPE_DIRECTIVE_PREFIX +"\" expected", 1, directives.length); return StringUtil.unquoteString(directives[0]); diff --git a/idea/tests/org/jetbrains/jet/formatter/FormattingSettingsConfigurator.java b/idea/tests/org/jetbrains/jet/formatter/FormattingSettingsConfigurator.java index 93a723c3ffd..b93cac3d827 100644 --- a/idea/tests/org/jetbrains/jet/formatter/FormattingSettingsConfigurator.java +++ b/idea/tests/org/jetbrains/jet/formatter/FormattingSettingsConfigurator.java @@ -30,8 +30,8 @@ public class FormattingSettingsConfigurator { private final String[] settingsToFalse; public FormattingSettingsConfigurator(String fileText) { - settingsToTrue = InTextDirectivesUtils.findArrayWithPrefix(fileText, "// SET_TRUE:"); - settingsToFalse = InTextDirectivesUtils.findArrayWithPrefix(fileText, "// SET_FALSE:"); + settingsToTrue = InTextDirectivesUtils.findArrayWithPrefixes(fileText, "// SET_TRUE:"); + settingsToFalse = InTextDirectivesUtils.findArrayWithPrefixes(fileText, "// SET_FALSE:"); } public void configureSettings(CodeStyleSettings settings) { diff --git a/idea/tests/org/jetbrains/jet/plugin/codeInsight/codeTransformations/AbstractCodeTransformationTest.java b/idea/tests/org/jetbrains/jet/plugin/codeInsight/codeTransformations/AbstractCodeTransformationTest.java index 10cdfb5343b..cbe7fa96aaa 100644 --- a/idea/tests/org/jetbrains/jet/plugin/codeInsight/codeTransformations/AbstractCodeTransformationTest.java +++ b/idea/tests/org/jetbrains/jet/plugin/codeInsight/codeTransformations/AbstractCodeTransformationTest.java @@ -37,7 +37,7 @@ public abstract class AbstractCodeTransformationTest extends LightCodeInsightTes configureByFile(path); String fileText = FileUtil.loadFile(new File(path)); - String isApplicableString = InTextDirectivesUtils.findStringWithPrefix(fileText, "// IS_APPLICABLE: "); + String isApplicableString = InTextDirectivesUtils.findStringWithPrefixes(fileText, "// IS_APPLICABLE: "); boolean isApplicableExpected = isApplicableString == null || isApplicableString.equals("true"); assert isApplicableExpected == intentionAction.isAvailable(getProject(), getEditor(), getFile()) diff --git a/idea/tests/org/jetbrains/jet/plugin/codeInsight/surroundWith/AbstractSurroundWithTest.java b/idea/tests/org/jetbrains/jet/plugin/codeInsight/surroundWith/AbstractSurroundWithTest.java index c08ea8e4c1c..efbd8cf1cb5 100644 --- a/idea/tests/org/jetbrains/jet/plugin/codeInsight/surroundWith/AbstractSurroundWithTest.java +++ b/idea/tests/org/jetbrains/jet/plugin/codeInsight/surroundWith/AbstractSurroundWithTest.java @@ -82,7 +82,7 @@ public abstract class AbstractSurroundWithTest extends LightCodeInsightTestCase configureByFile(path); String fileText = FileUtil.loadFile(new File(path)); - String isApplicableString = InTextDirectivesUtils.findStringWithPrefix(fileText, "// IS_APPLICABLE: "); + String isApplicableString = InTextDirectivesUtils.findStringWithPrefixes(fileText, "// IS_APPLICABLE: "); if (isApplicableString != null) { boolean isApplicableExpected = toString().equals("true"); diff --git a/idea/tests/org/jetbrains/jet/plugin/navigation/NavigationTestUtils.java b/idea/tests/org/jetbrains/jet/plugin/navigation/NavigationTestUtils.java index 76bc2494cd3..8e94dd73d53 100644 --- a/idea/tests/org/jetbrains/jet/plugin/navigation/NavigationTestUtils.java +++ b/idea/tests/org/jetbrains/jet/plugin/navigation/NavigationTestUtils.java @@ -49,7 +49,7 @@ public final class NavigationTestUtils { public static void assertGotoImplementations(Editor editor, GotoTargetHandler.GotoData gotoData) { // Get expected references from the tested document - List expectedReferences = InTextDirectivesUtils.findListWithPrefix(editor.getDocument().getText(), "// REF:"); + List expectedReferences = InTextDirectivesUtils.findListWithPrefixes(editor.getDocument().getText(), "// REF:"); Collections.sort(expectedReferences); if (gotoData != null) { @@ -72,11 +72,11 @@ public final class NavigationTestUtils { public static void assertGotoSymbol(@NotNull Project project, @NotNull Editor editor) { - List searchTextList = InTextDirectivesUtils.findListWithPrefix(editor.getDocument().getText(), "// SEARCH_TEXT:"); + List searchTextList = InTextDirectivesUtils.findListWithPrefixes(editor.getDocument().getText(), "// SEARCH_TEXT:"); Assert.assertFalse("There's no search text in test data file given. Use '// SEARCH_TEXT:' directive", searchTextList.isEmpty()); - List expectedReferences = InTextDirectivesUtils.findListWithPrefix(editor.getDocument().getText(), "// REF:"); + List expectedReferences = InTextDirectivesUtils.findListWithPrefixes(editor.getDocument().getText(), "// REF:"); String searchText = searchTextList.get(0); diff --git a/idea/tests/org/jetbrains/jet/plugin/quickfix/QuickFixActionsUtils.java b/idea/tests/org/jetbrains/jet/plugin/quickfix/QuickFixActionsUtils.java index 2b756f81b47..751f34b3367 100644 --- a/idea/tests/org/jetbrains/jet/plugin/quickfix/QuickFixActionsUtils.java +++ b/idea/tests/org/jetbrains/jet/plugin/quickfix/QuickFixActionsUtils.java @@ -59,7 +59,7 @@ public class QuickFixActionsUtils { } }); - List expectedErrorStrings = InTextDirectivesUtils.findLinesWithPrefixRemoved(file.getText(), "// ERROR:"); + List expectedErrorStrings = InTextDirectivesUtils.findLinesWithPrefixesRemoved(file.getText(), "// ERROR:"); Collections.sort(expectedErrorStrings); UsefulTestCase.assertOrderedEquals( @@ -69,7 +69,7 @@ public class QuickFixActionsUtils { public static void checkAvailableActionsAreExpected(JetFile file, Collection availableActions) { List validActions = Ordering.natural().sortedCopy( - Sets.newHashSet(InTextDirectivesUtils.findLinesWithPrefixRemoved(file.getText(), "// ACTION:"))); + Sets.newHashSet(InTextDirectivesUtils.findLinesWithPrefixesRemoved(file.getText(), "// ACTION:"))); Collection actualActions = Ordering.natural().sortedCopy( Collections2.transform(availableActions, new Function() { diff --git a/idea/tests/org/jetbrains/jet/search/AbstractSearcherTest.java b/idea/tests/org/jetbrains/jet/search/AbstractSearcherTest.java index 34003f8f554..2550bc16e1d 100644 --- a/idea/tests/org/jetbrains/jet/search/AbstractSearcherTest.java +++ b/idea/tests/org/jetbrains/jet/search/AbstractSearcherTest.java @@ -50,7 +50,7 @@ public abstract class AbstractSearcherTest extends LightCodeInsightFixtureTestCa } protected void checkResult(Query actual) throws IOException { - List expected = InTextDirectivesUtils.findListWithPrefix(FileUtil.loadFile(new File(getPathToFile())), "// SEARCH: "); + List expected = InTextDirectivesUtils.findListWithPrefixes(FileUtil.loadFile(new File(getPathToFile())), "// SEARCH: "); List actualModified = new ArrayList(); for (Object member : actual) { actualModified.add(member.toString()); diff --git a/idea/tests/org/jetbrains/jet/search/AnnotatedMembersSearchTest.java b/idea/tests/org/jetbrains/jet/search/AnnotatedMembersSearchTest.java index 3864496debe..0d035002d92 100644 --- a/idea/tests/org/jetbrains/jet/search/AnnotatedMembersSearchTest.java +++ b/idea/tests/org/jetbrains/jet/search/AnnotatedMembersSearchTest.java @@ -42,7 +42,8 @@ public class AnnotatedMembersSearchTest extends AbstractSearcherTest { private void doTest() throws IOException { myFixture.configureByFile(getFileName()); - List directives = InTextDirectivesUtils.findListWithPrefix(FileUtil.loadFile(new File(getPathToFile())), "// ANNOTATION: "); + List directives = InTextDirectivesUtils.findListWithPrefixes(FileUtil.loadFile(new File(getPathToFile())), + "// ANNOTATION: "); assertFalse("Specify ANNOTATION directive in test file", directives.isEmpty()); String annotationClassName = directives.get(0); PsiClass psiClass = getPsiClass(annotationClassName); diff --git a/idea/tests/org/jetbrains/jet/search/ClassInheritorsSearchTest.java b/idea/tests/org/jetbrains/jet/search/ClassInheritorsSearchTest.java index abbf4ec14ae..ea73776df2b 100644 --- a/idea/tests/org/jetbrains/jet/search/ClassInheritorsSearchTest.java +++ b/idea/tests/org/jetbrains/jet/search/ClassInheritorsSearchTest.java @@ -38,7 +38,7 @@ public class ClassInheritorsSearchTest extends AbstractSearcherTest { private void doTest() throws IOException { myFixture.configureByFile(getFileName()); - List directives = InTextDirectivesUtils.findListWithPrefix(FileUtil.loadFile(new File(getPathToFile())), "// CLASS: "); + List directives = InTextDirectivesUtils.findListWithPrefixes(FileUtil.loadFile(new File(getPathToFile())), "// CLASS: "); assertFalse("Specify CLASS directive in test file", directives.isEmpty()); String superClassName = directives.get(0); PsiClass psiClass = getPsiClass(superClassName); diff --git a/idea/tests/org/jetbrains/jet/search/JUnitMembersSearcherTest.java b/idea/tests/org/jetbrains/jet/search/JUnitMembersSearcherTest.java index 42509b50115..ec98fae180e 100644 --- a/idea/tests/org/jetbrains/jet/search/JUnitMembersSearcherTest.java +++ b/idea/tests/org/jetbrains/jet/search/JUnitMembersSearcherTest.java @@ -49,7 +49,7 @@ public class JUnitMembersSearcherTest extends AbstractSearcherTest { private void doJUnit3test() throws IOException { myFixture.configureByFile(getFileName()); - List directives = InTextDirectivesUtils.findListWithPrefix(FileUtil.loadFile(new File(getPathToFile())), "// CLASS: "); + List directives = InTextDirectivesUtils.findListWithPrefixes(FileUtil.loadFile(new File(getPathToFile())), "// CLASS: "); assertFalse("Specify CLASS directive in test file", directives.isEmpty()); String superClassName = directives.get(0); PsiClass psiClass = getPsiClass(superClassName); @@ -58,7 +58,8 @@ public class JUnitMembersSearcherTest extends AbstractSearcherTest { private void doJUnit4test() throws IOException { myFixture.configureByFile(getFileName()); - List directives = InTextDirectivesUtils.findListWithPrefix(FileUtil.loadFile(new File(getPathToFile())), "// ANNOTATION: "); + List directives = InTextDirectivesUtils.findListWithPrefixes(FileUtil.loadFile(new File(getPathToFile())), + "// ANNOTATION: "); assertFalse("Specify ANNOTATION directive in test file", directives.isEmpty()); String annotationClassName = directives.get(0); PsiClass psiClass = getPsiClass(annotationClassName);