[Lubuild] fix multiple bugs: case sensitive issue, default version issue and exception not throw out issue

[feich-ms]

Fix several bugs:

1. As known to us, values of utterances/patterns are automatically converted to lower case in luis portal even though typed words are upper case. When comparing app json from lu file with the one from remote existing luis app, we should to ignore cases. To achieve this, I converted the objects to string, then to lower case so that the comparison works insensitively.
2. Some properties in our luis definition don't align with the object pulled from luis portal. Fixed in this PR
3. Default version 0.1 was included by previous PR when no version is specified which caused some bugs when updating version. In this PR, version from lu will count only if it's value is over than existing app. For example, if the app with the same app id in luis portal is version 0.1, and the version from the lu file is 0.3, 0.3 > 0.1, then 0.3 will count, otherwise, 0.1 will auto increase by 0.1 to 0.2
4. Add test cases for above bug fixes
5. Add test cases for luis:build refinements #548
6. Fix bug Exceptions are not returned properly in luis:build #582

[vishwacsena]

Can you please add tests as well?

[feich-ms]

@vishwacsena, sure, will add tests soon.

[feich-ms]

Tests added @vishwacsena

[vishwacsena]

@munozemilio it might make sense to expose compareApps as a public method on LUIS?

[munozemilio]

CIL

[feich-ms]

@munozemilio, comments resolved. Also added test cases for luConfig support of #548. Fixed bug of #582 as well.

[munozemilio]

Some comments in line

[feich-ms]

Comments resolved. Thanks @munozemilio

[vishwacsena]

@feich-ms I tried this E2E and pushed up two sets of deltas up to your branch. I'm not familiar with nock and some of the lubuild tests are failing. Can you take a look and address?

[vishwacsena]

@munozemilio except for addressing tests, it would be good if you get a review as well. Thanks!

[munozemilio]

CIL

[feich-ms]

@munozemilio, tests failures are fixed. Please help to approve. Thanks.

The merge is blocked by 'Request to change'. Could you approve if there are no other issues.

[feich-ms]

@munozemilio, I added more modules in index.js and composerindex.js in the latest commit. My intention is to expose modules that composer client/broswer consumes in composerindex.js and other modules in index.js. Parser and SectionHandler is consumed by composer client and luBuild is consumed by composer server. Please let me know if this is not the reasonable way. Thanks.

[munozemilio]

Approved with a comment