Closed Bug 930992 Opened 10 years ago Closed 10 years ago

Create a System app region for "app install confirmation"


(Firefox OS Graveyard :: Gaia::UI Tests, defect, P4)



(b2g-v1.2 affected)

Tracking Status
b2g-v1.2 --- affected


(Reporter: zcampbell, Assigned: bsilverberg)



(Whiteboard: [gaia-ui-test][mentor=zac][lang=py])


(4 files)

A few tests use this confirmation prompt, eg test_homescreen_delete_app, test_homescreen_launch_app.

We can make all these tests use the single confirmation region.
Hello, this is the first thing I will be working on for Mozilla (if I get to start). I know only python and have no prior experience in open-source projects. Can I be of any help?
Yes you sure can Aman!

What we want is for a "App install confirmation" class in "tests/python/gaia-ui-tests/gaiatest/apps/system/region" folder.

It will be very similar to this in the homepage app.

Once you've created that you'll need to search through all of the tests to find the where the region can be used and update the tests and app objects to use it.

If you flick through our existing tests you'll be able to see the style of code that we use.

If you have any more questions you can email me or find us in the #mozwebqa IRC channel.
Assignee: nobody → aman.abhishek.01
Reassigning to bitgeeky :)
Assignee: aman.abhishek.01 → mozpankaj1994
:zac just a raw idea of the new class ConfirmInstall in , also please let me know some test where 
is in use , so that i can get a better understanding of how to make changes in tests and use the new ConfirmInstall class.
Attachment #829711 - Flags: feedback?(zcampbell)
Flags: needinfo?(zcampbell)
Comment on attachment 829711 [details]
Raw idea of the new class ConfirmInstall in

Hey bitgeeky, that's perfect!

If you search for 'app-install-install-button' in the /gaia-ui-tests/ folder then you'll be able to search backwards and find the tests and app objects that are using it.
Attached patch 930992.patchSplinter Review
:zac please have a look at this attachment , and suggest changes if any :)
Attachment #830479 - Flags: feedback?(zcampbell)
Comment on attachment 830479 [details] [diff] [review]

Review of attachment 830479 [details] [diff] [review]:

bitgeeky, this looks great.
Now you need to file it as a pull request so that we can run it on devices and run it on Travis.

Almost ready to merge :)
Attachment #830479 - Flags: feedback?(zcampbell) → feedback+
Attachment #829711 - Flags: feedback?(zcampbell)
Attachment #830851 - Flags: review?(zcampbell)
Comment on attachment 830851 [details]
Pointer to Github pull request:

Nice pull, it works perfectly both on a device and on desktopb2g!
Attachment #830851 - Flags: review?(zcampbell) → review+

Let's bring this into v1.2 too.
It looks to be clean but marking ? just as not 100% sure.
status-b2g-v1.2: --- → ?
Flags: needinfo?(zcampbell)
thanks :zac :)
Taking for v1.2 uplift
Assignee: mozpankaj1994 → bob.silverberg
Comment on attachment 831259 [details]
Pointer to Github pull request:

Uplift for v1.2
Attachment #831259 - Flags: review?(trifandreialin)
Attachment #831259 - Flags: review?(moz.teodosia)
Attachment #831259 - Flags: review?(florin.strugariu)
Attachment #831259 - Flags: review?(andrei.hutusoru)
Attachment #831259 - Flags: review?(moz.teodosia) → review-
Attachment #831259 - Flags: review?(florin.strugariu) → review+
Blocks: 938550
Attachment #831259 - Flags: review?(trifandreialin)
Attachment #831259 - Flags: review?(andrei.hutusoru)
Attachment #831259 - Flags: review-
Attachment #831259 - Flags: review+
Merged in v1.2:
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.