Check error on devel on all platforms ref@.Data not equal to res@.Data

[sneumann]

Hi,
We have a testthat error on all platforms (also in release #804).
Yours, Steffen

https://bioconductor.org/checkResults/devel/bioc-LATEST/xcms/nebbiolo2-checksrc.html

  ══ Failed tests ════════════════════════════════════════════════════════════════
  ── Failure ('test_XcmsExperiment.R:1427:5'): chromPeaksChromatograms,XcmsExperiment works ──
  ref@.Data not equal to res@.Data.
  Attributes: < Component "dimnames": Component 2: 1 string mismatch >

in

xcms/tests/testthat/test_XcmsExperiment.R

Line 1425 in 5f77f39

expect_equal(ref, res)

[jorainer]

Thanks - I will look into this. Seems there are some row/colnames different. Could be related to some updates in other packages (MsExperiment?). I'll figure it out.

[jorainer]

Note: I'm pretty sure it's related to the recent change in readMsExperiment() of *MsExperiment* that sets the row names of the sampleData()(which was added to have a better support for conversion toSummarizedExperiment). In that case it's all just about the unit tests - where the *reference* does not have rownames while the actual sampleData()/rowData()/pData()` does.