Merge pull request #94 from WebSpellChecker/handle-wrong-bundle-loading

Handle wrong bundle loading

Author: Bohdan-Chui

src/wproofreader.js

@@ -150,7 +150,12 @@ export default class WProofreader extends Plugin {
 	_loadWscbundle() {
 		const scriptLoader = new ScriptLoader(this._userOptions.srcUrl);
 
-		return scriptLoader.load();
+		return scriptLoader.load()
+			.then(() => {
+				if (!window.WEBSPELLCHECKER) {
+					throw new Error('WEBSPELLCHECKER is not defined.');
+				}
+			});
 	}
 
 	/**
@@ -278,7 +283,7 @@ export default class WProofreader extends Plugin {
 	 * @private
 	 */
 	_createInstance(root) {
-		WEBSPELLCHECKER.init(this._mergeOptions(root), this._handleInstanceCreated.bind(this));
+		window.WEBSPELLCHECKER.init(this._mergeOptions(root), this._handleInstanceCreated.bind(this));
 	}
 
 	/**

tests/mocks/mock-script.js

@@ -0,0 +1 @@
+(function() { })()

tests/wproofreader.js

@@ -452,6 +452,26 @@ describe('WProofreader', () => {
 					expect(editor.editing.view.getDomRoot('main')).to.not.be.equal(span);
 				});
 		});
+
+		describe('_loadWscbundle() method', () => {
+			it('should throw an error if WEBSPELLCHECKER is not defined', () => {
+				const options = { srcUrl: 'http://localhost:3000/tests/mocks/mock-script.js' };
+				const copy = window.WEBSPELLCHECKER;
+
+				delete window.WEBSPELLCHECKER;
+
+				const wproofreader = new WProofreader();
+				wproofreader._userOptions = options;
+
+				return wproofreader._loadWscbundle()
+					.catch((e) => {
+						expect(e.message).to.be.equal('WEBSPELLCHECKER is not defined.');
+					})
+					.finally(() => {
+						window.WEBSPELLCHECKER = copy;
+					});
+			});
+		});
 	});
 
 	describe('in CKEditor 5 restricted editing mode', () => {