Browse Source

Avoided using document.body for Editor tests.

fredck 10 năm trước cách đây
mục cha
commit
a18fb22088
1 tập tin đã thay đổi với 14 bổ sung13 xóa
  1. 14 13
      packages/ckeditor5-engine/tests/editor/editor.js

+ 14 - 13
packages/ckeditor5-engine/tests/editor/editor.js

@@ -3,27 +3,32 @@
  * For licensing, see LICENSE.md.
  */
 
-/* globals describe, it, expect, sinon, document */
+/* globals describe, it, expect, beforeEach, sinon, document */
 
 'use strict';
 
 var modules = bender.amd.require( 'editor' );
 
-describe( 'constructor', function() {
-	it( 'should create a new editor instance', function() {
-		var Editor = modules.editor;
+var editor;
+var element;
+
+beforeEach( function() {
+	var Editor = modules.editor;
 
-		var editor = new Editor( document.body );
+	element = document.createElement( 'div' );
+	document.body.appendChild( element );
+
+	editor = new Editor( element );
+} );
 
-		expect( editor ).to.have.property( 'element' ).to.equal( document.body );
+describe( 'constructor', function() {
+	it( 'should create a new editor instance', function() {
+		expect( editor ).to.have.property( 'element' ).to.equal( element );
 	} );
 } );
 
 describe( 'destroy', function() {
 	it( 'should fire "destroy"', function() {
-		var Editor = modules.editor;
-
-		var editor = new Editor( document.body );
 		var spy = sinon.spy();
 
 		editor.on( 'destroy', spy );
@@ -34,10 +39,6 @@ describe( 'destroy', function() {
 	} );
 
 	it( 'should delete the "element" property', function() {
-		var Editor = modules.editor;
-
-		var editor = new Editor( document.body );
-
 		editor.destroy();
 
 		expect( editor ).to.not.have.property( 'element' );