Remove @HardwareAgentRequired annotation

All Perf Unit tests are run on the hardware agents to ensure stability of the metrics.
So the annotation is not needed anymore and it doesn't work on JUnit5 tests so the same test is run twice and since test execution time depends on the bucketing, metrics are jumping.

GitOrigin-RevId: fc83a368447d09d6ad1b269c88fe4f6c79067a91

java/java-tests/testSrc/com/intellij/codeInsight/daemon/impl/DaemonRespondToChangesPerformanceTest.java

@@ -7,7 +7,6 @@ import com.intellij.codeInsight.daemon.DaemonCodeAnalyzer;
 import com.intellij.diagnostic.PerformanceWatcher;
 import com.intellij.diagnostic.ThreadDumper;
 import com.intellij.ide.highlighter.JavaFileType;
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.lang.LanguageAnnotators;
 import com.intellij.lang.annotation.AnnotationHolder;
 import com.intellij.lang.annotation.HighlightSeverity;
@@ -51,7 +50,6 @@ import java.util.concurrent.atomic.AtomicReference;
 /**
  * tests the daemon performance during highlighting interruptions/typing
  */
-@HardwareAgentRequired
 public class DaemonRespondToChangesPerformanceTest extends DaemonAnalyzerTestCase {
   private static final boolean DEBUG = false;
 

json/tests/test/com/jetbrains/jsonSchema/JsonSchemaPerformanceTest.java

@@ -1,7 +1,6 @@
 // Copyright 2000-2017 JetBrains s.r.o. Use of this source code is governed by the Apache 2.0 license that can be found in the LICENSE file.
 package com.jetbrains.jsonSchema;
 
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.json.JsonFileType;
 import com.intellij.json.psi.JsonFile;
 import com.intellij.json.psi.JsonObject;
@@ -26,7 +25,6 @@ import java.util.Collections;
 
 import static com.jetbrains.jsonSchema.JsonSchemaHighlightingTestBase.registerJsonSchema;
 
-@HardwareAgentRequired
 public class JsonSchemaPerformanceTest extends JsonSchemaHeavyAbstractTest {
   public static final String BASE_PATH = "/tests/testData/jsonSchema/performance/";
 

platform/platform-tests/testSrc/com/intellij/openapi/vfs/VfsUtilPerformanceTest.java

@@ -3,7 +3,6 @@ package com.intellij.openapi.vfs;
 
 import com.intellij.concurrency.JobLauncher;
 import com.intellij.concurrency.JobSchedulerImpl;
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.openapi.application.ex.ApplicationManagerEx;
 import com.intellij.openapi.command.WriteCommandAction;
 import com.intellij.openapi.diagnostic.FrequentEventDetector;
@@ -51,7 +50,6 @@ import static org.junit.Assert.*;
 
 @RunFirst
 @SkipSlowTestLocally
-@HardwareAgentRequired
 public class VfsUtilPerformanceTest extends BareTestFixtureTestCase {
   @Rule public TempDirectory tempDir = new TempDirectory();
 

platform/platform-tests/testSrc/com/intellij/ui/IconsLoadTimePerformanceTest.java

@@ -1,7 +1,6 @@
 // Copyright 2000-2023 JetBrains s.r.o. and contributors. Use of this source code is governed by the Apache 2.0 license.
 package com.intellij.ui;
 
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.internal.IconsLoadTime;
 import com.intellij.internal.IconsLoadTime.StatData;
 import com.intellij.openapi.diagnostic.Logger;
@@ -30,7 +29,6 @@ import static org.junit.Assume.assumeTrue;
  *
  * @author tav
  */
-@HardwareAgentRequired
 public class IconsLoadTimePerformanceTest {
   private static final Logger LOG = Logger.getInstance(IconsLoadTimePerformanceTest.class);
   private static final int SVG_ICON_AVERAGE_LOAD_TIME_EXPECTED_NO_CACHE = 100; // ms

platform/platform-tests/testSrc/com/intellij/util/io/PersistentMapPerformanceTest.java

@@ -1,7 +1,6 @@
 // Copyright 2000-2023 JetBrains s.r.o. and contributors. Use of this source code is governed by the Apache 2.0 license.
 package com.intellij.util.io;
 
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.openapi.util.io.FileUtil;
 import com.intellij.openapi.util.text.StringUtil;
 import com.intellij.tools.ide.metrics.benchmark.Benchmark;
@@ -22,7 +21,6 @@ import java.util.concurrent.atomic.AtomicInteger;
  * @author Eugene Zhuravlev
  */
 @SkipSlowTestLocally
-@HardwareAgentRequired
 public class PersistentMapPerformanceTest extends PersistentMapTestBase {
   interface MapConstructor<T, T2> {
     PersistentHashMap<T, T2> createMap(File file) throws IOException;

platform/testFramework/core/src/com/intellij/TestCaseLoader.java

@@ -2,7 +2,6 @@
 package com.intellij;
 
 import com.intellij.idea.ExcludeFromTestDiscovery;
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.idea.IJIgnore;
 import com.intellij.idea.IgnoreJUnit3;
 import com.intellij.nastradamus.NastradamusClient;
@@ -371,9 +370,6 @@ public class TestCaseLoader {
 
     if (checkForExclusion) {
       if (shouldExcludeTestClass(moduleName, testCaseClass)) return false;
-
-      boolean isHardwareAgentRequired = getAnnotationInHierarchy(testCaseClass, HardwareAgentRequired.class) != null;
-      if (isHardwareAgentRequired != HARDWARE_AGENT_REQUIRED) return false;
     }
 
     if (TestCase.class.isAssignableFrom(testCaseClass) || TestSuite.class.isAssignableFrom(testCaseClass)) {

platform/testFramework/core/src/com/intellij/idea/HardwareAgentRequired.java

@@ -1,14 +0,0 @@
-// Copyright 2000-2021 JetBrains s.r.o. and contributors. Use of this source code is governed by the Apache 2.0 license that can be found in the LICENSE file.
-package com.intellij.idea;
-
-import com.intellij.idea.extensions.HardwareAgentRequiredExecutionCondition;
-import org.junit.jupiter.api.extension.ExtendWith;
-
-import java.lang.annotation.*;
-
-@Retention(RetentionPolicy.RUNTIME)
-@Target(ElementType.TYPE)
-@Inherited
-@ExtendWith(HardwareAgentRequiredExecutionCondition.class)
-public @interface HardwareAgentRequired {
-}

platform/testFramework/core/src/com/intellij/idea/extensions/HardwareAgentRequiredExecutionCondition.java

@@ -1,30 +0,0 @@
-// Copyright 2000-2021 JetBrains s.r.o. and contributors. Use of this source code is governed by the Apache 2.0 license that can be found in the LICENSE file.
-package com.intellij.idea.extensions;
-
-import com.intellij.TestCaseLoader;
-import com.intellij.idea.HardwareAgentRequired;
-import org.junit.jupiter.api.extension.ConditionEvaluationResult;
-import org.junit.jupiter.api.extension.ExecutionCondition;
-import org.junit.jupiter.api.extension.ExtensionContext;
-import org.junit.platform.commons.support.AnnotationSupport;
-
-public class HardwareAgentRequiredExecutionCondition implements ExecutionCondition {
-
-  private static final boolean RUN_WITH_HARDWARE_REQUIREMENT = Boolean.getBoolean(TestCaseLoader.HARDWARE_AGENT_REQUIRED_FLAG);
-  private static final boolean IS_LOCAL_RUN = System.getenv("TEAMCITY_VERSION") == null;
-
-  private static final ConditionEvaluationResult ENABLED_LOCALLY = ConditionEvaluationResult.enabled("Enabled locally");
-  private static final ConditionEvaluationResult MET_HARDWARE_REQUIREMENT = ConditionEvaluationResult.enabled("Hardware requirement met");
-  private static final ConditionEvaluationResult UNMET_HARDWARE_REQUIREMENT = ConditionEvaluationResult.disabled("Unmet hardware requirement");
-
-  @Override
-  public ConditionEvaluationResult evaluateExecutionCondition(ExtensionContext context) {
-    if (IS_LOCAL_RUN) {
-      return ENABLED_LOCALLY;
-    }
-    boolean hasAnnotation = AnnotationSupport.findAnnotation(context.getTestClass(), HardwareAgentRequired.class).isPresent();
-    return hasAnnotation == RUN_WITH_HARDWARE_REQUIREMENT
-           ? MET_HARDWARE_REQUIREMENT 
-           : UNMET_HARDWARE_REQUIREMENT;
-  }
-}

plugins/ant/tests/src/com/intellij/lang/ant/AntHighlightingPerformanceTest.java

@@ -3,7 +3,6 @@ package com.intellij.lang.ant;
 
 import com.intellij.codeInsight.daemon.DaemonAnalyzerTestCase;
 import com.intellij.codeInsight.daemon.impl.HighlightInfo;
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.openapi.application.PluginPathManager;
 import com.intellij.testFramework.ExpectedHighlightingData;
 import com.intellij.tools.ide.metrics.benchmark.Benchmark;
@@ -12,7 +11,6 @@ import org.jetbrains.annotations.NotNull;
 import java.io.IOException;
 import java.util.Collection;
 
-@HardwareAgentRequired
 public class AntHighlightingPerformanceTest extends DaemonAnalyzerTestCase {
   @NotNull
   @Override

plugins/java-i18n/testSrc/com/intellij/lang/properties/PropertiesPerformanceTest.java

@@ -2,7 +2,6 @@
 package com.intellij.lang.properties;
 
 import com.intellij.codeInsight.JavaCodeInsightTestCase;
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.openapi.application.PluginPathManager;
 import com.intellij.openapi.module.Module;
 import com.intellij.openapi.roots.ModuleRootManager;
@@ -19,7 +18,6 @@ import java.io.File;
 import java.io.FileWriter;
 import java.io.IOException;
 
-@HardwareAgentRequired
 public class PropertiesPerformanceTest extends JavaCodeInsightTestCase {
   @Override
   protected void setUp() throws Exception {

xml/dom-tests/tests/com/intellij/util/xml/DomPerformanceTest.java

@@ -15,7 +15,6 @@
  */
 package com.intellij.util.xml;
 
-import com.intellij.idea.HardwareAgentRequired;
 import com.intellij.openapi.application.ApplicationManager;
 import com.intellij.openapi.command.WriteCommandAction;
 import com.intellij.openapi.module.Module;
@@ -32,7 +31,6 @@ import org.jetbrains.annotations.NotNull;
 import java.io.IOException;
 import java.util.List;
 
-@HardwareAgentRequired
 public class DomPerformanceTest extends DomHardCoreTestCase {
   public void testVisitorPerformance() {
     Ref<MyElement> ref = new Ref<>();