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

Replace test data #32

Merged
merged 1 commit into from
Sep 24, 2018
Merged

Replace test data #32

merged 1 commit into from
Sep 24, 2018

Conversation

tadeboro
Copy link
Contributor

@tadeboro tadeboro commented Sep 20, 2018

Old test data was missing quite a few pieces of information, which made it hard to test the inventory refresh process thoroughly.

New set of data contains data of better quality:

  • has more than one server,
  • servers actually contain some disks,
  • servers are placed in racks/chassis,
  • chassis are nested,
  • components contains location data.

Depends on #31

@miq-bot miq-bot changed the title Replace test data [WIP] Replace test data Sep 20, 2018
@miq-bot miq-bot added the wip label Sep 20, 2018
Old test data was missing quite a few pieces of information, which
made it hard to test the inventory refresh process thoroughly.

New set of data contains data of better quality:

  * has more than one server,
  * servers actually contain some disks,
  * servers are placed in racks/chassis,
  * chassis are nested,
  * components contains location data.
@tadeboro
Copy link
Contributor Author

@miq-bot remove_label wip
@miq-bot assign @agrare

@miq-bot miq-bot changed the title [WIP] Replace test data Replace test data Sep 24, 2018
@miq-bot miq-bot removed the wip label Sep 24, 2018
@miq-bot
Copy link
Member

miq-bot commented Sep 24, 2018

Checked commit xlab-si@cfbf65f with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
10 files checked, 1 offense detected

**

  • 💣 💥 🔥 🚒 - Linter/Yaml - missing config files

@agrare
Copy link
Member

agrare commented Sep 24, 2018

@tadeboro can you look at the spec failure?

@tadeboro tadeboro closed this Sep 24, 2018
@tadeboro tadeboro reopened this Sep 24, 2018
@agrare agrare added the test label Sep 24, 2018
@agrare agrare merged commit d8ba414 into ManageIQ:master Sep 24, 2018
@agrare agrare added this to the Sprint 95 Ending Sep 24, 2018 milestone Sep 24, 2018
@tadeboro tadeboro deleted the replace-test-data branch September 24, 2018 20:48
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