use nyc for coverage (#3652)

* use nyc for coverage

* removes unnecessary logss

* Disable RedisCacheAdapter tests when not in specific environment

spec/.eslintrc.json

@@ -16,6 +16,7 @@
       "notEqual": true,
       "it_exclude_dbs": true,
       "describe_only_db": true,
+      "describe_only": true,
       "on_db": true,
       "defaultConfiguration": true,
       "expectSuccess": true,

spec/ParseACL.spec.js

@@ -755,14 +755,12 @@ describe('Parse.ACL', () => {
       user2.set("password", "burger");
       return user2.signUp();
     }).then(() => {
-      console.log(user2.getSessionToken());
       return object.destroy({sessionToken: user2.getSessionToken() });
-    }).then((res) => {
-      console.log(res);
+    }).then(() => {
       fail('should not be able to destroy the object');
       done();
     }, (err) => {
-      console.error(err);
+      expect(err).not.toBeUndefined();
       done();
     });
   });

spec/RedisCacheAdapter.spec.js

@@ -1,6 +1,12 @@
 var RedisCacheAdapter = require('../src/Adapters/Cache/RedisCacheAdapter').default;
-
-describe('RedisCacheAdapter', function() {
+/*
+To run this test part of the complete suite
+set PARSE_SERVER_TEST_CACHE='redis'
+and make sure a redis server is available on the default port
+ */
+describe_only(() => {
+  return process.env.PARSE_SERVER_TEST_CACHE === 'redis';
+})('RedisCacheAdapter', function() {
   var KEY = 'hello';
   var VALUE = 'world';
 

spec/helper.js

@@ -419,6 +419,14 @@ global.describe_only_db = db => {
   }
 }
 
+global.describe_only = (validator) =>{
+  if (validator()) {
+    return describe;
+  } else {
+    return xdescribe;
+  }
+};
+
 
 var libraryCache = {};
 jasmine.mockLibrary = function(library, name, mock) {