test(attribute): assert subclass hydration through the public API

[abnegate]

What

The three attribute-hydration tests in tests/unit/AttributeSubclassTest.php reached past the public surface: they stubbed the Adapter, then invoked the private Database::createDocumentInstance() through ReflectionMethod. That asserted the shape of an internal method, not the behaviour callers depend on — renaming the method would have broken the tests while the feature stayed correct, and a regression anywhere else in the read path would have gone unnoticed.

They now write one attribute row to the Memory adapter and read it back with getDocument(), via a small store() helper. Same three properties are covered:

• testReadingBackHydratesTheMappedSubclass — the mapped document type hydrates to that subclass.
• testReadingBackHydratesTheSubclassTheStoredTypeNames — with Attribute::class mapped, the stored type selects the subclass.
• testStoredTypeWinsOverTheMappedClass — a stored Integer type beats a mapped StringType.

This removes the last Reflection usage from the file.

Verification

• vendor/bin/phpunit tests/unit/AttributeSubclassTest.php — OK (174 tests, 697 assertions)
• composer lint — passed

Tests only; no production code touched.

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (2)

• main
• 0.69.x

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 7f1824f2-db11-4940-8a6f-f2e2175818a4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[greptile-apps]

Confidence Score: 5/5

The test-only PR appears safe to merge, with no outstanding correctness or repository-rule issues.

Summary

• Persists attribute rows using the Memory adapter and reads them with getDocument().
• Verifies mapped-subclass hydration, stored-type subclass selection, and stored-type precedence.
• Corrects the previously reported test-name wording issue.

_{Reviews (2) · Last reviewed commit: "test(attribute): assert subclass hydrati..."}