mllg / checkmate

Fast and versatile argument checks
https://mllg.github.io/checkmate/
Other
261 stars 30 forks source link

2 tests in test_checkIntegerish.R fail on 32-bit platforms [ FAIL 2 | WARN 0 | SKIP 1 | PASS 3004 ] #241

Open barracuda156 opened 1 year ago

barracuda156 commented 1 year ago

Example with ppc:

R version 4.2.3 (2023-03-15) -- "Shortstop Beagle"
Copyright (C) 2023 The R Foundation for Statistical Computing
Platform: powerpc-apple-darwin10.8.0 (32-bit)

> library(testthat)
> test_check("checkmate")
Loading required package: checkmate
[ FAIL 2 | WARN 0 | SKIP 1 | PASS 3004 ]

══ Skipped tests ═══════════════════════════════════════════════════════════════
• On CRAN (1)

══ Failed tests ════════════════════════════════════════════════════════════════
── Failure ('test_checkIntegerish.R:73:3'): informative error messages ─────────
Check on 'x' failed: Must contain substring 'element 5'
Backtrace:
    ▆
 1. └─checkmate::expect_string(x, fixed = "element 5") at test_checkIntegerish.R:73:2
 2.   └─checkmate::makeExpectation(x, res, info, label)
── Failure ('test_checkIntegerish.R:77:3'): informative error messages ─────────
Check on 'x' failed: Must contain substring 'element 1'
Backtrace:
    ▆
 1. └─checkmate::expect_string(x, fixed = "element 1") at test_checkIntegerish.R:77:2
 2.   └─checkmate::makeExpectation(x, res, info, label)

[ FAIL 2 | WARN 0 | SKIP 1 | PASS 3004 ]
Error: Test failures
Execution halted
tillea commented 1 year ago

I confirm this issue also for all 32 bit architectures as it is reported in an according Debian bug which also links to the list of all supportet architectures CI check. If you follow the Regression links you get a full test log which reproduces the error message above.

barracuda156 commented 1 year ago

@tillea Thank you for the info, I have updated the issue title.

tillea commented 1 year ago

@barracuda156 Thanks for fixing the title. Some info from authors would be helpful. If it turns out to be a hard issue that will not be fixed quickly I'd exclude these tests for the moment.