1010401 - Make TBPL recognize OOP b2gdesktop tests

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Jonathan Griffin (:jgriffin)]

TBPL should distinguish b2gdesktop tests run in OOP mode; currently they appear as duplicate letters on cedar (e.g., G G, one for "regular" G and the other for OOP G).

I propose to group these using "oop", so TBPL will show something like:

  M(1) R G oop(M1, R, G)

etc.

Comment 1

[Jonathan Griffin (:jgriffin)]

Attachment: oop.diff

Looks ok on a local vagrant box with the tests already running on cedar.

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8422689 [details] [diff] [review]
oop.diff

Let's do this like we're doing the desktop e10s tests (M-e10s, etc). I'm not crazy about lumping all the different test types into one grouping as it would break with all existing conventions.

Comment 3

[Jonathan Griffin (:jgriffin)]

Attachment: Make TBPL distinguish OOP tests,

I think this should do what you want.

Comment 4

[Jonathan Griffin (:jgriffin)]

Attachment: Make TBPL distinguish OOP tests,

Oops, missed a line in the previous patch.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8424238 [details] [diff] [review]
Make TBPL distinguish OOP tests,

Pushed with some minor tweaks:
- Adjusted for the reftest job being renamed to reftest-sanity
- Tweak the UI so that M-oop displays with just the chunk number.

https://hg.mozilla.org/webtools/tbpl/rev/718d944e6511

I want to clean some of this up, but that's fodder for another bug. I'll check tbpl-dev later tonight to make sure everything looks OK and push it to production.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

In production :)