Skip to content
This repository has been archived by the owner on May 20, 2020. It is now read-only.

Tests #299

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

Tests #299

wants to merge 60 commits into from

Conversation

yoogottamk
Copy link
Collaborator

All requested tests work.

Currently not testing on RTE.

Based on #258.

yoogottamk and others added 30 commits July 3, 2019 08:18
…rks only for single snippet [problems with async/await]
switch to for..of and concise destructuring
add troubleshooting note in tests.md
attempting to fix connection refusal,
problem is possibly because multiple test urls can't run in parallel
also assert.js no longer req, and logging pass/fail gives error
remove test.js as it was being detected by jest but w/o test suites
tried to remove page.close because it is not needed?
reverting to a loop, and
trying to close the page nicely :(
we should not close the page as it is the only page we have
closing it means we can't goto the next url
fix don't make functions within a loop
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants