test run fix

Author: csviri

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/InformerWrapper.java

@@ -66,6 +66,9 @@ public void start() throws OperatorException {
       }
       try {
         var start = informer.start();
+        // note that in case we don't put here timeout and stopOnInformerErrorDuringStartup is
+        // false, and there is a rbac issue the get never returns; therefore operator never really
+        // starts
         start.toCompletableFuture().get(configService.cacheSyncTimeout(), TimeUnit.MILLISECONDS);
       } catch (TimeoutException | ExecutionException e) {
         log.warn("Informer startup error. Informer: {}", informer, e);

operator-framework/src/test/java/io/javaoperatorsdk/operator/InformerRelatedBehaviorTest.java renamed to operator-framework/src/test/java/io/javaoperatorsdk/operator/InformerRelatedBehaviorITS.java

@@ -31,11 +31,10 @@
  * that with periodical watch reconnect the permission is tested again.
  * </p>
  * <p>
- * The test does not end with "IT" since that did not work with tags.
+ * The test ends with "ITS" (Special) since it needs to run separately from other ITs
  * </p>
  */
-@Tag("WatchPermissionAwareTest")
-class InformerRelatedBehaviorTest {
+class InformerRelatedBehaviorITS {
 
   public static final String TEST_RESOURCE_NAME = "test1";
 

pom.xml

@@ -373,7 +373,6 @@
                                 <exclude>**/*Test.java</exclude>
                                 <exclude>**/*E2E.java</exclude>
                             </excludes>
-                            <excludedGroups>WatchPermissionAwareTest</excludedGroups>
                         </configuration>
                     </plugin>
                 </plugins>
@@ -388,7 +387,14 @@
                         <groupId>org.apache.maven.plugins</groupId>
                         <artifactId>maven-surefire-plugin</artifactId>
                         <configuration>
-                            <groups>WatchPermissionAwareTest</groups>
+                            <includes>
+                                <include>**/*ITS.java</include>
+                            </includes>
+                            <excludes>
+                                <exclude>**/*Test.java</exclude>
+                                <exclude>**/*E2E.java</exclude>
+                                <exclude>**/*IT.java</exclude>
+                            </excludes>
                         </configuration>
                     </plugin>
                 </plugins>
@@ -418,14 +424,14 @@
             <id>release</id>
             <build>
                 <plugins>
-
                     <plugin>
                         <groupId>org.apache.maven.plugins</groupId>
                         <artifactId>maven-surefire-plugin</artifactId>
                         <configuration>
                             <excludes>
                                 <exclude>**/*IT.java</exclude>
                                 <exclude>**/*E2E.java</exclude>
+                                <exclude>**/InformerRelatedBehaviorTest.java</exclude>
                             </excludes>
                         </configuration>
                     </plugin>