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

Toaster results show several modules' tests rely on buggy Num stringification #1736

Closed
zoffixznet opened this issue Apr 14, 2018 · 5 comments
Assignees
Labels
BLOCKER Preventing the next release of rakudo, or just needing attention before the release good first issue This issue is likely easy for beginners to resolve

Comments

@zoffixznet
Copy link
Contributor

zoffixznet commented Apr 14, 2018

Filing in hopes of getting more helping hands on this.

There are several modules on https://toaster.perl6.party/ at the top, with "Fail" in the right column. If you click on "Fail" you'll see test results. And in a bunch of modules, you'll see they test a specific stringification of a Num.

Presumably, they're testing a num using is routine, when they should be using is-approx routine or, if the nums are actually meant to be exactly equal, with cmp ...,'==', ... routine.

So we need volunteers to PR fixes to those modules' tests, swapping to correct testing functions or fixing the failures by some other means.

If you send a PR, please comment on this issue the name of the module, so others would know it's fixed already.

@zoffixznet zoffixznet added the good first issue This issue is likely easy for beginners to resolve label Apr 14, 2018
@sylints
Copy link

sylints commented Apr 15, 2018

Changed tests on Stats module, which should resolve the failure in the BioInfo module as well.

@sylints
Copy link

sylints commented Apr 15, 2018

Changed tests in ScaleVec module, which should also resolve the failure in Music::Engine

@AlexDaniel
Copy link
Contributor

Thank you, @sylints!

@AlexDaniel AlexDaniel added the BLOCKER Preventing the next release of rakudo, or just needing attention before the release label Apr 16, 2018
@AlexDaniel
Copy link
Contributor

Not a blocker per se, but we try to fix issues in the ecosystem before the release.

@AlexDaniel
Copy link
Contributor

AlexDaniel commented Apr 21, 2018

I did my own toaster run today, and here are the results:

Module Note
Algorithm::GooglePolylineEncoding Scimon/p6-Algorithm-GooglePolylineEncoding#1
👪 BioInfo depends on Stats
👪 CSS::Declarations depends on CSS::Specification
👪 CSS::Module depends on CSS::Specification
🐦 CSS::Specification toaster flop, installs fine locally
Chemistry::Elements rakudobug #1753
🐦 DB::Model::Easy toaster flop, installs fine locally
👪 Music::Engine depends on ScaleVec
Ops::SI scriptkitties/perl6-Ops-SI#2
Perl6-Math-Quaternion Util/Perl6-Math-Quaternion#7
ScaleVec samgwise/p6-ScaleVec#1 (@sylints++)
Stats MattOates/Stats#7 (@sylints++)
🐦 ANTLR4::Grammar toaster flop, installs fine locally
🐦 Cro::HTTP::Session::Redis toaster flop, installs fine locally
🐦 Number::Denominate toaster flop, installs fine locally

Those marked with ✓ had an issue described in this ticket and were fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLOCKER Preventing the next release of rakudo, or just needing attention before the release good first issue This issue is likely easy for beginners to resolve
Projects
None yet
Development

No branches or pull requests

3 participants