Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Add karma & chrome/firefox headless browser fixtures #3146

Merged
merged 7 commits into from
Oct 22, 2019
Merged

Add karma & chrome/firefox headless browser fixtures #3146

merged 7 commits into from
Oct 22, 2019

Conversation

cgewecke
Copy link
Collaborator

@cgewecke cgewecke commented Oct 19, 2019

Part of #3098

  • Adds new CI job to run the existing E2E tests in headless chrome, headless firefox
  • When browser testing, loads Web3 from packages/web3/dist/web3.min.js

There are various ways of setting this up. phantomjs is deprecated now fwiw.

Picked karma because it's well supported by Google/Angular, runs mocha tests easily, has lots of plugins, stack-overflow/github help, and should be low-maintenance.

Reference Documentation

@coveralls
Copy link

coveralls commented Oct 20, 2019

Coverage Status

Coverage remained the same at 84.005% when pulling edd52b9 on cgewecke:tests/headless-browser into a2027aa on ethereum:1.x.

@nivida nivida self-requested a review October 21, 2019 09:59
@nivida nivida added 1.x 1.0 related issues CI In Progress Currently being worked on labels Oct 21, 2019
Copy link
Contributor

@nivida nivida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just added 1-2 comments for adding helper functions instead of duplicating the conditions :)

@cgewecke
Copy link
Collaborator Author

@nivida Made requested changes, thanks for reviewing.

@nivida nivida removed the In Progress Currently being worked on label Oct 22, 2019
@nivida nivida merged commit a7235c8 into web3:1.x Oct 22, 2019
nachomazzara pushed a commit to nachomazzara/web3.js that referenced this pull request Jun 4, 2020
* Add karma & chrome/firefox headless browser fixtures

* Add test helper for conditional web3 require

* Add websocket port test helper

* Move build/lint to their own CI job

* Fix package-lock from rebase

* Fix ci.sh

* Skip all websockets tests for geth.instamine
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
1.x 1.0 related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants