[DAEMON-175] Windows Plugin: Expected gawked object error when updating data

GitHub Issue	n/a
Type	Bug
Priority	Critical
Status	Resolved
Resolution	Fixed
Resolution Date	2017-11-21T14:57:46.000+0000
Affected Version/s	Appc Daemon 1.0.0
Fix Version/s	Appc Daemon 1.0.0
Components	appcd-plugin-windows
Labels	n/a
Reporter	Ewan Harris
Assignee	Ewan Harris
Created	2017-11-21T10:30:03.000+0000
Updated	2017-11-21T14:57:46.000+0000

Description

The types dont match up with the gawked object, windows-sdk -> windows, windows-phone-sdks ->windowsphone

2017-11-21T10:20:02.595Z appcd:plugin:host:53884 > windows@1.0.0-13 TypeError: Expected destination to be an object
2017-11-21T10:20:02.595Z appcd:plugin:host:53884 > windows@1.0.0-13     at Function.set (C:\Users\ewan\.appcelerator\install\7.0.0-master.34\package\node_modules\gawk\dist\index.js:413:9)
2017-11-21T10:20:02.595Z appcd:plugin:host:53884 > windows@1.0.0-13     at callback.then.result (C:\Users\ewan\.appcelerator\install\7.0.0-master.34\package\node_modules\appcd-plugin-windows\dist\src\windows-info-service.js:69:6)
2017-11-21T10:20:02.595Z appcd:plugin:host:53884 > windows@1.0.0-13     at <anonymous>
2017-11-21T10:20:02.597Z appcd:plugin:host:53884 > windows@1.0.0-13 Updating data for windows-sdks
2017-11-21T10:20:02.597Z appcd:plugin:host:53884 > windows@1.0.0-13 TypeError: Expected destination to be an object
2017-11-21T10:20:02.597Z appcd:plugin:host:53884 > windows@1.0.0-13     at Function.set (C:\Users\ewan\.appcelerator\install\7.0.0-master.34\package\node_modules\gawk\dist\index.js:413:9)
2017-11-21T10:20:02.597Z appcd:plugin:host:53884 > windows@1.0.0-13     at callback.then.result (C:\Users\ewan\.appcelerator\install\7.0.0-master.34\package\node_modules\appcd-plugin-windows\dist\src\windows-info-service.js:69:6)
2017-11-21T10:20:02.597Z appcd:plugin:host:53884 > windows@1.0.0-13     at <anonymous>

Steps to reproduce

`\[appc\] appcd start --debug`

`\[appc\] appcd exec /windows/latest/info`

Actual

Above error

Expected

No error, results returned

Comments

Ewan Harris 2017-11-21 I assumed the change was intentional If so, merge https://github.com/appcelerator/appc-daemon/pull/172 If not, merge https://github.com/appcelerator/appc-daemon/pull/173

JSON Source