Groups options test and removes file existence check

lazd · Mar 3, 2016 · a96a72e · a96a72e
1 parent 682dc61
commit a96a72e
Showing 1 changed file with 34 additions and 37 deletions.
diff --git a/test/main.js b/test/main.js
@@ -215,50 +215,47 @@ describe('gulp-replace', function() {
       });
     });
 
-    it('should ignore binary files when skipBinary is enabled', function(done) {
-      var file = new File({
-        path: 'test/fixtures/binary.png',
-        cwd: 'test/',
-        base: 'test/fixtures',
-        contents: fs.readFileSync('test/fixtures/binary.png')
-      });
+    describe('options', function () {
+      describe('skipBinary', function () {
+        it('should ignore binary files when skipBinary is enabled', function(done) {
+          var file = new File({
+            path: 'test/fixtures/binary.png',
+            cwd: 'test/',
+            base: 'test/fixtures',
+            contents: fs.readFileSync('test/fixtures/binary.png')
+          });
 
-      var stream = replacePlugin('world', 'person', {skipBinary: true});
-      stream.on('data', function(newFile) {
-        should.exist(newFile);
-        should.exist(newFile.contents);
+          var stream = replacePlugin('world', 'person', {skipBinary: true});
+          stream.on('data', function(newFile) {
+            newFile.contents.should.eql(fs.readFileSync('test/expected/binary.png'));
+            done();
+          });
 
-        newFile.contents.should.eql(fs.readFileSync('test/expected/binary.png'));
-        done();
-      });
+          stream.write(file);
+          stream.end();
+        });
 
-      stream.write(file);
-      stream.end();
-    });
+        it('should replace string on non binary files when skipBinary is enabled', function(done) {
+          var file = new File({
+            path: 'test/fixtures/helloworld.txt',
+            cwd: 'test/',
+            base: 'test/fixtures',
+            contents: fs.createReadStream('test/fixtures/helloworld.txt')
+          });
 
-    it('should replace string on non binary files when skipBinary is enabled', function(done) {
-      var file = new File({
-        path: 'test/fixtures/helloworld.txt',
-        cwd: 'test/',
-        base: 'test/fixtures',
-        contents: fs.createReadStream('test/fixtures/helloworld.txt')
-      });
+          var stream = replacePlugin('world', 'person', {skipBinary: true});
+          stream.on('data', function(newFile) {
+            newFile.contents.pipe(concatStream({encoding: 'string'}, function(data) {
+              data.should.equal(fs.readFileSync('test/expected/helloworld.txt', 'utf8'));
+              done();
+            }));
+          });
 
-      var stream = replacePlugin('world', 'person', {skipBinary: true});
-      stream.on('data', function(newFile) {
-        should.exist(newFile);
-        should.exist(newFile.contents);
+          stream.write(file);
+          stream.end();
+        });
 
-        newFile.contents.pipe(concatStream({encoding: 'string'}, function(data) {
-          data.should.equal(fs.readFileSync('test/expected/helloworld.txt', 'utf8'));
-          done();
-        }));
       });
-
-      stream.write(file);
-      stream.end();
     });
-
-
   });
 });