511929 - [Packaged tests] Xpcshell test suite name: s/xpcshell/xpcshell-tests/g

Status: RESOLVED WONTFIX

(Release Engineering :: General, defect, P5)

Description

[Serge Gautherie (:sgautherie)]

Packaged or not, test suites should have the same name.

Comment 1

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

If I understand correctly, you mean:


on http://tinderbox.mozilla.org/showbuilds.cgi?tree=Firefox, the string displayed is "xpcshell-tests"

on http://tinderbox.mozilla.org/showbuilds.cgi?tree=Firefox-Unittest, the string displayed is "xpcshell"

...and you would like the strings to be identical on the two waterfalls? Is that summary correct?

(fyi: my understanding was that as soon as the unittest-separate-from-build was running smoothly, we'd turn off doing build-and-unittest jobs. This would also solve the string mis-match!)

Comment 2

[Serge Gautherie (:sgautherie)]

(In reply to comment #1)
> If I understand correctly, you mean:

Yes.

> ...and you would like the strings to be identical on the two waterfalls? Is
> that summary correct?

Yes: on waterfall, in the logs, everywhere.

> (fyi: my understanding was that as soon as the unittest-separate-from-build was
> running smoothly, we'd turn off doing build-and-unittest jobs. This would also

(Yes and I would think this could be done yet ... but maybe I'm missing something.)

> solve the string mis-match!)

The mismatch, sure.
Still, I think the intended name is actually 'xpcshell-tests', like the command...

Comment 3

[Chris Cooper [:coop] (he/him)]

Mass move of bugs from Release Engineering:Future -> Release Engineering. See
http://coop.deadsquid.com/2010/02/kiss-the-future-goodbye/ for more details.

Comment 4

[Chris Cooper [:coop] (he/him)]

Serge: is this still an issue?

Comment 5

[Serge Gautherie (:sgautherie)]

(In reply to comment #4)
> Serge: is this still an issue?

Yes, it is.

Comment 6

[[e-mail] (:bhearsum)]

I'm calling this WONTFIX. Non-packaged tests are deprecated. IMHO xpcshell-tests should be the new name (matches other suites better, which also say "test" or "tests" in them). It's not worth renaming the tests on the old branches.