We're updating the issue view to help you get more done. 

[iOS] Tidepool Mobile fails when shared account is deleted

Description

Problem

Tidepool Mobile iOS v2.1.10 crashes when attempting to view data shared from an account that has been deleted. If the account is already logged in, tapping the icon causes a crash and dumps user to home screen. If not logged in, the correct password does not work. The user has no chance to change active profile to an account that still exists.

Steps to reproduce the bug:

  • Create account 1, upload data

  • Create account 2

  • Share account 1 with account 2

  • Login to account 2 with Tidepool Mobile and view account 1 data

  • Delete account 1

  • Tidepool Mobile crashes on launch, and does not allow login after reinstall

This bug was found while verifying delete user functionality (https://trello.com/c/ER64SIAH, https://trello.com/c/dWWfWy3V).

Eric Luhrs and Becky Cooney tested this functionality on RN Tidepool Mobile on both iOS and Android and the RN app handles the delete properly on both devices. iOS Native bug.

Requested By

  • Eric Luhrs

App [limit to one]

  • Tidepool Mobile

Scope

  • [ XS / S / M / L ]

  • Reference:

  • XS: <= 1 day

  • S: 2-3 days

  • M: <= 1 week

  • L: > 1 week

Risk assessment

Reference: https://docs.google.com/document/d/17mb--4AeFVAgNJLEIE6obXIVjeu0ST7i2cYAsziu8Vc/edit?usp=sharing

NOTE: Risk assessment is the risk (probability and severity) of the adverse event occurring, not the probability of the bug occurring. See above for details.

  • Probability of an adverse event: Baseline risk because the user can no longer access Tidepool on Tidepool Mobile

  • Severity of the potential adverse event - Data becomes inaccessible, baseline risk

  • Risk Level: 0

  • Mitigation Plan: fix this bug

User Experience Assessment
Reference: Draft Doc: https://docs.google.com/document/d/1t3ludwd-eMoFdiK2IW_s-HXrGCDSMUW7EDgyoO9HZqg/edit#

  • Severity: Inaccessible

  • Issue Volume:
    A - # of distinct Clinician Reports: 0
    B - # of distinct User Reports: 0
    C - # of discoveries by Tidepool employees (testing, dogfooding, etc) 1
    D - # of occurrences according to logs (Rollbar, Kissmetrics, etc). 0

Issue Volume Index = 3A + B + 1 + D

  • Issue Level: 12

Done Criteria

  • When a shared account is deleted, user should be returned to primary account within Tidepool mobile.

Test Strategies

  • Create account 1, upload data

  • Create account 2

  • Share account 1 with account 2

  • Login to account 2 with Tidepool Mobile and view account 1 data

  • Delete account 1

  • Verify that Tidepool Mobile, rather than crashing, allows user to switch back to the primary account.

Verification Test

  • [Link to test document]

Approved for Production By

  • [Howard / Brandon]

Tags & Release Notes

  • TBD

Done Criteria

None

Design Specifications:

None

Accessibility Specifications

None

Test Strategy

None

Environment

None

Activity

Show:
Janet Diehl
June 26, 2019, 9:26 PM

Eric Luhrs Mar 31 at 12:51 PM (edited)

This bug was re-tested due to platform and mobile updates that were deployed just at testing concluded for delete user functionality (<0:ux 5:dev 2:tst> [platform] Delete account and all related data).

Second round of testing yielded the same result. Note that Tidepool Mobile-RN does not suffer from the same bug. Rather, it gives "Unknown error" when switching profiles, but does not fail to load.

Add Link as Attachment - Delete

Janet Diehl transferred this card from Active: Tidepool Mobile

Mar 27 at 10:35 AM

Brandon Arbiter Mar 25 at 7:40 AM

Moving to the Tidepool Mobile board.

Eric Luhrs Mar 22 at 6:00 PM

Pinging @tjotala @brandonarbiter for prioritization and assignment.

Delete

Eric Luhrs Mar 22 at 5:53 PM

There was a complex test case that led to the discovery of this bug. I believe that the steps are identified correctly above, but if not, see CASE 11 in the Delete User verification doc (linked in the description).

Delete

Eric Luhrs joined this card

Mar 22 at 4:56 PM

Priority

Assignee

Eric Luhrs

QA Assignee

None

Reporter

Janet Diehl

Requested By (External)

None

Fix versions

None

T-Shirt Size

None

Risk Level

0

User Experience Assessment Level

12

UEA: Issue Volume

None

UEA: Category

None

Components

Design Review Required?

None

Design Approved By:

None

QA done during validation of another ticket?

None

Accessibility Review Required?

None

Accessibility Review Approved By

None

Approved for Production By:

None

Marketing Comms

None

Support Follow Up

None

KB Update

None

Pull Request Needed

None

Resolved Build #

None
Configure