Browse Source

Fixed tests and API docs for `MarkerCollection`.

Maciej Bukowski 7 years ago
parent
commit
dbc36879ea

+ 9 - 9
packages/ckeditor5-engine/src/model/markercollection.js

@@ -333,29 +333,29 @@ class Marker {
 		this.name = name;
 
 		/**
-		 * Flag indicates if the marker is managed using operations or not.
+		 * Range marked by the marker.
 		 *
 		 * @protected
-		 * @member {Boolean}
+		 * @member {module:engine/model/liverange~LiveRange}
 		 */
-		this._managedUsingOperations = managedUsingOperations;
+		this._liveRange = this._attachLiveRange( liveRange );
 
 		/**
-		 * Specifies whether the marker affects the data produced by the data pipeline
-		 * (is persisted in the editor's data).
+		 * Flag indicates if the marker is managed using operations or not.
 		 *
 		 * @private
 		 * @member {Boolean}
 		 */
-		this._affectsData = affectsData;
+		this._managedUsingOperations = managedUsingOperations;
 
 		/**
-		 * Range marked by the marker.
+		 * Specifies whether the marker affects the data produced by the data pipeline
+		 * (is persisted in the editor's data).
 		 *
 		 * @private
-		 * @member {module:engine/model/liverange~LiveRange}
+		 * @member {Boolean}
 		 */
-		this._liveRange = this._attachLiveRange( liveRange );
+		this._affectsData = affectsData;
 	}
 
 	/**

+ 4 - 4
packages/ckeditor5-engine/tests/model/markercollection.js

@@ -369,11 +369,11 @@ describe( 'Marker', () => {
 
 		expect( marker.managedUsingOperations ).to.false;
 
-		marker._managedUsingOperations = true;
+		model.markers._set( 'name', range, true );
 
 		expect( marker.managedUsingOperations ).to.true;
 
-		marker._managedUsingOperations = false;
+		model.markers._set( 'name', range, false );
 
 		expect( marker.managedUsingOperations ).to.false;
 	} );
@@ -384,11 +384,11 @@ describe( 'Marker', () => {
 
 		expect( marker.affectsData ).to.false;
 
-		marker._affectsData = true;
+		model.markers._set( 'name', range, false, true );
 
 		expect( marker.affectsData ).to.true;
 
-		marker._affectsData = false;
+		model.markers._set( 'name', range, false, false );
 
 		expect( marker.affectsData ).to.false;
 	} );