Fixing the unit test for expect not visible

Author: beatfactor

lib/api/expect/visible.js

@@ -30,7 +30,7 @@ VisibleAssertion.prototype.executeCommand = function(callback) {
 };
 
 VisibleAssertion.prototype.elementFound = function() {
-  this.passed = this.negate ? !this.resultValue : this.resultValue;
+  this.passed = this.negate ? this.resultValue === false : this.resultValue;
   this.expected = this.negate ? 'not visible' : 'visible';
   this.actual = this.resultValue ? 'visible' : 'not visible';
 

test/src/api/expect/testExpectVisible.js

@@ -157,15 +157,15 @@ module.exports = {
     },
 
     'to not be visible with waitFor [FAILED]': function (done) {
-      this.client.api.globals.waitForConditionPollInterval = 10;
+      this.client.api.globals.waitForConditionPollInterval = 15;
 
-      Nocks.elementFound().visible().visible().visible();
+      Nocks.elementFound().visible().visible();
 
       var expect = this.client.api.expect.element('#weblogin').to.not.be.visible.before(25);
       this.client.once('nightwatch:finished', function (results, errors) {
         assert.equal(expect.assertion.waitForMs, 25);
-        assert.equal(expect.assertion.passed, false);
         assert.equal(expect.assertion.message, 'Expected element <#weblogin> to not be visible in 25ms');
+        assert.equal(expect.assertion.passed, false);
         done();
       });