From dd577e43efb3fa2c98cdd75386dc96a2cb4a5608 Mon Sep 17 00:00:00 2001
From: Dannon Baker <dannon.baker@gmail.com>
Date: Mon, 31 Aug 2020 15:58:06 -0400
Subject: [PATCH] Disable XMLHttpRequest for testing; I don't understand what
 in rxjs (?) was making this an issue, but this should fix the outstanding
 tests.

---
 client/src/components/Tags/tagService.test.js | 54 +++++++++----------
 client/tests/jest/jest.setup.js               |  3 ++
 2 files changed, 30 insertions(+), 27 deletions(-)

diff --git a/client/src/components/Tags/tagService.test.js b/client/src/components/Tags/tagService.test.js
index 417f103c9d3..d109adbd469 100644
--- a/client/src/components/Tags/tagService.test.js
+++ b/client/src/components/Tags/tagService.test.js
@@ -2,8 +2,8 @@ import axios from "axios";
 import MockAdapter from "axios-mock-adapter";
 import { TagService } from "./tagService";
 import { createTag } from "./model";
-//import { interval } from "rxjs";
-//import { take, takeUntil } from "rxjs/operators";
+import { interval } from "rxjs";
+import { take, takeUntil } from "rxjs/operators";
 // test response
 import autocompleteResponse from "./testData/autocompleteResponse.txt";
 
@@ -91,34 +91,34 @@ describe("Tags/tagService.js", () => {
 
         // hit the search input with multiple entries, only one ajax call
         // should result because of debouncing
-        //it("should debounce autocomplete search inputs", (done) => {
-        //    // stub ajax request to return the success response if the
-        //    // searchString is the expected input
+        it("should debounce autocomplete search inputs", (done) => {
+            // stub ajax request to return the success response if the
+            // searchString is the expected input
 
-        //    // take first emission, debouncing should guarantee the first
-        //    // emission is the last thing we sent, should be searchString
-        //    const timer$ = interval(100).pipe(take(1));
-        //    const option$ = svc.autocompleteOptions.pipe(takeUntil(timer$));
+            // take first emission, debouncing should guarantee the first
+            // emission is the last thing we sent, should be searchString
+            const timer$ = interval(100).pipe(take(1));
+            const option$ = svc.autocompleteOptions.pipe(takeUntil(timer$));
 
-        //    const nextHandler = jest.fn();
+            const nextHandler = jest.fn();
 
-        //    option$.subscribe({
-        //        next: nextHandler,
-        //        error: (err) => console.warn("error", err),
-        //        complete: () => {
-        //            expect(nextHandler).toHaveBeenCalledTimes(1);
-        //            expect(axiosMock.history.get.length).toBe(1);
-        //            done();
-        //        },
-        //    });
+            option$.subscribe({
+                next: nextHandler,
+                error: (err) => console.warn("error", err),
+                complete: () => {
+                    expect(nextHandler).toHaveBeenCalledTimes(1);
+                    expect(axiosMock.history.get.length).toBe(1);
+                    done();
+                },
+            });
 
-        //    // spam a bunch of key inputs followed by the correct input
-        //    const spamCount = 2 + Math.floor(Math.random() * 10);
-        //    for (let i = 0; i < spamCount; i++) {
-        //        const spamVal = new String(Math.random());
-        //        svc.autocompleteSearchText = spamVal;
-        //    }
-        //    svc.autocompleteSearchText = searchString;
-        //});
+            // spam a bunch of key inputs followed by the correct input
+            const spamCount = 2 + Math.floor(Math.random() * 10);
+            for (let i = 0; i < spamCount; i++) {
+                const spamVal = new String(Math.random());
+                svc.autocompleteSearchText = spamVal;
+            }
+            svc.autocompleteSearchText = searchString;
+        });
     });
 });
diff --git a/client/tests/jest/jest.setup.js b/client/tests/jest/jest.setup.js
index d0de870dc55..dd8c81d1e15 100644
--- a/client/tests/jest/jest.setup.js
+++ b/client/tests/jest/jest.setup.js
@@ -1 +1,4 @@
 import "@testing-library/jest-dom";
+/* still don't understand what was invoking the following, but nothing should,
+and this makes the tag tests work correctly */
+global.XMLHttpRequest = undefined;
\ No newline at end of file