Skip to content

docs: update App::perlbrew plan -- 71/73 tests pass#458

Closed
fglock wants to merge 1 commit intomasterfrom
feature/perlbrew-phase8-docs
Closed

docs: update App::perlbrew plan -- 71/73 tests pass#458
fglock wants to merge 1 commit intomasterfrom
feature/perlbrew-phase8-docs

Conversation

@fglock
Copy link
Copy Markdown
Owner

@fglock fglock commented Apr 7, 2026

Summary

  • Re-investigated the 7 remaining App::perlbrew test failures from Phase 7.4
  • Found that 5 of 7 were stale-build artifacts (Module::Build::Tiny not installed to ~/.perlonjava/lib). After a fresh build, these all pass
  • Updated status from 66/73 to 71/73 tests passing
  • All CPAN dependencies (local::lib, Path::Class, Capture::Tiny, Test2::Mock, Test2::Plugin::IOEvents) install from CPAN and work correctly -- no PerlOnJava-specific versions needed

Remaining 2 failures

Test Root Cause Complexity
command-info.t (3/4 subtests) jperl -V missing Compiled at line that perlbrew parses Easy fix in ArgumentParser.java
installation2.t (1/1 subtest) DESTROY not implemented -- Test2::Mock cannot restore methods when mock goes out of scope Systemic (needs DESTROY)

Added to plan

  • Phase 9.1: detailed fix plan for adding Compiled at to jperl -V output
  • Phase 9.2: analysis of DESTROY limitation and possible approaches (PhantomReference, scope-based cleanup, or accept limitation)

Test plan

  • Verified 71/73 pass via jperl Build test in App::perlbrew build directory
  • Confirmed root causes for both remaining failures with reproduction scripts

Generated with Devin

…ning failures

Phase 8 re-investigation found that 5 of 7 failures were stale-build
artifacts (Module::Build::Tiny not installed). After fresh build, only 2
tests actually fail:

- command-info.t: jperl -V missing Compiled at line (easy fix)
- installation2.t: DESTROY not implemented, Test2::Mock cannot restore
  methods when mock objects go out of scope (systemic limitation)

All CPAN dependencies (local::lib, Path::Class, Capture::Tiny,
Test2::Mock, Test2::Plugin::IOEvents) install from CPAN and work
correctly -- no PerlOnJava-specific versions needed.

Added Phase 9 plan with detailed fix analysis for both remaining failures.

Generated with Devin (https://cli.devin.ai/docs)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@fglock
Copy link
Copy Markdown
Owner Author

fglock commented Apr 8, 2026

Superseded by #459 which includes the actual code fix for command-info.t

@fglock fglock closed this Apr 8, 2026
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