Skip to content

Commit

Permalink
Merge pull request #11438 from creative-commoners/pulls/5.3/fix-unit-…
Browse files Browse the repository at this point in the history
…test

MNT Fix unit test
  • Loading branch information
GuySartorelli authored Oct 22, 2024
2 parents d7fa531 + 6df8732 commit fe39a2a
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions tests/php/Security/VersionedMemberAuthenticatorTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,14 @@ protected function setUp(): void
return;
}

// Explicity add the Versioned extension to Member, even though it's already in $required_extensions.
// This is done to call `unset(self::class::$extra_methods[strtolower($subclass)]);` in
// Extensible::add_extension() so when CustomMethods::getExtraMethodConfig() updates the $extra_methods
// it will include methods of Versioned such as publishSingle()
// This issue will only occur when running subsequent unit test classes in the same process, rather than this
// this unit test class in isolation
Member::add_extension(Versioned::class);

// Enforce dummy validation (this can otherwise be influenced by recipe config)
PasswordValidator::singleton()
->setMinLength(0)
Expand Down

0 comments on commit fe39a2a

Please sign in to comment.