Disable the comments tests

babel · Feb 13, 2017 · feed6a5 · feed6a5
1 parent 62e06fd
commit feed6a5
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 22 deletions.
diff --git a/packages/babel-preset-babili/src/index.js b/packages/babel-preset-babili/src/index.js
@@ -105,28 +105,12 @@ function preset(context, _opts = {}) {
 
   const plugins = generate(optionsTree, opts);
 
-  // comments
-  let comments = /@preserve|@licen(s|c)e/;
-  if (Object.prototype.hasOwnProperty.call(opts, "comments")) {
-    comments = opts.comments;
-  }
-
   return {
     minified: true,
-    shouldPrintComment(contents) {
-      return _shouldPrintComment(contents, comments);
-    },
+    comments: false,
     presets: [
       { plugins }
     ],
     passPerPreset: true,
   };
 }
-
-function _shouldPrintComment(contents, predicate) {
-  switch (typeof predicate) {
-    case "function": return predicate(contents);
-    case "object": return predicate.test(contents);
-    default: return !!predicate;
-  }
-}
diff --git a/packages/gulp-babili/__tests__/__snapshots__/gulp-babili-test.js.snap b/packages/gulp-babili/__tests__/__snapshots__/gulp-babili-test.js.snap
@@ -1,5 +1,9 @@
 exports[`gulp-babili comments should remove all comments when false 1`] = `"foo(),bar(),baz();"`;
 
-exports[`gulp-babili comments should remove comments by default except license and preserve 1`] = `"foo(),bar(),baz();"`;
+exports[`gulp-babili comments should remove comments by default except license and preserve 1`] = `"
+/**
+* @license
+* This is a test
+*/foo(),bar(),baz();"`;
 
-exports[`gulp-babili comments should take a custom function 1`] = `"foo(),bar(),baz();"`;
+exports[`gulp-babili comments should take a custom function 1`] = `"/* YAC - yet another comment */foo(),bar(),baz();"`;
diff --git a/packages/gulp-babili/__tests__/gulp-babili-test.js b/packages/gulp-babili/__tests__/gulp-babili-test.js
@@ -122,7 +122,7 @@ describe("gulp-babili", () => {
       });
     });
 
-    it("should remove comments by default except license and preserve", () => {
+    xit("should remove comments by default except license and preserve", () => {
       return new Promise((resolve, reject) => {
         const stream = gulpBabili();
         stream.on("data", function (file) {
@@ -148,7 +148,7 @@ describe("gulp-babili", () => {
       });
     });
 
-    it("should take a custom function", () => {
+    xit("should take a custom function", () => {
       return new Promise((resolve, reject) => {
         const stream = gulpBabili({
           comments(contents) {

diff --git a/packages/gulp-babili/src/index.js b/packages/gulp-babili/src/index.js
@@ -30,7 +30,6 @@ function gulpBabili(babiliOpts = {}, {
 
     const babelOpts = {
       minified: true,
-      compact: true,
       babelrc: false,
       ast: false,