peterclink / oauth

Automatically exported from code.google.com/p/oauth
0 stars 0 forks source link

[PHP] Several tests broken #131

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
On a fresh svn checkout (r1131) phpunit produces the following output:

PHPUnit 3.4.0 by Sebastian Bergmann.

.....F.............FF...F.F.....F.........FF...........

Time: 0 seconds

There were 8 failures:

1) OAuthRequestTest::testCreateRequestFromConsumerAndToken
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-bar
+

/mnt/hgfs/workspace/php-oauth/tests/OAuthRequestTest.php:92

2) OAuthRequestTest::testBuildSignature
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-kd94hf93k423kf44&pfkkdhi9sl3r4s00
+kd94hf93k423kf44%26pfkkdhi9sl3r4s00

/mnt/hgfs/workspace/php-oauth/tests/OAuthRequestTest.php:277

3) OAuthRequestTest::testSign
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-kd94hf93k423kf44&pfkkdhi9sl3r4s00
+kd94hf93k423kf44%26pfkkdhi9sl3r4s00

/mnt/hgfs/workspace/php-oauth/tests/OAuthRequestTest.php:305

4) OAuthServerTest::testRejectRequestWithMissingParameters
Allowed a request without `oauth_signature_method`

/mnt/hgfs/workspace/php-oauth/tests/OAuthServerTest.php:78

5) OAuthServerTest::testRejectFutureTimestamp
Expected exception OAuthException

6) OAuthServerTest::testRejectUnknownVersion
Expected exception OAuthException

7) OAuthSignatureMethodPlaintextTest::testBuildSignature
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-cs&
+cs%26

/mnt/hgfs/workspace/php-oauth/tests/OAuthSignatureMethodPlaintextTest.php:22

8) OAuthSignatureMethodPlaintextTest::testVerifySignature
Failed asserting that <boolean:false> is true.

/mnt/hgfs/workspace/php-oauth/tests/OAuthSignatureMethodPlaintextTest.php:52

FAILURES!
Tests: 55, Assertions: 139, Failures: 8.

Original issue reported on code.google.com by ithinkihaveacat on 5 Oct 2009 at 9:14

GoogleCodeExporter commented 8 years ago
This was because I committed my updated unit-tests way before the updated 
library. Fixed by revision  r1163.

Original comment by morten.f...@gmail.com on 5 Jan 2010 at 11:32