insightsengineering / rlistings

Value formatting and ASCII rendering infrastructure for tables and listings.
https://insightsengineering.github.io/rlistings/
25 stars 5 forks source link

R CMD CHECK error #228

Closed pawelru closed 2 weeks ago

pawelru commented 4 weeks ago

https://github.com/r-universe/pharmaverse/actions/runs/11285110100

* checking tests ...
  Running ‘testthat.R’ [10s/10s]
 [11s/10s] ERROR
Running the tests in ‘tests/testthat.R’ failed.
Last 13 lines of output:
   4.     └─formatters::`num_rep_cols<-`(`*tmp*`, value = rep_cols)
   5.       └─formatters:::mf_update_cinfo(obj, colwidths = NULL, rep_cols = value)
  ── Error ('test-export.R:113:3'): export_as_txt works with listings with all key cols ──
  Error in `rep(0, length(r_colwidths) - nrepcols - 1)`: invalid 'times' argument
  Backtrace:
      ▆
   1. └─formatters::export_as_txt(lsting) at test-export.R:113:3
   2.   └─formatters::paginate_to_mpfs(...)
   3.     ├─formatters::`num_rep_cols<-`(`*tmp*`, value = rep_cols)
   4.     └─formatters::`num_rep_cols<-`(`*tmp*`, value = rep_cols)
   5.       └─formatters:::mf_update_cinfo(obj, colwidths = NULL, rep_cols = value)

  [ FAIL 2 | WARN 0 | SKIP 13 | PASS 91 ]
  Error: Test failures
  Execution halted
pawelru commented 2 weeks ago

our rhub and dependency tests also failed with that message: https://github.com/insightsengineering/rlistings/actions/runs/11537358833

shajoezhu commented 2 weeks ago

see https://github.com/insightsengineering/rlistings/issues/229 and https://github.com/insightsengineering/rlistings/actions/runs/11623913471/job/32371701879, this issue will be fixed there