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

Add tests for related content score #2214

Merged
merged 1 commit into from
Oct 20, 2019
Merged

Add tests for related content score #2214

merged 1 commit into from
Oct 20, 2019

Conversation

MariaCheca
Copy link
Contributor

@MariaCheca MariaCheca commented Dec 21, 2017 •
edited by javierm
Loading

Objectives

Add tests for related content score validations.

@MariaCheca MariaCheca force-pushed the related-content-tests branch 2 times, most recently from 876d0c4 to 1103ad1 Compare January 8, 2018 15:24
@voodoorai2000 voodoorai2000 changed the title Added 'happy path' tests for related contents Increase test coverage for Related Content Sep 26, 2018
houndci-bot
houndci-bot reviewed Oct 20, 2019
View reviewed changes
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
spec/shared/features/relationable.rb Outdated Show resolved Hide resolved
@javierm javierm self-assigned this Oct 20, 2019
@javierm javierm added the Specs label Oct 20, 2019
@javierm javierm added this to Reviewing in Roadmap via automation Oct 20, 2019
@javierm javierm force-pushed the related-content-tests branch 3 times, most recently from 12fac4f to 28c0ff4 Compare October 20, 2019 12:11
@javierm javierm changed the title Increase test coverage for Related Content Add tests for related content score Oct 20, 2019
Roadmap automation moved this from Reviewing to Testing Oct 20, 2019
@javierm javierm merged commit a9110d2 into master Oct 20, 2019
Roadmap automation moved this from Testing to Release 1.1.0 Oct 20, 2019
@javierm javierm deleted the related-content-tests branch October 20, 2019 16:15
smarques pushed a commit to venetochevogliamo/consul that referenced this pull request Apr 29, 2020
@javierm
Merge pull request consuldemocracy#2214 from consul/related-content-t…
85a46f9 
…ests

Add tests for related content score
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@houndci-bot houndci-bot houndci-bot left review comments

@javierm javierm javierm approved these changes

@bertocq bertocq Awaiting requested review from bertocq

Assignees

@javierm javierm

@MariaCheca MariaCheca

Labels
Specs
Projects
No open projects
Roadmap
  
Release 1.1.0
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@MariaCheca @javierm @houndci-bot