Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'test_author' 'make' target (and alias 'atest') #175

Closed
wants to merge 2 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion MANIFEST
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ t/50-errno.t
t/60-forkdepth.t
t/70-subname.t
t/80-version.t
t/90-pod.t
t/lib/NYTProfTest.pm
t/nytprof_11-reader.out.txt
t/nytprof_12-data.out.txt
Expand Down Expand Up @@ -251,7 +252,6 @@ xt/61-cputime.t
xt/68-hashline.t
xt/71-moose.t
xt/72-autodie.t
xt/90-pod.t
xt/91-pod_coverage.t
xt/92-file_port.t
xt/test23-strevalxs.p
Expand Down
8 changes: 6 additions & 2 deletions Makefile.PL
Original file line number Diff line number Diff line change
Expand Up @@ -283,9 +283,13 @@ perltidy:
ctidy_bcpp:
bcpp -f 2 -i 4 -bcl -qb 10 -ylcnc -yb NYTProf.xs

NYTPROF_TEST_SHORT=""
test_short:
NYTPROF_TEST_SHORT=1 make test
NYTPROF_TEST_SHORT=1 NYTPROF_AUTHOR_TESTING=1 make test

test_author:
NYTPROF_AUTHOR_TESTING=1 make test

atest: test_author

}
}
Expand Down
1 change: 1 addition & 0 deletions t/12-data.t
Original file line number Diff line number Diff line change
Expand Up @@ -263,6 +263,7 @@ is(scalar(@noneval_fileinfos), 1, "got 1 noneval_fileinfo");

{
SKIP: {
skip "NYTPROF_AUTHOR_TESTING only", 3 unless $ENV{NYTPROF_AUTHOR_TESTING};
skip "Bad interaction when trace_level is set", 3
if trace_level();
my $profile;
Expand Down
16 changes: 16 additions & 0 deletions t/90-pod.t
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
#!perl -w

use Test::More;
unless ($ENV{NYTPROF_AUTHOR_TESTING}) {
plan skip_all => "NYTPROF_AUTHOR_TESTING only";
}
else {
diag("Relevant envvar is true; proceeding to testing POD");
}

eval "use Test::Pod 1.00";
plan skip_all => "Test::Pod 1.00 required for testing POD" if $@;

all_pod_files_ok();

1;
9 changes: 0 additions & 9 deletions xt/90-pod.t

This file was deleted.