feat(assert): Add `Assert::notSame` #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

roxblnfk merged 3 commits into php-testo:master from petrdobr:feat/assert-not-same

Oct 24, 2025

Contributor

petrdobr commented Oct 23, 2025

What was changed

Added Assert::notSame(mixed $expected, mixed $actual, string $message = '').
Added AssertException::notSame(...) with consistent payload/message.
Covered with an assertion in existing test case (simpleAssertions()).

Why?

Complements assertSame with a strict non-identity check; common need in specs and keeps reporter/collector output consistent.

Checklist

Tested
- Tested manually
- Unit tests added (expanded existing Assert test)
Documentation


          feat(assert): Add Assert::notSame (+fix typo)

6aa4818

petrdobr force-pushed the feat/assert-not-same branch from 8823d7a to 6aa4818 Compare

October 23, 2025 21:07

roxblnfk requested changes

View reviewed changes

src/Assert/State/AssertException.php Outdated Show resolved Hide resolved


          fix: use AssertException::same() for Assert::notSame fail scenario

e049227

roxblnfk reviewed

View reviewed changes

src/Assert.php Outdated Show resolved Hide resolved


          refactor(assert): Rename AssertException::same to compare and upd…

096b1f7

…ate related assertions

roxblnfk merged commit e635949 into php-testo:master

This was referenced Oct 24, 2025

Open

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet