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

Add testing infrastructure #42

Closed
humphd opened this issue Nov 5, 2019 · 2 comments
Closed

Add testing infrastructure #42

humphd opened this issue Nov 5, 2019 · 2 comments
Projects

Comments

@humphd
Copy link
Contributor

@humphd humphd commented Nov 5, 2019

We're starting to get some code in the repo, and it's time to think about how we'll test all the features we add.

I'd like to see us get an initial test in place, so we can figure out our infrastructure, and let others add more tests in future PRs.

Let's use Mocha or Jest to automate our tests. At first, we need a simple npm test run that runs one test against something like the feed parser function, or something else really basic: call into a function and it expects to get back a given result.

@Reza-Rajabi Reza-Rajabi added this to Suggested/Discussion in Main Nov 5, 2019
@Reza-Rajabi Reza-Rajabi moved this from Suggested/Discussion to To do in Main Nov 6, 2019
@ImmutableBox

This comment has been minimized.

Copy link
Collaborator

@ImmutableBox ImmutableBox commented Nov 6, 2019

I would like to try my hand at this. After some research/testing I decide to use Jest. My reasoning:

  • Jest setup was easier to setup.
  • Jest sent a warning message about problems with async operations while Mocha did not.

Mocha does seem to be more flexible, but the setup takes some time. Maybe in later development there can be a switch, but for a starting point ill be using Jest .

@ImmutableBox

This comment has been minimized.

Copy link
Collaborator

@ImmutableBox ImmutableBox commented Nov 7, 2019

Merged with #55.

Main automation moved this from To do to Closed Nov 7, 2019
@Reza-Rajabi Reza-Rajabi added the on-board label Nov 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Main
Closed
3 participants
You can’t perform that action at this time.