Update Windows tests to cover normalizedPath

__tests__/normalize.test.ts

@@ -1,33 +1,46 @@
 import { expect, test } from 'vitest';
 import { normalizePath } from '../src/normalize.path';
-import { resolve } from 'path';
 
 test('should normalize path ending with slash', () => {
  const path = '/path/to/folder/';
  const normalizedPath: string = normalizePath(path);
- expect(normalizedPath).to.equal('/path/to/folder/');
+ if (process.platform === 'win32') {
+ expect(normalizedPath).to.equal('C:\\path\\to\\folder\\');
+ } else {
+ expect(normalizedPath).to.equal('/path/to/folder/');
+ }
 });
 
 test('should normalize path without ending slash', () => {
  const path = '/path/to/folder';
  const normalizedPath: string = normalizePath(path);
- expect(normalizedPath).to.equal('/path/to/folder/');
+ if (process.platform === 'win32') {
+ expect(normalizedPath).to.equal('C:\\path\\to\\folder\\');
+ } else {
+ expect(normalizedPath).to.equal('/path/to/folder/');
+ }
 });
 
 test('should normalize empty path', () => {
  const path = '';
- const normalizedPath: string = normalizePath(path);
- expect(normalizedPath).to.equal(`${resolve('./')}/`);
+ expect(() => normalizePath(path)).to.throw('Path cannot be empty');
 });
 
 test('should normalize root path', () => {
  const path = '/';
  const normalizedPath: string = normalizePath(path);
- expect(normalizedPath).to.equal('/');
+
+ if (process.platform === 'win32') {
+ expect(normalizedPath).to.equal('C:\\');
+ } else {
+ expect(normalizedPath).to.equal('/');
+ }
 });
 
-test('should append trailing backslash if path ends with backslash on Windows systems', () => {
- const path = 'C:\\Users\\test\\Projects\\pdf-to-png-converter\\src\\';
- const normalizedPath = normalizePath(path);
- expect(normalizedPath).to.equal('C:\\Users\\test\\Projects\\pdf-to-png-converter\\src\\');
-});
+if (process.platform === 'win32') {
+ test('should append trailing backslash if path ends with backslash on Windows systems', () => {
+ const path = 'C:\\Windows\\';
+ const normalizedPath = normalizePath(path);
+ expect(normalizedPath).to.equal('C:\\Windows\\');
+ });
+}

dockerfile

@@ -3,6 +3,4 @@ RUN apt-get update && apt-get install -y build-essential libcairo2-dev libpango1
 WORKDIR /usr/pkg/
 COPY . .
 RUN npm ci
-RUN npm run build
-
 CMD npm run docker:test

src/compare.png.ts

@@ -2,6 +2,14 @@ import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { parse } from 'node:path';
 import comparePng, { ComparePngOptions } from 'png-visual-compare';
 
+/**
+ * Compares a PNG image with an expected PNG image.
+ * If the expected image does not exist, it creates it by copying the actual image.
+ * @param actualFilePathOrBuffer - The path or buffer of the actual PNG image.
+ * @param expectedFilePath - The path of the expected PNG image.
+ * @param opts - Optional compare options.
+ * @returns A promise that resolves to the comparison result.
+ */
 export function comparePNG(
  actualFilePathOrBuffer: string | Buffer,
  expectedFilePath: string,

src/normalize.path.ts

@@ -1,8 +1,18 @@
-import { resolve, join, normalize } from "path/posix";
+import { normalize, resolve } from 'node:path';
 
+/**
+ * Normalizes a given path by ensuring it ends with the appropriate path separator.
+ * 
+ * @param path - The path to be normalized.
+ * @returns The normalized path.
+ * @throws Error if the path is empty.
+ */
 export function normalizePath(path: string): string {
- const resolvedPath: string = normalize(resolve(join(path)));
- console.log(process.platform)
+ if (path === '') {
+ throw new Error('Path cannot be empty');
+ }
+ const resolvedPath: string = (normalize(resolve( path)));
+
  if (process.platform === 'win32') {
  if (resolvedPath.endsWith('\\')) {
  return resolvedPath;