Ad
  • Custom User Avatar
    • the change of output type for single value is a bad idea
    • assertion system needs to be rewritten with assertDeepEquals stuff (edit: assertion system is actually total bullshit and don't test the correctness of the output)
    • random tests with uppercase letters are missing
    • not enough random tests
    • random tests may fail because of the change of output type (the assertion function expects an array of strings)
    • the description doesn't state the ranges used for each input type