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 tests for Controllers #865

Closed
murny opened this issue Sep 18, 2018 · 2 comments
Closed

Add tests for Controllers #865

murny opened this issue Sep 18, 2018 · 2 comments

Comments

@murny
Copy link
Contributor

murny commented Sep 18, 2018

Some of our controllers are missing tests (DownloadsController and ItemsController are probably the two biggest but there is a bunch of others here (profiles/robots/static pages/etc))

We should be testing these actions and verifying they are coming back with the proper responses (in DownloadsController case, are the proper disposition and content_type headers set properly?).

@ConnorSheremeta
Copy link
Contributor

It appears as though DownloadsController has had tests added already (although simplecov is still showing 0% coverage for downloads_controller.rb for some reason.) Would it be safe to assume that this issue can be closed?

@ConnorSheremeta ConnorSheremeta removed their assignment Apr 12, 2019
@pgwillia
Copy link
Member

pgwillia commented Apr 12, 2019

Yeah, looks like the tests for downloads controller were added the day after the issue was created. 2b814d4

There are differences to coverage/index.html after running bundle exec rails test and bundle exec rails test:system

Download Controller looks good
Screenshot from 2019-04-12 13-32-59

There are some gaps that we could improve as part of this ticket (I've renamed it)
Screenshot from 2019-04-12 13-31-54

@pgwillia pgwillia changed the title Add tests for DownloadsController Add tests for Controllers Apr 12, 2019
@ConnorSheremeta ConnorSheremeta self-assigned this Apr 29, 2019
@weiweishi weiweishi reopened this Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants