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 test comparing classic and graph refresh for vmware #360

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Jan 29, 2019

Still left to fix: ems_folders (network folder is not hidden in classic refresh), switches, and lans

@agrare agrare force-pushed the full_db_comparison_classic_graph_refresh branch 2 times, most recently from eda277d to ee5a98e Compare January 30, 2019 01:15
@agrare agrare force-pushed the full_db_comparison_classic_graph_refresh branch from ee5a98e to edc0d77 Compare January 30, 2019 14:48
@agrare agrare self-assigned this Jan 30, 2019
@agrare agrare force-pushed the full_db_comparison_classic_graph_refresh branch from edc0d77 to 71dd146 Compare January 30, 2019 14:51
Copy link
Contributor

@Ladas Ladas left a comment

Choose a reason for hiding this comment

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

👍 awesome

@miq-bot
Copy link
Member

miq-bot commented Jan 30, 2019

Checked commits agrare/manageiq-providers-vmware@9f00043~...71dd146 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
4 files checked, 8 offenses detected

spec/models/manageiq/providers/vmware/infra_manager/inventory/collector_spec.rb

@Ladas Ladas added test bug and removed test labels Jan 30, 2019
@Ladas Ladas merged commit d104f70 into ManageIQ:master Jan 30, 2019
@Ladas Ladas added this to the Sprint 104 Ending Feb 4, 2019 milestone Jan 30, 2019
@agrare agrare deleted the full_db_comparison_classic_graph_refresh branch January 30, 2019 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants