1097040 - Write a test to verify that the SMS auto-save draft when message app terminated

Status: RESOLVED FIXED

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

Description

[Viorela Ioia [:viorela]]

Add an automated test for moztrap test case: https://moztrap.mozilla.org/manage/case/7806/

This case is created to verify if message app will save the draft automatically

Steps:
Open message app and create new SMS.
Long press on home button.
Show card view of tasks
Close message app and leave card view.
Re-open message app.
Last edit message thread is tagged as "draft"
Click draft message thread.
Last message draft is shown correctly

Comment 1

[Viorela Ioia [:viorela]]

Attachment: Github PR: https://github.com/mozilla-b2g/gaia/pull/26296

Comment 2

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8525328 [details] [review]
Github PR: https://github.com/mozilla-b2g/gaia/pull/26296

Looks generally good to me. A couple of nits to fix.

Comment 3

[Johan Lorenzo [:jlorenzo]]

Merged on master: https://github.com/mozilla-b2g/gaia/commit/7fdfe2ea29384eb5d6a691d30a4e9c5a356917a2

Comment 4

[Carsten Book [:Tomcat]]

reverted for test failures like https://treeherder.mozilla.org/ui/logviewer.html#?job_id=880220&repo=b2g-inbound btw this was also on the gaia try results

Comment 5

[Viorela Ioia [:viorela]]

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

Comment 6

[Viorela Ioia [:viorela]]

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

Comment 7

[Florin Strugariu [:Bebe]]

This was causing multiple fails 
Reverted in https://github.com/mozilla-b2g/gaia/commit/1374d2bd4bae6013feddcf85d4bd2b03a2f52661

Comment 8

[Viorela Ioia [:viorela]]

(In reply to Florin Strugariu [:Bebe] from comment #7)
> This was causing multiple fails 
> Reverted in
> https://github.com/mozilla-b2g/gaia/commit/
> 1374d2bd4bae6013feddcf85d4bd2b03a2f52661

The patch caused intermittent failures in test_sms_contact.py because of the changes done in New message app object (see Bug 1107927). 
I will open a new PR to fix this.

Comment 9

[Viorela Ioia [:viorela]]

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

Comment 10

[Viorela Ioia [:viorela]]

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

Merged: https://github.com/mozilla-b2g/gaia/commit/32fff7a88c0246eb8833b9f8dea668d4e32d8871