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

Add basic unit test setup #3078

Conversation

@euantorano
Copy link
Member

@euantorano euantorano commented Mar 24, 2018

This configures PHPUnit and PHPSpec, along with PHPCS for the inc/src folder only. It also adds Travis CI.

There is only one example test provided at the moment, but the plan is to add unit tests for new classes and functions as time goes on.

This configures PHPUnit and PHPSpec, along with PHPCS for the inc/src folder only. It also adds Travis CI.

Signed-off-by: Euan Torano <euantorano@gmail.com>
@euantorano euantorano force-pushed the develop/1.9-add-travis-ci-and-unit-tests branch from db7577a to fe2c04c Mar 24, 2018
@euantorano
Copy link
Member Author

@euantorano euantorano commented Mar 24, 2018

Also included is some code reformatting using phpcbf to ensure the PSR-2 code style is met.

Loading

Signed-off-by: Euan Torano <euantorano@gmail.com>
@euantorano euantorano force-pushed the develop/1.9-add-travis-ci-and-unit-tests branch from 4eecdf6 to 97ca7c6 Mar 24, 2018
@euantorano
Copy link
Member Author

@euantorano euantorano commented Mar 24, 2018

Looks like there's an issue with the lowest build variant, I'll need to try and work out what the problem is - we may use PHPUnit 6 instead of 7 to resolve it...

Loading

Signed-off-by: Euan Torano <euantorano@gmail.com>
@euantorano euantorano merged commit 462893a into mybb:develop/1.9 Mar 24, 2018
1 check passed
Loading
@euantorano euantorano deleted the develop/1.9-add-travis-ci-and-unit-tests branch Mar 24, 2018
euantorano added a commit to euantorano/mybb that referenced this issue Mar 23, 2019
* Add basic unit test setup

This configures PHPUnit and PHPSpec, along with PHPCS for the inc/src folder only. It also adds Travis CI.

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Remove PHP 7.0 from Travis build

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Set minimum stability to dev

Signed-off-by: Euan Torano <euantorano@gmail.com>
euantorano added a commit to euantorano/mybb that referenced this issue Mar 23, 2019
* Add basic unit test setup

This configures PHPUnit and PHPSpec, along with PHPCS for the inc/src folder only. It also adds Travis CI.

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Remove PHP 7.0 from Travis build

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Set minimum stability to dev

Signed-off-by: Euan Torano <euantorano@gmail.com>
euantorano added a commit that referenced this issue Jun 2, 2019
* Add basic unit test setup

This configures PHPUnit and PHPSpec, along with PHPCS for the inc/src folder only. It also adds Travis CI.

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Remove PHP 7.0 from Travis build

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Set minimum stability to dev

Signed-off-by: Euan Torano <euantorano@gmail.com>
euantorano added a commit that referenced this issue Jun 2, 2019
* Add basic unit test setup

This configures PHPUnit and PHPSpec, along with PHPCS for the inc/src folder only. It also adds Travis CI.

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Remove PHP 7.0 from Travis build

Signed-off-by: Euan Torano <euantorano@gmail.com>

* Set minimum stability to dev

Signed-off-by: Euan Torano <euantorano@gmail.com>
@euantorano euantorano mentioned this pull request Jan 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant