improve: moving some integration test into packages, some @sample cleanup (#3119)

Signed-off-by: Attila Mészáros <[email protected]>

Author: csviri

operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/ConcurrencyIT.java

@@ -25,7 +25,6 @@
 import org.slf4j.LoggerFactory;
 
 import io.fabric8.kubernetes.api.model.ConfigMap;
-import io.javaoperatorsdk.annotation.Sample;
 import io.javaoperatorsdk.operator.baseapi.simple.TestCustomResource;
 import io.javaoperatorsdk.operator.baseapi.simple.TestReconciler;
 import io.javaoperatorsdk.operator.junit.LocallyRunOperatorExtension;
@@ -34,15 +33,6 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.awaitility.Awaitility.await;
 
-@Sample(
-    tldr = "Concurrent Reconciliation of Multiple Resources",
-    description =
-        """
-        Demonstrates the operator's ability to handle concurrent reconciliation of multiple \
-        resources. The test creates, updates, and deletes many resources simultaneously to \
-        verify proper handling of concurrent operations, ensuring thread safety and correct \
-        resource state management under load.
-        """)
 class ConcurrencyIT {
   public static final int NUMBER_OF_RESOURCES_CREATED = 50;
   public static final int NUMBER_OF_RESOURCES_DELETED = 30;

operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/StopWithoutStartIT.java

@@ -19,22 +19,12 @@
 import org.junit.jupiter.api.Timeout;
 
 import io.fabric8.kubernetes.api.model.ConfigMap;
-import io.javaoperatorsdk.annotation.Sample;
 import io.javaoperatorsdk.operator.Operator;
 import io.javaoperatorsdk.operator.api.reconciler.Context;
 import io.javaoperatorsdk.operator.api.reconciler.ControllerConfiguration;
 import io.javaoperatorsdk.operator.api.reconciler.Reconciler;
 import io.javaoperatorsdk.operator.api.reconciler.UpdateControl;
 
-@Sample(
-    tldr = "Stop Operator Without Starting",
-    description =
-        """
-        Demonstrates that an operator can be stopped without being started. This is important \
-        for cleanup scenarios where an operator is created and registered with reconcilers but \
-        never started due to initialization failures or other conditions. The stop() method \
-        properly cleans up thread pools even when the operator was never started.
-        """)
 class StopWithoutStartIT {
 
   @Test

…baseapi/InformerErrorHandlerStartIT.java …rterror/InformerErrorHandlerStartIT.javaoperator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/InformerErrorHandlerStartIT.java renamed to operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/informerstarterror/InformerErrorHandlerStartIT.java operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/InformerErrorHandlerStartIT.java renamed to operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/informerstarterror/InformerErrorHandlerStartIT.java

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package io.javaoperatorsdk.operator.baseapi;
+package io.javaoperatorsdk.operator.baseapi.informerstarterror;
 
 import java.time.Duration;
 

…/baseapi/LeaderElectionPermissionIT.java …rmission/LeaderElectionPermissionIT.javaoperator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/LeaderElectionPermissionIT.java renamed to operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/leaderelectionpermission/LeaderElectionPermissionIT.java operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/LeaderElectionPermissionIT.java renamed to operator-framework/src/test/java/io/javaoperatorsdk/operator/baseapi/leaderelectionpermission/LeaderElectionPermissionIT.java

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package io.javaoperatorsdk.operator.baseapi;
+package io.javaoperatorsdk.operator.baseapi.leaderelectionpermission;
 
 import org.junit.jupiter.api.Test;