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

(Gsoc'21) (Week-8,9,10) Adding headless testing #641

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

satyasaibhushan
Copy link
Collaborator

  • Added mocha-chrome as a grunt task, which is borrowed from p5.js.
  • I changed the test filing architecture and the initialisation of expect in all test files so that all the files goes to one spec array.
  • Added karma-js as it seems more consistent with tests.
  • We currently, cannot use it in github automations as it is still not 100 % reliable. Some tests are still failing in some cases.
  • We can add retry functionality to some tests which can be identified as failing and maybe one day in future, we can add it to github actions.

Copy link
Contributor

@Abhijay007 Abhijay007 left a comment

Choose a reason for hiding this comment

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

@satyasaibhushan can you please look into the merge conflicts, if you are busy I can resolve those conflicts and make a new PR fixing merge conflicts around the same :) it will be great if we are able to implement headless testing, as you mentioned various uses cases of the same in the PR description.

@satyasaibhushan
Copy link
Collaborator Author

@Abhijay007 Yeah. Go ahead and resolve the conflicts.

@Abhijay007
Copy link
Contributor

@Abhijay007 Yeah. Go ahead and resolve the conflicts.

sure :)

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.

2 participants