From 16468b1216f14ca60f74c2ec951ea230c4b41e8d Mon Sep 17 00:00:00 2001
From: Raul Ochoa <rochoaf@gmail.com>
Date: Fri, 18 Sep 2015 16:45:35 +0200
Subject: [PATCH] remove console.log

---
 test/acceptance/multilayer.js            | 7 -------
 test/acceptance/ported/multilayer.js     | 3 ---
 test/acceptance/ported/server_gettile.js | 6 ++----
 test/unit/cartodb/req2params.test.js     | 2 --
 4 files changed, 2 insertions(+), 16 deletions(-)

diff --git a/test/acceptance/multilayer.js b/test/acceptance/multilayer.js
index 17609b9c..b6409d98 100644
--- a/test/acceptance/multilayer.js
+++ b/test/acceptance/multilayer.js
@@ -174,7 +174,6 @@ suite(suiteName, function() {
           var errors = [];
           if ( err ) {
             errors.push(err.message);
-            console.log("Error: " + err);
           }
           redis_client.keys("map_cfg|" + expected_token, function(err, matches) {
               if ( err ) {
@@ -278,7 +277,6 @@ suite(suiteName, function() {
           var errors = [];
           if ( err ) {
             errors.push(err.message);
-            console.log("Error: " + err);
           }
           redis_client.keys("map_cfg|" + expected_token, function(err, matches) {
               if ( err ) {
@@ -500,7 +498,6 @@ suite(suiteName, function() {
           var errors = [];
           if ( err ) {
             errors.push(err.message);
-            console.log("Error: " + err);
           }
           redis_client.keys("map_cfg|" + expected_token, function(err, matches) {
               if ( err ) {
@@ -831,7 +828,6 @@ suite(suiteName, function() {
           var errors = [];
           if ( err ) {
             errors.push(err.message);
-            console.log("Error: " + err);
           }
           redis_client.keys("map_cfg|" + expected_token, function(err, matches) {
               if ( err ) {
@@ -950,7 +946,6 @@ suite(suiteName, function() {
           var errors = [];
           if ( err ) {
             errors.push(err.message);
-            console.log("Error: " + err);
           }
           redis_client.keys("map_cfg|" + expected_token, function(err, matches) {
               if ( err ) {
@@ -1114,7 +1109,6 @@ suite(suiteName, function() {
           var errors = [];
           if ( err ) {
             errors.push(err.message);
-            console.log("Error: " + err);
           }
           redis_client.keys("map_cfg|" + expected_token, function(err, matches) {
               if ( err ) {
@@ -1219,7 +1213,6 @@ suite(suiteName, function() {
         function finish(err) {
           if ( err ) {
             errors.push(err.message);
-            console.log("Error: " + err);
           }
           if ( errors.length ) {
               done(new Error(errors));
diff --git a/test/acceptance/ported/multilayer.js b/test/acceptance/ported/multilayer.js
index fe1f3b47..8572a1c7 100644
--- a/test/acceptance/ported/multilayer.js
+++ b/test/acceptance/ported/multilayer.js
@@ -41,7 +41,6 @@ describe('multilayer', function() {
           fs.readFile(filename, "binary", function(err, file) {
             if ( err ) {
               response.writeHead(404, {'Content-Type': 'text/plain'});
-              console.log("File '" + filename + "' not found");
               response.write("404 Not Found\n");
             } else {
               response.writeHead(200);
@@ -499,7 +498,6 @@ describe('multilayer', function() {
         },
         function do_get_tile(err)
         {
-          console.log(err);
           assert.ifError(err);
           var next = this;
           assert.response(server, {
@@ -742,7 +740,6 @@ describe('multilayer', function() {
           return null;
         },
         function finish(err) {
-//if ( err ) console.log(err.stack);
           var errors = [];
           if ( err ) {
               errors.push(''+err);
diff --git a/test/acceptance/ported/server_gettile.js b/test/acceptance/ported/server_gettile.js
index b185728a..3e95b4a6 100644
--- a/test/acceptance/ported/server_gettile.js
+++ b/test/acceptance/ported/server_gettile.js
@@ -201,10 +201,8 @@ describe('server_gettile', function() {
     };
     var test_strict_lbl = "unused directives are not tolerated if strict";
     if ( semver.satisfies(mapnik.versions.mapnik, '2.3.x') ) {
-      // Strictness handling changed in 2.3.x, possibly a bug:
-      // see http://github.com/mapnik/mapnik/issues/2301
-      console.warn("Strictness test skipped due to http://github.com/mapnik/mapnik/issues/2301");
-      it.skip(test_strict_lbl,  test_strictness);
+      // Strictness handling changed in 2.3.x, possibly a bug: see http://github.com/mapnik/mapnik/issues/2301
+      it.skip('[skipped due to http://github.com/mapnik/mapnik/issues/2301]' + test_strict_lbl,  test_strictness);
     }
     else {
       it(test_strict_lbl,  test_strictness);
diff --git a/test/unit/cartodb/req2params.test.js b/test/unit/cartodb/req2params.test.js
index f3187d86..43f79ae0 100644
--- a/test/unit/cartodb/req2params.test.js
+++ b/test/unit/cartodb/req2params.test.js
@@ -61,7 +61,6 @@ suite('req2params', function() {
       var req = {headers: { host:'localhost' }, query: {} };
       baseController.req2params(prepareRequest(req), function(err, req) {
           if ( err ) { done(err); return; }
-          //console.dir(req);
           assert.ok(_.isObject(req.query), 'request has query');
           assert.ok(!req.query.hasOwnProperty('dbuser'), 'dbuser was removed from query');
           assert.ok(req.hasOwnProperty('params'), 'request has params');
@@ -76,7 +75,6 @@ suite('req2params', function() {
       var req = {headers: { host:'localhost' }, query: {map_key: '1234'} };
       baseController.req2params(prepareRequest(req), function(err, req) {
           if ( err ) { done(err); return; }
-          //console.dir(req);
           assert.ok(_.isObject(req.query), 'request has query');
           assert.ok(!req.query.hasOwnProperty('dbuser'), 'dbuser was removed from query');
           assert.ok(req.hasOwnProperty('params'), 'request has params');