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

Optional parameter is not initialized properly in Eclipse - not all <parameter> tags are being added to testng-customsuite.xml #455

Closed
yherasym opened this issue Oct 21, 2013 · 10 comments

Comments

@yherasym
Copy link

Eclipse Juno Service Realease 2
TestNG plugin 6.8.6.20130914_0724

Preconditions: a project has two testng xml files; a test-class uses parameters with Optional default values (simple project has been attached);
Scenario:

  1. Run TestNG test by Right mouse click on test-class (see attachment YH-246
    yh-246
    );
    As result Eclipse will ask which testng xml has to be used as expected (see attachment YH-247
    yh-247
    );
  2. Select some file.
    ActualResult: all optional parameters are initialized with "not-found" values;
    see YH-248
    yh-248

ExpectedResult: the optional parameters are initialized with their specified default values;

@yherasym
Copy link
Author

Cannot attach zip file. Providing testng file by it screenshot

@yherasym
Copy link
Author

yh-250

@yherasym
Copy link
Author

I have found a root of the problem: temporary testng-customsuite.xml is not forming correctly in part of tag;
In my case the file has been formed as:

------------------------

but my original is

-----------------------------------------

As you can see that only first tag has been added to the temporary xml file;
ExpecteResult: all tags have to be in the temporary xml file.

If it is not testng-eclipse plugin issue - pls state and I will move the issue against Eclipse.

@niranjankhare
Copy link

I am also facing this issue. has this issue been acknowledged? Any updates on this?

@bryankindleysonos
Copy link

@yherasym comment is spot on. You can see the testng-customsuite.xml being used will not contain all the params from the xml you select from the pop up. Selecting a template XML will get around this, but that is not as convenient as selecting from the pop up. I would think ALL params from the xml you select should make it to the testng-customsuite.xml which would solve this problem.

@ahoehma
Copy link

ahoehma commented Aug 12, 2016

I confirm this problem too.

@juherr
Copy link
Member

juherr commented Aug 12, 2016

Ping @missedone

@juherr
Copy link
Member

juherr commented Aug 17, 2016

Fixed by testng-team/testng-eclipse/pull/275

@juherr juherr closed this as completed Aug 17, 2016
@missedone
Copy link
Contributor

hi guys,
this is fixed in latest beta of testng eclipse plugin, you can install from the beta update site: http://testng.org/eclipse-beta, please feedback me how is this doing for you.
thanks

@hasaziz
Copy link

hasaziz commented Aug 17, 2016

thanks for fixing this bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants