Opened on 01/18/2016 at 08:37:41 AM

Closed on 02/23/2016 at 03:32:10 PM

Last modified on 02/24/2016 at 07:26:41 AM

#3525 closed change (fixed)

Add possibility for testers to crash application

Reported by: mario Assignee:
Priority: P3 Milestone: Adblock-Browser-for-iOS-1.3.0
Module: Adblock-Browser-for-iOS Keywords: salsita
Cc: Blocked By:
Blocking: Platform: Adblock Browser for iOS
Ready: yes Confidential: no
Tester: Scheer Verified working: yes
Review URL(s):

Description (last modified by mario)

Background

#3272 introduced a crash reporter and #3436 extends its functionality by more configuration options. In order to perform proper QA for #3436 (and other upcoming issues related to crashes/crash reports) we need to be able to reliably crash the application. Thus we'd like to introduce said option which is only visible for development builds.

What to change

  1. Add a new button labeled "Crash Application Now" at the bottom of the crash report settings screen (Settings -> Crash Report)
  2. Tapping this button causes the App to immediately crash, triggering a crash report.

Special Requirements

  1. Make this button/functionality only available in development builds. It must never be visible in release builds.
  2. Make the crash report distinguishable from other crash reports in regard to content so it cannot be confused with real crash reports.

Attachments (0)

Change History (9)

comment:1 Changed on 01/18/2016 at 09:22:54 AM by mario

  • Description modified (diff)

comment:2 Changed on 01/18/2016 at 03:50:03 PM by jand

  • Resolution set to fixed
  • Status changed from new to closed

comment:3 Changed on 02/09/2016 at 04:25:18 PM by mario

  • Milestone set to Adblock-Browser-for-iOS-next

Batch modify: added Adblock-Browser-for-iOS-next milestone.

comment:4 Changed on 02/19/2016 at 01:04:40 PM by scheer

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Tester changed from Unknown to Scheer

Above it states -

'Add a new button labeled "Crash Application Now" at the bottom of the crash report settings screen (Settings -> Crash Report)'

But, the test crash option in the app actually states 'Generate Test Crash'. Is 'Generate Test Crash' or 'Crash Application Now' the correct text?

Also stated is -

'Make this button/functionality only available in development builds. It must never be visible in release builds.'

I would not be able to check this until this feature went live, of course. Could I ask how it will be removed during the release process? Will there be some sort of check put in place for this to be looked at? A reminder?

Also -

'Make the crash report distinguishable from other crash reports in regard to content so it cannot be confused with real crash reports.'

Here, it would be great to have some idea of how to identify these. Is there some particular wording that is shown in some particular area of the crash report?

comment:5 Changed on 02/19/2016 at 03:30:51 PM by mario

  • Ready set

comment:6 Changed on 02/23/2016 at 10:37:33 AM by jand

I fixed wrong title. Crash report generated by this button can be easily recognised in HockeyApp.

If the crash button will make to production, it will not be working, since this hockeyApp functionality is disabled in App Store.

@mario: Do you want to localize this button? I think it does not make sense so that I made it unlozalizable.

comment:7 Changed on 02/23/2016 at 10:44:40 AM by mario

No objections, as this will always only be available in devbuilds.

If the crash button will make to production, it will not be working [...]

We need to make sure that this can't ever happen. Even though it wouldn't function, this feature is meant to be displayed in devbuilds only.

comment:8 Changed on 02/23/2016 at 03:32:10 PM by jand

  • Resolution set to fixed
  • Status changed from reopened to closed

Pavel must use release configuration during creation of release build, but it created by default, so it should be ok.

comment:9 Changed on 02/24/2016 at 07:26:41 AM by scheer

  • Verified working set

@Mario - I would say we need something in place to check the release build before it hits the App Store for this then.

  • The 'Crash Application Now' button now correctly crashes the app and sends a report to HockeyApp.

ABB 1.3.0-qa (824)
iPhone 6 Plus - iOS 9.2.1

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.