1149434 - Missing marionette notification tests

Status: RESOLVED INVALID

(Firefox OS Graveyard :: Gaia::System, defect)

Description

[:gerard-majax]

So, after being surprised by bug 1144495 I checked our Gaia marionnette tests, and I've noticed that we now have some marionette.ini file I never noticed. And that apps/system/test/marionette/notification_resend_block_test.js was not listed in it.

So bug 1103559 was never checked for ...

Comment 1

[:gerard-majax]

$ for f in apps/system/test/marionette/notification_*.js; do echo "$(basename $f):"; git grep "\[$(basename $f)\]" apps/system/test/marionette/manifest.ini; done;
notification_actions_test.js:
notification_behavior_test.js:
apps/system/test/marionette/manifest.ini:[notification_behavior_test.js]
notification_count_test.js:
notification_events_test.js:
apps/system/test/marionette/manifest.ini:[notification_events_test.js]
notification_get_test.js:
apps/system/test/marionette/manifest.ini:[notification_get_test.js]
notification_resend_block_test.js:
notification_resend_test.js:
apps/system/test/marionette/manifest.ini:[notification_resend_test.js]
notification_test.js:
apps/system/test/marionette/manifest.ini:[notification_test.js]

Comment 2

[:gerard-majax]

The manifest.ini was introduced in bug 1039140

Comment 3

[GitHub Autolander Bot]

Attachment: [gaia] lissyx:bug1149434 > mozilla-b2g:master

Comment 4

[:gerard-majax]

Andrew, I'm confused by bug 1039140: is it live ? Should we add manifest.ini content ?

Comment 5

[:gerard-majax]

green: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=39d3d2707a7417b5dc265b47eadd483cb0515b64&exclusion_profile=false

Comment 6

[Andrew Halberstadt [:ahal]]

I'm not actually sure either, I was pulled off that project when we had the B2G re-org. I suspect those manifests have been dropped, but maybe :gaye in #fxos-automation would know for sure.

Comment 7

[:gerard-majax]

Gareth, can you help me here?

Comment 8

[:gerard-majax]

I have been told on IRC that this is useless.