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

Upgrade from FactoryGirl to FactoryBot #558

Merged
merged 1 commit into from
Mar 7, 2019
Merged

Upgrade from FactoryGirl to FactoryBot #558

merged 1 commit into from
Mar 7, 2019

Conversation

juliancheal
Copy link
Member

No description provided.

@djberg96
Copy link
Contributor

djberg96 commented Mar 5, 2019

Hm, thought I did this already in #534. I guessed I missed one?

Ah, Libor added it in a485c9a#diff-7d8aa00e248fc954da2ec96ccc62faf3 after I had done the main change.

@kbrock
Copy link
Member

kbrock commented Mar 5, 2019

  1) ConversionHosts API delete can delete a conversion host via DELETE
     Failure/Error: factory_exists?(factory) ? super : class_from_symbol(factory).create!(*args)
     
     ActiveRecord::RecordInvalid:
       Validation failed: ConversionHost: Resource Feature not available/supported
     # ./spec/manageiq/spec/support/missing_factory_helper.rb:10:in `create'

@djberg96
Copy link
Contributor

djberg96 commented Mar 5, 2019

@kbrock I'm not seeing that locally, though. Maybe @juliancheal needs to rebase?

@juliancheal
Copy link
Member Author

Rebased :)

@djberg96
Copy link
Contributor

djberg96 commented Mar 5, 2019

Yeah, now you're seeing the same two errors that I saw locally.

@kbrock
Copy link
Member

kbrock commented Mar 6, 2019

@juliancheal Looks like master had the same problem. This was fixed 20 hours ago - please re-kick

This error feels very unrelated

@bdunne bdunne closed this Mar 7, 2019
@bdunne bdunne reopened this Mar 7, 2019
@miq-bot
Copy link
Member

miq-bot commented Mar 7, 2019

Checked commit https://github.com/juliancheal/manageiq-api/commit/e609822b5e7ab26ac167e8365bdaeb785290dc00 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@bdunne bdunne merged commit 272ca98 into ManageIQ:master Mar 7, 2019
@bdunne bdunne self-assigned this Mar 7, 2019
@bdunne bdunne added this to the Sprint 107 Ending Mar 18, 2019 milestone Mar 7, 2019
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