Closed Bug 1215524 Opened 9 years ago Closed 9 years ago

Convert Bookmark marionette tests to Pinning

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
2.6 S2 - 12/4

People

(Reporter: mikehenrty, Assigned: mikehenrty)

References

Details

(Whiteboard: [systemsfe])

When we enabled pinning the web, we had to keep it disabled for the old bookmark tests. Let's convert these tests over since they might check for and find real bugs.

	modified:   apps/bookmark/test/marionette/install_app_test.js
	modified:   apps/homescreen/test/marionette/bookmark_edit_test.js
	modified:   apps/homescreen/test/marionette/bookmark_invalid_favicon_test.js
	modified:   apps/homescreen/test/marionette/bookmark_order_test.js
	modified:   apps/homescreen/test/marionette/bookmark_test.js
	modified:   apps/homescreen/test/marionette/bookmark_uninstall_test.js
	modified:   apps/verticalhome/test/marionette/bookmark_invalid_favicon_test.js
	modified:   apps/verticalhome/test/marionette/bookmark_test.js
	modified:   apps/verticalhome/test/marionette/bookmark_uninstall_test.js
Depends on: 1223374
Assignee: nobody → mhenretty
Target Milestone: --- → 2.6 S1 - 11/20
Depends on: 1223778
Depends on: 1223792
Depends on: 1224236
Depends on: 1225073
Target Milestone: 2.6 S1 - 11/20 → 2.6 S2 - 12/4
Closing this as all the new homescreen tests have been converted. We will not fix the vertical homescreen tests since it's legacy. And apps/bookmark/test/marionette/install_app_test.js is covered by pinning_the_web_test.js.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.