? ? Success

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
4 Aug 2016

This is a fork/rebase/redoing of the great work started by @wilsonge in #4071

**incomplete - but creating the PR so that travis can start to churn the tests **

avatar joomla-cms-bot joomla-cms-bot - change - 4 Aug 2016
Category Unit Tests
avatar PhilETaylor PhilETaylor - open - 4 Aug 2016
avatar PhilETaylor PhilETaylor - change - 4 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Aug 2016
Labels Added: ? ?
avatar wilsonge
wilsonge - comment - 4 Aug 2016

Love it!

avatar PhilETaylor
PhilETaylor - comment - 5 Aug 2016

THE TRAVIS BUILD PASSED - WOOT WOOT !!! Even PHP 5.3!!!

avatar brianteeman
brianteeman - comment - 5 Aug 2016

Hope these links help to collect a list of scenario to test

https://github.com/joomla/joomla-cms/pull/4182#issuecomment-53395318
https://github.com/joomla/joomla-cms/pull/3735#issue-35215540
avatar PhilETaylor
PhilETaylor - comment - 5 Aug 2016

Ok I think this can be tested and merged so that @wilsonge or @dgt41 can run with it, Im on holiday next week and this weekend is manic with the kids activities

There is inline documentation to understand the concepts

It doesnt provide full 100% unit coverage, its meant to be an example and framework for someone to run with.

To run just these tests, go to your joomla-cms folder and run

phpunit --testsuite plugins --stop-on-failure

avatar PhilETaylor
PhilETaylor - comment - 5 Aug 2016

Hope these links help to collect a list of scenario to test //@brianteeman

Thanks - I added them as comments inline

avatar wilsonge wilsonge - change - 5 Aug 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-05 15:17:41
Closed_By wilsonge
avatar wilsonge
wilsonge - comment - 5 Aug 2016

As this doesn't affect any production code I'm merging this to run with it over the weekend. Thanks for picking up on that old code Phil!!

avatar PhilETaylor
PhilETaylor - comment - 5 Aug 2016

No worries - enjoy, like I said its far from a finished work and doesnt really "cover" the class its testing at all, but it shows how you could implemented what you were aiming for in a correct phpunit way with assertions.

Add a Comment

Login with GitHub to post a comment