Add simplifyFileSize option

[KroshSputnik]

I added a new option to enable to show the files sizes in B, MB, GB, TB, PB.

Enable it in the options by doing
'simplifyFileSize':true

Not sure if I need to divide the size by 1024 or 1000.
Also would I need to round the final size, as it seems the size returned for the file size is already rounded.
Feedback will be appreciated.

[UlisesGascon]

Thanks for this proposal @KroshSputnik! Can you include tests and documentation?

[UlisesGascon]

Currently we support [email protected] so new ES6 syntax is not allowed:

> [email protected] test /home/runner/work/serve-index/serve-index
> mocha --reporter spec --bail --check-leaks test/


/home/runner/work/serve-index/serve-index/index.js:308
      const fileSizes = ['B', 'KB', 'MB', 'GB', 'TB', 'PB']
      ^^^^^
SyntaxError: Use of const in strict mode.
    at Module._compile (module.js:439:25)
    at Object.Module._extensions..js (module.js:474:[10](https://github.com/expressjs/serve-index/actions/runs/13029380620/job/36428408044?pr=114#step:10:11))
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/home/runner/work/serve-index/serve-index/test/test.js:8:18)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at /home/runner/work/serve-index/serve-index/node_modules/mocha/lib/mocha.js:220:27
    at Array.forEach (native)
    at Mocha.loadFiles (/home/runner/work/serve-index/serve-index/node_modules/mocha/lib/mocha.js:217:14)
    at Mocha.run (/home/runner/work/serve-index/serve-index/node_modules/mocha/lib/mocha.js:469:10)
    at Object.<anonymous> (/home/runner/work/serve-index/serve-index/node_modules/mocha/bin/_mocha:404:18)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:[11](https://github.com/expressjs/serve-index/actions/runs/13029380620/job/36428408044?pr=114#step:10:12)9:16)
    at node.js:945:3
npm ERR! weird error 8
npm ERR! not ok code 0

Full log

[KroshSputnik]

Thanks for this proposal @KroshSputnik! Can you include tests and documentation?

Hi @UlisesGascon
I'm not sure how to add a test, do I need to create a new one in test.js?
I have never created one before, could you walk it through?

I've added docs in readme and fixed the es6 issue.

Thanks

[KroshSputnik]

@UlisesGascon I have added test, for when the option is enabled and when it isnt.

Can you double check my pull request.

[bjohansebas]

It looks great, although we could probably delegate that logic to bytes instead of handling it ourselves.

[socket-security]

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/[email protected]	None	0	12.3 kB	dougwilson

View full report↗︎

[KroshSputnik]

@bjohansebas Updated the feature with your suggestion.

[bjohansebas]

Great! Once the tests are run, I'll be able to give my LGTM

[UlisesGascon]

Thanks for include the tests and make the changes to support [email protected] @KroshSputnik! :)

I added some suggestions, I think that we will need to modify the HISTORY.md to include the new dependency and the feature too 👍

[KroshSputnik]

@UlisesGascon All tasks completed. If need anything else, let me know.

[UlisesGascon]

mmm.. not sure why one of the new tests is failing 🤔. Can you have a look @KroshSputnik ?

[KroshSputnik]

I think the issues due to the differences of how linux and windows calculate bytes.

I downgraded to node 0.8, but all tests pass successfully.