This is an automated email from the ASF dual-hosted git repository.
ibessonov pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git
The following commit(s) were added to refs/heads/main by this push:
new 41007ff42d IGNITE-22395 Remove assertions for paranoid leak detection
property (#3864)
41007ff42d is described below
commit 41007ff42df58e9651aeb3f9e0e520a27d5ff093
Author: Phillippko <[email protected]>
AuthorDate: Mon Jun 3 17:05:50 2024 +0400
IGNITE-22395 Remove assertions for paranoid leak detection property (#3864)
---
.../cli/ParanoidLeakDetectionLevelTest.java | 33 ----------------------
.../ParanoidLeakDetectionLevelPropertyTest.java | 31 --------------------
.../network/ParanoidLeakDetectionLevelTest.java | 33 ----------------------
.../ignite/internal/IgniteIntegrationTest.java | 8 ------
4 files changed, 105 deletions(-)
diff --git
a/modules/cli/src/test/java/org/apache/ignite/internal/cli/ParanoidLeakDetectionLevelTest.java
b/modules/cli/src/test/java/org/apache/ignite/internal/cli/ParanoidLeakDetectionLevelTest.java
deleted file mode 100644
index 68770a9db0..0000000000
---
a/modules/cli/src/test/java/org/apache/ignite/internal/cli/ParanoidLeakDetectionLevelTest.java
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You 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 org.apache.ignite.internal.cli;
-
-import static io.netty.util.ResourceLeakDetector.Level.PARANOID;
-import static org.hamcrest.MatcherAssert.assertThat;
-import static org.hamcrest.Matchers.is;
-
-import io.netty.util.ResourceLeakDetector;
-import org.junit.jupiter.api.Test;
-
-/** Test checking that leak detection level is set to PARANOID for CLI unit
tests. */
-public class ParanoidLeakDetectionLevelTest {
- @Test
- public void testParanoidLeakDetectionLevel() {
- assertThat(ResourceLeakDetector.getLevel(), is(PARANOID));
- }
-}
diff --git
a/modules/jdbc/src/test/java/org/apache/ignite/internal/jdbc/ParanoidLeakDetectionLevelPropertyTest.java
b/modules/jdbc/src/test/java/org/apache/ignite/internal/jdbc/ParanoidLeakDetectionLevelPropertyTest.java
deleted file mode 100644
index 1abab7214e..0000000000
---
a/modules/jdbc/src/test/java/org/apache/ignite/internal/jdbc/ParanoidLeakDetectionLevelPropertyTest.java
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You 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 org.apache.ignite.internal.jdbc;
-
-import static org.hamcrest.MatcherAssert.assertThat;
-import static org.hamcrest.Matchers.is;
-
-import org.junit.jupiter.api.Test;
-
-/** Test checking that leak detection level property is set to PARANOID for
JDBC unit tests. */
-public class ParanoidLeakDetectionLevelPropertyTest {
- @Test
- public void testParanoidLeakDetectionLevelProperty() {
- assertThat(System.getProperty("io.netty.leakDetectionLevel"),
is("paranoid"));
- }
-}
diff --git
a/modules/network/src/test/java/org/apache/ignite/internal/network/ParanoidLeakDetectionLevelTest.java
b/modules/network/src/test/java/org/apache/ignite/internal/network/ParanoidLeakDetectionLevelTest.java
deleted file mode 100644
index 3f7d1046c1..0000000000
---
a/modules/network/src/test/java/org/apache/ignite/internal/network/ParanoidLeakDetectionLevelTest.java
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You 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 org.apache.ignite.internal.network;
-
-import static io.netty.util.ResourceLeakDetector.Level.PARANOID;
-import static org.hamcrest.MatcherAssert.assertThat;
-import static org.hamcrest.Matchers.is;
-
-import io.netty.util.ResourceLeakDetector;
-import org.junit.jupiter.api.Test;
-
-/** Test checking that leak detection level is set to PARANOID for network
unit tests. */
-public class ParanoidLeakDetectionLevelTest {
- @Test
- public void testParanoidLeakDetectionLevel() {
- assertThat(ResourceLeakDetector.getLevel(), is(PARANOID));
- }
-}
diff --git
a/modules/runner/src/testFixtures/java/org/apache/ignite/internal/IgniteIntegrationTest.java
b/modules/runner/src/testFixtures/java/org/apache/ignite/internal/IgniteIntegrationTest.java
index 4e1d5f8dea..9fa5710043 100644
---
a/modules/runner/src/testFixtures/java/org/apache/ignite/internal/IgniteIntegrationTest.java
+++
b/modules/runner/src/testFixtures/java/org/apache/ignite/internal/IgniteIntegrationTest.java
@@ -17,13 +17,9 @@
package org.apache.ignite.internal;
-import static org.hamcrest.MatcherAssert.assertThat;
-import static org.hamcrest.Matchers.is;
-
import org.apache.ignite.internal.junit.StopAllIgnitesAfterTests;
import org.apache.ignite.internal.testframework.BaseIgniteAbstractTest;
import org.apache.ignite.internal.testframework.WorkDirectoryExtension;
-import org.junit.jupiter.api.BeforeAll;
import org.junit.jupiter.api.extension.ExtendWith;
/**
@@ -32,8 +28,4 @@ import org.junit.jupiter.api.extension.ExtendWith;
// The order is important here.
@ExtendWith({WorkDirectoryExtension.class, StopAllIgnitesAfterTests.class})
public abstract class IgniteIntegrationTest extends BaseIgniteAbstractTest {
- @BeforeAll
- public static void assertParanoidLeakDetectionProperty() {
- assertThat(System.getProperty("io.netty.leakDetectionLevel"),
is("paranoid"));
- }
}