#87/fix: fixed handling undefined and null in the configurations (#25)

CHANGELOG.md

@@ -1,6 +1,12 @@
 # Changelog
 
-## 1.2.0  2023-11-11
+## 1.2.1 - 2023-11-24
+
+### Fixed
+
+-Fixed issue with undefined and null values ([#87](https://github.com/gherking/gherking/issues/87))
+
+## 1.2.0 - 2023-11-11
 
 ### Added
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "gherkin-formatter",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "Tool to format gherkin-ast model to gherkin string",
   "main": "index.js",
   "types": "index.d.ts",

src/index.ts

@@ -15,42 +15,47 @@ export interface FormatOptions {
 }
 
 const DEFAULT_OPTIONS: FormatOptions = {
-  oneTagPerLine: false,
-  separateStepGroups: false,
-  compact: false,
-  lineBreak: null,
-  indentation: "  ",
-  tagFormat: TagFormat.FUNCTIONAL,
+    oneTagPerLine: false,
+    separateStepGroups: false,
+    compact: false,
+    lineBreak: null,
+    indentation: "  ",
+    tagFormat: TagFormat.FUNCTIONAL,
 };
 
-export const config = (options: Partial<FormatOptions>): FormatOptions => {
-  debug("config(options: %o)", options);
-  return {
-    ...DEFAULT_OPTIONS,
-    ...(options || {}),
-  };
+export const config = (options?: Partial<FormatOptions>): FormatOptions => {
+    debug("config(options: %o)", options);
+    options = options ?? {};
+    return {
+        oneTagPerLine: options.oneTagPerLine ?? DEFAULT_OPTIONS.oneTagPerLine,
+        separateStepGroups: options.separateStepGroups ?? DEFAULT_OPTIONS.separateStepGroups,
+        compact: options.compact ?? DEFAULT_OPTIONS.compact,
+        lineBreak: options.lineBreak ?? DEFAULT_OPTIONS.lineBreak,
+        indentation: options.indentation ?? DEFAULT_OPTIONS.indentation,
+        tagFormat: options.tagFormat ?? DEFAULT_OPTIONS.tagFormat,
+    };
 };
 
 export const format = (document: Document, options?: Partial<FormatOptions>): string => {
-  debug("format(document: %s, options: %o)", document?.constructor.name, options);
-  if (!document) {
-    throw new Error("Document must be set!");
-  }
-  if (!(document instanceof Document)) {
-    throw new TypeError(`The passed object is not a GherkinDocument! ${document}`);
-  }
-  options = config(options);
-  astConfig.set({
-    tagFormat: options.tagFormat,
-  });
-  setDefaultOptions({
-    eol: options.lineBreak,
-    indent: options.indentation,
-    skipEmpty: options.compact,
-    skipFirstLevelIndent: true,
-    indentEmpty: false,
-    trimLeft: false,
-    trimRight: true,
-  });
-  return formatGherkinDocument(document, options);
+    debug("format(document: %s, options: %o)", document?.constructor.name, options);
+    if (!document) {
+        throw new Error("Document must be set!");
+    }
+    if (!(document instanceof Document)) {
+        throw new TypeError(`The passed object is not a GherkinDocument! ${document}`);
+    }
+    options = config(options);
+    astConfig.set({
+        tagFormat: options.tagFormat,
+    });
+    setDefaultOptions({
+        eol: options.lineBreak,
+        indent: options.indentation,
+        skipEmpty: options.compact,
+        skipFirstLevelIndent: true,
+        indentEmpty: false,
+        trimLeft: false,
+        trimRight: true,
+    });
+    return formatGherkinDocument(document, options);
 };