Skip to content
This repository has been archived by the owner on Jan 6, 2023. It is now read-only.

Add tests for config_options and provider_options #23

Merged
merged 1 commit into from May 30, 2020
Merged

Add tests for config_options and provider_options #23

merged 1 commit into from May 30, 2020

Conversation

apatard
Copy link
Member

@apatard apatard commented May 2, 2020

This pull request depends on #16 since it's the missing test cases for the two bugs fixed by it.

This means that:

  • the test will fail now but will have to be triggered again once the other pull request is merged
  • it should not be merged now (hopefully, the CI will prevent that)

@apatard
molecule_vagrant/test/: Add tests for config_options and provider_opt…
532792e 
…ions

Add tests to make sure that config_options and provider_options
are taken into account.

Signed-off-by: Arnaud Patard <apatard@hupstream.com>
@apatard apatard requested a review from ssbarnea May 2, 2020 14:06
@apatard
Copy link
Member Author

apatard commented May 4, 2020

recheck

1 similar comment
@apatard
Copy link
Member Author

apatard commented May 4, 2020

recheck

@ssbarnea ssbarnea added the gate Gate PR in Zuul CI label May 4, 2020
@apatard
Copy link
Member Author

apatard commented May 29, 2020

recheck

2 similar comments
@apatard
Copy link
Member Author

apatard commented May 29, 2020

recheck

@apatard
Copy link
Member Author

apatard commented May 30, 2020

recheck

@ansible-zuul ansible-zuul bot merged commit 9f96a17 into ansible-community:master May 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ssbarnea ssbarnea ssbarnea approved these changes

Assignees
No one assigned
Labels
gate Gate PR in Zuul CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@apatard @ssbarnea