Closed almakantara closed 3 years ago
This pull request introduces 5 alerts when merging 9bba1d1b2fd252bd8a2aac65d17a9da7e931ff80 into b903b80a5e53eac831030e31bf2898656311658a - view on LGTM.com
new alerts:
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
main/com.smartnews/index.js | 14 | 32.53% | ||
<!-- | Total: | 14 | --> |
Totals | |
---|---|
Change from base Build 1109: | -6.5% |
Covered Lines: | 913 |
Relevant Lines: | 1767 |
Sorry, I tried to read your script and drowned in the confusion of command-line flags, so I rewrote it entirely. I ended up with two scripts:
./scripts/run-home-assistant.sh
that will install and run./scripts/setup-ha-virtual-devices.js
that will upload the virtual devices to Home AssistantI also added the test integration so the Home Assistant device and the associated org.thingpedia.iot.* devices become available to unit and scenario tests. I'm going to squash and merge all of this, and you can start working on the actual tests.
Took me quite a bit of debugging on Travis, because Travis's python setup is quirky, but it seems to work now. I'm going to squash everything and merge.
There is a node core script which does the job, and could be launched with different parameters to achieve different result, to get the help just launch it with the "-M" parameter.
The second script is a Bash .sh that runs the node, it has an interactive mode, that asks all the needed parameter for starting, and a manual mode that asks directly for the arguments to pass to the node script. There is also a further feature without any request by the script, that just launch what passed as argument to the bash script, and is usable by prepend al the argument by "p" or "p" right after the sh filename.