'reading and writing models works' test fails about fldgen HOT 6 CLOSED

Right now it does not look like we have access to the data needed for this test. It looks like it was something that might have been developed to run on a local test and it is unclear if there is a record about how the test data was generated. @crvernon, @FeralFlows, and @abigailsnyder do you think I could comment out this test for now to get the github actions up and running?

@kdorheim I think commenting it out so that we can see if there are any other errors to deal with on the github actions make sense, and then we can circle back to figuring this one out. I do think it's important that we figure it out (I know you do too), but whether we figure it out and then deal with github actions or deal with github actions and then figure it out...I don't think it matters too much

from fldgen.

Follow up notes:
The offending test is one making sure that results previously saved as .rda agree with results saved as .rds.

It's failing because the old .rda data is not being carried anymore. We also never actually use the loadmodel or savemodel functions this test is for, so I'm stripping those functions and this test out of the code in an upcoming PR.

from fldgen.

Right now it does not look like we have access to the data needed for this test. It looks like it was something that might have been developed to run on a local test and it is unclear if there is a record about how the test data was generated. @crvernon, @FeralFlows, and @abigailsnyder do you think I could comment out this test for now to get the github actions up and running?

from fldgen.

error details (from Kalyn's machine and githubs test on a windows machine):

> test_check("fldgen")

  -- 1. Error: reading and writing models works (@test_writedata.R#59)  ----------

  cannot open the connection

  Backtrace:

    1. testthat::expect_silent(testmodel1 <- loadmodel(oldfile, oldfmt = TRUE))

    9. fldgen::loadmodel(oldfile, oldfmt = TRUE)

   10. base::load(file)

   11. base::readChar(con, 5L, useBytes = TRUE)

  

  == testthat results  ===========================================================

  [ OK: 233 | SKIPPED: 0 | WARNINGS: 0 | FAILED: 1 ]

  1. Error: reading and writing models works (@test_writedata.R#59) 

Depending on what we figure out is causing the test failure, it's possible this may be related to missing data / weird vertical striping in results that @crvernon @FeralFlows are seeing. It may also be completely unrelated and just a test quirk due to updated packages at some point or something.

from fldgen.

Agree with @abigailsnyder

from fldgen.

@kdorheim I wonder if this error has anything with changes made in this commit
ab31dcf

might be worth checking when we do circle back to digging into the write test

from fldgen.