emilio-martinez / is-datatype / 207

100%

push

travis-ci

perf: refactor to achieve various data access and size improvements

The following improvements were made:
* Sorted `DataType` properties to easily differentiate between primitives and not — will come in handy in the future
* Avoid zero `DataType` index to avoid falsy comparisons
* Reduce redundancies in `null` test
* Create a sister `const enum` for `DataType` for internal use
  This change should improve performance due to removing the overhead of object access in favor of primitive use in comparisons. Additionally, because `DATATYPE` is compiled into just numbers, there will be a size reduction.
  `DATATYPE` will not be exported to JS, only `DataType`.
  A test has been added to ensure that both `DATATYPE` and `DataType` stay in sync.
* Simplify `isValidOptions` to not be switch-based
* Deduplicate number bounds logic into `testNumberWithinBounds`
* Remove unnecessary `typeof` object check for Array

The overall results are a smaller output with a bit more performant checks, especially due to less comparison overhead.

156 of 156 branches covered (100.0%)

102 of 102 relevant lines covered (100.0%)

2025.39 hits per line