manageiq
manageiq copied to clipboard
[WIP] Allow creation of non-leaf providers in specs
We don't want to allow the creation of non-leaf providers in production/development but for specs this requirement pushes us to having specific provider plugins be the prototypical providers for these types which goes against our general "no specific providers in specs" even if it doesn't look like it.
This is where I was suggesting that we include the DummyProvider in test, and then the DummyProvider can have one of each kind of manager/provider in it.
Would that be a better approach? Or is this more of a "phase 1" to getting there?
Agreed with the DummyProvider
suggestion.
I wrote stub_supports
to aid in the use of Dummy's.
This pull request has been automatically marked as stale because it has not been updated for at least 3 months.
If these changes are still valid, please remove the stale
label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s)
Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.
This pull request has been automatically marked as stale because it has not been updated for at least 3 months.
If these changes are still valid, please remove the stale
label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s)
Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.
This pull request has been automatically marked as stale because it has not been updated for at least 3 months.
If these changes are still valid, please remove the stale
label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).
This pull request has been automatically marked as stale because it has not been updated for at least 3 months.
If these changes are still valid, please remove the stale
label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).
This pull request has been automatically marked as stale because it has not been updated for at least 3 months.
If these changes are still valid, please remove the stale
label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).