Closed Bug 1213199 Opened 9 years ago Closed 9 years ago

[Messages] 'Message saved as draft' notification is displayed even if draft has been removed afterwards before going to Inbox

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::SMS
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(blocking-b2g:2.5+, b2g-v2.2 unaffected, b2g-master verified)

Status:
VERIFIED FIXED
Project Flags:
blocking-b2g 2.5+
Tracking Flags:
Tracking Status
b2g-v2.2 --- unaffected
b2g-master --- verified

People

(Reporter: yelanying, Assigned: azasypkin)

References

Details

(Keywords: regression, Whiteboard: [2.5-aries-test-run-3][sms-sprint-FxOS-S9])

Attachments

(4 files)

logcat_1144.txt
434.45 KB, text/plain
Details
Aries_KK v2.5.png
28.66 KB, image/png
Details
[gaia] azasypkin:bug-1213199-saved-draft-notification > mozilla-b2g:master
46 bytes, text/x-github-pull-request
julienw
: review+
julienw
: feedback+
Details | Review
verified_Aries KK v2.5.3gp
2.18 MB, video/3gpp
Details
Attached file logcat_1144.txt — 
[1.Description]:
[Flame KK v2.5][Aries KK v2.5][Message] Launch message app and send a MMS. After the MMS was sent successfully, long tap it to delete. Then wong suggestions "Message saved as draft" pop up.
Time: 11:44
See attachments: Aries_KK v2.5.png and logcat_1144.txt

[2.Testing Steps]: 
Precondition: There is a SIM card in the DUT.
1.Launch message app and enter new message composer.
2.Add an attachment.
3.Tap sending button.
4.After sending it successfully, long tap the MMS.
5.Select "Delete" from message options.
6.Tap "Delete".

[3.Expected Result]: 
6."Message is deleted" pops up, or no any prompt.

[4.Actual Result]: 
6."Message saved as draft" pops up.

[5.Reproduction build]: 
Device: Aries KK v2.5 (Affected)
Build ID               20151008233852
Gaia Revision          e698df503ff700eb5782e3d50c6eb753567d3451
Gaia Date              2015-10-08 17:26:52
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/23b7f289df923c01e692299fcd4be7029de8b155
Gecko Version          44.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20151008.225742
Firmware Date          Thu Oct  8 22:57:51 UTC 2015
Bootloader             s1

Device: Flame KK v2.5 (Affected)
Build ID               20151008150210
Gaia Revision          e698df503ff700eb5782e3d50c6eb753567d3451
Gaia Date              2015-10-08 17:26:52
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/e5f1bc63ad52d0eb86f7fb838226ca6036774660
Gecko Version          44.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20151008.182756
Firmware Date          Thu Oct  8 18:28:08 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device: Flame KK v2.2 (Unaffected)
Build ID               20151008032505
Gaia Revision          885647d92208fb67574ced44004ab2f29d23cb45
Gaia Date              2015-10-07 13:05:24
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/ab6c34bfacf7
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20151008.064723
Firmware Date          Thu Oct  8 06:47:34 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

[6.Reproduction Frequency]: 
Always Recurrence,10/10

[7.TCID]: 
Free test
Attached image Aries_KK v2.5.png — 

    
  

          status-b2g-v2.2:
          --- → unaffected

          status-b2g-master:
          --- → affected
Keywords: regression

    
    

    
  
I'm pretty sure it's regressed by bug 1176976, likely we need to reset "showDraftSavedNotice" if draft has been deleted (once message is sent) after it was saved (once user adds attachment).
Blocks: 1176976
Flags: needinfo?(azasypkin)
blocking-b2g: --- → 2.5?

    
    

    
  
Comment on attachment 8672620 [details] [review]
[gaia] azasypkin:bug-1213199-saved-draft-notification > mozilla-b2g:master

Hey Julien,

How does this solution look for you (no test changes yet)?

