[DBIx-Class-Devel] [Perl5/DBIx-Class] Make t/53lean_startup.t robust wrt stacktrace details (PR #143)

Leon Timmermans notifications at github.com
Sat Feb 19 15:57:21 GMT 2022


This test checks if none of DBIC is loading anything unexpected. It
contains logic to skip over indirect loaders such as base, parent and
Class::C3::Componentised. However, this erroneously skips over direct
dependencies of such classes. In particular, it assigns the MRO::Compat
dependency in Class::C3::Componentised to whatever class it thinks it
loads CCC. Before ad89278 this worked out fine, because it would be
assigned to main, which while meaningless isn't part of DBIx::Class so
ignored. After ad89278, it's assigned to the class that loads CCC, which
is DBIx::Class::Componentised, which is part of DBIC, and hence fails
the test.

This test will cause it to not skip on for any BEGIN blocks inside an
indirect loader, therefor the MRO::Compat dependency will correctly be
assigned to Class::C3::Componentised.
You can view, comment on, or merge this pull request online at:

  https://github.com/Perl5/DBIx-Class/pull/143

-- Commit Summary --

  * Make t/53lean_startup.t robust wrt stacktrace details

-- File Changes --

    M t/53lean_startup.t (2)

-- Patch Links --

https://github.com/Perl5/DBIx-Class/pull/143.patch
https://github.com/Perl5/DBIx-Class/pull/143.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/Perl5/DBIx-Class/pull/143
You are receiving this because you are subscribed to this thread.

Message ID: <Perl5/DBIx-Class/pull/143 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scsys.co.uk/pipermail/dbix-class-devel/attachments/20220219/69dc3387/attachment.htm>


More information about the DBIx-Class-Devel mailing list