[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