Using an absolute path added events do not fire

I can add these tests as a PR if you want. But first I wanted to make sure that my thought process on this was correct. These tests are simply copies of current tests. Or are such patterns not supported?

  addedByAbsolutePath: function(test) {
    test.expect(1);
    gaze(process.cwd() + '/**/*', function(err, watcher) {
      setTimeout(function() {
        test.ok(false, 'Ended without adding a file.');
        watcher.close();
      }, 1000);
      this.on('added', function(filepath) {
        var expected = path.relative(process.cwd(), filepath);
        test.equal(path.join('sub', 'tmp.js'), expected);
        watcher.close();
      });
      this.on('changed', function() { test.ok(false, 'changed event should not have emitted.'); });
      this.on('deleted', function() { test.ok(false, 'deleted event should not have emitted.'); });
      fs.writeFileSync(path.resolve(__dirname, 'fixtures', 'sub', 'tmp.js'), 'var tmp = true;');
      watcher.on('end', test.done);
    });
  },
  dontAddUnmatchedFilesByAbsolutePath: function(test) {
    // TODO: Code smell
    test.expect(2);
    gaze(process.cwd() + '/**/*.js', function(err, watcher) {
      setTimeout(function() {
        test.ok(true, 'Ended without adding a file.');
        watcher.close();
      }, 1000);
      this.on('added', function(filepath) {
        test.equal(path.relative(process.cwd(), filepath), path.join('sub', 'tmp.js'));
      });
      fs.writeFileSync(path.resolve(__dirname, 'fixtures', 'sub', 'tmp'), 'Dont add me!');
      fs.writeFileSync(path.resolve(__dirname, 'fixtures', 'sub', 'tmp.js'), 'add me!');
      watcher.on('end', test.done);
    });
  },

[seebees]

Ok. I see it here. The npm and README documentation needs to be updated. If I get time I’ll give you a PR.

But… When I test this I see that

cwd = /path/to/some/ and pattern = files/**/*js does not fire the add event but cwd = /path/to/some/files/ and pattern = **/*.js does

again, I can work around it but is the expectation that the first one will not fire add events? or do you want a test for that case?

[hedefalk]

+1, I tried to replace chokidar because it didn’t build on Atom/Electron because fsevents. I just wanted to watch an absolute file name, but that doesn’t seem possible. So I tried to break it down and use cwd, but that seems broken on empty folders too. I did an outside test:

https://gist.github.com/hedefalk/adccbaeb10c7285686d19e9263fda6c5

I’m gonna try something else 😃