lint(e2e): turn off remaining mocha lint violations

some of them may have value, but the majority of the value has been
captured now

Author: mikehardy

.eslintrc.js

@@ -41,6 +41,11 @@ module.exports = {
     '@typescript-eslint/no-empty-function': 'off',
     // off for validation tests
     '@typescript-eslint/ban-ts-ignore': 'off',
+    'mocha/no-skipped-tests': 'off', // we skip tests for a reason, frequently
+    'mocha/no-top-level-hooks': 'off', // potentially has value if anyone wants to refactor
+    'mocha/no-hooks-for-single-case': 'off', // potentially has value
+    'mocha/no-setup-in-describe': 'off', // potentially has value, large refactor here though
+    
   },
   globals: {
     __DEV__: true,

packages/app/e2e/app.e2e.js

@@ -53,6 +53,7 @@ describe('firebase', function() {
   });
 });
 
+// eslint-disable-next-line mocha/max-top-level-suites
 describe('firebase -> X', function() {
   it('apps should provide an array of apps', function() {
     should.equal(!!firebase.apps.length, true);