From b972110432afc15c95e463f3374afd257dee0ea3 Mon Sep 17 00:00:00 2001 From: peter Date: Wed, 10 Oct 2018 22:01:28 +0200 Subject: [PATCH] dom: remove testInternalDomLocksReadConsistency it takes a lot of time and doesn't seem to check anything useful nowadays --- .../util/xml/DomConcurrencyStressTest.java | 80 ------------------- 1 file changed, 80 deletions(-) diff --git a/xml/dom-tests/tests/com/intellij/util/xml/DomConcurrencyStressTest.java b/xml/dom-tests/tests/com/intellij/util/xml/DomConcurrencyStressTest.java index d0065dbdc619..1794e7620804 100644 --- a/xml/dom-tests/tests/com/intellij/util/xml/DomConcurrencyStressTest.java +++ b/xml/dom-tests/tests/com/intellij/util/xml/DomConcurrencyStressTest.java @@ -26,9 +26,7 @@ import com.intellij.semantic.SemService; import com.intellij.testFramework.PlatformTestUtil; import com.intellij.testFramework.Timings; import com.intellij.util.ConcurrencyUtil; -import com.intellij.util.TimeoutUtil; import com.intellij.util.containers.ContainerUtil; -import com.intellij.util.xml.impl.DomFileElementImpl; import com.intellij.util.xml.impl.DomTestCase; import com.intellij.util.xml.reflect.DomExtender; import com.intellij.util.xml.reflect.DomExtenderEP; @@ -44,65 +42,6 @@ import java.util.concurrent.CountDownLatch; * @author peter */ public class DomConcurrencyStressTest extends DomTestCase { - public void testInternalDomLocksReadConsistency() throws Throwable { - getDomManager().registerFileDescription(new DomFileDescription(MyElement.class, "a"), getTestRootDisposable()); - - registerExtender(MyElement.class, MyExtender.class); - - - final XmlFile file = createXmlFile("" + - "" + - "" + - "" + - " " + - " " + - " " + - " " + - "" + - "" + - "" + - " " + - " " + - " " + - " " + - "" + - "" + - " " + - " " + - " " + - "" + - "" + - "" + - "" + - ""); - - final int ITERATIONS = Timings.adjustAccordingToMySpeed(239, true); - System.out.println("ITERATIONS =" + ITERATIONS); - runThreads(42, new Runnable() { - @Override - public void run() { - for (int i = 0; i < ITERATIONS; i++) { - ApplicationManager.getApplication().runReadAction(new Runnable() { - @Override - public void run() { - final DomFileElementImpl element = getDomManager().getFileElement(file); - assertNotNull(element); - element.getRootElement().accept(new DomElementVisitor() { - @Override - public void visitDomElement(final DomElement element) { - if (DomUtil.hasXml(element)) { - element.acceptChildren(this); - } - } - }); - } - }); - - SemService.getSemService(getProject()).clearCache(); - } - } - }); - } private void registerExtender(final Class elementClass, final Class extenderClass) { final DomExtenderEP extenderEP = new DomExtenderEP(); @@ -162,25 +101,6 @@ public class DomConcurrencyStressTest extends DomTestCase { } - public static class MyExtender extends DomExtender { - private final Random myRandom = new Random(); - - @Override - public void registerExtensions(@NotNull MyElement myElement, @NotNull DomExtensionsRegistrar registrar) { - for (MyElement element : myElement.getFooElements()) { - element.getFooElements(); - } - if (myRandom.nextInt(20) < 2) { - TimeoutUtil.sleep(1); - } - registrar.registerFixedNumberChildExtension(new XmlName("custom-foo", null), MyElement.class); - myElement.getSomeChild().getFooElements(); - myElement.getFooChild().getFooChild().getAttr(); - myElement.getAttr(); - registrar.registerCollectionChildrenExtension(new XmlName("custom-bar", null), MyElement.class); - } - } - public void testBigCustomFile() throws Throwable { final int ITERATIONS = Timings.adjustAccordingToMySpeed(239, true); getDomManager().registerFileDescription(new DomFileDescription<>(MyAllCustomElement.class, "component"), getTestRootDisposable());