Skip to content

Conversation

@hellofromtonya
Copy link
Collaborator

@hellofromtonya hellofromtonya commented Mar 21, 2019

Adds the following:

  • Debug Toolkit - (replacing UpDevTools)
  • Adds Composer to Central Hub plugin
  • Adds bootstrapping for PHP Unit Test Suite to Central Hub plugin
  • Adds bootstrapping for PHP Integration Test Suite to Central Hub plugin

Unit Tests: runs the plugin by itself without WordPress and any other plugin.

Integration Tests: runs WordPress, the theme, and the plugin, i.e. to test the plugin in an integrated environment.

@hellofromtonya hellofromtonya changed the title [WIP] Bootstrap Test Suites for Central Hub plugin Bootstrap Test Suites for Central Hub plugin Mar 21, 2019
* License URI: https://www.gnu.org/licenses/gpl-2.0.html
* Text Domain: central_hub
* Requires WP: 4.8
* Requires PHP: 5.6
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rgadon107 I fixed the plugin's file header here as the formatting was off.

}
}

autoload();
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added empty line at the end of the PHP file, as this will be required for WPCS, i.e. once we add the code validators/sniffers.

colors="true"
verbose="true">
<testsuites>
<testsuite name="unit">
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This XML file is for the unit tests. It's in the root of the plugin.

@hellofromtonya hellofromtonya removed the request for review from rgadon107 March 22, 2019 19:23
@hellofromtonya hellofromtonya changed the title Bootstrap Test Suites for Central Hub plugin [WIP] Bootstrap Test Suites for Central Hub plugin Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants