Skip to content

test: fixtures name nobody and use a documentation address (greenhouse evidence/1192) - #122

Merged
rodrigoteamx merged 1 commit into
mainfrom
t-0104/no-personal-fixtures
Oct 11, 2026
Merged

rodrigoteamx merged 1 commit into
mainfrom
t-0104/no-personal-fixtures

Conversation

@rodrigoteamx

Copy link
Copy Markdown
Contributor

A public repository carries no identifier of a person or of a machine (greenhouse evidence/1144, evidence/1192).

where what it was what it becomes
tests/Data/DevToolsSourceTest.php, tests/Rendering/AdminHtmlRendererTest.php a principal with a person's user name, in three fixtures operator
tests/AdminPluginTest.php the request the loopback gate has to refuse came from an address of a real private network 192.0.2.47 (RFC 5737): as far from loopback

The tests assert what they asserted. No code changes.

Checked

php-cs-fixer clean · PHPStan no errors · 295 tests, 3,167 assertions, all green.

After the change

The scanner — every tracked file of this branch, and git archive HEAD (what the registry would serve) — returns zero lines of the classes it lists.

Nothing is released from here.

…e evidence/1192)

Three fixtures wrote a principal with a person's user name; they say operator now. The request that has to be refused
for not being loopback came from an address of a real private network; it comes from 192.0.2.47 (RFC 5737), which is as
far from loopback.

A public repository carries no identifier of a person or of a machine (greenhouse evidence/1144, evidence/1192).
@rodrigoteamx
rodrigoteamx merged commit 0b3367b into main Oct 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant