add missing default options and merge, #165

phetsims · Sep 20, 2021 · 48c9c4c · 48c9c4c
1 parent 549b407
commit 48c9c4c
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/js/wavepacket/model/WavePacket.js b/js/wavepacket/model/WavePacket.js
@@ -16,6 +16,8 @@
 import DerivedProperty from '../../../../axon/js/DerivedProperty.js';
 import NumberProperty from '../../../../axon/js/NumberProperty.js';
 import Range from '../../../../dot/js/Range.js';
+import merge from '../../../../phet-core/js/merge.js';
+import Tandem from '../../../../tandem/js/Tandem.js';
 import ArrayIO from '../../../../tandem/js/types/ArrayIO.js';
 import NumberIO from '../../../../tandem/js/types/NumberIO.js';
 import fourierMakingWaves from '../../fourierMakingWaves.js';
@@ -35,7 +37,9 @@ class WavePacket {
    */
   constructor( options ) {
 
-    //REVIEW: options.tandem looks required, acceptable to do a merge with an opt-out tandem here?
+    options = merge( {
+      tandem: Tandem.REQUIRED
+    }, options );
 
     // @public
     this.L = 1; // wavelength when component spacing is 2 * Math.PI, in m