write() resolves before content is written

[nknapp]

The following code-snippets seems not to work as expected in node 4.1:

var qfs = require('q-io/fs')
var fs = require('fs')

qfs.write('abc.txt', 'this is the file contents')
  // .delay(1000)
  .done(function () {
    console.log(fs.readFileSync('abc.txt', { encoding: 'utf-8'}))
  })

In nodejs 0.10.38, this outputs the line this is the file contents, in node 4.1.0, I only get an empty line in the output. It seems like the promise is resolved before the contents is written. Removing the comment in front of the .delay(1000) causes the correct line to appear.

[nknapp]

Using the read() function of q-io/fs also prints the correct line:

var qfs = require('q-io/fs')
qfs.write('abc.txt', 'this is the file contents')
  // .delay(1000)
  .done(function () {
    qfs.read('abc.txt').done(console.log)
  })

[kriskowal]

Thanks. A PR would be most welcome.

[nknapp]

Where is the write()-method defined? Shouldn't it be in fs.js somewhere?

[kriskowal]

Methods of streams are in one of the stream modules, and in particular interest for these purposes, the Node bindings for streams. I don’t recall the specific module off-hand. I may be of more assistance after business hours.

[gagern]

It isn't write which is broken, it's close which incorrectly assumes that we don't have a finish event, due to a broken version comparison.