PHPUnit, Selenium Basic Test Fails with Fatal Error

Question

I am running PHP 5.3.6 and the latest version of PHPUnit from Github. When I copy example 17.1 from the docs, it suffers a fatal error when the assertTitle fails. I get this error message:

Fatal error: Call to a member function toString() on a non-object in <path>/phpunit/phpunit-selenium/PHPUnit/Extensions/SeleniumTestCase.php on line 1041

When I change the assertion to pass, PHPUnit runs just fine.

I dug up the line and this is the snippet :

protected function onNotSuccessfulTest(Exception $e)
    {
        if ($e instanceof PHPUnit_Framework_ExpectationFailedException) {
            $buffer  = 'Current URL: ' . $this->drivers[0]->getLocation() .
                       "\n";
            $message = $e->getComparisonFailure()->toString();

$e->getComparisonFailure() returns NULL, not an object. What am I doing incorrectly?

UPDATE:

I found the reason for the failure, although a fix isn't on my horizon yet.

On line 92 of PHPUnit/Framework/Constraint.php, it calls using

$this->fail($other,$description)

which is defined as:

protected function fail($other, $description, PHPUnit_Framework_ComparisonFailure $comparisonFailure = NULL)

Since no PHPUnit_Framework_ComparisonFailure is passed, NULL is passed on line 141 of PHPUnit/Framework/Constraint.php

throw new PHPUnit_Framework_ExpectationFailedException( $failureDescription, $comparisonFailure );

which is fetched by getComparisonFailure() which is supposed to return an object, as described above.

Any more ideas?

Your saying the default, copy-paste, example of phpunit-selenium has a bug when the only assertion in it fails which when copy and pasted should fail? — Eric Cope
– Eric Cope, Commented Oct 31, 2011 at 14:19

Klaidi · Accepted Answer · 2011-11-01 21:20:16Z

4

Just replace $message = $e->getComparisonFailure()->toString(); with $message = $e->getComparisonFailure()->exceptionToString;

It works fine for me

answered Nov 1, 2011 at 21:20

Klaidi

561 bronze badge

Sign up to request clarification or add additional context in comments.

2 Comments

Eric Cope Over a year ago

that doesn't work for me because the object returned by $e->getComparisonFailure is NULL, not an object with "exceptionToString" as a method.

Eric Cope Over a year ago

I changed it to exceptionToString from exceptionToString() and it worked. Is this a bug in PHPUnit that requires a pull request?

HNygard · Accepted Answer · 2011-11-13 02:05:02Z

This problem relates to issue #66 on the PHPUnit-Selenium project on Github. I've made one commit that fixes this problem and one that fixes the "setCustomMessage()" problem which Matthew mentioned. I think both problems where introduced by one commit in PHPUnit 3.6

$message = $e->getComparisonFailure()->toString();

Where changed to

// ?: Is "comparison failure" set and therefore an object? (fixes issue #66)
if(is_object($e->getComparisonFailure())) {
    // -> Comparison failure is and object and should therefore have the toString method
    $message = $e->getComparisonFailure()->toString();
}
else {
    // -> Comparison failure is not an object. Lets use exception message instead
    $message = $e->getMessage();
}

And at the end of onNotSuccessfulTest():

$e->setCustomMessage($buffer);

Where changed to:

// ?: Does the setCustomMessage method exist on the exception object? (fixes issue #67)
if(method_exists($e, 'setCustomMessage')) {
    // -> Method setCustomMessage does exist on the object, this means we are using PHPUnit 
    //    between 3.4 and 3.6
    $e->setCustomMessage($buffer);
}
else {
    // -> Method setCustomerMessages does not exist on the object, must make a new exception to 
    //    add the new message (inc. current URL, screenshot path, etc)
    $e = new PHPUnit_Framework_ExpectationFailedException($buffer, $e->getComparisonFailure());
}

Hopefully both changes will be accepted into the PHPUnit-Selenium project. At least it fixed the problems I where having with the latest updates for PHPUnit and PHPUnit-Selenium from PEAR.

Matthew · Accepted Answer · 2011-11-04 07:53:40Z

1

I replaced $message = $e->getComparisonFailure()->toString(); to be:

if($e->getComparisonFailure() == NULL)
{
    $message = $e;
}
else
{
    $message = $e->getComparisonFailure()->toString();
}

I also reverted PHPUnit_Framework_ExpectationFailedException back to 3.5.3 so it has setCustomMessage()

answered Nov 4, 2011 at 7:53

Matthew

111 bronze badge

Collectives™ on Stack Overflow

PHPUnit, Selenium Basic Test Fails with Fatal Error

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related