Thanks!
Flags: needinfo?(azasypkin)

        Attachment #8672620 -
        Flags: feedback?(felash)

    
    

    
  
Just want to mention that deleting the message is not necessary.

Here is the minimal STR for this bug:
1. Enter a conversation.
2. Start writing a message (can be SMS).
3. Press "Home". Under the hood this will save a draft. There are other ways to trigger this though.
4. Enter the Messages app by tapping the Homescreen icon.
5. Send the message.
6. Press back. (note: you don't need to wait that the message is sent).

    
    

    
  
Comment on attachment 8672620 [details] [review]
[gaia] azasypkin:bug-1213199-saved-draft-notification > mozilla-b2g:master

yep, looks good !

        Attachment #8672620 -
        Flags: feedback?(felash) → feedback+
Assignee: nobody → azasypkin
Whiteboard: [2.5-aries-test-run-3] → [2.5-aries-test-run-3][sms-sprint-FxOS-S9]

    
    

    
  
Comment on attachment 8672620 [details] [review]
[gaia] azasypkin:bug-1213199-saved-draft-notification > mozilla-b2g:master

Thanks for the feedback, I think it should be ready for review! However I left one question regarding very edgy case at GitHub.

        Attachment #8672620 -
        Flags: review?(felash)

    
  
Summary: [Message] Wrong suggestions 'Message saved as draft' pop up after you select delete the MMS. → [Messages] 'Message saved as draft' notification is displayed even if draft has been removed afterwards before going to Inbox
blocking-b2g: 2.5? → 2.5+

    
  
See Also:  → 1080504

    
    

    
  
Comment on attachment 8672620 [details] [review]
[gaia] azasypkin:bug-1213199-saved-draft-notification > mozilla-b2g:master

I don't mind the edge case but I don't like the unit test, I think we're testing things that are too internal...

        Attachment #8672620 -
        Flags: review?(felash)

    
  
Status: NEW → ASSIGNED

    
    

    
  
Comment on attachment 8672620 [details] [review]
[gaia] azasypkin:bug-1213199-saved-draft-notification > mozilla-b2g:master

Hey Julien,

Does it look better now? :)

Thanks!

        Attachment #8672620 -
        Flags: review?(felash)

    
    

    
  
Comment on attachment 8672620 [details] [review]
[gaia] azasypkin:bug-1213199-saved-draft-notification > mozilla-b2g:master

r=me

thanks for the new tests !

        Attachment #8672620 -
        Flags: review?(felash) → review+

    
    

    
  
Master: https://github.com/mozilla-b2g/gaia/commit/c93bdb29e47e2cba9e81aceaf04f084d31e37a64
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED

    
    

    
  


  
According to the STR of Comment 0, this bug has been verified as pass on latest Flame KK v2.5 and Aries KK v2.5.

Actual results: No any prompt
See attachment: verified_Aries KK v2.5.3gp
Reproduce rate: 0/6


Device: Flame KK v2.5 (Pass)
Build ID               20151019150205
Gaia Revision          a87f947366c2e044bd6336e1982419ac45378969
Gaia Date              2015-10-19 15:22:08
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/9605da94e75d61598d3c00f01a12d1b6bc427a6c
Gecko Version          44.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20151019.182947
Firmware Date          Mon Oct 19 18:29:58 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0


Device: Aries KK v2.5 (Pass)
Build ID               20151019205841
Gaia Revision          a87f947366c2e044bd6336e1982419ac45378969
Gaia Date              2015-10-19 15:22:08
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/9605da94e75d61598d3c00f01a12d1b6bc427a6c
Gecko Version          44.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20151019.201730
Firmware Date          Mon Oct 19 20:17:38 UTC 2015
Bootloader             s1

    
  
Status: RESOLVED → VERIFIED
QA Whiteboard: [MGSEI-Triage+]

          status-b2g-master:
          affected → verified

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: