? ? Success

User tests: Successful: Unsuccessful:

avatar csthomas
csthomas
22 Feb 2016

Pull Request for Issue # .
No issue.

Summary of Changes

From time to time tests fails but everything is OK.

The problem is in /tests/unit/suites/libraries/joomla/log/JLogEntryTest on line 28:

$tmp = new JLogEntry('Lorem ipsum dolor sit amet');
$date = JFactory::getDate('now');

When $tmp->date is "2016-02-22 19:00:01"
and $date is one second later "2016-02-22 19:00:02" then assertion fails.

Testing Instructions

Random fails.

I had situation when my pull request did not pass one php version tests.
When I used commit --amend and pushed force again then all tests passed.

avatar csthomas csthomas - open - 22 Feb 2016
avatar csthomas csthomas - change - 22 Feb 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Feb 2016
Labels Added: ? ?
avatar wilsonge
wilsonge - comment - 23 Feb 2016

Rather than use use the $delta parameter of assertEquals I think rather than this (https://github.com/sebastianbergmann/phpunit/blob/4.8/src/Framework/Assert.php#L503-L514)

avatar csthomas
csthomas - comment - 24 Feb 2016

Ok, assertEquals

avatar wilsonge wilsonge - change - 24 Feb 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-02-24 09:22:08
Closed_By wilsonge
avatar wilsonge wilsonge - reference | 5ed9f2d - 24 Feb 16
avatar wilsonge wilsonge - merge - 24 Feb 2016
avatar wilsonge wilsonge - close - 24 Feb 2016
avatar wilsonge
wilsonge - comment - 24 Feb 2016

Merged - thanks!

avatar wilsonge wilsonge - change - 24 Feb 2016
Milestone Added:

Add a Comment

Login with GitHub to post a comment