Instead of assertResponse, system.module makes two calls to assertReponse. This effectively keeps a fresh installation of D7 from being able to pass its own tests.

CommentFileSizeAuthor
#1 656300-1-fix-assertreponse-typo.patch997 bytesAnonymous (not verified)

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs review
StatusFileSize
new 656300-1-fix-assertreponse-typo.patch997 bytes
ksenzee’s picture

ksenzee
she/her
commented
Status: Needs review » Reviewed & tested by the community

Ouch.

ksenzee’s picture

ksenzee
she/her
commented
Component: simpletest.module » system.module

Status: Reviewed & tested by the community » Needs review

sun requested that failed test be re-tested.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community 
+++ modules/system/system.test	(working copy)
@@ -554,7 +554,7 @@ class AccessDeniedTestCase extends DrupalWebTestCa
-    $this->assertReponse(403);
+    $this->assertResponse(403);

Effectively, this should have thrown a fatal error, and even with #653940: Clean-up: Tests do not report all errors, the testbot does not return fatal errors.

Good to see that the rest of the test still passes.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.