Integrated error reporting: closes #2 and closes #4

index.js

@@ -5,7 +5,12 @@ var GulpBrowserBrowserify;
         return function () {
             return through.obj(function (file, enc, cb) {
                 var bundleCallback = function (err, bufferedContent) {
+                    if (Buffer.isBuffer(bufferedContent)) {
                         file.contents = bufferedContent;
+                    }
+                    else {
+                        pr.beautylog.error("gulp-browser: .browserify() " + err.message);
+                    }
                     cb(null, file);
                 };
                 browserify(file)

package.json

@@ -7,7 +7,8 @@
     "test": "(cd ts/compile && node compile.js) && (node test.js)",
     "reinstall": "(rm -r node_modules && npm install)",
     "release": "(git pull origin master && npm version patch && git push origin master && git checkout release && git merge master && git push origin release && git checkout master)",
-    "startdev": "(git checkout master && git pull origin master)"
+    "update": "(git checkout master && git pull origin master && npm install)",
+    "upgrade": "(npm run update) && (ncu upgradeAll && npm install)"
   },
   "repository": {
     "type": "git",
@@ -31,9 +32,9 @@
     "through2": "2.0.0"
   },
   "devDependencies": {
-    "beautylog": "1.0.4",
+    "beautylog": "1.0.6",
     "gulp": "3.9.0",
     "gulp-callfunction": "0.0.10",
-    "gulp-typescript": "2.9.2"
+    "gulp-typescript": "2.10.0"
   }
 }

test.js

@@ -4,13 +4,24 @@ var gulpBrowser = require("./index.js");
 var pr = require("pushrocks");
 var gulpCallFunction = require("gulp-callfunction");
 var pipeWorked = function () {
-    pr.beautylog.success("Pipe didn't break! Ready for Primetime!");
+    pr.beautylog.info("Pipe didn't break! Ready for Primetime!");
 };
 pr.beautylog.log('Now trying to browserify a testfile...');
 gulp.task('gulpBrowserTest', function (cb) {
-    gulp.src('./test/*.js')
+    var stream = gulp.src('./test/browserifyGulpTest.js')
         .pipe(gulpBrowser.browserify())
         .pipe(gulp.dest("./test/result/"))
         .pipe(gulpCallFunction(pipeWorked));
+    return stream;
 });
-gulp.start.apply(gulp, ['gulpBrowserTest']);
+gulp.task('gulpBrowserTestError', function (cb) {
+    var stream = gulp.src('./test/browserifyGulpTestError.js')
+        .pipe(gulpBrowser.browserify())
+        .pipe(gulp.dest("./test/result/"))
+        .pipe(gulpCallFunction(pipeWorked));
+    return stream;
+});
+gulp.task("default", ['gulpBrowserTest', 'gulpBrowserTestError'], function () {
+    pr.beautylog.success("Test passed!");
+});
+gulp.start.apply(gulp, ['default']);

test/browserifyGulpTestError.js

@@ -0,0 +1 @@
+var myError = require("my-error");

ts/modulebrowserify.ts

@@ -4,7 +4,11 @@ module GulpBrowserBrowserify {
         return function() {
             return through.obj((file, enc, cb) => { //this is the trough object that gets returned by gulpBrowser.browserify();
                 var bundleCallback = function(err, bufferedContent) {
+                    if (Buffer.isBuffer(bufferedContent)){
                         file.contents = bufferedContent;
+                    } else {
+                        pr.beautylog.error("gulp-browser: .browserify() " + err.message);
+                    }
                     cb(null,file);
                 };
                 browserify(file)

ts/test.ts

@@ -5,14 +5,25 @@ var pr = require("pushrocks");
 var gulpCallFunction = require("gulp-callfunction");
 
 var pipeWorked = function() {
-    pr.beautylog.success("Pipe didn't break! Ready for Primetime!");
+    pr.beautylog.info("Pipe didn't break! Ready for Primetime!");
 };
 
 pr.beautylog.log('Now trying to browserify a testfile...');
 gulp.task('gulpBrowserTest',function(cb) {
-    gulp.src('./test/*.js')
+    var stream =gulp.src('./test/browserifyGulpTest.js')
         .pipe(gulpBrowser.browserify())
         .pipe(gulp.dest("./test/result/"))
         .pipe(gulpCallFunction(pipeWorked));
+    return stream;
 });
-gulp.start.apply(gulp, ['gulpBrowserTest']);
+gulp.task('gulpBrowserTestError',function(cb) {
+    var stream = gulp.src('./test/browserifyGulpTestError.js')
+        .pipe(gulpBrowser.browserify())
+        .pipe(gulp.dest("./test/result/"))
+        .pipe(gulpCallFunction(pipeWorked));
+    return stream;
+});
+gulp.task("default",['gulpBrowserTest','gulpBrowserTestError'],function(){
+    pr.beautylog.success("Test passed!");
+});
+gulp.start.apply(gulp, ['default']);