fix(mocha): mocha globals should be re-wrapped for every new suite

Closes #523, closes #962

Author: juliemr

lib/frameworks/mocha.js

@@ -13,6 +13,10 @@ exports.run = function(runner, specs, done) {
   // Mocha doesn't set up the ui until the pre-require event, so
   // wait until then to load mocha-webdriver adapters as well.
   mocha.suite.on('pre-require', function() {
+    // We need to re-wrap all of the global functions, which selenium-webdriver/
+    // testing only does when it is required. So first we must remove it from
+    // the cache.
+    delete require.cache[require.resolve('selenium-webdriver/testing')];
     var mochaAdapters = require('selenium-webdriver/testing');
     global.after = mochaAdapters.after;
     global.afterEach = mochaAdapters.afterEach;

spec/mocha/lib_spec.js

@@ -16,6 +16,10 @@ describe('no protractor at all', function() {
 });
 
 describe('protractor library', function() {
+  it.skip('should be able to skip tests', function() {
+    expect(true).to.equal(false);
+  });
+
   it('should expose the correct global variables', function() {
     expect(protractor).to.exist;
     expect(browser).to.exist;
@@ -25,6 +29,8 @@ describe('protractor library', function() {
   });
 
   it('should wrap webdriver', function() {
+    // Mocha will report the spec as slow if it goes over this time in ms.
+    this.slow(6000);
     browser.get('index.html');
     expect(browser.getTitle()).to.eventually.equal('My AngularJS App');
   });

spec/mochaConf.js

@@ -14,4 +14,8 @@ exports.config = {
   capabilities: env.capabilities,
 
   baseUrl: env.baseUrl,
+
+  mochaOpts: {
+    reporter: 'spec'
+  }
 };