refactor: refactor internal libs

Signed-off-by: Jan Kowalleck <jan.kowalleck@gmail.com>

libs/universal-node-xml/index.d.ts → libs/universal-node-xml/stringify.d.ts

@@ -28,9 +28,5 @@ declare interface ThrowError {
 }
 
 declare type Stringify = (element: SimpleXml.Element, options?: SerializerOptions) => string
-export declare const stringify: Stringify | ThrowError
-
-/*
-declare type Parse = (xml: string) => SimpleXml.Element
-export declare const parse: Parse | ThrowError
-*/
+declare const stringify: Stringify | ThrowError
+export = stringify

libs/universal-node-xml/index.js → libs/universal-node-xml/stringify.js

@@ -25,24 +25,25 @@ Copyright (c) OWASP Foundation. All Rights Reserved.
  * @type {[string, function():(Function|*)][]}
  */
 const possibleStringifiers = [
-  ['xmlbuilder2', () => require('./stringifiers/xmlbuilder2')]
+  ['xmlbuilder2', () => require('./__stringifiers/xmlbuilder2')]
+  // ... add others here, pull-requests welcome!
 ]
 /* eslint-enable jsdoc/valid-types */
 
-module.exports.stringify = function () {
+module.exports = function () {
   throw new Error(
     'No stringifier available.' +
     ' Please install any of the optional dependencies: ' +
     possibleStringifiers.map(kv => kv[0]).join(', ')
   )
 }
-module.exports.stringify.fails = true
+module.exports.fails = true
 
 for (const [, getStringifier] of possibleStringifiers) {
   try {
     const possibleStringifier = getStringifier()
     if (typeof possibleStringifier === 'function') {
-      module.exports.stringify = possibleStringifier
+      module.exports = possibleStringifier
       break
     }
   } catch {

libs/universal-node-xml/index.test.js → libs/universal-node-xml/stringify.test.js

@@ -20,20 +20,19 @@
 const assert = require('assert')
 const { suite, test } = require('mocha')
 
-const { stringify } = require('./')
+const stringify  = require('./stringify')
 
-suite('libs/universal-node-xml', () => {
-  suite('stringify', () => {
+suite('libs/universal-node-xml/stringify', () => {
     const dummyElem = Object.freeze({
       type: 'element',
       name: 'foo'
     })
     const dummyElemStringifiedRE = /<foo(:?\/>|><\/foo>)/
 
     if (stringify.fails) {
       test('call should fail/throw', () => {
         assert.throws(
           () => {
             stringify(dummyElem)
           },
           (err) => {
@@ -49,5 +48,4 @@
         assert.match(stringified, dummyElemStringifiedRE)
       })
     }
-  })
 })

src/serialize/xmlSerializer.node.ts

@@ -17,7 +17,7 @@ SPDX-License-Identifier: Apache-2.0
 Copyright (c) OWASP Foundation. All Rights Reserved.
 */
 
-import { stringify } from '../../libs/universal-node-xml'
+import * as stringify from '../../libs/universal-node-xml/stringify'
 import type { SerializerOptions } from './types'
 import type { SimpleXml } from './xml/types'
 import { XmlBaseSerializer } from './xmlBaseSerializer'