[Junit-trackers] [ junit-Feature Requests-1742040 ] Parameterized runner should include data in test names

[SourceForge.net <no...@so...>]

Feature Requests item #1742040, was opened at 2007-06-23 05:51
Message generated for change (Comment added) made by dasblinkenlight
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=365278&aid=1742040&group_id=15278

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Anli Shundi (anshnd)
Assigned to: Nobody/Anonymous (nobody)
Summary: Parameterized runner should include data in test names

Initial Comment:
Right now the test names are determined as follows in Parameterized:

@Override
protected String getName() {
 return String.format("[%s]", fParameterSetNumber);
}
		
@Override
protected String testName(final Method method) {
  return String.format("%s[%s]", method.getName(), fParameterSetNumber);
}

It would help of the value of the first parameter was included in the name.  Maybe Parameterized can be parameterized to contain the format for names?  I can submit a patch if needed.

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

Comment By: dasblinkenlight (dasblinkenlight)
Date: 2007-07-13 14:54

Message:
Logged In: YES 
user_id=1827208
Originator: NO

> It would help of the value of the first parameter was included in the
name.

I think that using the first parameter
(A) is too limiting because it may not be a meaningful string, and
(B) is not backward-compatible, because users will see "junk" names when
their first argument is a class that does not override Object#toString.

A better solution would be to have the TestClassRunnerForParameters call
the method with @TestName annotation in the test class, if one is provided,
and use the [%s] name by default. This way the users will have full control
over the generation of test names, and the results will be 100%
backward-compatible because @TestName annotation class does not exist yet.

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

Comment By: David Saff (dsaff)
Date: 2007-07-09 12:05

Message:
Logged In: YES 
user_id=325156
Originator: NO

Moved to feature requests.

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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=365278&aid=1742040&group_id=15278