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

Create test for feed-parser #72

Closed
Immutablevoid opened this issue Nov 7, 2019 · 1 comment · Fixed by #325
Closed

Create test for feed-parser #72

Immutablevoid opened this issue Nov 7, 2019 · 1 comment · Fixed by #325
Assignees
Labels
type: test Creation and development of test

Comments

@Immutablevoid
Copy link
Contributor

Immutablevoid commented Nov 7, 2019

A test is needed for feed-parser once #37 is merge. You can start working on this now, can be a basic check to see if feed-parser returns values.

Can check if the feed-parser is able to parser these blogs below:

feed-queue.test.js can also be removed once some actual tests are added since the test doesn't accomplish much #55.

This doesn't need to be all implemented in one pull request, can be separated into multiple issues.

Test are written in Jest.
Commands to run test: npm test or npm run jest

Save test under test folder.
Save test as testname.test.js replacing testname with the name of your test.

@Immutablevoid Immutablevoid changed the title Creating test for feed-parser Create test for feed-parser Nov 7, 2019
@nazneennahar
Copy link
Contributor

hey i would wanna work on this issue.

Thanks.

@Immutablevoid Immutablevoid added the type: test Creation and development of test label Nov 7, 2019
sukhbeersingh added a commit that referenced this issue Nov 25, 2019
Closes #72: Adds more test cases to feedParser.test.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: test Creation and development of test
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants