Changes Config iterator interface to Config#names() method.

ckeditor · Dec 5, 2019 · d3e2d35 · d3e2d35
1 parent f9bd16e
commit d3e2d35
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/config.js b/src/config.js
@@ -114,14 +114,14 @@ export default class Config {
 	}
 
 	/**
-	 * Iterable interface.
-	 *
 	 * Iterates over all top level configuration names.
 	 *
 	 * @returns {Iterable.<String>}
 	 */
-	[ Symbol.iterator ]() {
-		return Object.keys( this._config )[ Symbol.iterator ]();
+	* names() {
+		for ( const name of Object.keys( this._config ) ) {
+			yield name;
+		}
 	}
 
 	/**

diff --git a/tests/config.js b/tests/config.js
@@ -460,9 +460,9 @@ describe( 'Config', () => {
 		} );
 	} );
 
-	describe( 'iterator', () => {
-		it( 'should return top level keys of the configuration', () => {
-			expect( Array.from( config ) ).to.be.deep.equal( [ 'creator', 'language', 'resize', 'toolbar', 'options' ] );
+	describe( 'names()', () => {
+		it( 'should return an iterator of top level names of the configuration', () => {
+			expect( Array.from( config.names() ) ).to.be.deep.equal( [ 'creator', 'language', 'resize', 'toolbar', 'options' ] );
 		} );
 	} );
 } );