Skip to content

Refactor puppetserver spec testing#648

Merged
mmoll merged 1 commit intotheforeman:masterfrom
ekohl:refactor-puppetserver-spec-testing
Nov 2, 2018
Merged

Refactor puppetserver spec testing#648
mmoll merged 1 commit intotheforeman:masterfrom
ekohl:refactor-puppetserver-spec-testing

Conversation

@ekohl
Copy link
Copy Markdown
Member

@ekohl ekohl commented Nov 1, 2018

This is in preparation of dropping support for puppetserver < 2.7

@ekohl ekohl force-pushed the refactor-puppetserver-spec-testing branch from 46112fc to c76955c Compare November 2, 2018 00:05
@ekohl ekohl force-pushed the refactor-puppetserver-spec-testing branch from c76955c to fa12a9e Compare November 2, 2018 10:58
@ekohl
Copy link
Copy Markdown
Member Author

ekohl commented Nov 2, 2018

@mmoll the centos 6 acceptance test has become unstable but I think that doesn't block this PR.

@mmoll
Copy link
Copy Markdown
Contributor

mmoll commented Nov 2, 2018

on small question, 👍 otherwise

This uses params inheritance for much cleaner tests. We also test via
the main class and remove a lot of overrides that aren't needed.
@ekohl ekohl force-pushed the refactor-puppetserver-spec-testing branch from fa12a9e to 52397d3 Compare November 2, 2018 19:23
@mmoll mmoll merged commit 91baeb8 into theforeman:master Nov 2, 2018
@mmoll
Copy link
Copy Markdown
Contributor

mmoll commented Nov 2, 2018

merged, bedankt @ekohl!

@ekohl ekohl deleted the refactor-puppetserver-spec-testing branch November 19, 2018 12:51
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.

3 participants