Skip to content

test: fix disabled test-fs-largefile

Rodrigo Muino Tomonari requested to merge github/fork/Trott/large-file into master

Add common.refreshTmpDir() before using the tmp directory.

fs.writeSync no longer requires an integer for the position argument, so change test from assert.throws() to assert.doesNotThrow(). The test now passes.

Because it involves a 5 GB file, we're not going to activate the test, although that is possible if we add checking for appropriate available resources (definitely disk space, likely memory, maybe check that it's a 64-bit OS).

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test fs

Merge request reports