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

Make test short 20210408 #155

Merged
merged 2 commits into from
Apr 9, 2021
Merged

Conversation

jkeenan
Copy link
Collaborator

@jkeenan jkeenan commented Apr 8, 2021

Purpose of this pull request is to provide the developer a short-cut to running a full make test.

When set in environment, will cause only 1 of (generally) 8
options-combinations to be run, thereby speeding up a run of 'make test'
during development.  TODO:  transform this into a 'make' target.
This will set envvar NYTPROF_TEST_SHORT and cause only one combination
to run within run_test_group().
@jkeenan jkeenan merged commit 46396ac into timbunce:master Apr 9, 2021
@jkeenan jkeenan deleted the make-test-short-20210408 branch April 9, 2021 19:11
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request May 1, 2021
Upstream changes:
Changes in Devel::NYTProf 6.08 - DATE TO COME

B<There should be no observable differences in the performance of Devel-NYTProf
from that of the previous release.>  This release is maintenance-focused and
changes mainly concern improvements in test coverage, elimination of
build-time warnings and correction of small errors detected by end-users.

=head3 Substantive

Corrected 1 error in F<Makefile.PL>
(L<timbunce/devel-nytprof#158>; thanks to Kent Fredric).

Corrected other errors in F<NYTProf.xs> and F<ppport.h> at suggestion of Reini Urban
in earlier pull request.

Two subroutines in F<Devel::NYTProf::Data> -- C<packages_at_depth_subinfo()>
and C<package_fids()> -- which were not used anywhere in the codebase or test
suite were removed and placed in F<devstuff/superseded.pm>.  One function --
C<get_str_id()> defined in F<NYTProf.xs> but never used in the codebase was
similarly removed and placed in F<devstuff/superseded.xs>.  Anyone who was
using these functions may file a pull request to have them restored to the
codebase.

In C<Devel::NYTProf::Data::strip_prefix_from_paths()>, a branch concerned with
the possibility that an argument was a hash was not exercised anywhere in the
codebase or the test suite and was simply commented out.  Anyone who was using
this functionality may file a pull request to have it restored to the
codebase.

Eliminated all build-time warnings being generated during F<make> on testing
platforms Linux, FreeBSD, OpenBSD, Windows.

=head3 Test Suite

Improved coverage provided by test suite to each of the following files:

    lib/Devel/NYTProf/Data.pm
    lib/Devel/NYTProf/FileInfo.pm
    lib/Devel/NYTProf/Reader.pm
    lib/Devel/NYTProf/SubInfo.pm
    lib/Devel/NYTProf/Util.pm

This was accomplished mainly by adding 4 new test files:

    t/11-reader.t
    t/12-data.t
    t/13-fileinfo.t
    t/14-subinfo.t

... and their corresponding data files.  In addition, there were substantial
additions to F<t/30-util.t>.

Coverage was evaluated with C<trace_level> unset and C<trace_level> 5.

=head3 Housekeeping

Updated Travis and AppVeyor configuration files.

Updated F<MANIFEST> and F<MANIFEST.SKIP> so that F<make manifest> works as intended.

Moved the following 6 test files from F<t/> to F<xt/>:

    t/68-hashline.t
    t/71-moose.t
    t/72-autodie.t
    t/90-pod.t
    t/91-pod_coverage.t
    t/92-file_port.t

Reason:  They either (a) had been previously described as insufficiently
developed and were being skipped; or (b) are of concern only to NYTProf
developers.  Hence, a failure in any of them should not preclude installation
and use of Devel::NYTProf.

Added a F<make test_short> target to F<Makefile.PL> and environmental variable
C<NYTPROF_TEST_SHORT>.  These will enable NYTProf developers to cut testing
time without significant loss in accuracy
(L<timbunce/devel-nytprof#155>).

Corrected one date error in 6.07 release
(L<timbunce/devel-nytprof#152>; thanks to @shawnlaffan).

Applied several commits either cherry-picked from, or initiated by Reini Urban
in an older pull request.

=head2 Changes in Devel::NYTProf 6.07 - 6th April 2021

  Focus is to enable test suite to pass with perl-5.33.3 and higher.  (Changes
    in blead required changes in expectations set in test data files.)

  Correction to MANIFEST
    thanks to mattlaw. #142

  POD improvements
    thanks to tomhukins. #129

  Makefile.PL modernizations
    thanks to karenetheridge. #126

  .gitignore corrections
    thanks to jkeenan. #146

  Adapt tests to work with 5.33 and higher
    thanks to jkeenan. #147
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant