浏览代码

chore: fix loglevel import, logging in visual regression tests

see #706, new esModuleInterop flag in tsconfig.json
sschmid 5 年之前
父节点
当前提交
b0fb5e90e1
共有 27 个文件被更改,包括 26 次插入37 次删除
  1. 1 1
      src/MusicalScore/Graphical/BoundingBox.ts
  2. 1 1
      src/MusicalScore/Graphical/EngravingRules.ts
  3. 1 1
      src/MusicalScore/Graphical/GraphicalContinuousDynamicExpression.ts
  4. 1 1
      src/MusicalScore/Graphical/GraphicalInstantaneousDynamicExpression.ts
  5. 1 1
      src/MusicalScore/Graphical/GraphicalMusicSheet.ts
  6. 1 1
      src/MusicalScore/Graphical/MusicSheetCalculator.ts
  7. 1 1
      src/MusicalScore/Graphical/SkyBottomLineCalculator.ts
  8. 1 1
      src/MusicalScore/Graphical/VexFlow/VexFlowConverter.ts
  9. 1 1
      src/MusicalScore/Graphical/VexFlow/VexFlowMeasure.ts
  10. 1 1
      src/MusicalScore/Graphical/VexFlow/VexFlowMusicSheetCalculator.ts
  11. 1 1
      src/MusicalScore/Graphical/VexFlow/VexFlowOctaveShift.ts
  12. 1 1
      src/MusicalScore/Graphical/VexFlow/VexFlowTabMeasure.ts
  13. 1 1
      src/MusicalScore/MusicParts/MusicPartManagerIterator.ts
  14. 1 1
      src/MusicalScore/MusicSheet.ts
  15. 1 1
      src/MusicalScore/MusicSource/Repetition.ts
  16. 1 1
      src/MusicalScore/ScoreIO/InstrumentReader.ts
  17. 1 1
      src/MusicalScore/ScoreIO/MusicSheetReader.ts
  18. 1 1
      src/MusicalScore/ScoreIO/MusicSymbolModules/ArticulationReader.ts
  19. 1 1
      src/MusicalScore/ScoreIO/MusicSymbolModules/ChordSymbolReader.ts
  20. 1 1
      src/MusicalScore/ScoreIO/MusicSymbolModules/ExpressionReader.ts
  21. 1 1
      src/MusicalScore/ScoreIO/MusicSymbolModules/SlurReader.ts
  22. 1 1
      src/MusicalScore/ScoreIO/VoiceGenerator.ts
  23. 1 1
      src/MusicalScore/SubInstrument.ts
  24. 1 1
      src/MusicalScore/VoiceData/Expressions/InstantaneousDynamicExpression.ts
  25. 1 1
      src/MusicalScore/VoiceData/Notehead.ts
  26. 1 1
      src/OpenSheetMusicDisplay/OpenSheetMusicDisplay.ts
  27. 0 11
      test/Util/generateImages_browserless.js

+ 1 - 1
src/MusicalScore/Graphical/BoundingBox.ts

@@ -1,4 +1,4 @@
-import * as log from "loglevel";
+import log from "loglevel";
 import {ArgumentOutOfRangeException} from "../Exceptions";
 import {PointF2D} from "../../Common/DataObjects/PointF2D";
 import {SizeF2D} from "../../Common/DataObjects/SizeF2D";

+ 1 - 1
src/MusicalScore/Graphical/EngravingRules.ts

@@ -1,6 +1,6 @@
 import { PagePlacementEnum } from "./GraphicalMusicPage";
 //import {MusicSymbol} from "./MusicSymbol";
-import * as log from "loglevel";
+import log from "loglevel";
 import { TextAlignmentEnum } from "../../Common/Enums/TextAlignment";
 import { PlacementEnum } from "../VoiceData/Expressions/AbstractExpression";
 import { AutoBeamOptions, AlignRestOption, FillEmptyMeasuresWithWholeRests } from "../../OpenSheetMusicDisplay/OSMDOptions";

+ 1 - 1
src/MusicalScore/Graphical/GraphicalContinuousDynamicExpression.ts

