Skip to content

Commit 7f45bdd

Browse files
committed
fix getLibrariesLocations on unix/linux platforms
1 parent 77fb2c6 commit 7f45bdd

File tree

2 files changed

+22
-4
lines changed

2 files changed

+22
-4
lines changed

net.lecousin.core/src/main/java/net/lecousin/framework/application/libraries/DefaultLibrariesManager.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ public Library getLibrary(String groupId, String artifactId) {
156156

157157
@Override
158158
public List<File> getLibrariesLocations() {
159-
String[] paths = System.getProperty("java.class.path").split(";");
159+
String[] paths = System.getProperty("java.class.path").split(System.getProperty("path.separator"));
160160
ArrayList<File> list = new ArrayList<>(paths.length);
161161
for (String path : paths) {
162162
path = path.trim();

net.lecousin.core/src/test/java/net/lecousin/framework/core/tests/application/TestApplication.java

Lines changed: 21 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,18 @@
11
package net.lecousin.framework.core.tests.application;
22

3+
import java.io.File;
4+
import java.util.List;
35
import java.util.Locale;
46

5-
import org.junit.Assert;
6-
import org.junit.Test;
7-
87
import net.lecousin.framework.application.Application;
98
import net.lecousin.framework.application.LCCore;
109
import net.lecousin.framework.concurrent.Task;
1110
import net.lecousin.framework.core.test.LCCoreAbstractTest;
1211
import net.lecousin.framework.io.IO;
1312

13+
import org.junit.Assert;
14+
import org.junit.Test;
15+
1416
public class TestApplication extends LCCoreAbstractTest {
1517

1618
@Test
@@ -46,4 +48,20 @@ public void testResource() throws Exception {
4648
Assert.assertTrue(null == io);
4749
}
4850

51+
@Test
52+
public void testLibrariesLocations() {
53+
List<File> list = LCCore.getApplication().getLibrariesManager().getLibrariesLocations();
54+
Assert.assertNotEquals(0, list.size());
55+
boolean found = false;
56+
for (File f : list) {
57+
if (!f.isDirectory()) continue;
58+
File dir = new File(f, getClass().getPackage().getName().replace('.', '/'));
59+
if (!dir.exists()) continue;
60+
if (new File(dir, getClass().getSimpleName() + ".class").exists()) {
61+
found = true;
62+
break;
63+
}
64+
}
65+
Assert.assertTrue(found);
66+
}
4967
}

0 commit comments

Comments
 (0)