Skip to content
Snippets Groups Projects

Nightwatch config for selenium-server and gitignore changes

Merged Sean King requested to merge change/selenium-server into develop
All threads resolved!

Changes

  • The selenium-server path in the Nightwatch config uses require instead of hardcoding the path to selenium-server
  • Add logs/ to .gitignore so that Selenium server logs don't get added to the git repository

A concern was raised by tusooa regarding a mention of the package outside deps. Viewing this example, I figured we could use the require method used by chromedriver instead of hardcoding it if that makes sense.

Edited by Sean King

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Sean King changed the description

    changed the description

  • Author Developer

    For note, the Selenium server only runs if you have the same version of the Chromium browser as the chromedriver uses. See log below:

    Error
        An error occurred while creating a new Selenium Server session: [SessionNotCreatedError] session not created: This version of ChromeDriver only supports Chrome version 108
    Current browser version is 111.0.5563.64 with binary path /usr/bin/chromium

    And given the insanely fast pace that Chromium gets new major versions at, I'm not sure how we should approach this. Besides, the only unit test we have in Nightwatch fails because it checks that the default "Welcome to your Vue.js App" page is shown. See log below:

    TEST FAILURE (1m 44s):  
       - 1 assertions failed; 1 passed
    
       ✖ 1) test
    
       – default e2e tests (1m 42s)
    
       → ✖ NightwatchAssertError
       Testing if element <.hello> is present in 5000ms - expected "is present" but got: "not present" (5328ms)
    Edited by Sean King
  • HJ approved this merge request

    approved this merge request

  • tusooa resolved all threads

    resolved all threads

  • tusooa approved this merge request

    approved this merge request

  • merged

  • tusooa mentioned in commit 7a7d8027

    mentioned in commit 7a7d8027

Please register or sign in to reply
Loading