In developing #268 , I found some issues with the way weapon fire works when the weapon fire period is within a few multiples of the frame period. #268 fixes a bug where changes in weapon fire period below 100 ms are not possible, so this pull request depends on merging that one first.
This pull request adds test cases to demonstrate these issues. Continuous fire weapons appear to be working correctly, and their test cases pass.
I noticed on my machine that a previous click and hold test cases is now failing, but I don't know when that was broken, so we should fix it in this pull request as well.
In developing #268 , I found some issues with the way weapon fire works when the weapon fire period is within a few multiples of the frame period. #268 fixes a bug where changes in weapon fire period below 100 ms are not possible, so this pull request depends on merging that one first.
This pull request adds test cases to demonstrate these issues. Continuous fire weapons appear to be working correctly, and their test cases pass.
I noticed on my machine that a previous click and hold test cases is now failing, but I don't know when that was broken, so we should fix it in this pull request as well.