Fix FileAccessTreeTests#testDuplicateExclusivePaths to work on windows

libs/entitlement/src/test/java/org/elasticsearch/entitlement/runtime/policy/FileAccessTreeTests.java

@@ -365,17 +365,17 @@ public void testDuplicatePrunedPaths() {
 
     public void testDuplicateExclusivePaths() {
         // Bunch o' handy definitions
-        var originalFileData = FileData.ofPath(path("/a/b"), READ).withExclusive(true);
-        var fileDataWithWriteMode = FileData.ofPath(path("/a/b"), READ_WRITE).withExclusive(true);
+        var pathAB = path("/a/b");
+        var pathCD = path("/c/d");
+        var originalFileData = FileData.ofPath(pathAB, READ).withExclusive(true);
+        var fileDataWithWriteMode = FileData.ofPath(pathAB, READ_WRITE).withExclusive(true);
         var original = new ExclusiveFileEntitlement("component1", "module1", new FilesEntitlement(List.of(originalFileData)));
         var differentComponent = new ExclusiveFileEntitlement("component2", original.moduleName(), original.filesEntitlement());
         var differentModule = new ExclusiveFileEntitlement(original.componentName(), "module2", original.filesEntitlement());
         var differentPath = new ExclusiveFileEntitlement(
             original.componentName(),
             original.moduleName(),
-            new FilesEntitlement(
-                List.of(FileData.ofPath(path("/c/d"), originalFileData.mode()).withExclusive(originalFileData.exclusive()))
-            )
+            new FilesEntitlement(List.of(FileData.ofPath(pathCD, originalFileData.mode()).withExclusive(originalFileData.exclusive())))
         );
         var differentMode = new ExclusiveFileEntitlement(
             original.componentName(),
@@ -387,7 +387,7 @@ public void testDuplicateExclusivePaths() {
             original.moduleName(),
             new FilesEntitlement(List.of(originalFileData.withPlatform(WINDOWS)))
         );
-        var originalExclusivePath = new ExclusivePath("component1", Set.of("module1"), normalizePath(path("/a/b")));
+        var originalExclusivePath = new ExclusivePath("component1", Set.of("module1"), normalizePath(pathAB));
 
         // Some basic tests
 
@@ -409,12 +409,17 @@ public void testDuplicateExclusivePaths() {
             originalExclusivePath,
             new ExclusivePath("component2", Set.of(original.moduleName()), originalExclusivePath.path()),
             new ExclusivePath(original.componentName(), Set.of("module2"), originalExclusivePath.path()),
-            new ExclusivePath(original.componentName(), Set.of(original.moduleName()), normalizePath(path("/c/d")))
+            new ExclusivePath(original.componentName(), Set.of(original.moduleName()), normalizePath(pathCD))
         );
         var iae = expectThrows(IllegalArgumentException.class, () -> buildExclusivePathList(distinctEntitlements, TEST_PATH_LOOKUP));
+        var pathABString = pathAB.toAbsolutePath().toString();
         assertThat(
             iae.getMessage(),
-            equalTo("Path [/a/b] is already exclusive to [component1][module1], cannot add exclusive access for [component2][module1]")
+            equalTo(
+                "Path ["
+                    + pathABString
+                    + "] is already exclusive to [component1][module1], cannot add exclusive access for [component2][module1]"
+            )
         );
 
         var equivalentEntitlements = List.of(original, differentMode, differentPlatform);

libs/entitlement/src/test/java/org/elasticsearch/entitlement/runtime/policy/PolicyManagerTests.java

@@ -447,7 +447,9 @@ public void testDuplicateEntitlements() {
     public void testFilesEntitlementsWithExclusive() {
         var baseTestPath = Path.of("/base").toAbsolutePath();
         var testPath1 = Path.of("/base/test").toAbsolutePath();
+        var testPath1String = testPath1.toString();
         var testPath2 = Path.of("/base/test/foo").toAbsolutePath();
+        var testPath2String = testPath2.toAbsolutePath();
         var iae = expectThrows(
             IllegalArgumentException.class,
             () -> new PolicyManager(
@@ -494,7 +496,9 @@ public void testFilesEntitlementsWithExclusive() {
         assertThat(
             iae.getMessage(),
             equalTo(
-                "Path [/base/test] is already exclusive to [plugin1][test.module1],"
+                "Path ["
+                    + testPath1String
+                    + "] is already exclusive to [plugin1][test.module1],"
                     + " cannot add exclusive access for [plugin2][test.module2]"
             )
         );
@@ -547,8 +551,8 @@ public void testFilesEntitlementsWithExclusive() {
             Strings.format(
                 "duplicate/overlapping exclusive paths found in files entitlements: "
                     + "[[plugin1] [test] [%s]] and [[(server)] [test] [%s]]",
-                testPath1,
-                testPath2
+                testPath1String,
+                testPath2String
             ),
             iae.getMessage()
         );

muted-tests.yml

@@ -360,12 +360,6 @@ tests:
 - class: org.elasticsearch.env.NodeEnvironmentTests
   method: testIndexCompatibilityChecks
   issue: https://github.com/elastic/elasticsearch/issues/124388
-- class: org.elasticsearch.entitlement.runtime.policy.PolicyManagerTests
-  method: testFilesEntitlementsWithExclusive
-  issue: https://github.com/elastic/elasticsearch/issues/124420
-- class: org.elasticsearch.entitlement.runtime.policy.FileAccessTreeTests
-  method: testDuplicateExclusivePaths
-  issue: https://github.com/elastic/elasticsearch/issues/124437
 - class: org.elasticsearch.xpack.restart.FullClusterRestartIT
   method: testWatcherWithApiKey {cluster=UPGRADED}
   issue: https://github.com/elastic/elasticsearch/issues/124159