Add logger to troubleshot on Windows

asciidoctor · Jul 8, 2023 · c855de9 · c855de9
1 parent ffab601
commit c855de9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/preprocess.js b/src/preprocess.js
@@ -229,6 +229,7 @@ function readPlantUmlInclude (url, resource, includePaths, includeStack, vfs, lo
     logger.info(`Skipping preprocessing of PlantUML standard library include '${url}'`)
     skip = true
   } else if (includeStack.includes(url)) {
+    console.log({ includeStack, url })
     const message = `Preprocessing of PlantUML include failed, because recursive reading already included referenced file '${url}'`
     throw new Error(message)
   } else {
@@ -243,6 +244,7 @@ function readPlantUmlInclude (url, resource, includePaths, includeStack, vfs, lo
     } else {
       filePath = resolveIncludeFile(url, resource, includePaths, vfs)
       if (includeStack.includes(filePath)) {
+        console.log('after resolveIncludeFile', { includeStack, resource, includePaths, url })
         const message = `Preprocessing of PlantUML include failed, because recursive reading already included referenced file '${filePath}'`
         throw new Error(message)
       } else {
@@ -342,6 +344,7 @@ function getPlantUmlTextOrFirstBlock (text) {
  * @param {string[]} includeOnce
  */
 function checkIncludeOnce (text, filePath, includeOnce) {
+  console.log('checkIncludeOnce', { filePath, includeOnce })
   if (includeOnce.includes(filePath)) {
     const message = `Preprocessing of PlantUML include failed, because including multiple times referenced file '${filePath}' with '!include_once' guard`
     throw new Error(message)

diff --git a/test/preprocess.spec.js b/test/preprocess.spec.js
@@ -272,11 +272,11 @@ ${includedText}`
   })
 
   it('should throw an error for local file(s) referenced multiple times nested with "!include_once local-file-or-url"', () => {
-    const localExistingFileNameIncldudeOncePath = 'test/fixtures/plantuml/styles/style-include-once-general.iuml'
+    const localExistingFileNameIncludedOncePath = 'test/fixtures/plantuml/styles/style-include-once-general.iuml'
     const diagramTextWithExistingLocalIncludeOneFile = `
       !include_once ${localExistingFilePath}
       alice -> bob
-      !include ${localExistingFileNameIncldudeOncePath}`
+      !include ${localExistingFileNameIncludedOncePath}`
     const errorMessage = `Preprocessing of PlantUML include failed, because including multiple times referenced file '${localExistingFilePath}' with '!include_once' guard`
     expect(() => preprocessPlantUML(diagramTextWithExistingLocalIncludeOneFile, {})).to.throw(errorMessage)
   })