Move helper functions up.

Test methods are now private to the JS module. Only the Start() method is exported.
0xfe · Sep 2, 2021 · 6dad8b2 · 6dad8b2
1 parent 9e8fb99
commit 6dad8b2
Show file tree

Hide file tree

Showing 64 changed files with 14,504 additions and 14,576 deletions.
diff --git a/tests/accidental_tests.ts b/tests/accidental_tests.ts
diff --git a/tests/annotation_tests.ts b/tests/annotation_tests.ts
diff --git a/tests/articulation_tests.ts b/tests/articulation_tests.ts
diff --git a/tests/auto_beam_formatting_tests.ts b/tests/auto_beam_formatting_tests.ts
diff --git a/tests/bach_tests.ts b/tests/bach_tests.ts
diff --git a/tests/barline_tests.ts b/tests/barline_tests.ts
@@ -14,59 +14,58 @@ import { Barline, BarlineType } from 'stavebarline';
 const BarlineTests = {
   Start(): void {
     QUnit.module('Barline');
-    test('Enums', this.enums);
-
+    test('Enums', enums);
     const run = VexFlowTests.runTests;
-    run('Simple BarNotes', this.simple);
-    run('Style BarNotes', this.style);
+    run('Simple BarNotes', simple);
+    run('Style BarNotes', style);
   },
+};
 
-  enums(): void {
-    // VexFlow 4.0 renamed Barline.type => BarlineType.
-    // The old way still works, for backwards compatibility.
-    equal(Barline.type, BarlineType);
-  },
+function enums(): void {
+  // VexFlow 4.0 renamed Barline.type => BarlineType.
+  // The old way still works, for backwards compatibility.
+  equal(Barline.type, BarlineType);
+}
 
-  simple(options: TestOptions): void {
-    const f = VexFlowTests.makeFactory(options, 380, 160);
-    const stave = f.Stave();
+function simple(options: TestOptions): void {
+  const f = VexFlowTests.makeFactory(options, 380, 160);
+  const stave = f.Stave();
 
-    const notes = [
-      f.StaveNote({ keys: ['d/4', 'e/4', 'f/4'], stem_direction: -1, duration: '2' }),
-      f.BarNote({ type: 'single' }), // => f.BarNote({ type: BarlineType.SINGLE })
-      f
-        .StaveNote({ keys: ['c/4', 'f/4', 'a/4'], stem_direction: -1, duration: '2' })
-        .addAccidental(0, f.Accidental({ type: 'n' }))
-        .addAccidental(1, f.Accidental({ type: '#' })),
-    ];
+  const notes = [
+    f.StaveNote({ keys: ['d/4', 'e/4', 'f/4'], stem_direction: -1, duration: '2' }),
+    f.BarNote({ type: 'single' }), // => f.BarNote({ type: BarlineType.SINGLE })
+    f
+      .StaveNote({ keys: ['c/4', 'f/4', 'a/4'], stem_direction: -1, duration: '2' })
+      .addAccidental(0, f.Accidental({ type: 'n' }))
+      .addAccidental(1, f.Accidental({ type: '#' })),
+  ];
 
-    const voice = f.Voice().addTickables(notes);
-    f.Formatter().joinVoices([voice]).formatToStave([voice], stave);
-    f.draw();
+  const voice = f.Voice().addTickables(notes);
+  f.Formatter().joinVoices([voice]).formatToStave([voice], stave);
+  f.draw();
 
-    ok(true, 'Simple Test');
-  },
+  ok(true, 'Simple Test');
+}
 
-  style(options: TestOptions): void {
-    const f = VexFlowTests.makeFactory(options, 380, 160);
-    const stave = f.Stave();
+function style(options: TestOptions): void {
+  const f = VexFlowTests.makeFactory(options, 380, 160);
+  const stave = f.Stave();
 
-    const notes = [
-      f.StaveNote({ keys: ['d/4', 'e/4', 'f/4'], stem_direction: -1, duration: '2' }),
-      f.BarNote({ type: 'single' }), // => f.BarNote({ type: BarlineType.SINGLE })
-      f
-        .StaveNote({ keys: ['c/4', 'f/4', 'a/4'], stem_direction: -1, duration: '2' })
-        .addAccidental(0, f.Accidental({ type: 'n' }))
-        .addAccidental(1, f.Accidental({ type: '#' })),
-    ];
-    notes[1].setStyle({ shadowBlur: 15, shadowColor: 'blue', fillStyle: 'blue', strokeStyle: 'blue' });
+  const notes = [
+    f.StaveNote({ keys: ['d/4', 'e/4', 'f/4'], stem_direction: -1, duration: '2' }),
+    f.BarNote({ type: 'single' }), // => f.BarNote({ type: BarlineType.SINGLE })
+    f
+      .StaveNote({ keys: ['c/4', 'f/4', 'a/4'], stem_direction: -1, duration: '2' })
+      .addAccidental(0, f.Accidental({ type: 'n' }))
+      .addAccidental(1, f.Accidental({ type: '#' })),
+  ];
+  notes[1].setStyle({ shadowBlur: 15, shadowColor: 'blue', fillStyle: 'blue', strokeStyle: 'blue' });
 
-    const voice = f.Voice().addTickables(notes);
-    f.Formatter().joinVoices([voice]).formatToStave([voice], stave);
-    f.draw();
+  const voice = f.Voice().addTickables(notes);
+  f.Formatter().joinVoices([voice]).formatToStave([voice], stave);
+  f.draw();
 
-    ok(true, 'Style');
-  },
-};
+  ok(true, 'Style');
+}
 
 export { BarlineTests };