fix: remove the tests' inclusion in the plugin #82
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR removes the inclusion of the test as a dependency for the plugin and also changes the composer.json to map and autoload the test only in dev mode.
Context
We removed the addition of test files in a production plugin in #76, which generated problems because the test module was included in class-openedx-commerce.php. Problem: #62 (comment)
Testing instructions
Download this plugin with these changes: ZIP version, that was generated by this workflow.
Install and activate the plugin in a clean WordPress instance (E. g., You can use Docker or Local)
If you want to go further, you can configure the plugin and test other functions.
Additional information
The URL of the zip file I created for testing this PR has the tag v2.0.3, but it is irrelevant because the workflow uploads the zip file in the latest release of my repo which is using this updated branch.
Ref
Video demo (Using Local)
Installing and activating the plugin in a clean installation
https://github.com/openedx/openedx-wordpress-ecommerce/assets/35668326/e9fb881a-8095-473e-a6b4-6ce1a2acc3bb
Checklist for Merge