[Koha-bugs] [Bug 7994] Syntax error in yaml (syspref) files

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7994

--- Comment #22 from claire.hernandez at biblibre.com <claire.hernandez at biblibre.com> ---
(In reply to comment #18)

> 1/ Usage is not correct:
> +=head1 USAGE
> +
> +From Koha root directory:
> +
> +prove -v xt/tt_valid.t

Yep done.

> 2/ It not possible to launch the test file from the koha home directory
> (prove xt/yaml_valid.t) (see the FindBin module) 

Done but too much code for me for adding this "feature" - nothing more sexy?

> 3/ Don't you think it would be more friendly to do a test file (thus using
> Test::More) for your pl file ?

I don't know how to give a parameter to a test file and I wanted to have a "-f"
option to test whatever you want (sudoc yaml file for example).

> 4/ You add a new dependency Test::YAML::Valid. I dont know if we have to add
> this dependency to koha (given that it is used for unit tests).

I think we should be free to add new test libraies if we need it for testing.
For discusion irl monday :)

-- 
You are receiving this mail because:
You are watching all bug changes.