"make fulltest" requires TAP::Harness::Archive
Jonathan "Duke" Leto
I was setting up Parrot on a new machine, which I have done many
times, but now I see that "make fulltest" now requires
TAP::Harness::Archive (even if you aren't sending to Smolder). Is this
on purpose? If so, the least we could do is give the user a nice
message about how to install or find TAP::Harness::Archive.
Our other option to only require THA if something is being sent to
Smolder, which seems to be more user-friendly behavior.
Duke
--
Jonathan "Duke" Leto <jona...@leto.net>
Leto Labs LLC
209.691.DUKE // http://labs.leto.net
NOTE: Personal email is only checked twice a day at 10am/2pm PST,
please call/text for time-sensitive matters.
_______________________________________________
http://lists.parrot.org/mailman/listinfo/parrot-dev
James E Keenan
> Our other option to only require THA if something is being sent to
> Smolder, which seems to be more user-friendly behavior.
>
Please review the kid51/fulltest_easy now on github.
On this branch I have tested 'make fulltest', 'make smolder_fulltest'
and 'perl t/fullharness' successfully -- but of course all the boxes I
have available already have the necessary modules installed. So it
would be good to test on a box which lacks TAP::Harness::Archive or
anything in TAP-Harness-Multiple.
Thank you very much.
Jim Keenan
_______________________________________________
http://lists.parrot.org/mailman/listinfo/parrot-dev
Jonathan "Duke" Leto
> Please review the kid51/fulltest_easy now on github.
>
> On this branch I have tested 'make fulltest', 'make smolder_fulltest' and
> 'perl t/fullharness' successfully -- but of course all the boxes I have
> available already have the necessary modules installed. So it would be good
> to test on a box which lacks TAP::Harness::Archive or anything in
> TAP-Harness-Multiple.
You can run test suites without modules with
https://metacpan.org/module/Test::Without::Module
like so:
perl -MTest::Without::Module=Some::Module -w -Iblib/lib t/SomeModule.t
I will test this out on my machine soon.
Jonathan "Duke" Leto
Just a few caveats about this module:
1) It does not support lexical scopes (it is file/package-wide, not sure which)
2) Certain core XS modules cannot be unloaded
Duke
Jonathan "Duke" Leto
I just merged master in and ran the tests on my machine and pushed it back to
your branch, but I haven't ran it correctly with the modules it checks for.
Please test the latest stuff in this branch and let me know.
Duke
--
James E Keenan
> Howdy,
>
>> Please review the kid51/fulltest_easy now on github.
>>
>> On this branch I have tested 'make fulltest', 'make smolder_fulltest' and
>> 'perl t/fullharness' successfully -- but of course all the boxes I have
>> available already have the necessary modules installed. So it would be good
>> to test on a box which lacks TAP::Harness::Archive or anything in
>> TAP-Harness-Multiple.
>
> You can run test suites without modules with
> https://metacpan.org/module/Test::Without::Module
>
> like so:
>
> perl -MTest::Without::Module=Some::Module -w -Iblib/lib t/SomeModule.t
>
Thanks for that suggestion. I installed that library and, with one
correction in the kid51/fulltest_easy branch to t/fullharness, this now
should work as intended:
$ perl -MTest::Without::Module=TAP::Harness::ReportByDescription
t/fullharness --archive --in-testing
To run make smolder_fulltest, please install
TAP::Harness::Archive and TAP::Harness::ReportByDescription from CPAN.
From the cpan shell: 'install TAP::Harness::ReportByDescription'.
With cpanminus : cpanm TAP::Harness::ReportByDescription
at t/fullharness line 68
$ perl -MTest::Without::Module=TAP::Harness::Archive t/fullharness
--archive --in-testing
To run make smolder_fulltest, please install
TAP::Harness::Archive and TAP::Harness::ReportByDescription from CPAN.
From the cpan shell: 'install TAP::Harness::ReportByDescription'.
With cpanminus : cpanm TAP::Harness::ReportByDescription
at t/fullharness line 68
$ perl -MTest::Without::Module=TAP::Harness::Archive t/fullharness
--in-testing
testb: running with: --runcore=bounds -D40 --gc-debug
testb__t/oo/attributes.t ...... ok
[snip]
headerizer: running with: -D40 --gc-debug
headerizer__t/tools/dev/headerizer/01_functions.t .. ok
headerizer__t/tools/dev/headerizer/02_methods.t .... ok
All tests successful.
Files=44, Tests=1357, 3 wallclock secs ( 0.23 usr 0.09 sys + 2.17
cusr 0.69 csys = 3.18 CPU)
Result: PASS
_______________________________________________
http://lists.parrot.org/mailman/listinfo/parrot-dev
James E Keenan
Are there any objections to merging this branch in?
Jonathan "Duke" Leto
--
James E Keenan
> +1 to merge now
>
I have merged the kid51/fulltest_easy branch into master.
When subsequently running make smolder_fulltest, I noticed a codingstd
failure:
$ prove -v t/codingstd/filenames.t
t/codingstd/filenames.t ..
1..3
not ok 1 - No multi-dot filenames
# Failed test 'No multi-dot filenames'
# at t/codingstd/filenames.t line 93.
# Multi-dot filename found in 1 files:
# .travis.yml
ok 2 - Portable characters in filenames
ok 3 - Filenames length
# Looks like you failed 1 test of 3.
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/3 subtests
Test Summary Report
-------------------
t/codingstd/filenames.t (Wstat: 256 Tests: 3 Failed: 1)
Failed test: 1
Non-zero exit status: 1
Files=1, Tests=3, 0 wallclock secs ( 0.03 usr 0.01 sys + 0.19 cusr
0.01 csys = 0.24 CPU)
Result: FAIL
Can you add an exception in t/codingstd/filenames.t to account for this
hidden file?