Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[#434] Acceptance tests for mysql_database #459

Merged
merged 1 commit into from Feb 20, 2014

Conversation

Projects
None yet
2 participants
Member

dcarley commented Feb 20, 2014

Includes a test for the regression fixed in 30ce3e0.

NB: The ensure => present on the resource without any charset or
collate arguments is intentional because of issue described in #458

[#434] Acceptance tests for mysql_database
Includes a test for the regression fixed in 30ce3e0.

NB: The `ensure => present` on the resource without any `charset` or
`collate` arguments is intentional because of issue described in #458
Contributor

apenney commented Feb 20, 2014

So many weird things to fix, soon as we're done with all this CI work I'm going to get time to finally get back to fixing mysql properly and make it less of an embarrassment! Thanks for this test.

apenney pushed a commit that referenced this pull request Feb 20, 2014

Merge pull request #459 from dcarley/434-acceptance_tests_for_mysql_d…
…atabase

[#434] Acceptance tests for mysql_database

@apenney apenney merged commit 1454a5b into puppetlabs:master Feb 20, 2014

1 check was pending

default The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment