Skip to content

Comments

GPII-4231: JJ's rewrite of SystemSettingsHandler for Colour Vision merged up with master#311

Merged
amb26 merged 45 commits intoGPII:masterfrom
amb26:GPII-4231
Apr 27, 2020
Merged

GPII-4231: JJ's rewrite of SystemSettingsHandler for Colour Vision merged up with master#311
amb26 merged 45 commits intoGPII:masterfrom
amb26:GPII-4231

Conversation

@amb26
Copy link
Member

@amb26 amb26 commented Apr 8, 2020

No description provided.

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job failed: https://ci.gpii.net/job/windows-tests/1344/

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

Acceptance test hang again:

02:08:18.508:  Writing journal file to path C:\Users\vagrant\AppData\Local\Temp/gpii/journal-2020-04-24T090607.987Z.json
02:08:18.530:  UserLogonRequest completes successfully with the payload: User with GPII key os_win was successfully logged in.
Build was aborted
Aborted by unknown
Finished: ABORTED

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job failed: https://ci.gpii.net/job/windows-tests/1345/

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

Run 1345: Worrying, massive failure to restore the value of most settings - screen resolution, narrator registry settings, onscreen keyboard, sound sentry, mouse settings, sticky keys, and touchpad. See gist at https://gist.github.com/amb26/485c08897742d8013af0ac66d7045617

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job failed: https://ci.gpii.net/job/windows-tests/1347/

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

Unit test failure in process handling:

06:45:28.002:  Executing \\vboxsvr\vagrant\gpii\node_modules\processHandling\test\test-window.exe -window
06:45:28.788:  jq: FAIL: Module "gpii.tests.windows.processHandling" Test name "Testing closeProcessByName" - Message: Expected 3 assertions, but 2 were run
06:45:28.790:  jq: Source:     at Object.asyncTest (\\vboxsvr\vagrant\node_modules\infusion\tests\lib\qunit\js\qunit.js:406:9)
    at Object.<anonymous> (\\vboxsvr\vagrant\gpii\node_modules\processHandling\test\testProcessHandling.js:165:8)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Module.replacementCompile (\\vboxsvr\vagrant\node_modules\append-transform\index.js:58:13)
    at Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Object.<anonymous> (\\vboxsvr\vagrant\node_modules\append-transform\index.js:62:4)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (\\vboxsvr\vagrant\tests\UnitTests.js:24:1)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Module.replacementCompile (\\vboxsvr .... [output suppressed at 1024 chars - for more output, increase fluid.logObjectRenderChars]
06:45:28.794:  jq: Test concluded - Module "gpii.tests.windows.processHandling" Test name "Testing closeProcessByName": 2/3 passed - FAIL
06:45:28.918:  Executing C:\Users\vagrant\AppData\Local\Temp\gpii-process-handling-test.exe closeProcessByNameTest /T 30
06:45:28.934:  Exit code:5
06:45:29.686:  Process gpii-process-handling-test.exe will not close cleanly: No windows belong to the process
06:45:30.058:  jq: FAIL: Module "gpii.tests.windows.processHandling" Test name "Testing closeProcessByName (window-less process)" - Message: Expected 3 assertions, but 4 were run
06:45:30.058:  jq: Source:     at Object.asyncTest (\\vboxsvr\vagrant\node_modules\infusion\tests\lib\qunit\js\qunit.js:406:9)
    at Object.<anonymous> (\\vboxsvr\vagrant\gpii\node_modules\processHandling\test\testProcessHandling.js:201:8)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Module.replacementCompile (\\vboxsvr\vagrant\node_modules\append-transform\index.js:58:13)
    at Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Object.<anonymous> (\\vboxsvr\vagrant\node_modules\append-transform\index.js:62:4)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (\\vboxsvr\vagrant\tests\UnitTests.js:24:1)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Module.replacementCompile (\\vboxsvr .... [output suppressed at 1024 chars - for more output, increase fluid.logObjectRenderChars]
06:45:30.059:  jq: Test concluded - Module "gpii.tests.windows.processHandling" Test name "Testing closeProcessByName (window-less process)": 4/5 passed - FAIL

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1348/

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1349/

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1350/

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1351/

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1352/

@amb26
Copy link
Member Author

amb26 commented Apr 24, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1353/

@amb26
Copy link
Member Author

amb26 commented Apr 26, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1354/

@amb26
Copy link
Member Author

amb26 commented Apr 26, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1355/

@amb26
Copy link
Member Author

amb26 commented Apr 26, 2020

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1356/

@amb26
Copy link
Member Author

amb26 commented Apr 27, 2020

ok to test

@amb26 amb26 merged commit 2e04c69 into GPII:master Apr 27, 2020
@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/1357/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants