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 a spec for chart click-through to hosts. #2751

Merged
merged 2 commits into from
Nov 20, 2017

Conversation

martinpovolny
Copy link
Member

@martinpovolny martinpovolny commented Nov 16, 2017

spec for:
#2740

requires:
#2727

pending rebase

@miq-bot add_label gaprindashvili/yes
@miq-bot add_label test

@martinpovolny
Copy link
Member Author

ping @karelhala

@martinpovolny martinpovolny mentioned this pull request Nov 16, 2017
@miq-bot
Copy link
Member

miq-bot commented Nov 20, 2017

This pull request is not mergeable. Please rebase and repush.

@martinpovolny
Copy link
Member Author

@karelhala, @romanblanco : please, review!

@miq-bot
Copy link
Member

miq-bot commented Nov 20, 2017

Some comments on commits martinpovolny/manageiq-ui-classic@40cb120~...f90ff05

spec/controllers/host_controller_spec.rb

  • ⚠️ - 300 - Detected expect_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.

@miq-bot
Copy link
Member

miq-bot commented Nov 20, 2017

Checked commits martinpovolny/manageiq-ui-classic@40cb120~...f90ff05 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍰

@mzazrivec mzazrivec self-assigned this Nov 20, 2017
@mzazrivec mzazrivec added this to the Sprint 74 Ending Nov 27, 2017 milestone Nov 20, 2017
@mzazrivec mzazrivec merged commit da20aec into ManageIQ:master Nov 20, 2017
simaishi pushed a commit that referenced this pull request Nov 21, 2017
Add a spec for chart click-through to hosts.
(cherry picked from commit da20aec)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 9cf3d6437c810f2a1667e4820a854c593a9fdc33
Author: Milan Zázrivec <[email protected]>
Date:   Mon Nov 20 16:56:20 2017 +0100

    Merge pull request #2751 from martinpovolny/click_through_spec
    
    Add a spec for chart click-through to hosts.
    (cherry picked from commit da20aec420b3b6a12be4b024a8da6c80af37debd)

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.

4 participants