From 07d8e659df10539ea2525edd205cbfe0f9313d35 Mon Sep 17 00:00:00 2001 From: jahorton Date: Mon, 9 Dec 2019 10:14:14 +0700 Subject: [PATCH] fix(common): Re-enables two unit tests now functional b/c worker polyfill --- .../in_browser/cases/worker-dummy-integration.js | 6 ------ .../unit_tests/in_browser/cases/worker-trie-integration.js | 7 ------- 2 files changed, 13 deletions(-) diff --git a/common/predictive-text/unit_tests/in_browser/cases/worker-dummy-integration.js b/common/predictive-text/unit_tests/in_browser/cases/worker-dummy-integration.js index 5b274abd66..4702709d03 100644 --- a/common/predictive-text/unit_tests/in_browser/cases/worker-dummy-integration.js +++ b/common/predictive-text/unit_tests/in_browser/cases/worker-dummy-integration.js @@ -47,12 +47,6 @@ describe('LMLayer using dummy model', function () { describe('Wordbreaking', function () { it('will perform (default) wordbreaking and return word at caret', function () { - if(navigator.userAgent.indexOf('MSIE') !== -1 || navigator.appVersion.indexOf('Trident/') > -1) { - // Our wordbreaking uses the IE-unsupported .codePointAt() function. - console.warn("Bypassing wordbreak test on IE."); - this.skip(); - } - this.timeout(config.timeouts.standard * 3); // This one makes multiple subsequent calls across // the WebWorker boundary, so we should be generous here. var lmLayer = new LMLayer(helpers.defaultCapabilities); diff --git a/common/predictive-text/unit_tests/in_browser/cases/worker-trie-integration.js b/common/predictive-text/unit_tests/in_browser/cases/worker-trie-integration.js index 6d8ca3bc00..d5441823f9 100644 --- a/common/predictive-text/unit_tests/in_browser/cases/worker-trie-integration.js +++ b/common/predictive-text/unit_tests/in_browser/cases/worker-trie-integration.js @@ -14,13 +14,6 @@ describe('LMLayer using the trie model', function () { // the WebWorker boundary, so we should be generous here. var lmLayer = new LMLayer(helpers.defaultCapabilities); - // As noted in worker-dummy-integration as well. - if(navigator.userAgent.indexOf('MSIE') !== -1 || navigator.appVersion.indexOf('Trident/') > -1) { - // Our wordbreaking uses the IE-unsupported .codePointAt() function. - console.warn("Bypassing wordbreak test on IE."); - this.skip(); - } - // We're testing many as asynchronous messages in a row. // this would be cleaner using async/await syntax, but // alas some of our browsers don't support it.