Behavior with brace expansion

[rovjuvano]

4 tests, the two with brace expansion fail

  it('should get a base name from character class pattern', function(done) {
    var globber = new glob.Glob("js/t[a-z]st}/*.js", {cwd: __dirname});
    glob2base(globber).should.equal("js/");
    done();
  });
  // passes

  it('should get a base name from brace , expansion', function(done) {
    var globber = new glob.Glob("js/{src,test}/*.js", {cwd: __dirname});
    glob2base(globber).should.equal("js/");
    done();
  });
  // AssertionError: expected 'js/src/' to be 'js/

  it.only('should get a base name from brace .. expansion', function(done) {
    var globber = new glob.Glob("js/test{0..9}/*.js", {cwd: __dirname});
    glob2base(globber).should.equal("js/");
    done();
  });
  // AssertionError: expected 'js/test0/' to be 'js/'

  it('should get a base name from extglob', function(done) {
    var globber = new glob.Glob("js/t+(wo|est)/*.js", {cwd: __dirname});
    glob2base(globber).should.equal("js/");
    done();
  });
  // passes

The behavior when involving brace expansion:

1. Does not conform to the behavior with other types of patterns (it includes instead of excludes the directory matched). This can be argued as the correct behavior.
2. Only produces a base for the first expansion and not any of the subsequent ones in the pattern. Due to this behavior, modules such as glob-stream will produce Files with a path outside the base. (e.g. path: CWD/test/main.js, base: CWD/src/)

[yocontra]

The base in all of those examples should be js/ - Will look into this tonight

[yocontra]

Fixed in 0.0.5 - an npm update should fix this right up. Thanks for the report!