Fix measure index in /groove endpoint

infojunkie · Aug 10, 2024 · 54cead0 · 54cead0
1 parent acd3841
commit 54cead0
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "musicxml-midi",
-  "version": "2.5.1",
+  "version": "2.5.2",
   "description": "MusicXML to MIDI converter",
   "type": "module",
   "directories": {

diff --git a/src/js/server.js b/src/js/server.js
@@ -175,7 +175,7 @@ app.post('/groove', async (req, res, next) => {
   const chords = params['chords'].split(',').map(s => s.trim())
   const measures = [...Array(parseInt(params['count']))].map((_, index) => {
     return `
-MidiMark 0 Measure:${index+1}: $Duration
+MidiMark 0 Measure:${index}: $Duration
 ${chords[index % chords.length]}
     `.trim()
   }).join('\n')

diff --git a/test/server.test.js b/test/server.test.js
@@ -99,6 +99,8 @@ describe('MusicXML to MIDI conversion server', () => {
     const track = midi.tracks.find(track => !!track.find(event => event.type === 'marker' && event.text.includes('Measure:')))
     expect(track.filter(event => event.type === 'marker' && event.text.includes('Measure:')).length)
       .toEqual(4)
+    const measure = track.filter(event => event.type === 'marker' && event.text.includes('Measure:'))[0]
+    expect(measure.text.split(':')[1]).toEqual('0')
   })
 })