1721101 - Intermittent comm/mail/components/extensions/test/browser/browser_ext_contentScripts.js | backgroundColor - "rgb(0, 255, 0)" == "rgba(0, 0, 0, 0)" - got "rgb(0, 255, 0)", expected "rgba(0, 0, 0, 0)" (operator ==)

Status: RESOLVED FIXED

(Thunderbird :: Add-Ons: Extensions API, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: geoff [at] darktrojan.net
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=345468394&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/KLows4jMTciS8hA_CkFCmA/runs/0/artifacts/public/logs/live_backing.log

---

My new test isn't as reliable as it should be.

Comment 1

[Intermittent Failures Robot]

5 failures in 2224 pushes (0.002 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 5

Platform and build breakdown:

• linux64: 2
  • opt: 2
• linux64-shippable: 3
  • opt: 3

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1721101&startday=2021-07-12&endday=2021-07-18&tree=all

Comment 2

[Intermittent Failures Robot]

24 failures in 2940 pushes (0.008 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 24

Platform and build breakdown:

• linux64-shippable: 13
  • opt: 13
• linux64: 11
  • opt: 11

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1721101&startday=2021-07-19&endday=2021-07-25&tree=all

Comment 3

[Geoff Lankow (:darktrojan)]

The test is wrong. It expects the lack of permissions to stop the script from running, but actually permissions don't do that, and the script usually isn't running because the page is loaded before the extension. Sometimes the page load happens after the extension load and the script runs (because it should), failing the test.

Comment 4

[Geoff Lankow (:darktrojan)]

Also there's a second failure which I think is also caused by the extension loading before the page does. I don't actually understand why, it shouldn't be happening.

Comment 5

[Geoff Lankow (:darktrojan)]

Attachment: Bug 1721101 - Correct behaviour of test browser_ext_contentScripts.js. r=rnons

Depends on D120916

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/88e19d69c40a
Correct behaviour of test browser_ext_contentScripts.js. r=rnons

Comment 7

[Rob Lemley [:rjl]]

Comment on attachment 9233228 [details]
Bug 1721101 - Correct behaviour of test browser_ext_contentScripts.js. r=rnons

[Approval Request Comment]
Low risk, only affects test code. This does show up on beta91 builds.

Comment 8

[Wayne Mery (:wsmwk)]

Comment on attachment 9233228 [details]
Bug 1721101 - Correct behaviour of test browser_ext_contentScripts.js. r=rnons

[Triage Comment]
Approved for beta

Comment 9

[Rob Lemley [:rjl]]

Thunderbird 91.0b5:
https://hg.mozilla.org/releases/comm-beta/rev/709382c6c070

Comment 10

[Geoff Lankow (:darktrojan)]

I didn't request uplift because I'm still seeing a problem similar to this one on comm-central. Not that I'm opposed to it being uplifted or anything. But there may be more.

Comment 11

[Intermittent Failures Robot]

3 failures in 3131 pushes (0.001 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 3

Platform and build breakdown:

• linux64-shippable: 2
  • opt: 2
• linux64: 1
  • opt: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1721101&startday=2021-07-26&endday=2021-08-01&tree=all

Comment 12

[Geoff Lankow (:darktrojan)]

I suppose I should open a new bug instead since there was an uplift.

Comment 13

[Intermittent Failures Robot]

1 failures in 3025 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 1

Platform and build breakdown:

• linux64-shippable: 1
  • opt: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1721101&startday=2021-08-09&endday=2021-08-15&tree=all