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

Unable to install current version 0.20 without skipping tests #32

Open
nicolasfranck opened this issue Aug 22, 2024 · 3 comments
Open

Comments

@nicolasfranck
Copy link
Contributor

Culprit test is t/20-xslt.t

Not exactly sure why, but I expect some of the example oai archives used
are no longer there.

@phochste
Copy link
Member

Would it be possible to give some context so that I can help you? Perl version? Output of commands? The xslt tets does not need network connection. On my system install works.

@nicolasfranck
Copy link
Contributor Author

Os: mac os x
Perl version: 5.22.1
Installation command: cpanm Catmandu::OAI

cpanm (App::cpanminus) 1.7046 on perl 5.022001 built for darwin-2level
Work directory is /Users/njfranck/.cpanm/work/1724337750.71912
You have make /usr/bin/make
You have LWP 6.68
You have /usr/bin/tar: bsdtar 3.5.3 - libarchive 3.5.3 zlib/1.2.12 liblzma/5.4.3 bz2lib/1.0.8 
You have /usr/bin/unzip
Searching Catmandu::OAI () on cpanmetadb ...
--> Working on Catmandu::OAI
Fetching http://www.cpan.org/authors/id/H/HO/HOCHSTEN/Catmandu-OAI-0.20.tar.gz
-> OK
Unpacking Catmandu-OAI-0.20.tar.gz
Entering Catmandu-OAI-0.20
Checking configure dependencies from META.json
Checking if you have Module::Build::Tiny 0.034 ... Yes (0.039)
Configuring Catmandu-OAI-0.20
Running Build.PL
Creating new 'Build' script for 'Catmandu-OAI' version '0.20'
-> OK
Checking dependencies from MYMETA.json ...
Checking if you have XML::Struct 0.18 ... Yes (0.27)
Checking if you have Catmandu::XML 0.16 ... Yes (0.17)
Checking if you have Test::Exception 0.32 ... Yes (0.43)
Checking if you have Test::More 1.001003 ... Yes (1.302192)
Checking if you have Test::Simple 1.001003 ... Yes (1.302192)
Checking if you have Moo 1.0 ... Yes (2.005005)
Checking if you have URI 0 ... Yes (5.17)
Checking if you have Test::Pod 0 ... Yes (1.52)
Checking if you have IO::String 0 ... Yes (1.08)
Checking if you have HTTP::OAI 4.03 ... Yes (4.12)
Checking if you have IO::Socket::SSL 1.993 ... Yes (2.083)
Checking if you have XML::LibXML 0 ... Yes (2.0208)
Checking if you have MODS::Record 0.11 ... Yes (0.13)
Checking if you have Catmandu 1.0306 ... Yes (1.2020)
Building and testing Catmandu-OAI-0.20
cp lib/Catmandu/Importer/OAI/Parser/struct.pm blib/lib/Catmandu/Importer/OAI/Parser/struct.pm
cp lib/Catmandu/Importer/OAI/Parser/marcxml.pm blib/lib/Catmandu/Importer/OAI/Parser/marcxml.pm
cp lib/Catmandu/Store/OAI/Bag.pm blib/lib/Catmandu/Store/OAI/Bag.pm
cp lib/Catmandu/OAI.pm blib/lib/Catmandu/OAI.pm
cp lib/Catmandu/Importer/OAI/Parser/oai_dc.pm blib/lib/Catmandu/Importer/OAI/Parser/oai_dc.pm
cp lib/Catmandu/Importer/OAI/Parser/raw.pm blib/lib/Catmandu/Importer/OAI/Parser/raw.pm
cp lib/Catmandu/Store/OAI.pm blib/lib/Catmandu/Store/OAI.pm
cp lib/Catmandu/Importer/OAI/Parser/mods.pm blib/lib/Catmandu/Importer/OAI/Parser/mods.pm
cp lib/Catmandu/Importer/OAI.pm blib/lib/Catmandu/Importer/OAI.pm
t/00-load.t ............ ok
t/01-importer.t ........ ok
t/02-handler.t ......... ok
t/20-xslt.t ............
All 1 subtests passed 
t/author-pod-syntax.t .. skipped: these tests are for testing by the author

Test Summary Report
-------------------
t/20-xslt.t          (Wstat: 11 Tests: 1 Failed: 0)
  Non-zero wait status: 11
  Parse errors: No plan found in TAP output
Files=5, Tests=5,  0 wallclock secs ( 0.01 usr  0.01 sys +  0.54 cusr  0.07 csys =  0.63 CPU)
Result: FAIL
-> FAIL Installing Catmandu::OAI failed. See /Users/njfranck/.cpanm/work/1724337750.71912/build.log for details. Retry with --force to force install it.

I do not really see any reason why it should fail, only that it does fail on that one test

@phochste
Copy link
Member

I still can't replicate this on osx with 5.22.1. But, I see the xslt test skips some tests when not all dependencies are met ..which are mandatory. I remove this requirement and released version 0.21 to cpan. Hopefully this fixes your issue.

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

No branches or pull requests

2 participants