fix(common): Re-enables two unit tests now functional b/c worker polyfill

This commit is contained in:
jahorton 2019-12-09 10:14:14 +07:00
parent 110715a30d
commit 07d8e659df
2 changed files with 0 additions and 13 deletions

View file

@ -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);

View file

@ -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.