?
Related to # 3123

User tests: Successful: Unsuccessful:

avatar fanno
fanno
4 May 2015

This is a redone pull request per request.

avatar fanno fanno - open - 4 May 2015
avatar nikosdion
nikosdion - comment - 4 May 2015

This PR is fine by me :) :+1:

avatar fanno
fanno - comment - 4 May 2015

not sure if it matter or not but i tried to run a unit test:

Time: 45.27 seconds, Memory: 184.25Mb


FAILURES!
Tests: 5704, Assertions: 10160, Failures: 13, Errors: 216, Incomplete: 1, Skipped: 69.

Process finished with exit code 2

looks like it is ome database thing incorrect i and trying to figure it out

avatar zero-24 zero-24 - change - 4 May 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 4 May 2015
Labels Added: ?
avatar brianteeman brianteeman - change - 4 May 2015
Rel_Number 0 3123
Relation Type Related to
avatar zero-24 zero-24 - change - 4 May 2015
Category Authentication Plugins
avatar zero-24 zero-24 - change - 4 May 2015
Status New Pending
avatar nikosdion
nikosdion - comment - 5 May 2015

Do you get Unit Test failures for the code you touched, or any code which uses the code you touched?

avatar fanno
fanno - comment - 5 May 2015

only thing i toutch other than than this pt, is to run composer, copy "phpunit.xml" and uncomment settings for DB inside. and run it. (i have db created, and user), still looking in to what i do wrong.

I don't see and code for auth plugin?

avatar nikosdion
nikosdion - comment - 5 May 2015

You should start with looking at the PHPUnit output. See which tests are failing. Track them down and see if these are failures that have to do with your code or not.

avatar fanno
fanno - comment - 5 May 2015

I fixed the incorrect UT config setup

Time: 54.6 seconds, Memory: 180.00Mb


FAILURES!
Tests: 5704, Assertions: 10212, Failures: 1, Incomplete: 1, Skipped: 264.

Process finished with exit code 1
Failed asserting that false matches expected '�bc �de EUR100'.
Expected :�bcde EUR100
Actual   :false
 <Click to see difference>

 /home/fanno/workspace/phpstorm/fanno/joomla-cms/tests/unit/suites/libraries/joomla/string/JStringTest.php:815
 phar:///home/fanno/workspace/phpunit/phpunit-lts.phar/phpunit/TextUI/Command.php:176

should now be gtg.

avatar nikosdion
nikosdion - comment - 5 May 2015

Yes, indeed. It looks that the only failing test is one which depends on unicode string manipulation that has nothing to do with your PR.

avatar fanno
fanno - comment - 5 May 2015

I had enabled all databases but only have system configured for mysql, and the utuser premissions was incorrect =) been a long time since i ran this ut last

avatar wilsonge wilsonge - change - 5 May 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-05-05 23:11:59
Closed_By wilsonge
avatar wilsonge wilsonge - close - 5 May 2015
avatar wilsonge wilsonge - reference | 075d827 - 5 May 15
avatar wilsonge wilsonge - merge - 5 May 2015
avatar wilsonge wilsonge - close - 5 May 2015
avatar wilsonge
wilsonge - comment - 5 May 2015

Merged on review

avatar wilsonge wilsonge - change - 5 May 2015
Milestone Added:

Add a Comment

Login with GitHub to post a comment