Skip to content

Fix/93 moo meta no moose - #94

Merged
geofffranks merged 2 commits into
mainfrom
fix/93-moo-meta-no-moose
Jun 15, 2026
Merged

Fix/93 moo meta no moose#94
geofffranks merged 2 commits into
mainfrom
fix/93-moo-meta-no-moose

Conversation

@geofffranks

Copy link
Copy Markdown
Owner

Fixes #93

geofffranks and others added 2 commits June 15, 2026 18:27
_meta_for short-circuits on Moo::HandleMoose::FakeMetaClass (matched by
exact ref, not ->isa) before any ->isa check, so Moo's lazy Moose
metaclass inflation never fires. Previously, mocking a Moo class without
Moose installed died inside Moo::HandleMoose; now Moo classes
consistently fall back to plain symbol-table mocking.

Adds t/moo_meta_for.t (unit) and t/moo_basic.t (functional mock/unmock,
including accessors and inherited methods), declares Moo as a recommended
optional test dep, and adds an oo-deps CI matrix exercising the full
{Moo, Moose, Mouse} dependency cross-product (8 profiles) on a clean
Perl. The moo profile -- Moo present, Moose absent -- reproduces GH #93.

Closes GH #93

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@geofffranks
geofffranks force-pushed the fix/93-moo-meta-no-moose branch from 7972bf1 to fa8bee9 Compare June 15, 2026 22:28
@geofffranks
geofffranks merged commit 90a5f2d into main Jun 15, 2026
32 checks passed
@github-actions github-actions Bot mentioned this pull request Jun 15, 2026
@geofffranks
geofffranks deleted the fix/93-moo-meta-no-moose branch June 15, 2026 22:52
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Jun 17, 2026
https://build.opensuse.org/request/show/1360007
by user tinita + anag_factory
- updated to 0.185.3 (v0.185.3)
   see /usr/share/doc/packages/perl-Test-MockModule/Changes
  ## 0.185.3 — 2026-06-15
  ## What's Changed
  * Fix/93 moo meta no moose by @geofffranks in geofffranks/test-mockmodule#94
  **Full Changelog**: geofffranks/test-mockmodule@0.185.2...0.185.3
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.

Moose required when mocking Moo

1 participant