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.

Sign up for GitHub

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

Jump to bottom

Improve test coverage by writing more tests #453

Open
compwron opened this issue Jun 9, 2020 · 3 comments · May be fixed by #776
Open

Improve test coverage by writing more tests #453

compwron opened this issue Jun 9, 2020 · 3 comments · May be fixed by #776
Labels
dev quality of life Improves the ability or comfort of developers to make changes friendly making arooo easier to contribute to good first issue A small change that is a good place to start for someone new to this app and/or coding in general help wanted Issue does not require DU membership to complete maintenance-updates Work to make sure that the app keeps up to date with language and dependencies up-for-grabs Getting listed on https://up-for-grabs.net to get more dev help

Comments

@compwron
Copy link
Member

compwron commented Jun 9, 2020

To view coverage, run rake spec and then open the file coverage/index.html

Screen Shot 2020-06-08 at 8 49 57 PM
@compwron compwron added maintenance-updates Work to make sure that the app keeps up to date with language and dependencies dev quality of life Improves the ability or comfort of developers to make changes friendly making arooo easier to contribute to good first issue A small change that is a good place to start for someone new to this app and/or coding in general help wanted Issue does not require DU membership to complete up-for-grabs Getting listed on https://up-for-grabs.net to get more dev help labels Jun 9, 2020
@uzorjchibuzor
Copy link

Is help still needed on this issue?

@fchatterji
Copy link

Hey, I´d like to help on this issue :)

@fchatterji fchatterji mentioned this issue Feb 13, 2023
Closed
@fchatterji
Copy link

I added a PR with some tests. I think I could go further and add more tests, but this seems to be a good start

@fchatterji fchatterji linked a pull request Feb 20, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dev quality of life Improves the ability or comfort of developers to make changes friendly making arooo easier to contribute to good first issue A small change that is a good place to start for someone new to this app and/or coding in general help wanted Issue does not require DU membership to complete maintenance-updates Work to make sure that the app keeps up to date with language and dependencies up-for-grabs Getting listed on https://up-for-grabs.net to get more dev help
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve test coverage fchatterji/arooo
added controller specs fchatterji/arooo
4 participants
@compwron @fchatterji @uzorjchibuzor @membership-at-du