Skip to content

Port old test changes to new pytest test_core.py from PR 60514#60588

Merged
garethgreenaway merged 1 commit intosaltstack:masterfrom
dmurphy18:upd_test_core_60587
Jul 21, 2021
Merged

Port old test changes to new pytest test_core.py from PR 60514#60588
garethgreenaway merged 1 commit intosaltstack:masterfrom
dmurphy18:upd_test_core_60587

Conversation

@dmurphy18
Copy link
Copy Markdown
Contributor

What does this PR do?

Adds recent changes to old test file from PR 60514 to new pytest migrated version of test

What issues does this PR fix or reference?

Fixes: #60587

New Behavior

Include changes for Solaris from old test

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

@dmurphy18 dmurphy18 added bug broken, incorrect, or confusing behavior solaris affects this operating system labels Jul 21, 2021
@dmurphy18 dmurphy18 requested a review from a team as a code owner July 21, 2021 16:10
@garethgreenaway garethgreenaway merged commit d66cd3c into saltstack:master Jul 21, 2021
@dmurphy18 dmurphy18 deleted the upd_test_core_60587 branch May 3, 2023 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug broken, incorrect, or confusing behavior solaris affects this operating system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Need to pull PR https://github.com/saltstack/salt/pull/60514 in new pytest test_core

2 participants