glyphnote_tests TODOs fixed

0xfe · Sep 10, 2021 · 75f6437 · 75f6437
1 parent 29035aa
commit 75f6437
Showing 1 changed file with 3 additions and 12 deletions.
diff --git a/tests/glyphnote_tests.ts b/tests/glyphnote_tests.ts
@@ -3,15 +3,6 @@
 //
 // GlyphNote Tests
 
-/* eslint-disable */
-// @ts-nocheck
-
-// TODO: Factory.GlyphNote()'s third argument should be optional.
-// TODO: Factory.RepeatNote()'s second and third arguments should both be optional.
-// TODO: Line 42 - Object literal may only specify known properties, and 'options' does not exist in type 'Partial<SystemOptions>'
-// TODO: ChordSymbol.addGlyphOrText(text: string[], ...) should instead take a string (not an array)!
-//       The implementation looks like text is an array, but we are actually just pulling out individual characters from the string.
-
 import { VexFlowTests, TestOptions } from './vexflow_test_helpers';
 import { ChordSymbol } from 'chordsymbol';
 import { Glyph } from 'glyph';
@@ -40,7 +31,7 @@ function chordChanges(options: TestOptions): void {
     width: 250,
     debugFormatter: options.params.debug,
     noPadding: options.params.noPadding,
-    options: { alpha: options.params.alpha },
+    details: { alpha: options.params.alpha },
   });
 
   const score = f.EasyScore();
@@ -82,7 +73,7 @@ function basic(options: TestOptions): void {
     width: 250,
     debugFormatter: options.params.debug,
     noPadding: options.params.noPadding,
-    options: { alpha: options.params.alpha },
+    details: { alpha: options.params.alpha },
   });
 
   const score = f.EasyScore();
@@ -120,7 +111,7 @@ function repeatNote(options: TestOptions): void {
     width: 250,
     debugFormatter: options.params.debug,
     noPadding: options.params.noPadding,
-    options: { alpha: options.params.alpha },
+    details: { alpha: options.params.alpha },
   });
 
   const score = f.EasyScore();