short-circuit valuePattern validation when stringTest query parameter…

… is provided, phetsims/masses-and-springs#344, #472
phetsims · Feb 27, 2019 · 4654f48 · 4654f48
1 parent 373008a
commit 4654f48
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/js/NumberSpinner.js b/js/NumberSpinner.js
@@ -88,7 +88,8 @@ define( function( require ) {
     // validate options
     assert && assert( _.includes( ARROWS_POSITION_VALUES, options.arrowsPosition ), 'invalid arrowsPosition: ' + options.arrowsPosition );
     assert && assert( _.includes( VALUE_ALIGN_VALUES, options.valueAlign ), 'invalid valueAlign: ' + options.valueAlign );
-    assert && assert( options.valuePattern.indexOf( SunConstants.VALUE_NAMED_PLACEHOLDER ) !== -1,
+    assert && assert( !!phet.chipper.queryParameters.stringTest ||
+                      options.valuePattern.indexOf( SunConstants.VALUE_NAMED_PLACEHOLDER ) !== -1,
       'missing value placeholder in options.valuePattern: ' + options.valuePattern );
 
     Tandem.indicateUninstrumentedCode();