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

Support other versions of remi repo on redhat #495

Merged
merged 0 commits into from
Dec 11, 2022

Conversation

edestecd
Copy link
Contributor

No description provided.

@bastelfreak
Copy link
Member

Hi @edestecd, thanks for the PR. Can you provide acceptance tests that use different values for the parameter?

@bastelfreak bastelfreak added enhancement New feature or request needs-tests labels Nov 2, 2018
@edestecd
Copy link
Contributor Author

edestecd commented Nov 6, 2018

@bastelfreak tests added

@Dan33l
Copy link
Member

Dan33l commented Nov 6, 2018

Hi @edestecd you added unit tests, and you can let the file in the PR. This is interesting to have more tests. But @bastelfreak talk about acceptance tests not unit tests.
If you need some guidance, fell free to join us on slack or IRC #Voxpupuli for interactive talks.

@edestecd
Copy link
Contributor Author

edestecd commented Nov 6, 2018

@Dan33l I didn't see any acceptance tests for debian or ubuntu for example. SO i don't know how you usually write them. I'll try and have a look at some others.

@Dan33l
Copy link
Member

Dan33l commented Nov 6, 2018

In this module, you can find :

  • the helper_acceptance. The role of the helper is to prepare system under tests (SUT) to be ready for being able to receive tests.
  • 2 spec files making tests php56_spec.rb and php_spec.rb
  • the file .travis.yml that permit to run tests defined by previous items. All lines with BEAKER are acceptance tests executed for each PR.

So tests about all supported OSes should be present. For instance this line launch tests under CentOS7 with puppet5

You can run tests locally on you work station. The CONTRIBUTING.md file gives some guidance about this in the section intergation tests

You can find also some guidance on serverspec.org about how to tests items that are expected to be present or absent on the SUT.

@vox-pupuli-tasks
Copy link

Dear @edestecd, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@kenyon kenyon merged commit 0d33a0e into voxpupuli:master Dec 11, 2022
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.

4 participants