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

Update deprecated Beaker methods #2675

Merged
merged 2 commits into from
Feb 10, 2024

Conversation

mhashizume
Copy link
Contributor

@mhashizume mhashizume commented Jan 30, 2024

This PR updates several deprecated Beaker methods that were removed entirely in Beaker 5. This is in preparation for moving Facter to using Beaker 5 for tests.

@mhashizume mhashizume added the maintenance Maintenance chores are excluded from changelogs label Jan 30, 2024
@mhashizume mhashizume requested a review from a team as a code owner January 30, 2024 19:10
@mhashizume mhashizume marked this pull request as draft January 30, 2024 19:11
@mhashizume mhashizume force-pushed the PA-5022/main/beaker-deprecated branch from 7190022 to e99bd35 Compare January 31, 2024 19:15
@mhashizume mhashizume changed the title Update deprecated Beaker method Update deprecated Beaker methods Jan 31, 2024
@mhashizume mhashizume marked this pull request as ready for review January 31, 2024 19:16
@mhashizume mhashizume force-pushed the PA-5022/main/beaker-deprecated branch from e99bd35 to a41f964 Compare January 31, 2024 23:23
The assert_no_match method in Beaker was long deprecated then eventually
removed altogether in
voxpupuli/beaker@6282311

This commit updates all instances of assert_no_match in Facter's Beaker
tests with refute_match.
@mhashizume mhashizume force-pushed the PA-5022/main/beaker-deprecated branch from a41f964 to 824c0a5 Compare February 1, 2024 23:56
@mhashizume mhashizume force-pushed the PA-5022/main/beaker-deprecated branch 2 times, most recently from 6b9d078 to 9c16c33 Compare February 6, 2024 17:38
@tvpartytonight tvpartytonight mentioned this pull request Feb 9, 2024
@mhashizume mhashizume force-pushed the PA-5022/main/beaker-deprecated branch from 9c16c33 to 8a08f51 Compare February 9, 2024 23:40
Previously in Beaker, you could use standalone stdout and stderr methods
to access output from remote machines.

These methods were deprecated in 2013 with voxpupuli/beaker@28b2510 and
dropped entirely in voxpupuli/beaker@73a31c7.

This commit removes these methods from Facter tests in favor of calling
stdout and stderr methods on Result objects.
@mhashizume mhashizume force-pushed the PA-5022/main/beaker-deprecated branch from 8a08f51 to e4e4f32 Compare February 10, 2024 00:02
@mhashizume mhashizume merged commit f0a6704 into puppetlabs:main Feb 10, 2024
17 checks passed
@mhashizume mhashizume deleted the PA-5022/main/beaker-deprecated branch February 10, 2024 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Maintenance chores are excluded from changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants