Merge 649cd46 into f271c49

scripts/utils/getkarmaconfig.js

@@ -70,11 +70,11 @@ module.exports = function getKarmaConfig() {
 		customLaunchers: {
 			CHROME_TRAVIS_CI: {
 				base: 'Chrome',
-				flags: [ '--no-sandbox', '--disable-background-timer-throttling' ]
+				flags: [ '--disable-background-timer-throttling', '--js-flags="--expose-gc"' ]
 			},
 			CHROME_LOCAL: {
 				base: 'Chrome',
-				flags: [ '--disable-background-timer-throttling' ]
+				flags: [ '--disable-background-timer-throttling', '--js-flags="--expose-gc"' ]
 			}
 		},
 
@@ -123,7 +123,10 @@ module.exports = function getKarmaConfig() {
 		webpackConfig.module.rules.push( {
 			test: /\.jsx?$/,
 			loader: 'istanbul-instrumenter-loader',
-			include: /src/,
+			include: [
+				/src/,
+				/_utils-tests/
+			],
 			exclude: [
 				/node_modules/
 			],

src/ckeditor.jsx

@@ -21,13 +21,15 @@ export default class CKEditor extends React.Component {
 		/**
 		 * An instance of EditorWatchdog or an instance of EditorWatchdog-like adapter for ContextWatchdog.
 		 *
-		 * @type {EditorWatchdog|EditorWatchdogAdapter}
+		 * @type {module:watchdog/watchdog~Watchdog|EditorWatchdogAdapter}
 		 */
 		this.watchdog = null;
 	}
 
 	/**
 	 * An editor instance.
+	 *
+	 * @type {module:core/editor/editor~Editor|null}
 	 */
 	get editor() {
 		if ( !this.watchdog ) {
@@ -37,17 +39,20 @@ export default class CKEditor extends React.Component {
 		return this.watchdog.editor;
 	}
 
-	// The CKEditor component should not be updated by React itself.
-	// However, if the component identifier changes, the whole structure should be created once again.
+	/**
+	 * The CKEditor component should not be updated by React itself.
+	 * However, if the component identifier changes, the whole structure should be created once again.
+	 *
+	 * @param {Object} nextProps
+	 * @return {Boolean}
+	 */
 	shouldComponentUpdate( nextProps ) {
 		if ( !this.editor ) {
 			return false;
 		}
 
-		// Only when the editor component changes the whole structure should be restarted.
+		// Only when the component identifier changes the whole structure should be re-created once again.
 		if ( nextProps.id !== this.props.id ) {
-			this._destroyEditor();
-
 			return true;
 		}
 
@@ -62,22 +67,33 @@ export default class CKEditor extends React.Component {
 		return false;
 	}
 
-	// Initialize the editor when the component is mounted.
+	/**
+	 * Initialize the editor when the component is mounted.
+ 	 */
 	componentDidMount() {
 		this._initializeEditor();
 	}
 
-	// Initialize the editor when the component is updated as it should be destroyed before the update.
+	/**
+	 * Re-render the entire component once again. The old editor will be destroyed and the new one will be created.
+	 */
 	componentDidUpdate() {
+		this._destroyEditor();
 		this._initializeEditor();
 	}
 
-	// Destroy the editor before unmouting the component.
+	/**
+	 * Destroy the editor before unmounting the component.
+ 	 */
 	componentWillUnmount() {
 		this._destroyEditor();
 	}
 
-	// Render a <div> element which will be replaced by CKEditor.
+	/**
+	 * Render a <div> element which will be replaced by CKEditor.
+	 *
+	 * @return {JSX.Element}
+	 */
 	render() {
 		return (
 			<div ref={ this.domContainer }></div>
@@ -86,12 +102,14 @@ export default class CKEditor extends React.Component {
 
 	/**
 	 * Initializes the editor by creating a proper watchdog and initializing it with the editor's configuration.
+	 *
+	 * @private
 	 */
 	_initializeEditor() {
 		if ( this.context instanceof ContextWatchdog ) {
 			this.watchdog = new EditorWatchdogAdapter( this.context );
 		} else {
-			this.watchdog = new EditorWatchdog( this.editor );
+			this.watchdog = new CKEditor._EditorWatchdog( this.props.editor );
 		}
 
 		this.watchdog.setCreator( ( el, config ) => this._createEditor( el, config ) );
@@ -107,6 +125,7 @@ export default class CKEditor extends React.Component {
 	/**
 	 * Creates an editor from the element and configuration.
 	 *
+	 * @private
 	 * @param {HTMLElement} element The source element.
 	 * @param {Object} config CKEditor 5 editor configuration.
 	 * @returns {Promise}
@@ -157,6 +176,8 @@ export default class CKEditor extends React.Component {
 
 	/**
 	 * Destroys the editor by destroying the watchdog.
+	 *
+	 * @private
 	 */
 	_destroyEditor() {
 		// It may happen during the tests that the watchdog instance is not assigned before destroying itself. See: #197.
@@ -172,7 +193,8 @@ export default class CKEditor extends React.Component {
 	/**
 	 * Returns true when the editor should be updated.
 	 *
-	 * @param {*} nextProps React's properties.
+	 * @private
+	 * @param {Object} nextProps React's properties.
 	 * @returns {Boolean}
 	 */
 	_shouldUpdateEditor( nextProps ) {
@@ -192,6 +214,12 @@ export default class CKEditor extends React.Component {
 		return true;
 	}
 
+	/**
+	 * Returns the editor configuration.
+	 *
+	 * @private
+	 * @return {Object}
+	 */
 	_getConfig() {
 		if ( this.props.data && this.props.config.initialData ) {
 			console.warn(
@@ -277,12 +305,7 @@ class EditorWatchdogAdapter {
 	 * An editor instance.
 	 */
 	get editor() {
-		// TODO - try/catch should not be necessary as `getItem` could return `null` instead of throwing errors.
-		try {
-			return this._contextWatchdog.getItem( this._id );
-		} catch ( err ) {
-			return null;
-		}
+		return this._contextWatchdog.getItem( this._id );
 	}
 }
 
@@ -313,3 +336,7 @@ CKEditor.defaultProps = {
 	config: {},
 	onError: ( error, details ) => console.error( error, details )
 };
+
+// Store the API in the static property to easily overwrite it in tests.
+// Too bad dependency injection does not work in Webpack + ES 6 (const) + Babel.
+CKEditor._EditorWatchdog = EditorWatchdog;

tests/_utils-tests/context.js

@@ -0,0 +1,40 @@
+/**
+ * @license Copyright (c) 2003-2021, CKSource - Frederico Knabben. All rights reserved.
+ * For licensing, see LICENSE.md.
+ */
+
+import Context from '../_utils/context';
+
+describe( 'Context', () => {
+	describe( 'constructor()', () => {
+		it( 'saves the config', () => {
+			const config = { foo: 'bar' };
+			const context = new Context( config );
+
+			expect( context.config ).to.equal( config );
+		} );
+	} );
+
+	describe( 'destroy()', () => {
+		it( 'should return a promise that resolves properly', () => {
+			return Context.create()
+				.then( context => {
+					const promise = context.destroy();
+
+					expect( promise ).to.be.an.instanceof( Promise );
+
+					return promise;
+				} );
+		} );
+	} );
+
+	describe( 'create()', () => {
+		it( 'should return a promise that resolves properly', () => {
+			const promise = Context.create();
+
+			expect( promise ).to.be.an.instanceof( Promise );
+
+			return promise;
+		} );
+	} );
+} );

tests/_utils-tests/turnoffdefaulterrorcatching.js

@@ -1,19 +1,22 @@
 /**
  * @license Copyright (c) 2003-2021, CKSource - Frederico Knabben. All rights reserved.
- * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
+ * For licensing, see LICENSE.md.
  */
 
 /* global window */
 
-/**
- * Turns off the default error catching
- * so Mocha won't complain about errors caused by the called function.
- */
-export default async function turnOffDefaultErrorCatching( fn ) {
-	const originalOnError = window.onerror;
-	window.onerror = null;
+import turnOffDefaultErrorCatching from '../_utils/turnoffdefaulterrorcatching';
+
+describe( 'turnOffDefaultErrorCatching()', () => {
+	it( 'should catch the error', () => {
+		const onErrorStub = sinon.stub( window, 'onerror' );
+
+		turnOffDefaultErrorCatching( () => {
+			window.onerror( 'Foo', null, 0 );
+		} );
 
-	await fn();
+		onErrorStub.restore();
 
-	window.onerror = originalOnError;
-}
+		expect( onErrorStub.called ).to.equal( false );
+	} );
+} );

tests/_utils/context.js

@@ -1,13 +1,22 @@
-
 /**
  * @license Copyright (c) 2003-2021, CKSource - Frederico Knabben. All rights reserved.
  * For licensing, see LICENSE.md.
  */
 
+/**
+ * Mock of class that representing the Context feature.
+ *
+ * @see: https://ckeditor.com/docs/ckeditor5/latest/api/module_core_context-Context.html
+ */
 export default class ContextMock {
+	constructor( config ) {
+		this.config = config;
+	}
+
 	static create( config ) {
 		return Promise.resolve( new ContextMock( config ) );
 	}
+
 	static destroy() {
 		return Promise.resolve();
 	}

tests/_utils/turnoffdefaulterrorcatching.js

@@ -0,0 +1,19 @@
+/**
+ * @license Copyright (c) 2003-2021, CKSource - Frederico Knabben. All rights reserved.
+ * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
+ */
+
+/* global window */
+
+/**
+ * Turns off the default error catching
+ * so Mocha won't complain about errors caused by the called function.
+ */
+export default async function turnOffDefaultErrorCatching( fn ) {
+	const originalOnError = window.onerror;
+	window.onerror = () => {};
+
+	await fn();
+
+	window.onerror = originalOnError;
+}