Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test container plugin dependencies should be added automatically #79

Closed
kapexx opened this issue May 14, 2024 · 0 comments · Fixed by #82
Closed

Test container plugin dependencies should be added automatically #79

kapexx opened this issue May 14, 2024 · 0 comments · Fixed by #82

Comments

@kapexx
Copy link
Contributor

kapexx commented May 14, 2024

For junit-plugin configurations, the documentation of container says "If the container points to a plugin project, the plugin will be added to the dependency list as well". This partly works, the plugin is added to the dependencies. But that plugin's transitive dependencies are not added automatically. It seems like the container plugin is not part of the automatic dependency resolution.

Example: Assuming we have two plugin projects, org.example.demo.test which depends on org.example.demo. In the following configuration org.example.demo.test is added as dependency because the container points to that plugin project. Launching it will fail because org.example.demo is missing though.

junit-plugin configuration Demo {
    test {
        runner junit5;
        container '/org.example.demo.test';
    }
    application org.eclipse.pde.junit.runtime.coretestapplication;
    plugin slf4j.simple;
}

Only if we add the container plugin redundantly in the dependencies, then org.example.demo is added automatically as expected.

junit-plugin configuration Demo {
    test {
        runner junit5;
        container '/org.example.demo.test';
    }
    application org.eclipse.pde.junit.runtime.coretestapplication;
    plugin slf4j.simple;
    plugin org.example.demo.test;
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant