瀏覽代碼

Used ...spread operator in Model#bind tests. Minor refactoring in Model#bind tests.

Aleksander Nowodzinski 10 年之前
父節點
當前提交
de8f911c52
共有 1 個文件被更改,包括 8 次插入12 次删除
  1. 8 12
      packages/ckeditor5-ui/tests/mvc/model/model.js

+ 8 - 12
packages/ckeditor5-ui/tests/mvc/model/model.js

@@ -681,16 +681,11 @@ describe( 'Model', () => {
 	}
 
 	function assertStructure( model, expectedBound, expectedModels, expectedBindings ) {
+		const boundModels = [ ...model._boundTo.keys() ];
+
 		// Check model._bound object.
 		expect( model._bound ).to.be.deep.equal( expectedBound );
 
-		// TODO: Should be boundModels = [ ...model._boundTo.boundModels() ]
-		const boundModels = [];
-
-		for ( let boundModel of model._boundTo.keys() ) {
-			boundModels.push( boundModel );
-		}
-
 		// Check model._boundTo models.
 		expect( boundModels ).to.have.members( expectedModels );
 
@@ -701,12 +696,13 @@ describe( 'Model', () => {
 
 		// Check model._boundTo model bindings.
 		expectedBindings.forEach( ( binding, index ) => {
-			expect( model._boundTo.get( expectedModels[ index ] ) )
-				.to.have.keys( Object.keys( binding ) );
+			const bindingKeys = Object.keys( binding );
+
+			expect( model._boundTo.get( expectedModels[ index ] ) ).to.have.keys( bindingKeys );
 
-			Object.keys( binding ).forEach( b => {
-				expect( Array.from( model._boundTo.get( expectedModels[ index ] )[ b ] ) )
-					.to.have.members( binding[ b ] );
+			bindingKeys.forEach( k => {
+				expect( Array.from( model._boundTo.get( expectedModels[ index ] )[ k ] ) )
+					.to.have.members( binding[ k ] );
 			} );
 		} );
 	}