Added unit tests for ZzFX.js and ZzFXMicro.js - Githubissues

KilledByAPixel / ZzFX

A Tiny JavaScript Sound FX System

https://zzfx.3d2k.com

MIT License

585 stars 36 forks source link

Added unit tests for ZzFX.js and ZzFXMicro.js #14

Closed devloai closed 8 months ago

devloai commented 8 months ago

Summary

This pull request introduces unit tests for ZzFx.js and ZzFXMicro.js.

Changes

Added the following test cases for ZzFX.js:

play sound with default parameters
play sound with custom parameters
get frequency of a musical note
zzfx function with various parameters for sound synthesis
sound synthesis with negative values for parameters

Added the following test cases for ZzFXMicro.js:

Test sound generation with default parameters
Test sound generation with specific non-default parameters
Test sound generation with extreme parameter values
Test sound generation verifying the effect of each parameter individually

Coverage report

File	% Stmts	% Branch	% Funcs	% Lines
ZzFX.js	100 (up from 0)	91.66 (up from 0)	100 (up from 0)	100 (up from 0)
ZzFXMicro.js	98.14 (up from 0)	96.55 (up from 0)	80 (up from 0)	100 (up from 0)

@KilledByAPixel

KilledByAPixel commented 8 months ago

Thank you but unit tests are not needed for this project at this time.

I will keep your changes in mind if we decide to add it in the future!