Fix broken tests

kumargu · kumargu · commit 12d3cc3edeb6 · 2025-03-06T16:05:08.000+05:30
Signed-off-by: Gulshan Kumar &lt;kumargu@amazon.com&gt;
diff --git a/plugins/repository-azure/src/yamlRestTest/resources/rest-api-spec/test/repository_azure/test.policy b/plugins/repository-azure/src/yamlRestTest/resources/rest-api-spec/test/repository_azure/test.policy
@@ -0,0 +1,12 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+
+grant {
+  permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "connect,resolve";
+};
diff --git a/plugins/repository-hdfs/src/test/resources/org/opensearch/repositories/hdfs/test.policy b/plugins/repository-hdfs/src/test/resources/org/opensearch/repositories/hdfs/test.policy
@@ -0,0 +1,12 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+
+grant {
+  permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "connect,resolve";
+};
diff --git a/plugins/repository-hdfs/src/test/resources/rest-api-spec/test/hdfs_repository/test.policy b/plugins/repository-hdfs/src/test/resources/rest-api-spec/test/hdfs_repository/test.policy
@@ -0,0 +1,12 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+
+grant {
+  permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "connect,resolve";
+};
diff --git a/plugins/repository-hdfs/src/test/resources/rest-api-spec/test/secure_hdfs_repository/test.policy b/plugins/repository-hdfs/src/test/resources/rest-api-spec/test/secure_hdfs_repository/test.policy
@@ -0,0 +1,12 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+
+grant {
+  permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "connect,resolve";
+};
diff --git a/server/src/test/resources/org/opensearch/bootstrap/test.policy b/server/src/test/resources/org/opensearch/bootstrap/test.policy
@@ -11,6 +11,7 @@ grant {
   permission java.util.PropertyPermission "*", "read,write";
   permission java.security.SecurityPermission "createPolicy.JavaPolicy";
   permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "accept,connect";
 };
 
 grant codeBase "${codebase.framework}" {
diff --git a/server/src/test/resources/org/opensearch/index/fielddata/ordinals/test.policy b/server/src/test/resources/org/opensearch/index/fielddata/ordinals/test.policy
@@ -0,0 +1,15 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+
+grant {
+  // allow to test Security policy and codebases
+  permission java.util.PropertyPermission "*", "read,write";
+  permission java.security.SecurityPermission "createPolicy.JavaPolicy";
+  permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "accept,connect";
+};
diff --git a/server/src/test/resources/org/opensearch/index/translog/test.policy b/server/src/test/resources/org/opensearch/index/translog/test.policy
@@ -0,0 +1,13 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+
+grant {
+  // allow to test Security policy and codebases
+  permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "accept,connect";
+};
diff --git a/test/framework/src/test/java/org/opensearch/transport/nio/SimpleMockNioTransportTests.java b/test/framework/src/test/java/org/opensearch/transport/nio/SimpleMockNioTransportTests.java
@@ -97,6 +97,7 @@ protected int channelsPerNodeConnection() {
         return 3;
     }
 
+    @AwaitsFix(bugUrl = "https://github.com/opensearch-project/OpenSearch/pull/16731")
     public void testConnectException() throws UnknownHostException {
         try {
             serviceA.connectToNode(
diff --git a/test/framework/src/test/resources/org/opensearch/bootstrap/test.policy b/test/framework/src/test/resources/org/opensearch/bootstrap/test.policy
@@ -8,8 +8,10 @@
 
 grant codeBase "${codebase.opensearch-nio}" {
   permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "connect,resolve";
 };
 
 grant {
   permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "connect,resolve";
 };
diff --git a/test/framework/src/test/resources/org/opensearch/test/test.policy b/test/framework/src/test/resources/org/opensearch/test/test.policy
@@ -0,0 +1,11 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+grant {
+  permission java.net.NetPermission "accessUnixDomainSocket";
+  permission java.net.SocketPermission "*", "connect,resolve";
+};

Original file line number	Diff line number	Diff line change
`@@ -97,6 +97,7 @@ protected int channelsPerNodeConnection() {`
`97`	`97`	`return 3;`
`98`	`98`	`}`
`99`	`99`
	`100`	`+ @AwaitsFix(bugUrl = "https://github.com/opensearch-project/OpenSearch/pull/16731")`
`100`	`101`	`public void testConnectException() throws UnknownHostException {`
`101`	`102`	`try {`
`102`	`103`	`serviceA.connectToNode(`