Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TIMOB-25827] Error when fresh install with forceUnInstall #84

Merged
merged 1 commit into from Mar 5, 2018
Merged

Conversation

infosia
Copy link
Contributor

@infosia infosia commented Mar 2, 2018

TIMOB-25827

Expected:

  1. Build an application for wp-emulator with the emulator closed (clean state) with --forceUnInstall
  2. Build should not fail

Also make sure TIMOB-23800 doesn't happen again

  1. Build an application
  2. Without uninstalling an app, build an app with same <guid> with different <id> in tiapp.xml
  3. Build should error out with A debug application is already installed, please remove existing debug application.
  4. After uninstalling an app manually, build should not fail then.

@infosia infosia requested a review from ewanharris March 2, 2018 03:59
Copy link

@ewanharris ewanharris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CR/FT Pass!

Build no longer errors when using --forceUnInstall flag with no app installed, tested building to emulator and device. No regressions around previous behaviour where existing installed app is no the same as being built

Merging

@ewanharris ewanharris merged commit 785d86c into appcelerator-archive:master Mar 5, 2018
@infosia infosia deleted the TIMOB-25827 branch March 6, 2018 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants