1080169 - Add a test that sends an sms with picture from Gallery attached

Status: RESOLVED FIXED

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

Description

[Peter Bylenga [:PBylenga]]

Currently the test does not test cropping images during the attachment process like Manual does.  Currently Automation is not encountering https://bugzilla.mozilla.org/show_bug.cgi?id=1063658 or https://bugzilla.mozilla.org/show_bug.cgi?id=1077106.

https://moztrap.mozilla.org/manage/cases/?pagenumber=1&pagesize=20&sortfield=created_on&sortdirection=desc&filter-id=10742

Manual STRs:


    Open Message app

    Message app opens and displays correctly, all previous threads will be display if they have not been removed.

    Tap on the Compose icon

    New Message screen will appear and display correctly.

    Enter a recipient in the "To:" text field

    Recipient will be added to the text box.

    Tap on Paper Clip icon

    Context menu will display with options: - Video - Music - Gallery - Camera

    Select Gallery

    Pictures will be populated from the phone.

    Select a picture

    Crop screen will display the photo selected and the user will be able to adjust the photo.

    Save the cropped image by tapping on the icon in the top right

    A preview of the picture selected will be displayed in the text field on the bottom. MMS text will display above the "Send" button.

    Tap on the image, and select 'View'

    Image shown.

    Tap on Send.

    MMS will be sent to recipient.

    Tap on the image just sent.

    Image shown.

Comment 1

[Florin Strugariu [:Bebe]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

@Peater @zac I don't think we can actually move the crop area with marionette

At the moment we just tap done in the crop view.

Zac do you have any suggestion on how to interact with the crop view?

Comment 2

[Zac C (:zac)]

Bebe, it's a canvas element and we can't currently interact with those very reliably. I'd say we have to skpi that step.

Comment 3

[Zac C (:zac)]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

f? as per comment #2

Comment 4

[Viorela Ioia [:viorela]]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

There are 2 steps from https://moztrap.mozilla.org/manage/case/10742/ that are not added in this PR. 
That would be: 
- Step 8: Tap on the image, and select 'View' and 
- Step 10:  Tap on the image just sent.
As for step 8, we are currently blocked by Bug 1080020. 
Please update your PR by adding these steps.

Comment 5

[Florin Strugariu [:Bebe]]

Updated the pull with some of the steps.


This still need some work as we are blocked by Bug 1080020

Comment 6

[Florin Strugariu [:Bebe]]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

Viorela can you take a look over the changes

Comment 7

[Viorela Ioia [:viorela]]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

Bug 1080020 is fixed, can you update your PR and ask for review? Thanks!

Comment 8

[Florin Strugariu [:Bebe]]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

Updated with all the steps.

Please review

Comment 9

[Viorela Ioia [:viorela]]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

comments in the PR

Comment 10

[Geo Mealer [:geo] -- This account is inactive after 2015-07-07]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

Looks like you already have two active reviewers. I'm removing myself as an r?, but I'm going to continue to eavesdrop for the learning.

Comment 11

[Viorela Ioia [:viorela]]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

Looks good, but there are some comments that need to be addressed.
Also, I started an adhoc run: http://jenkins1.qa.scl3.mozilla.com/job/flame-kk.ui.adhoc/256/console

Comment 12

[Viorela Ioia [:viorela]]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

r+

Comment 13

[Robert Chira [:RobertC]]

Comment on attachment 8504071 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25092

Merged on master: https://github.com/mozilla-b2g/gaia/commit/6ef24c53f0f44a41749b3a8cd72e0fb239760a6a

Can we uplift to v2.1?

Comment 14

[Florin Strugariu [:Bebe]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25704

Comment 15

[Zac C (:zac)]

I have backed this out because it has been reviewed poorly:
https://github.com/mozilla-b2g/gaia/commit/10c41953f753e2c39818080d7d9294aa1bf4fa29

Problems (also comments in git pull):
1. Explicit frame switching - wrong
2. Typo in `visible`
3. poorly coded header tap

Please fix them and re-land it.

Comment 16

[Florin Strugariu [:Bebe]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25764

Updated the pull Zac can you review it pls

Comment 17

[Zac C (:zac)]

Comment on attachment 8515910 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25764

r+

Comment 18

[Florin Strugariu [:Bebe]]

Merged in https://github.com/mozilla-b2g/gaia/commit/cd9ad5bbc2a26b77c94477639c8c2442701ae4cb

It has been reviewed and R+