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

Fixes #19761 - revert use of new call for applicability #6812

Merged
merged 1 commit into from
Jun 8, 2017

Conversation

jlsherrill
Copy link
Member

This reverts commit 7e9e55c.

@mention-bot
Copy link

@jlsherrill, thanks for your PR! By analyzing the history of the files in this pull request, we identified @parthaa, @beav and @johnpmitsch to be potential reviewers.

@@ -8,13 +8,13 @@ class ConsumerTestBase < ActiveSupport::TestCase
include VCR::TestCase

let(:uuid) { 'uuid' }
let(:consumer_name) { 'gregor' }
let(:name) { 'name' }
Copy link
Contributor

@beav beav Jun 6, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would leave this change in. If you define a var named 'name', it will make the test pass 100% of the time, even with failures.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but I 💓 auto-passing tests!

@beav
Copy link
Contributor

beav commented Jun 6, 2017

APJ

@ehelms
Copy link
Member

ehelms commented Jun 6, 2017

Test failure fun

@jlsherrill jlsherrill merged commit b83f828 into Katello:KATELLO-3.4 Jun 8, 2017
@jlsherrill jlsherrill deleted the 19761 branch June 8, 2017 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants