songsheet 6.0.0 → 6.2.0

package/test/parser.test.js

@@ -112,6 +112,14 @@ describe('parse spent-some-time-in-buffalo', () => {
     expect(song.author).toBe('WILLIE COTTON')
   })
 
+  it('extracts BPM', () => {
+    expect(song.bpm).toBe(100)
+  })
+
+  it('has null timeSignature when not specified', () => {
+    expect(song.timeSignature).toBeNull()
+  })
+
   it('counts sections correctly', () => {
     expect(song.sections.verse.count).toBe(3)
     expect(song.sections.chorus.count).toBe(3)
@@ -180,6 +188,14 @@ describe('parse song-of-myself', () => {
     expect(song.author).toBe('WILLIE AND WALT')
   })
 
+  it('extracts time signature', () => {
+    expect(song.timeSignature).toEqual({ beats: 3, value: 4 })
+  })
+
+  it('has null BPM when not specified', () => {
+    expect(song.bpm).toBeNull()
+  })
+
   it('counts sections correctly', () => {
     expect(song.sections.verse.count).toBe(1)
     expect(song.sections.chorus.count).toBe(1)
@@ -238,6 +254,54 @@ describe('parse song-of-myself', () => {
   })
 })
 
+describe('time signature parsing', () => {
+  it('returns null timeSignature when only BPM is present', () => {
+    const song = parse('SONG - AUTHOR\n(120 BPM)\n\nG\nLyrics')
+    expect(song.bpm).toBe(120)
+    expect(song.timeSignature).toBeNull()
+  })
+
+  it('parses time signature without BPM', () => {
+    const song = parse('SONG - AUTHOR\n(3/4 time)\n\nG\nLyrics')
+    expect(song.bpm).toBeNull()
+    expect(song.timeSignature).toEqual({ beats: 3, value: 4 })
+  })
+
+  it('parses both BPM and time signature together', () => {
+    const song = parse('SONG - AUTHOR\n(100 BPM, 3/4 time)\n\nG\nLyrics')
+    expect(song.bpm).toBe(100)
+    expect(song.timeSignature).toEqual({ beats: 3, value: 4 })
+  })
+
+  it('parses reversed order: time signature before BPM', () => {
+    const song = parse('SONG - AUTHOR\n(3/4 time, 100 BPM)\n\nG\nLyrics')
+    expect(song.bpm).toBe(100)
+    expect(song.timeSignature).toEqual({ beats: 3, value: 4 })
+  })
+
+  it('parses 6/8 time', () => {
+    const song = parse('SONG - AUTHOR\n(6/8 Time)\n\nG\nLyrics')
+    expect(song.timeSignature).toEqual({ beats: 6, value: 8 })
+  })
+
+  it('is case insensitive for TIME', () => {
+    const song = parse('SONG - AUTHOR\n(3/4 TIME)\n\nG\nLyrics')
+    expect(song.timeSignature).toEqual({ beats: 3, value: 4 })
+  })
+
+  it('returns both null when no metadata parens present', () => {
+    const song = parse('SONG - AUTHOR\n\nG\nLyrics')
+    expect(song.bpm).toBeNull()
+    expect(song.timeSignature).toBeNull()
+  })
+
+  it('does not match non-metadata parens like (live)', () => {
+    const song = parse('SONG - AUTHOR\n(live)\n\nG\nLyrics')
+    expect(song.bpm).toBeNull()
+    expect(song.timeSignature).toBeNull()
+  })
+})
+
 describe('bar line parsing', () => {
   it('parses bar lines in chord lines', () => {
     const song = parse('TEST SONG - AUTHOR\n\n| G | C | D |\nSome lyrics here')

package/test/transpose.test.js

@@ -56,6 +56,19 @@ describe('transposeChord', () => {
     expect(transposeChord({ root: 'B', type: '' }, 1, false)).toEqual({ root: 'C', type: '' })
     expect(transposeChord({ root: 'C', type: '' }, -1, false)).toEqual({ root: 'B', type: '' })
   })
+
+  it('transposes slash chord bass note', () => {
+    expect(transposeChord({ root: 'G', type: '', bass: 'B' }, 2, false)).toEqual({ root: 'A', type: '', bass: 'C#' })
+  })
+
+  it('transposes slash chord bass note with flats', () => {
+    expect(transposeChord({ root: 'C', type: 'maj7', bass: 'Bb' }, 2, true)).toEqual({ root: 'D', type: 'maj7', bass: 'C' })
+  })
+
+  it('does not add bass when not present', () => {
+    const result = transposeChord({ root: 'G', type: '' }, 2, false)
+    expect(result.bass).toBeUndefined()
+  })
 })
 
 describe('transpose full song', () => {