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

Tests: Ensure options are not set when tests run. #95

Merged

Conversation

costdev
Copy link
Contributor

@costdev costdev commented Nov 2, 2024

Pull Request

What changed?

  1. The instantiation of \AspireUpdate\Controller is guarded so that it doesn't run when the tests are running.
  2. A new object is used for each test so that state doesn't spill into other tests.

Why did it change?

\AspireUpdate\Controller was instantiated in the main plugin file.
This affected test runs because options were already set.

Did you fix any specific issues?

Fixes #94

CERTIFICATION

By opening this pull request, I do agree to abide by
the CODE OF CONDUCT and be bound by the terms
of the Contribution Guidelines in effect on the date and time
of my contribution as proven by the
revision information in GitHub. I also agree that any previous contributions shall be deemed subject to the terms of the
version in effect on the date and time of this pull request, or any future revisions for pull requests I may submit.
Further, I certify that this work is my own, is original, does not violate the intellectual property of any other person
or entity, and I am not violating any license agreements or contracts I have with any person or entity. Finally, I agree
that this code may be licensed under any license deemed appropraite by AspirePress, including but not
limited to open source, closed source, proprietary or custom licenses, and that such license terms neither violate my
rights or my copyright to this code.

@asirota asirota merged commit bb0d77a into aspirepress:main Nov 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tests: The instantiation of \AspireUpdate\Controller affects tests.
2 participants