-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Description
For now, I'm just logging the problem instead of trying to hunt down the root cause. I think the correct way forward is to write integration tests, which would hopefully expose the bug. This will also enable the ability to refactor the class with more confidence that I'm not regressing anything.
Originally posted by @jaredjj3 in #264
Write integration tests that assert that playback.Sequence objects are being created correctly. Make sure there's coverage for scores that have parts with multiple staves. A lot of the problems that came up in #264 were due to edge cases for parts with multiple staves.
After, consider refactoring depending on what bugs were found.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels