Commit cc25226
committed
Merge branch 'GPII-581'
* GPII-581: (22 commits)
GPII-581: Removed console.log for SPI, added fluid.log for process waiting
GPII-581: Tidy up of waitForSpi
GPII-581: Tidy up of waitForProcessState Made options immutable, renamed waitForProcessStateChange to waitForProcessState
GPII-581: Removed redundant waitForTask() Replaced with windows.waitForProcessStart()
GPII-581: Created waitForProcessStateChange function, and waitForProcessTermination/Start wrappers. Provides the ability to wait for a process to start or end.
GPII-581: Removed assertion of the waitfor process being closed on demand. For some reason it was unable to send the signal, but it's ok because it will die after a few seconds. Not going to fail the test just because waitfor didn't work.
GPII-581: Made the API test asynchronous (and work). The promise returned from 'set' is no longer synchronous.
GPII-581: Made the API test asynchronous. The promise returned from 'set' is no longer synchronous
GPII-581: Fixed indentation.
GPII-581: Added comment to waitForTask helper function.
GPII-581: Removed some declarations that are no longer required. "WaitForSingleObject" and related constants.
GPII-581: Added 'options' parameter to waitForProcessTermination And other tidy-ups
GPII-581: Added comment for waitForSpi.
GPII-581: Wait for sethc.exe to finish before calling SPI, too. Acceptance tests where still causing 2 instances of sethc.
GPII-581: SPI High contrast detection no longer spawns a new process. Moved process related code into processHandling.js, updated kill tests to also make use of it.
GPII-581: Created processHandling, and consumed killProcess. Putting process related functionality together
GPII-581: Put WindowsUtilitiesCore.js back into WindowsUtilities.js No longer needs to be in a separate file.
GPII-581: Environment variable not being set for child process Options for fork now added to instead of replaced
GPII-581: Made SpiHighContrastCompletion.js more pleasant. Added background information, refactored convoluted loops and nesting into separate functions.
GPII-581: Moved the win api definitions to WindowsUtilitiesCore.js Taken the ffi funciton definitions from WindowsUtilities.js into its own file, so they can be included by the child-process in SpiHighContrastCompletion.js without the need to load universal.
...File tree
10 files changed
+481
-203
lines changed- gpii/node_modules
- WindowsUtilities
- killProcess
- test
- processHandling
- test
- spiSettingsHandler
- src
- test
- tests
10 files changed
+481
-203
lines changedSome generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
This file was deleted.
Binary file not shown.
This file was deleted.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments