testng-eclipse icon indicating copy to clipboard operation
testng-eclipse copied to clipboard

need to add more test cases for regression and to improve the coverage

Open missedone opened this issue 9 years ago • 3 comments

for now there is very few unit test for plugin. we need add more for the core plugin and m2e integration.

test cases for launching relate function might he good start:

  • [ ] launching with different message protocol.
  • [ ] test cases for covering the m2e integration
    • [ ] launch a plain java project (with TestNG Library container, not maven managed)
    • [ ] launch a maven base java project with properties var in maven-surefire-plugin configuration
    • [ ] launch a maven base java project with maven-surefire-plugin in specific maven profile

with this test cases available, it can be used as a reference/template if others will contribute extension like gradle integration, etc.

missedone avatar Feb 20 '16 04:02 missedone

well, it's not a trivial task, at the moment, i have no idea how to run the launchConfiguration and verify it in Eclipse Plugin Unit Test framework.

i see there is no any test in junit eclipse plugin at all.

i'll keep this story open and have more time to think about this, and any suggestion or pull requests are welcome:)

link to #229

missedone avatar Feb 21 '16 02:02 missedone

FWIW, now that testng-eclipse uses latest version of tycho, tests can be written and executed with testng itself, which seems appropriate :-)

If you'd like, I can try submitting a PR with a test plugin that will execute such tests at build time.

mbooth101 avatar Mar 16 '17 11:03 mbooth101

@mbooth101, that will be great you can start this. at the first step, I think you only need to add a few critical tests first, for example, launching the TestNG process, etc. Or, you can identify some other case that might be a good example. once this test plugin framework is done, we can add more and more cases later. thx

missedone avatar Mar 16 '17 12:03 missedone