COM-2479:

- add rounding to getCompressionRatio
- rename getPixelRations -> getPixelRatios
- refactoring use getOriginalExtensionSrcSet

Author: vkosinov

src/utils/__tests__/getCompressionRatio.ts

@@ -10,8 +10,8 @@ test('Pixel ratios 1x, 2x', () => {
 
 test('Pixel ratios 1x, 2x, 3x', () => {
   expect(getCompressionRatio(['1x', '2x', '3x'])).toStrictEqual({
-    '1x': 0.3333333333333333,
-    '2x': 0.6666666666666666,
+    '1x': 0.33333,
+    '2x': 0.66667,
     '3x': 0,
   });
 });

src/utils/__tests__/getOriginalExtensionSrcSet.ts

@@ -9,7 +9,7 @@ export const getCompressionRatio = (pixelRatios: Dpr[]): CompressionRatio => {
       return acc;
     }
 
-    acc[item] = (index + 1) / length;
+    acc[item] = Number(((index + 1) / length).toFixed(5));
     return acc;
   }, {} as CompressionRatio);
 };

src/utils/getCompressionRatio.ts

@@ -1,6 +1,6 @@
 import { Dpr } from '../types';
 
-export const getPixelRations = (originalPixelRatio: Dpr): Dpr[] => {
+export const getPixelRatios = (originalPixelRatio: Dpr): Dpr[] => {
   switch (originalPixelRatio) {
     case '1x':
       return ['1x'];

src/utils/getOriginalExtensionSrcSet.ts

@@ -3,6 +3,5 @@ export { getBreakpointMedia } from './getBreakpointMedia';
 export { getSrcSetString } from './getSrcSetString';
 export { getSources } from './getSources';
 export { getOriginal } from './getOriginal';
-export { getPixelRations } from './getPixelRations';
+export { getPixelRatios } from './getPixelRatios';
 export { getCompressionRatio } from './getCompressionRatio';
-export { getOriginalExtensionSrcSet } from './getOriginalExtensionSrcSet';

src/utils/getPixelRations.ts src/utils/getPixelRatios.tssrc/utils/getPixelRations.ts renamed to src/utils/getPixelRatios.ts

@@ -6,7 +6,7 @@ import { getImgproxyUrlBuilder } from './imgproxyUrlBuilder';
 import { Breakpoint, OrderedBreakpointSource, SrcSet, Dpr } from '../types';
 import { imageUrls } from './plugin';
 import { schema } from './loaderOptionsSchema';
-import { getBreakpointMedia, getPixelRations, getOriginalExtensionSrcSet } from '../utils';
+import { getBreakpointMedia, getPixelRatios } from '../utils';
 
 // Такое имя используется, если нужна одна картинка для всех разрешений
 // В таком случаем не будут сгенерированы медиа выражения для разных breakpoint'ов
@@ -30,7 +30,7 @@ export const loader = function (this: webpack.loader.LoaderContext, source: stri
 
   validateOptions(schema, options, { name: 'Imgproxy responsive loader', baseDataPath: 'options' });
 
-  const pixelRatios: Dpr[] = getPixelRations(options.originalPixelRatio);
+  const pixelRatios: Dpr[] = getPixelRatios(options.originalPixelRatio);
   const breakpoints: Breakpoint[] = options.breakpoints;
   // Такой результат приходит от file-loader 'module.exports = "/build/myImage/mobile.all-4b767a7b.png";'
   // Получаем оригинальное имя файла изображения (originalImageFileName = mobile.all.png)
@@ -76,7 +76,10 @@ export const loader = function (this: webpack.loader.LoaderContext, source: stri
       srcSets: [
         {
           extension: originalExtension,
-          srcSet: getOriginalExtensionSrcSet(pixelRatios, outputImagePath),
+          srcSet: pixelRatios.reduce((acc, item): SrcSet => {
+            acc[item] = outputImagePath;
+            return acc;
+          }, {} as SrcSet),
         },
       ],
     };