Opened 5 years ago

Closed 5 years ago

#838 closed defect (fixed)

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 (1)

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

Download all attachments as: .zip

Change History (6)

comment:1 Changed 5 years ago by rhebus

  • Owner set to rhebus
  • Status changed from new to assigned

comment:2 Changed 5 years ago by azawawi

  • Owner changed from rhebus to azawawi

comment:3 Changed 5 years ago by azawawi

  • Owner changed from azawawi to rhebus

Changed 5 years ago by rhebus

Patch adding checks to xt/*

comment:4 Changed 5 years 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.

comment:5 Changed 5 years ago by azawawi

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed in r10378

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

Note: See TracTickets for help on using tickets.