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

Migrate LaunchSuite and SuiteGenerator classes from "org.testng.xml" in testng codebase to here #562

Closed
krmahadevan opened this issue Dec 28, 2023 · 6 comments · Fixed by #564

Comments

@krmahadevan
Copy link
Member

The below two classes reside in TestNG but it is being ONLY used by the eclipse plugin. So it would make sense that we move these classes from the TestNG codebase to the testng eclipse plugin codebase.

Classes in question:

The above classes are being referred to in

  • org.testng.eclipse.launch.TestNGLaunchConfigurationDelegate
  • org.testng.eclipse.ui.util.ConfigurationHelper
  • org.testng.eclipse.util.CustomSuite
  • org.testng.eclipse.util.SuiteGenerator

Please confirm once the migration is complete and I will go ahead and remove off the above two classes from the testng codebase.

ping @missedone

@missedone
Copy link
Collaborator

sure, i will raise the PR after the 7.9.0 release

@krmahadevan
Copy link
Member Author

@missedone - Here's the PR that I have raised for TestNG testng-team/testng#3031

krmahadevan added a commit to testng-team/testng that referenced this issue Jan 3, 2024
@lahodaj
Copy link

lahodaj commented Apr 17, 2024

I would like to ask how it was determined these classes are not used anywhere outside of the Eclipse plugin. Because the LaunchSuite and SuiteGenerator are used inside NetBeans:
https://github.com/apache/netbeans/blob/0090cf959e52e236fbba307f4b75e3cd4e91d0be/java/testng/src/org/netbeans/modules/testng/spi/XMLSuiteSupport.java#L52

@juherr
Copy link
Member

juherr commented Apr 17, 2024

@lahodaj Thanks for the alert.
We are not able to know all usages of the api.
About netbeans, we tried to contact them years ago when we released testng-remote.
Are you part of the netbeans team?

@lahodaj
Copy link

lahodaj commented Apr 17, 2024

@juherr

NetBeans is an Apache project now, and I am member of its PMC. Best way to contact the project is to write to dev@netbeans.apache.org.

Regarding testng-remote - I don't think I am an expert on test framework support in NetBeans. I peeked at the repository, but I am not completely clear on what this is doing. Are there some examples? Thanks!

@missedone
Copy link
Collaborator

missedone commented Apr 17, 2024

@lahodaj regards to this particular case, you don't have to learn about testng-remote, instead, you can refer to the PR #564 how the testng-eclipse incorporates with the breaking change in testng itself, that said, simply copy the relate source code to NetBeans (both are apache license)

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.

4 participants