1062579 - get_symbol() can return an empty job symbol for mochitests with no part number

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P3)

Description

[Ed Morley [:emorley]]

The fix for bug 1061758 had to wait until the fix for bug 1046743 landed, since get_symbol() doesn't protect against returning an empty symbol in certain cases.

Now that bug 1046743 has landed, we don't have to hardcode the name of each job type that has multiple parts (eg jobfoo-1, jobfoo-2), so as long as the naming convention isn't changed, we're less likely to hit this case. 

However, we may still hit it if a new mochitest job were to be added on a say a new platform, that doesn't have multiple parts.

eg: Android 9.0 mozilla-central opt test mochitest"

Since this would get identified as a mochitest, it hits:

    # Mochitests, Mochitest-e10s and Mochitest OOP are the only ones
    # that display as just a number (with no letters)
    if s in ["M", "M-e10s", "M-oop"]:
        s = ""

But with no part number we end up with no symbol:

    n = ""
    nummatch = NUMBER_RE.match(bn)
    if nummatch:
        n = nummatch.group(1)
    return "{0}{1}".format(s, n)

Comment 1

[Ed Morley [:emorley]]

Attachment: Ensure get_symbol() cannot return an empty job symbol

In the case where 'n' wasn't found for jobs whose symbol letter was
suppressed (eg: a mochitest job that isn't split into multiple parts),
get_symbol() could have previously returned an empty symbol.
The suppression now only takes place if 'n' was found.

Comment 2

[Ed Morley [:emorley]]

https://github.com/mozilla/treeherder-service/commit/554b233fb3d9e3fe93ff0ecb96ceae6ee5151257