Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8248418: jpackage fails to extract main class and version from app mo…
…dule linked in external runtime

Reviewed-by: herrick, almatvee
  • Loading branch information
Alexey Semenyuk committed Mar 29, 2021
1 parent fd45694 commit 128c0c9
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 23 deletions.
2 changes: 0 additions & 2 deletions test/jdk/ProblemList.txt
Expand Up @@ -837,8 +837,6 @@ jdk/jfr/startupargs/TestStartDuration.java 8214685 windows-

# jdk_jpackage

tools/jpackage/share/jdk/jpackage/tests/ModulePathTest3.java#id0 8248418 generic-all

############################################################################

# Client manual tests
Expand Down
Expand Up @@ -57,18 +57,6 @@
* --jpt-run=jdk.jpackage.tests.ModulePathTest3
*/

/*
* @test
* @summary jpackage for app's module linked in external runtime
* @library ../../../../helpers
* @build jdk.jpackage.test.*
* @modules jdk.jpackage/jdk.jpackage.internal
* @compile ModulePathTest3.java
* @run main/othervm/timeout=360 -Xmx512m jdk.jpackage.test.Main
* --jpt-run=jdk.jpackage.tests.ModulePathTest3
* --jpt-exclude=test8248418
*/

public final class ModulePathTest3 {

public ModulePathTest3(String jlinkOutputSubdir, String runtimeSubdir) {
Expand All @@ -85,15 +73,6 @@ public void test8248254() throws XPathExpressionException, IOException {
testIt("me.mymodule/me.mymodule.Main");
}

/**
* Test case for JDK-8248418.
* App's module with version specified in runtime directory.
*/
@Test
public void test8248418() throws XPathExpressionException, IOException {
testIt("me.mymodule/me.mymodule.Main@3.7");
}

private void testIt(String mainAppDesc) throws XPathExpressionException,
IOException {
final JavaAppDesc appDesc = JavaAppDesc.parse(mainAppDesc);
Expand Down

1 comment on commit 128c0c9

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.