Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GPII-3922: May 2019 security updates and updated linting. #5

Merged
merged 4 commits into from
May 17, 2019

Conversation

the-t-in-rtf
Copy link
Collaborator

See GPII-3922 for details.

@amb26
Copy link
Member

amb26 commented May 16, 2019

Consistent hang on Windows in with the browser tests in Chrome:

DevTools listening on ws://127.0.0.1:12972/devtools/browser/2c6caf6b-5f6c-47c8-bb64-013cb0469fcc
12:53:17.926:  Test case listener has not responded after 5000ms - at sequence pos 5 of 17 sequence element {
    "event": "{testEnvironment}.webdriver.events.onExecuteScriptComplete",
    "listener": "{testEnvironment}.webdriver.executeScript",
    "args": [
        { Function

        },
        "fluid.getGlobalValue",
        "locationBarComponent.model.number"
    ]
} of fixture chrome: Confirm that backward and forward navigation work within a single page...

Some time after the timeout I then get a bizarrely worded message written by the characteristic schizoid nerd who writes these kinds of things assuming that we even know which "Program" he is talking about:

******** ERROR ********************************************************
This program is improperly using Base::Config by not initializing config
with the Load functions
See the documentation in Base/Dev/index.html for usage instructions.
At the minimum, LoadCommandLine should be use with a dummy command line
consisting of the program name.
************************************************************************

@the-t-in-rtf
Copy link
Collaborator Author

Thanks, @amb26. I can see various problems in a Windows Vagrant VM, will troubleshoot and let you know when this is ready for further review. At a minimum, this needs to be updated to use nyc instead of istanbul, and likely the test environment can be simplified, I'm not really even sure why express is necessary.

@the-t-in-rtf
Copy link
Collaborator Author

@amb26, I have updated the tests to use updated dependencies and a more modern take on instrumentation/coverage. As a bonus, the coverage reports are now actually meaningful.

@amb26
Copy link
Member

amb26 commented May 17, 2019

I now get coverage data, but the failure and output is as before - the test hangs during the "forward and backward navigation" test and the Chrome automation window stays open. Next, the coverage data appears, and then finally the schizoid message is output

@amb26 amb26 merged commit 44ad609 into fluid-project:master May 17, 2019
@the-t-in-rtf the-t-in-rtf deleted the GPII-3922 branch May 20, 2019 07:58
@the-t-in-rtf
Copy link
Collaborator Author

Thanks, @amb26, I have cut the release and updated the infusion-docs pull to use the new version. As we've discussed in chat, there may indeed be issues with particular combinations of chromedriver and Chrome, but it does work in the Vagrant Windows 10 VM. Once we have CI for this and other repos that use gpii-webdriver, we will at least have a good sense of whether the latest versions in chocolatey work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants