Displaying the descriptions in the test list seems to be a missing feature and would be handy so future version of the testbot could display the descriptions.

CommentFileSizeAuthor
#6 drupal-test_list_description-939550-6.patch1.33 KBmbrett5062
#1 939550-list-description.patch2.63 KBboombatower

Comments

boombatower’s picture

boombatower commented
Assigned: Unassigned » boombatower
Status: Active » Needs review
StatusFileSize
new 939550-list-description.patch2.63 KB

--list

Actions
 - Actions executing in a potentially infinite loop (ActionLoopTestCase)
 - Actions configuration (ActionsConfigurationTestCase)
Aggregator
 - Add feed functionality (AddFeedTestCase)
 - Update on cron functionality (AggregatorCronTestCase)
 - Checks display of aggregator items (AggregatorRenderingTestCase)
 - Categorize feed item functionality (CategorizeFeedItemTestCase)
 - Categorize feed functionality (CategorizeFeedTestCase)
 - Feed parser functionality (FeedParserTestCase)
 - Import feeds from OPML functionality (ImportOPMLTestCase)
 - Remove feed item functionality (RemoveFeedItemTestCase)
 - Remove feed functionality (RemoveFeedTestCase)
 - Update feed item functionality (UpdateFeedItemTestCase)
 - Update feed functionality (UpdateFeedTestCase)

--list-description

Available test groups & classes
-------------------------------

Actions
 - Actions executing in a potentially infinite loop (ActionLoopTestCase) - Tests actions executing in a loop, and makes sure they abort properly.
 - Actions configuration (ActionsConfigurationTestCase) - Tests complex actions configuration by adding, editing, and deleting a complex action.
Aggregator
 - Add feed functionality (AddFeedTestCase) - Add feed test.
 - Update on cron functionality (AggregatorCronTestCase) - Update feeds on cron.
 - Checks display of aggregator items (AggregatorRenderingTestCase) - Checks display of aggregator items on the page.
 - Categorize feed item functionality (CategorizeFeedItemTestCase) - Test feed item categorization.
 - Categorize feed functionality (CategorizeFeedTestCase) - Categorize feed test.
 - Feed parser functionality (FeedParserTestCase) - Test the built-in feed parser with valid feed samples.
 - Import feeds from OPML functionality (ImportOPMLTestCase) - Test OPML import.
 - Remove feed item functionality (RemoveFeedItemTestCase) - Remove feed items from a feed.
 - Remove feed functionality (RemoveFeedTestCase) - Remove feed test.
 - Update feed item functionality (UpdateFeedItemTestCase) - Update feed items from a feed.
 - Update feed functionality (UpdateFeedTestCase) - Update feed test.
boombatower’s picture

boombatower commented
Version: 7.x-dev » 8.x-dev
mbrett5062’s picture

mbrett5062 commented

Do we still want to do this. If so, I can re-roll this against 8.x and get it moving again. Just assign to me if you would still like this in. If not we should close it.

cameron tod’s picture

cameron tod commented

I don't see why not, it seems a useful feature and won't have any impact on anything else. You can assign it to yourself if you are happy to work on it.

mbrett5062’s picture

mbrett5062 commented
Assigned: boombatower » mbrett5062
Status: Needs review » Needs work

Re-rolling against latest 8.x, patch to follow.

mbrett5062’s picture

mbrett5062 commented
Status: Needs work » Needs review
StatusFileSize
new drupal-test_list_description-939550-6.patch1.33 KB

Here is patch for 8.x.

mbrett5062’s picture

mbrett5062 commented

Manually tested, works and does what it says on the box. If bot test comes back clean and someone else can review, I think this can be moved to RTBC.

boombatower’s picture

boombatower commented
Status: Needs review » Reviewed & tested by the community

Thanks for re-rolling.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Hm. I don't think this addition makes sense.

By moving to PSR-0 classes, we've essentially turned getInfo() into nonsense. The current test "name" became needless and should be dropped, and the description should take over its place, when/where appropriate.

I've started to investigate that in #697760: Replace getInfo() in tests with native phpDoc + annotations (following PHPUnit) already, but wasn't able to get it done yet.

In essence, the current "list" should output the test class names + description (instead of name, because name ceases to exist).

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Needs review

Looks like this still needs discussion.

mbrett5062’s picture

mbrett5062 commented
Assigned: mbrett5062 » Unassigned
Issue summary: View changes

Sorry, should have dropped this some while ago. I guess a lot has changed since end of 2012, and not able to work on this at the moment. If it's still relevant I hope someone will pick it up.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Closed (duplicate)

Meanwhile, #697760: Replace getInfo() in tests with native phpDoc + annotations (following PHPUnit) is ready for prime time.