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

Feature: Domain Mapping Unit Tests #88

Merged
merged 33 commits into from
May 1, 2022

Conversation

cameronterry
Copy link
Owner

@cameronterry cameronterry commented Apr 18, 2022

Adds unit testing to the plugin and the first round of unit tests. At the moment the setup is primarily aimed at working with VVV, and can be run through composer using the following scripts:

# Ensure the dependencies are in place.
composer install

# Setup the unit testing environment
composer run test-setup

# Run the unit tests
composer run test

Also added a Github action which should fire on the develop branch as well as any release/* branch and any PRs pointed at them. Naturally, due to the issues discovered below, it will fail for the time being.

So far, the following issues have been discovered:

@cameronterry cameronterry mentioned this pull request Apr 24, 2022
@cameronterry cameronterry marked this pull request as ready for review April 30, 2022 18:25
@cameronterry
Copy link
Owner Author

Noting the failed unit tests, however merging with the release branch to begin the fixes noted in the description.

Once these are fixed, the unit test action should begin passing.

@cameronterry cameronterry merged commit 1dd34da into release/2.3.0 May 1, 2022
@cameronterry cameronterry deleted the feature/domain-unit-testing branch May 1, 2022 11:41
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.

None yet

1 participant