ZF-10869: Zend_Http_UserAgent_AbstractDeviceTest::testMatchMobileOtherHeaders() unit test failure.

Issue Type: Unit Tests: Problem Created: 2010-12-27T04:27:24.000+0000 Last Updated: 2010-12-28T11:53:33.000+0000 Status: Resolved Fix version(s): - 1.11.2 (30/Dec/10)

Reporter: Alexander Veremyev (alexander) Assignee: Matthew Weier O'Phinney (matthew) Tags: - Zend_Http_UserAgent

Related issues: Attachments:


{panel} 1) Zend_Http_UserAgent_AbstractDeviceTest::testMatchMobileOtherHeaders Failed asserting that is true. {panel}

The same error is generated by different code within trunk and 1.11 release branch:

trunk: {panel} $userAgent = 'Opera/5.12 (Macintosh;US;PPC) [en]'; $this->assertTrue(Zend_Http_UserAgent_Mobile::match($userAgent, array( 'HTTP_USER_AGENT' => $userAgent ))); {panel}

{panel} $server = array( 'http_accept' => 'midp' ); $this->assertTrue(Zend_Http_UserAgent_Mobile::match($userAgent, $server)); {panel}

It seems, that the problem is produced by some of latest fixes, but corresponding changes were not merged into release branch.


Posted by Alexander Veremyev (alexander) on 2010-12-27T04:31:39.000+0000

PS It looks like large enough part of new commits were not merged into release branch yet. That would be good to have it done, since new release is coming soon.

Posted by Matthew Weier O'Phinney (matthew) on 2010-12-28T11:53:29.000+0000

Fixed on the 1.11 release branch; had to merge 3 commits from trunk to the branch.

Have you found an issue?

See the Overview section for more details.


© 2006-2018 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.