TrigenSoftware / srcset / 33745645564

94%

push

github

fix(imgproxy,cloudflare): select `src` the way a generation rule does (#39)

* fix(imgproxy,cloudflare): select `src` the way a generation rule does

`src`, and with it `url`, was the largest width of the last format of the
rule, so the format list had to be written in the opposite order to a
build-time rule: the widely supported format first there, last here.

It is now the variant of the source format - read from the url extension -
at the largest width, falling back to the first format of the list when the
source format is not in it. That is the selection a generation rule makes,
minus the source width, which is unknown at runtime.

* refactor(imgproxy,cloudflare): deduplicate in `toUniqArray`

`toArray` left the deduplication to every call site, which spelled it as
`[...new Set(toArray(...))]` and `for (const width of new Set(widths))`.

610 of 658 branches covered (92.71%)

Branch coverage included in aggregate %.

15 of 15 new or added lines in 4 files covered. (100.0%)

843 of 889 relevant lines covered (94.83%)

79446.03 hits per line