path get/set for recording path change #19

gulpjs · Jun 28, 2014 · f3f9be0 · f3f9be0
1 parent 83bd747
commit f3f9be0
Show file tree

Hide file tree

Showing 2 changed files with 73 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -11,6 +11,9 @@ var cloneBuffer = require('./lib/cloneBuffer');
 function File(file) {
   if (!file) file = {};
 
+  // record path change
+  this.history = [];
+
   // TODO: should this be moved to vinyl-fs?
   this.cwd = file.cwd || process.cwd();
   this.base = file.base || this.cwd;
@@ -123,4 +126,18 @@ Object.defineProperty(File.prototype, 'relative', {
   }
 });
 
+Object.defineProperty(File.prototype, 'path', {
+  get: function() {
+    return this.history[this.history.length - 1];
+  },
+  set: function(path) {
+    // record history only when path changed
+    if (path && path !== this.path) {
+      this.history.push(path);
+    }
+
+    this._path = path;
+  }
+});
+
 module.exports = File;
diff --git a/test/File.js b/test/File.js
@@ -80,7 +80,7 @@ describe('File', function() {
       done();
     });
   });
-  
+
   describe('isBuffer()', function() {
     it('should return true when the contents are a Buffer', function(done) {
       var val = new Buffer("test");
@@ -382,7 +382,7 @@ describe('File', function() {
       process.nextTick(done);
     });
   });
-  
+
   describe('inspect()', function() {
     it('should return correct format when no contents and no path', function(done) {
       var file = new File();
@@ -445,7 +445,7 @@ describe('File', function() {
       done();
     });
   });
-  
+
   describe('contents get/set', function() {
     it('should work with Buffer', function(done) {
       var val = new Buffer("test");
@@ -537,4 +537,57 @@ describe('File', function() {
     });
   });
 
+  describe('path get/set', function() {
+
+    it('should record history when instantiation', function() {
+      var file = new File({
+        cwd: '/',
+        path: '/test/test.coffee'
+      });
+
+      file.path.should.eql('/test/test.coffee');
+      file.history.should.eql(['/test/test.coffee']);
+    });
+
+    it('should record history when path change', function() {
+      var file = new File({
+        cwd: '/',
+        path: '/test/test.coffee'
+      });
+
+      file.path = '/test/test.js';
+      file.path.should.eql('/test/test.js');
+      file.history.should.eql(['/test/test.coffee', '/test/test.js']);
+
+      file.path = '/test/test.coffee';
+      file.path.should.eql('/test/test.coffee');
+      file.history.should.eql(['/test/test.coffee', '/test/test.js', '/test/test.coffee']);
+    });
+
+    it('should not record history when set the same path', function() {
+      var file = new File({
+        cwd: '/',
+        path: '/test/test.coffee'
+      });
+
+      file.path = '/test/test.coffee';
+      file.path = '/test/test.coffee';
+      file.path.should.eql('/test/test.coffee');
+      file.history.should.eql(['/test/test.coffee']);
+    });
+
+    it('should not record history when path is null', function() {
+      var file = new File({
+        cwd: '/',
+        path: null
+      });
+
+      should.not.exist(file.path)
+      file.history.should.eql([]);
+
+      file.path = '/test/test.coffee';
+      file.path.should.eql('/test/test.coffee');
+    });
+  });
+
 });