@@ -7,7 +7,7 @@ import { AbstractGraphicalExpression } from "./AbstractGraphicalExpression";
 import { PlacementEnum } from "../VoiceData/Expressions/AbstractExpression";
 import { SkyBottomLineCalculator } from "./SkyBottomLineCalculator";
 import { ISqueezable } from "./ISqueezable";
-import * as log from "loglevel";
+import log from "loglevel";
 
 /**
  * This class prepares the graphical elements for a continuous expression. It calculates the wedges and

+ 1 - 1
src/MusicalScore/Graphical/GraphicalInstantaneousDynamicExpression.ts

@@ -4,7 +4,7 @@ import { GraphicalMeasure } from "./GraphicalMeasure";
 import { AbstractGraphicalExpression } from "./AbstractGraphicalExpression";
 import { SkyBottomLineCalculator } from "./SkyBottomLineCalculator";
 import { PlacementEnum } from "../VoiceData/Expressions/AbstractExpression";
-import * as log from "loglevel";
+import log from "loglevel";
 
 export class GraphicalInstantaneousDynamicExpression extends AbstractGraphicalExpression {
     protected mInstantaneousDynamicExpression: InstantaneousDynamicExpression;

+ 1 - 1
src/MusicalScore/Graphical/GraphicalMusicSheet.ts

@@ -17,7 +17,7 @@ import {GraphicalNote} from "./GraphicalNote";
 import {Instrument} from "../Instrument";
 import {BoundingBox} from "./BoundingBox";
 import {MusicSheetCalculator} from "./MusicSheetCalculator";
-import * as log from "loglevel";
+import log from "loglevel";
 //import Dictionary from "typescript-collections/dist/lib/Dictionary"; // unused for now
 import {CollectionUtil} from "../../Util/CollectionUtil";
 import {SelectionStartSymbol} from "./SelectionStartSymbol";

+ 1 - 1
src/MusicalScore/Graphical/MusicSheetCalculator.ts

@@ -47,7 +47,7 @@ import { AccidentalCalculator } from "./AccidentalCalculator";
 import { MidiInstrument } from "../VoiceData/Instructions/ClefInstruction";
 import { Staff } from "../VoiceData/Staff";
 import { OctaveShift } from "../VoiceData/Expressions/ContinuousExpressions/OctaveShift";
-import * as log from "loglevel";
+import log from "loglevel";
 import Dictionary from "typescript-collections/dist/lib/Dictionary";
 import { GraphicalLyricEntry } from "./GraphicalLyricEntry";
 import { GraphicalLyricWord } from "./GraphicalLyricWord";

+ 1 - 1
src/MusicalScore/Graphical/SkyBottomLineCalculator.ts

@@ -7,7 +7,7 @@ import { PointF2D } from "../../Common/DataObjects/PointF2D";
 import { CanvasVexFlowBackend } from "./VexFlow/CanvasVexFlowBackend";
 import { VexFlowMeasure } from "./VexFlow/VexFlowMeasure";
 import { unitInPixels } from "./VexFlow/VexFlowMusicSheetDrawer";
-import * as log from "loglevel";
+import log from "loglevel";
 import { BoundingBox } from "./BoundingBox";
 /**
  * This class calculates and holds the skyline and bottom line information.

+ 1 - 1
src/MusicalScore/Graphical/VexFlow/VexFlowConverter.ts

@@ -15,7 +15,7 @@ import {SystemLinesEnum} from "../SystemLinesEnum";
 import {FontStyles} from "../../../Common/Enums/FontStyles";
 import {Fonts} from "../../../Common/Enums/Fonts";
 import {OutlineAndFillStyleEnum, OUTLINE_AND_FILL_STYLE_DICT} from "../DrawingEnums";
-import * as log from "loglevel";
+import log from "loglevel";
 import { ArticulationEnum, StemDirectionType } from "../../VoiceData/VoiceEntry";
 import { SystemLinePosition } from "../SystemLinePosition";
 import { GraphicalVoiceEntry } from "../GraphicalVoiceEntry";

+ 1 - 1
src/MusicalScore/Graphical/VexFlow/VexFlowMeasure.ts

@@ -19,7 +19,7 @@ import StaveConnector = Vex.Flow.StaveConnector;
 import StaveNote = Vex.Flow.StaveNote;
 import StemmableNote = Vex.Flow.StemmableNote;
 import NoteSubGroup = Vex.Flow.NoteSubGroup;
-import * as log from "loglevel";
+import log from "loglevel";
 import {unitInPixels} from "./VexFlowMusicSheetDrawer";
 import {Tuplet} from "../../VoiceData/Tuplet";
 import {RepetitionInstructionEnum, RepetitionInstruction, AlignmentType} from "../../VoiceData/Instructions/RepetitionInstruction";

+ 1 - 1
src/MusicalScore/Graphical/VexFlow/VexFlowMusicSheetCalculator.ts

@@ -21,7 +21,7 @@ import { Tuplet } from "../../VoiceData/Tuplet";
 import { VexFlowMeasure } from "./VexFlowMeasure";
 import { VexFlowTextMeasurer } from "./VexFlowTextMeasurer";
 import Vex from "vexflow";
-import * as log from "loglevel";
+import log from "loglevel";
 import { unitInPixels } from "./VexFlowMusicSheetDrawer";
 import { VexFlowGraphicalNote } from "./VexFlowGraphicalNote";
 import { TechnicalInstruction } from "../../VoiceData/Instructions/TechnicalInstruction";

+ 1 - 1
src/MusicalScore/Graphical/VexFlow/VexFlowOctaveShift.ts

@@ -4,7 +4,7 @@ import { OctaveShift, OctaveEnum } from "../../VoiceData/Expressions/ContinuousE
 import { BoundingBox } from "../BoundingBox";
 import { GraphicalStaffEntry } from "../GraphicalStaffEntry";
 import { VexFlowVoiceEntry } from "./VexFlowVoiceEntry";
-import * as log from "loglevel";
+import log from "loglevel";
 
 /**
  * The vexflow adaptation of a graphical shift.

+ 1 - 1
src/MusicalScore/Graphical/VexFlow/VexFlowTabMeasure.ts

@@ -9,7 +9,7 @@ import { GraphicalVoiceEntry } from "../GraphicalVoiceEntry";
 import { VexFlowVoiceEntry } from "./VexFlowVoiceEntry";
 import { Arpeggio } from "../../VoiceData/Arpeggio";
 import { Voice } from "../../VoiceData/Voice";
-import * as log from "loglevel";
+import log from "loglevel";
 
 export class VexFlowTabMeasure extends VexFlowMeasure {
     constructor(staff: Staff, sourceMeasure: SourceMeasure = undefined, staffLine: StaffLine = undefined) {

+ 1 - 1
src/MusicalScore/MusicParts/MusicPartManagerIterator.ts

@@ -14,7 +14,7 @@ import {ContinuousDynamicExpression} from "../VoiceData/Expressions/ContinuousEx
 import {InstantaneousDynamicExpression} from "../VoiceData/Expressions/InstantaneousDynamicExpression";
 import {MultiTempoExpression} from "../VoiceData/Expressions/MultiTempoExpression";
 import {AbstractExpression} from "../VoiceData/Expressions/AbstractExpression";
-import * as log from "loglevel";
+import log from "loglevel";
 
 export class MusicPartManagerIterator {
     constructor(manager: MusicPartManager, startTimestamp?: Fraction, endTimestamp?: Fraction) {

+ 1 - 1
src/MusicalScore/MusicSheet.ts

@@ -16,7 +16,7 @@ import {EngravingRules} from "./Graphical/EngravingRules";
 import {NoteState} from "./Graphical/DrawingEnums";
 import {Note} from "./VoiceData/Note";
 import {VoiceEntry} from "./VoiceData/VoiceEntry";
-import * as log from "loglevel";
+import log from "loglevel";
 
 // FIXME Andrea: Commented out some unnecessary/not-ported-yet code, have a look at (*)
 

+ 1 - 1
src/MusicalScore/MusicSource/Repetition.ts

@@ -4,7 +4,7 @@ import {Fraction} from "../../Common/DataObjects/Fraction";
 import {MusicSheet} from "../MusicSheet";
 import {RepetitionInstruction} from "../VoiceData/Instructions/RepetitionInstruction";
 import {PartListEntry} from "./PartListEntry";
-import * as log from "loglevel";
+import log from "loglevel";
 
 export class Repetition extends PartListEntry /*implements IRepetition*/ {
     constructor(musicSheet: MusicSheet, virtualOverallRepetition: boolean) {

+ 1 - 1
src/MusicalScore/ScoreIO/InstrumentReader.ts

@@ -17,7 +17,7 @@ import {RhythmSymbolEnum} from "../VoiceData/Instructions/RhythmInstruction";
 import {KeyEnum} from "../VoiceData/Instructions/KeyInstruction";
 import {IXmlAttribute} from "../../Common/FileIO/Xml";
 import {ChordSymbolContainer} from "../VoiceData/ChordSymbolContainer";
-import * as log from "loglevel";
+import log from "loglevel";
 import {MidiInstrument} from "../VoiceData/Instructions/ClefInstruction";
 import {ChordSymbolReader} from "./MusicSymbolModules/ChordSymbolReader";
 import {ExpressionReader} from "./MusicSymbolModules/ExpressionReader";

+ 1 - 1
src/MusicalScore/ScoreIO/MusicSheetReader.ts

@@ -6,7 +6,7 @@ import {IXmlElement} from "../../Common/FileIO/Xml";
 import {Instrument} from "../Instrument";
 import {ITextTranslation} from "../Interfaces/ITextTranslation";
 import {MusicSheetReadingException} from "../Exceptions";
-import * as log from "loglevel";
+import log from "loglevel";
 import {IXmlAttribute} from "../../Common/FileIO/Xml";
 import {RhythmInstruction} from "../VoiceData/Instructions/RhythmInstruction";
 import {RhythmSymbolEnum} from "../VoiceData/Instructions/RhythmInstruction";

+ 1 - 1
src/MusicalScore/ScoreIO/MusicSymbolModules/ArticulationReader.ts

@@ -1,6 +1,6 @@
 import {ArticulationEnum, VoiceEntry} from "../../VoiceData/VoiceEntry";
 import {IXmlAttribute, IXmlElement} from "../../../Common/FileIO/Xml";
-import * as log from "loglevel";
+import log from "loglevel";
 import {TechnicalInstruction, TechnicalInstructionType} from "../../VoiceData/Instructions/TechnicalInstruction";
 import {OrnamentContainer, OrnamentEnum} from "../../VoiceData/OrnamentContainer";
 import {PlacementEnum} from "../../VoiceData/Expressions/AbstractExpression";

+ 1 - 1
src/MusicalScore/ScoreIO/MusicSymbolModules/ChordSymbolReader.ts

@@ -4,7 +4,7 @@ import {ChordDegreeText, ChordSymbolContainer, ChordSymbolEnum, Degree} from "..
 import {AccidentalEnum, NoteEnum, Pitch} from "../../../Common/DataObjects/Pitch";
 import {KeyInstruction} from "../../VoiceData/Instructions/KeyInstruction";
 import {ITextTranslation} from "../../Interfaces/ITextTranslation";
-import * as log from "loglevel";
+import log from "loglevel";
 
 export class ChordSymbolReader {
     public static readChordSymbol(xmlNode: IXmlElement, musicSheet: MusicSheet, activeKey: KeyInstruction): ChordSymbolContainer {

+ 1 - 1
src/MusicalScore/ScoreIO/MusicSymbolModules/ExpressionReader.ts

@@ -15,7 +15,7 @@ import {UnknownExpression} from "../../VoiceData/Expressions/UnknownExpression";
 import {PlacementEnum} from "../../VoiceData/Expressions/AbstractExpression";
 import {TextAlignmentEnum} from "../../../Common/Enums/TextAlignment";
 import {ITextTranslation} from "../../Interfaces/ITextTranslation";
-import * as log from "loglevel";
+import log from "loglevel";
 
 export class ExpressionReader {
     private musicSheet: MusicSheet;

+ 1 - 1
src/MusicalScore/ScoreIO/MusicSymbolModules/SlurReader.ts

@@ -2,7 +2,7 @@
 import { IXmlElement, IXmlAttribute } from "../../../Common/FileIO/Xml";
 import { Slur } from "../../VoiceData/Expressions/ContinuousExpressions/Slur";
 import { Note } from "../../VoiceData/Note";
-import * as log from "loglevel";
+import log from "loglevel";
 import { ITextTranslation } from "../../Interfaces/ITextTranslation";
 
 export class SlurReader {

+ 1 - 1
src/MusicalScore/ScoreIO/VoiceGenerator.ts

@@ -19,7 +19,7 @@ import { NoteEnum } from "../../Common/DataObjects/Pitch";
 import { Staff } from "../VoiceData/Staff";
 import { StaffEntryLink } from "../VoiceData/StaffEntryLink";
 import { VerticalSourceStaffEntryContainer } from "../VoiceData/VerticalSourceStaffEntryContainer";
-import * as log from "loglevel";
+import log from "loglevel";
 import { Pitch } from "../../Common/DataObjects/Pitch";
 import { IXmlAttribute } from "../../Common/FileIO/Xml";
 import { CollectionUtil } from "../../Util/CollectionUtil";

+ 1 - 1
src/MusicalScore/SubInstrument.ts

@@ -1,6 +1,6 @@
 import {Instrument} from "./Instrument";
 import {MidiInstrument} from "./VoiceData/Instructions/ClefInstruction";
-import * as log from "loglevel";
+import log from "loglevel";
 
 export class SubInstrument {
 

+ 1 - 1
src/MusicalScore/VoiceData/Expressions/InstantaneousDynamicExpression.ts

@@ -3,7 +3,7 @@ import {MultiExpression} from "./MultiExpression";
 import {DynamicExpressionSymbolEnum} from "./DynamicExpressionSymbolEnum";
 //import {ArgumentOutOfRangeException} from "../../Exceptions";
 import {InvalidEnumArgumentException} from "../../Exceptions";
-import * as log from "loglevel";
+import log from "loglevel";
 
 export class InstantaneousDynamicExpression extends AbstractExpression {
     constructor(dynamicExpression: string, soundDynamics: number, placement: PlacementEnum, staffNumber: number) {

+ 1 - 1
src/MusicalScore/VoiceData/Notehead.ts

@@ -1,5 +1,5 @@
 import { Note } from "./Note";
-import * as log from "loglevel";
+import log from "loglevel";
 
 /**
  * A note head with shape and fill information belonging to a [[Note]].

+ 1 - 1
src/OpenSheetMusicDisplay/OpenSheetMusicDisplay.ts

@@ -12,7 +12,7 @@ import { Cursor } from "./Cursor";
 import { MXLHelper } from "../Common/FileIO/Mxl";
 import { Promise } from "es6-promise";
 import { AJAX } from "./AJAX";
-import * as log from "loglevel";
+import log from "loglevel";
 import { DrawingParametersEnum, DrawingParameters, ColoringModes } from "../MusicalScore/Graphical/DrawingParameters";
 import { IOSMDOptions, OSMDOptions, AutoBeamOptions, BackendType } from "./OSMDOptions";
 import { EngravingRules, PageFormat } from "../MusicalScore/Graphical/EngravingRules";

+ 0 - 11
test/Util/generateImages_browserless.js

@@ -67,17 +67,6 @@ async function init () {
     // eslint-disable-next-line no-global-assign
     document = dom.window.document
 
-    // OSMD console output in nodejs broke after using "esModuleInterop": true in tsconfig to update Vexflow to 1.2.91.
-    // the virtualConsole could be used to get logs from OSMD, but then we need to pass our whole script in HTML to JSDOM.
-    // it would probably be cleaner to implement a more powerful logging system in OSMD you can hook into.
-    // var virtualConsole = new jsdom.VirtualConsole()
-    // virtualConsole.sendTo(console);
-    // var dom = global.document = new jsdom.JSDOM('', {
-    //     features: {
-    //         virtualConsole: virtualConsole
-    //     }
-    // })
-
     // eslint-disable-next-line no-global-assign
     global.window = dom.window
     // eslint-disable-next-line no-global-assign