[Perl/perl5] 29f0eb: t/run/todo.t, t/op/glob.t: Revise TODO status of g...

[email protected] (James E Keenan via perl5-changes)
Newsgroups perl.perl5.changes
Message-ID <Perl/perl5/push/refs/heads/blead/[email protected]>
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 29f0eb34760effe8cedf13c40d5fdca28019ce3b
      https://github.com/Perl/perl5/commit/29f0eb34760effe8cedf13c40d5fdca28019ce3b
  Author: James E Keenan <[email protected]>
  Date:   2026-08-17 (Mon, 17 Aug 2026)

  Changed paths:
    M t/op/glob.t
    M t/run/todo.t

  Log Message:
  -----------
  t/run/todo.t, t/op/glob.t: Revise TODO status of glob test

The "GH 16869" test added to t/run/todo.t in de313d8e84 (Aug 23 2025)
was written in the belief that an assertion failure in glob3 would be
encountered on all debugging builds.  Examination of test results
suggests that this is only true on Linux.  If so, then we can have this
test run "live" (i.e., without TODO or SKIP conditions) on all
non-debugging builds on all platforms and on debugging builds other than
on Linux.

To test this hypothesis, we'll restrict the scope of the GH 16869 TODO
condition to debugging builds on Linux and run the test without TODO or
SKIP conditions from t/op/glob.t.  Should the test in t/op/glob.t now
start to fail, we'll know we have deeper problems which warrant renewed
examination in GH #16869.

Also: let's use 'done_testing()' rather than 'plan()' because the exact
number of tests to be run in todo.t is inherently quite variable.


  Commit: b789382b869a0cc3f9989a1293ae8c9ad1e95dda
      https://github.com/Perl/perl5/commit/b789382b869a0cc3f9989a1293ae8c9ad1e95dda
  Author: James E Keenan <[email protected]>
  Date:   2026-08-17 (Mon, 17 Aug 2026)

  Changed paths:
    M t/op/glob.t
    M t/run/todo.t

  Log Message:
  -----------
  cygwin: Use same test restrictions as on linux

Our GH CI had a test failure on cygwin (which is set up for debugging):
https://github.com/Perl/perl5/actions/runs/31833414125/job/94876841306

Let's see if treating this the same as Linux clears this up.


Compare: https://github.com/Perl/perl5/compare/044d9bdf5d2c...b789382b869a

To unsubscribe from these emails, change your notification settings at https://github.com/Perl/perl5/settings/notifications
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.