Make test short 20210408 #155
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
force-pushed
the
make-test-short-20210408
branch
from
c30bb71
to
918ef7a
Compare
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of this pull request is to provide the developer a short-cut to running a full
make test.