From a8ebab2069d884b5fb2fde836070f41faaf81619 Mon Sep 17 00:00:00 2001 From: Alexey Kudravtsev Date: Wed, 20 Nov 2013 20:02:54 +0400 Subject: [PATCH] test for Cls parsing duplicates --- .../testSrc/com/intellij/JavaTestUtil.java | 4 +- .../com/intellij/psi/ClsDuplicatesTest.java | 99 +++++++++++++++++++ 2 files changed, 101 insertions(+), 2 deletions(-) create mode 100644 java/java-tests/testSrc/com/intellij/psi/ClsDuplicatesTest.java diff --git a/java/java-tests/testSrc/com/intellij/JavaTestUtil.java b/java/java-tests/testSrc/com/intellij/JavaTestUtil.java index 43e38e07b0c0..4dfce7cc9438 100644 --- a/java/java-tests/testSrc/com/intellij/JavaTestUtil.java +++ b/java/java-tests/testSrc/com/intellij/JavaTestUtil.java @@ -1,5 +1,5 @@ /* - * Copyright 2000-2012 JetBrains s.r.o. + * Copyright 2000-2013 JetBrains s.r.o. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -53,7 +53,7 @@ public class JavaTestUtil { }); } - private static Sdk getTestJdk() { + public static Sdk getTestJdk() { try { ProjectJdkImpl jdk = (ProjectJdkImpl)JavaAwareProjectJdkTableImpl.getInstanceEx().getInternalJdk().clone(); jdk.setName("JDK"); diff --git a/java/java-tests/testSrc/com/intellij/psi/ClsDuplicatesTest.java b/java/java-tests/testSrc/com/intellij/psi/ClsDuplicatesTest.java new file mode 100644 index 000000000000..1d8ea1718fb0 --- /dev/null +++ b/java/java-tests/testSrc/com/intellij/psi/ClsDuplicatesTest.java @@ -0,0 +1,99 @@ +/* + * Copyright 2000-2013 JetBrains s.r.o. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package com.intellij.psi; + +import com.intellij.JavaTestUtil; +import com.intellij.openapi.projectRoots.Sdk; +import com.intellij.openapi.util.Comparing; +import com.intellij.openapi.util.text.StringUtil; +import com.intellij.psi.search.GlobalSearchScope; +import com.intellij.testFramework.PsiTestCase; +import com.intellij.usageView.UsageViewLongNameLocation; +import gnu.trove.THashSet; +import gnu.trove.TObjectHashingStrategy; + +import java.util.Set; + +public class ClsDuplicatesTest extends PsiTestCase { + @Override + protected Sdk getTestProjectJdk() { + return JavaTestUtil.getTestJdk(); + } + + public void testDuplicates() throws Exception { + final PsiPackage rootPackage = JavaPsiFacade.getInstance(getProject()).findPackage(""); + assert rootPackage != null; + final GlobalSearchScope scope = GlobalSearchScope.allScope(getProject()); + JavaRecursiveElementVisitor visitor = new JavaRecursiveElementVisitor() { + @Override + public void visitPackage(PsiPackage aPackage) { + System.out.println(aPackage.getQualifiedName()); + + visit(aPackage); + for (PsiPackage subPackage : aPackage.getSubPackages(scope)) { + visitPackage(subPackage); + } + for (PsiClass aClass : aPackage.getClasses(scope)) { + visitElement(aClass); + } + } + + @Override + public void visitElement(PsiElement element) { + if (element instanceof PsiNamedElement) { + visit((PsiNamedElement)element); + } + super.visitElement(element); + } + + @Override + public void visitClass(PsiClass aClass) { + if ("com.sun.xml.internal.bind.v2.runtime.unmarshaller.DomLoader.State".equals(aClass.getQualifiedName())) { + int i =0; + } + super.visitClass(aClass); + } + }; + rootPackage.accept(visitor); + } + + private final Set uniques = new THashSet(new TObjectHashingStrategy() { + @Override + public int computeHashCode(PsiNamedElement object) { + String name = object.getName(); + return name == null ? 0 : name.hashCode(); + } + + @Override + public boolean equals(PsiNamedElement o1, PsiNamedElement o2) { + boolean eq = o1.getParent() == o2.getParent() && StringUtil.equals(o1.getName(), o2.getName()) && o1.getClass() == o2.getClass() + && Comparing.equal(o1.getText(), o2.getText()); + + if (eq) { + return true; + } + else { + return false; + } + } + }); + + private void visit(PsiNamedElement element) { + if (!uniques.add(element)) { + fail("Duplicate Element: "+ElementDescriptionUtil.getElementDescription(element, UsageViewLongNameLocation.INSTANCE)+": "+element.getText() ); + } + } +}