[TIMOB-12107] CLI: Check tiapp.xml for sane values during build
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | Critical |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2012-12-22T02:29:15.000+0000 |
Affected Version/s | Release 3.0.0 |
Fix Version/s | Release 3.0.2, Release 3.1.0, 2012 Sprint 26 JS, 2012 Sprint 26 |
Components | CLI |
Labels | qe-cli, qe-port, qe-testadded |
Reporter | Chris Barber |
Assignee | Chris Barber |
Created | 2012-12-17T20:09:02.000+0000 |
Updated | 2013-08-01T22:36:39.000+0000 |
Description
It's a good idea to check various tiapp.xml values to make sure they are sane. This includes things like app name and such.
Master pull request: https://github.com/appcelerator/titanium_mobile/pull/3626 3.0.x pull request: https://github.com/appcelerator/titanium_mobile/pull/3627
Hi Chris--we need to create test cases for each of the things fixed in that pull request. How would you suggest we go about doing so?
There was a order of validation conflict that was causing the clean and create commands to error: Master pull request: https://github.com/appcelerator/titanium_mobile/pull/3629 3.0.x pull request: https://github.com/appcelerator/titanium_mobile/pull/3630
Wanted to add more error checking for missing tiapp.xml id, name, and guid elements: Master pull request: https://github.com/appcelerator/titanium_mobile/pull/3631
Closing as fixed. Tested and verified with: Titanium Studio, build: 3.0.1.201212181159 Titanium Studio, build: 3.0.2.v20130124094710 Titanium Studio, build: 3.1.0.v20130124094206