fix: tests running after deps upgrade

Author: jankapunkt

src/Meteor.js

@@ -54,6 +54,7 @@ const Meteor = {
     return {
       connected: !!this._reactiveDict.get('connected'),
       status: Data.ddp ? Data.ddp.status : 'disconnected',
+      hasDdp: !!Data.ddp,
       //retryCount: 0
       //retryTime:
       //reason:

test/hooks/mockServer.js

@@ -25,7 +25,7 @@ export async function mochaGlobalSetup() {
     }
   };
 
-  console.debug('mockserver wait for connected...')
+  console.debug('mockserver wait for connected...');
   await new Promise((resolve) => {
     _server.on('connection', (socket) => {
       console.debug('mockserver listening');
@@ -35,7 +35,7 @@ export async function mochaGlobalSetup() {
       resolve();
     });
   });
-  console.debug('end of setup')
+  console.debug('end of setup');
 }
 
 export async function mochaGlobalTeardown() {

test/src/Collection.tests.js

@@ -160,7 +160,6 @@ describe('Collection', function () {
           foo: 'bar',
           _version: 1,
         });
-        console.debug(server())
         server().message(null);
         done();
       });

test/src/Meteor.tests.js

@@ -1,6 +1,6 @@
 import { expect } from 'chai';
 import Meteor from '../../src/Meteor';
-import { awaitDisconnected, stub, restoreAll } from '../testHelpers';
+import { stub, restoreAll, asyncTimeout } from '../testHelpers';
 import DDP from '../../lib/ddp';
 
 describe('Meteor - integration', function () {
@@ -16,38 +16,6 @@ describe('Meteor - integration', function () {
   });
 
   describe(Meteor.connect.name, () => {
-    before(awaitDisconnected);
-
-    it('requires manual connect if autoConnect is  set to false', function (done) {
-      this.timeout(3000);
-      expect(Meteor.getData().ddp.status).to.equal('disconnected');
-      stub(DDP.prototype, 'on', () => {});
-      let connectCalled = 0;
-      stub(DDP.prototype, 'connect', () => {
-        done(new Error('should not automatically call connect'));
-      });
-
-      const AsyncStorage = {
-        getItem: async () => {},
-        setItem: async () => {},
-        removeItem: async () => {},
-      };
-
-      const endpoint = `ws://localhost:3000/websocket`;
-      Meteor.connect(endpoint, {
-        AsyncStorage,
-        NetInfo: null,
-        autoConnect: false,
-      });
-
-      // let's wait some time to make sure no internals
-      // unintentionally call ddp.connect before we do
-      setTimeout(() => {
-        expect(Meteor.getData().ddp.status).to.equal('disconnected');
-        done();
-      }, 2900);
-    });
-
     it('allows to bypass NetInfo', (done) => {
       stub(DDP.prototype, 'on', () => {});
       stub(DDP.prototype, 'connect', done);

test/testHelpers.js

@@ -38,17 +38,8 @@ export const restoreAll = () => {
   });
 };
 
-export const awaitDisconnected = async () => {
-  Meteor.disconnect();
-  await new Promise((resolve) => {
-    let timer = setInterval(() => {
-      if (Meteor.status().status === 'disconnected') {
-        clearInterval(timer);
-        resolve();
-      }
-    }, 100);
-  });
-};
+export const asyncTimeout = (ms) =>
+  new Promise((resolve) => setTimeout(resolve, ms));
 
 export const props = (obj) => {
   let p = [];