Fix build, cleanup

Author: helfer

package.json

@@ -17,8 +17,7 @@
     "react-scripts-ts": "^2.6.0",
     "ts-jest": "22.0.1",
     "tslint": "^5.11.0",
-    "typescript": "^2.6.2",
-    "zen-observable-ts": "^0.8.11"
+    "typescript": "^2.6.2"
   },
   "main": "build/dist/index.js",
   "module": "build/dist/index.js",

src/QueueLink.ts

@@ -7,7 +7,7 @@ import {
 import {
     Observable,
     Observer,
-    } from '@apollo/client/utilities';
+} from '@apollo/client/utilities';
 
 interface OperationQueueEntry {
     operation: Operation;
@@ -39,7 +39,7 @@ export default class QueueLink extends ApolloLink {
         if (operation.getContext().skipQueue) {
             return forward(operation);
         }
-        return new Observable(observer => {
+        return new Observable<FetchResult>((observer: Observer<FetchResult>) => {
             const operationEntry = { operation, forward, observer };
             this.enqueue(operationEntry);
             return () => this.cancelOperation(operationEntry);

src/TestUtils.ts

@@ -1,8 +1,9 @@
 import {
     ApolloLink,
+    FetchResult,
     Operation,
 } from '@apollo/client/link/core';
-import { Observable } from '@apollo/client/utilities';
+import { Observable, Observer } from '@apollo/client/utilities';
 import {
     ExecutionResult,
 } from 'graphql';
@@ -17,7 +18,7 @@ export class TestLink extends ApolloLink {
     public request (operation: Operation) {
         this.operations.push(operation);
         // TODO(helfer): Throw an error if neither testError nor testResponse is defined
-        return new Observable(observer => {
+        return new Observable<FetchResult>((observer: Observer<FetchResult>) => {
             if (operation.getContext().testError) {
                 setTimeout(() => observer.error(operation.getContext().testError), 0);
                 return;
@@ -49,14 +50,14 @@ export const assertObservableSequence = (
     }
     return new Promise((resolve, reject) => {
         const sub = observable.subscribe({
-            next: (value) => {
+            next: (value: FetchResult) => {
                 expect({ type: 'next', value }).toEqual(sequence[index]);
                 index++;
                 if (index === sequence.length) {
                     resolve(true);
                 }
             },
-            error: (value) => {
+            error: (value: FetchResult) => {
                 expect({ type: 'error', value }).toEqual(sequence[index]);
                 index++;
                 // This check makes sure that there is no next element in

tsconfig.json

@@ -8,7 +8,8 @@
         "removeComments": true,
         "preserveConstEnums": true,
         "outDir": "build/dist",
-        "sourceMap": true
+        "sourceMap": true,
+        "skipLibCheck": true
     },
     "include": [
         "src/**/*"

yarn.lock

@@ -8037,16 +8037,6 @@ yargs@~3.10.0:
     decamelize "^1.0.0"
     window-size "0.1.0"
 
-zen-observable-ts@^0.8.11:
-  version "0.8.11"
-  resolved "https://registry.yarnpkg.com/zen-observable-ts/-/zen-observable-ts-0.8.11.tgz#d54a27cd17dc4b4bb6bd008e5c096af7fcb068a9"
-  dependencies:
-    zen-observable "^0.8.0"
-
-zen-observable@^0.8.0:
-  version "0.8.11"
-  resolved "https://registry.yarnpkg.com/zen-observable/-/zen-observable-0.8.11.tgz#d3415885eeeb42ee5abb9821c95bb518fcd6d199"
-
 zen-observable@^0.8.14:
   version "0.8.15"
   resolved "https://registry.yarnpkg.com/zen-observable/-/zen-observable-0.8.15.tgz#96415c512d8e3ffd920afd3889604e30b9eaac15"