Ticket #838 (closed defect: fixed)

Opened 7 months ago

Last modified 7 months ago

Author tests should all check RELEASE_TESTING and/or AUTOMATED_TESTING

Reported by: rhebus Owned by: rhebus
Priority: minor Milestone:
Component: editor Version: 0.55
Keywords: Cc:

Description

Some of the xt/* tests check RELEASE_TESTING and/or AUTOMATED_TESTING but most do not. They should all make this check.

Mostly this can be implemented by copying Adam's template from  http://cpansearch.perl.org/src/ADAMK/Algorithm-Dependency-1.110/t/98_pod.t (see also  http://use.perl.org/~Alias/journal/38822 )

Note also the critic tests should probably only run under RELEASE_TESTING; we shouldn't be bothering CPAN testers with Perl::Critic failures.

Attachments

xt-with-release-checks.diff Download (4.7 KB) - added by rhebus 7 months ago.
Patch adding checks to xt/*

Change History

Changed 7 months ago by rhebus

  • owner set to rhebus
  • status changed from new to assigned

Changed 7 months ago by azawawi

  • owner changed from rhebus to azawawi

Changed 7 months ago by azawawi

  • owner changed from azawawi to rhebus

Changed 7 months ago by rhebus

Patch adding checks to xt/*

Changed 7 months ago by rhebus

The patch adds checks for RELEASE_TESTING and AUTOMATED_TESTING to all xt/* apart from Perl::Critic tests.
Perl::Critic tests only check for RELEASE_TESTING.

Changed 7 months ago by azawawi

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in r10378

I only did whitespace changes to it (spaces->tab). Thanks :)

Note: See TracTickets for help on using tickets.