Closed Bug 959053 Opened 6 years ago Closed 6 years ago

[settings] add marionette test for battery panel

Categories

(Firefox OS Graveyard :: Gaia, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(feature-b2g:2.0)

RESOLVED FIXED
1.4 S1 (14feb)
feature-b2g 2.0

People

(Reporter: gasolin, Assigned: gasolin)

References

Details

(Whiteboard: [ft:system-platform])

Attachments

(1 file)

add marionette test for battery panel and main page battery status
Blocks: 956210
Assignee: nobody → gasolin
test passed
Attachment #8359131 - Flags: review?(arthur.chen)
Comment on attachment 8359131 [details] [review]
pull request redirect to github

Thanks for the effort, Fred. Please check my github comment.
Attachment #8359131 - Flags: review?(arthur.chen)
Comment on attachment 8359131 [details] [review]
pull request redirect to github

fixed comments and use selector to detect checked item
Attachment #8359131 - Flags: review?(arthur.chen)
Comment on attachment 8359131 [details] [review]
pull request redirect to github

r=me. Thanks for the patch!
Attachment #8359131 - Flags: review?(arthur.chen) → review+
Reverting due to failing linter on travis: https://github.com/mozilla-b2g/gaia/commit/b1047ea52b0bf4828fb2d2c607da4e86fd7c829d

Please either fix the lint problems for JShint, or update the xfail.list file and re-land.
fix lint with PR https://github.com/mozilla-b2g/gaia/pull/15663 and wait for travis result
reland with all green https://github.com/mozilla-b2g/gaia/commit/ecee0ca82ddac60d077d8665b5c5a3e3a1a556e6
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
In the future, please do not consider adding a file to xfail.list as a solution to jshint problems.  The solution is to fix the problems in jshint, not revert to using gjslint.

Kevin, could you kindly not suggest it as a solution next time? Fred/Arthur, can we make sure to not add more files to xfail.list? Thanks!
Corey thanks for reminding.

current settings app integration test barks lots of errors in jshint.
I'd fire bug 964188 to fix it at once.
Blocks: 969265
Blocks: 973436
Whiteboard: [ft:system-platform]
Target Milestone: --- → 1.4 S1 (14feb)
feature-b2g: --- → 2.0
You need to log in before you can comment on or make changes to this bug.