remove TODOs, #60

phetsims · Jun 19, 2019 · e27658b · e27658b
1 parent 301d713
commit e27658b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/js/view/describers/ForceDescriber.js b/js/view/describers/ForceDescriber.js
@@ -365,11 +365,12 @@ define( require => {
      * Returns the appropriate changed direction for the vectors ('get bigger/smaller'), if no change, null is returned.
      *
      * @param {boolean} forceBiggerOverride - when true, just return the "get bigger" string.
-     * @returns {string|null} - TODO: this should never return null
+     * @returns {string}
      * @private
      */
     getChangeDirection( forceBiggerOverride ) {
       const index = forceBiggerOverride ? 2 : this.vectorChangeDirection + 1;
+      assert && assert( CHANGE_DIRECTIONS[ index ] !== null, 'Alert should not be called if no change in direction' );
       return CHANGE_DIRECTIONS[ index ];
     }