Thanks for looking into it, I will merge and look into adding a unit test to check that certain methods can be called . I think can add JUnit tests to cover the small use case. Anyone know what were the unit tests added with 5.1.0 to test the new feature about saving and restoring state for plugins? It would be easier for me to piggy back and add my condition in there
Richard in the JIRA Issue I added the setup to replicate with minimal effort. My use case didn't destroy the other activity it was just stopped. The use case was not about having a fully initialize Cordova Activity being destroy or stopped. That works On Thu, Jan 28, 2016 at 8:01 PM Richard Knoll <[email protected]> wrote: > I also think we should add tests for this. Carlos, do you have a setup for > this bug that you could share? It would be nice to have mobilespec tests > for save/restore in general, but that ends up being sort of tricky because > it requires enabling the "Don't Keep Activities" dev setting. > > Thanks, > Richard > > -----Original Message----- > From: Joe Bowser [mailto:[email protected]] > Sent: Thursday, January 28, 2016 1:39 PM > To: dev <[email protected]> > Subject: Re: [REVIEW] CB-10472 NullPointerException: > org.apache.cordova.PluginManager.onSaveInstanceState > > LGTM, but we should have a unit test to cover this behaviour somehow. > > On Thu, Jan 28, 2016 at 2:23 PM, Carlos Santana <[email protected]> > wrote: > > > It's being a while since I have done Android native dev, Can someone > > review the PR I sent > > https://github.com/apache/cordova-android/pull/255 > > > > This hit us today at work, we have a complex launch of multiple > > Cordova Activities to deal with security aspects and crosswalk. > > > > It's only in [email protected] due to the recent changes for > > saveState and restoreState for plugins > > > > It will be great to release a 5.1.1 next week with a couple of other > > bugs that are already seating in master. > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] >
