Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ac634edf authored by Hemant Gupta's avatar Hemant Gupta Committed by Andre Eisenbach
Browse files

MAP: Remove handling for ACTION_MESSAGE_DELIVERY

Precondition:
DUT connected to any remote device supporting MAP Client

Usecase
1) Send SMS to invalid number from MAP Client using DUT as MAP Server

Expectation
1) Message sending fails and error event received at MAP Client
2) Only 1 failed message entry is seen in failed message folder of DUT.

Observation
2 entries of failed message are seen in failed folder of DUT

Root Cause
Movement of message to Failed Folder is already handled in SMS app
so handling it again in BT MAP code causes duplicate failed entry for same
SMS

Fix
Remove handling of message movements for ACTION_MESSAGE_DELIVERY in BT MAP code
since as per MAP specification movement of SMS from outbox folder to failed folder
on Sending Failure event is only recommended and not mandatory.
Same is already handled in MMS application code both for success and failure
case.

Test: Duplicate entries are not created for SMS for failure or success cases.

Bug: 35625080
Change-Id: I3f74a0dd5504e13f65f1ab6bafac4115b1698fe3
parent 44cc7fd7
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment