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

Split comments and debates admin tests #3844

Merged
merged 2 commits into from
Nov 10, 2019
Merged

Split comments and debates admin tests #3844

merged 2 commits into from
Nov 10, 2019

Conversation

javierm
Copy link
Member

@javierm javierm commented Nov 10, 2019

References

Objectives

  • Have separate tests for comments and hidden comments, and for debates and hidden debates
  • Add tests for admin comments and debates index

@javierm javierm added the Specs label Nov 10, 2019
@javierm javierm self-assigned this Nov 10, 2019
@javierm javierm added this to Reviewing in Roadmap via automation Nov 10, 2019
@javierm javierm changed the title Split comment and debates admin tests Split comments and debates admin tests Nov 10, 2019
When we renamed hidden controllers in the admin section in commit
85722eb we forgot to rename the test files as well.
@javierm javierm moved this from Reviewing to Testing in Roadmap Nov 10, 2019
@javierm javierm merged commit 2228ac3 into master Nov 10, 2019
Roadmap automation moved this from Testing to Release 1.1.0 Nov 10, 2019
@javierm javierm deleted the hidden_tests branch November 10, 2019 12:00
smarques pushed a commit to venetochevogliamo/consul that referenced this pull request Apr 29, 2020
Split comments and debates admin tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Roadmap
  
Release 1.1.0
Development

Successfully merging this pull request may close these issues.

None yet

1 participant