visionmedia / bytes.js

node byte string parser
MIT License
458 stars 57 forks source link

Uppercase support, tests and graceful fail. #15

Closed joelabair closed 9 years ago

joelabair commented 10 years ago

The name of the package is bytes - so obviously we're not concerned with confusing bits and their associated abbrs. Why not support 5mb and 5MB without throwing an error.

I found that mocha@1.19.0 requires us to require should in our tests.
And I added a few tests for UC chars and graceful failure.

tj commented 9 years ago

fixed from ^