Merge pull request #1 from flovilmart/logging

Better password stripping tests
spenthil · Jul 15, 2016 · bc09f88 · bc09f88
2 parents 7ed9a7d + be22ef1
commit bc09f88
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/spec/FileLoggerAdapter.spec.js b/spec/FileLoggerAdapter.spec.js
@@ -59,7 +59,10 @@ describe('verbose logs', () => {
         level: 'verbose'
       });
     }).then((results) => {
-      expect(results[1].body.password).toEqual("********");
+      let logString = JSON.stringify(results);
+      expect(logString.match(/\*\*\*\*\*\*\*\*/g).length).not.toBe(0);
+      expect(logString.match(/moon-y/g)).toBe(null);
+
       var headers = {
         'X-Parse-Application-Id': 'test',
         'X-Parse-REST-API-Key': 'rest'
@@ -74,11 +77,16 @@ describe('verbose logs', () => {
           size: 100,
           level: 'verbose'
         }).then((results) => {
-          expect(results[1].url.includes('password=********')).toEqual(true);
+          let logString = JSON.stringify(results);
+          expect(logString.match(/\*\*\*\*\*\*\*\*/g).length).not.toBe(0);
+          expect(logString.match(/moon-y/g)).toBe(null);
           done();
         });
       });
-    });
+    }).catch((err) => {
+      fail(JSON.stringify(err));
+      done();
+    })
   });
 
   it("should not mask information in non _User class", (done) => {