Skip to content

Commit c9e894c

Browse files
author
Al Manning
committed
fix attachments windows
1 parent 23f5d23 commit c9e894c

File tree

3 files changed

+15
-40
lines changed

3 files changed

+15
-40
lines changed

src/publish/confluence/confluence-helper.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -647,11 +647,11 @@ export const findAttachments = (
647647
const assetInPublishFiles = publishFiles.find((assetPathParam) => {
648648
const assetPath = pathWithForwardSlashes(assetPathParam);
649649

650-
const toCheck = join(parentPath, assetFileName);
651-
650+
const toCheck = pathWithForwardSlashes(join(parentPath, assetFileName));
651+
console.log("toCheck", toCheck);
652652
return assetPath === toCheck;
653653
});
654-
654+
console.log("assetInPublishFiles", assetInPublishFiles);
655655
return assetInPublishFiles ?? assetFileName;
656656
});
657657
}

src/publish/confluence/confluence.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,7 @@ async function publish(
315315
return null;
316316
}
317317

318-
const fileName = attachmentPath;
318+
const fileName = pathWithForwardSlashes(attachmentPath);
319319

320320
const existingDuplicateAttachment = existingAttachments.find(
321321
(attachment: AttachmentSummary) => {

tests/unit/confluence.test.ts

Lines changed: 11 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ import {
5959
Space,
6060
} from "../../src/publish/confluence/api/types.ts";
6161

62-
const RUN_ALL_TESTS = true;
62+
const RUN_ALL_TESTS = false;
6363
const FOCUS_TEST = false;
6464

6565
const xtest = (
@@ -3503,6 +3503,15 @@ const runFindAttachments = () => {
35033503
check(expected, bodyValue, filePaths, path);
35043504
});
35053505

3506+
test(suiteLabel("single_image_lookup_relative_path_win2"), async () => {
3507+
const bodyValue: string =
3508+
'<ri:attachment ri:filename="images/elephant.png" ri:version-at-save="1" />';
3509+
const filePaths: string[] = ["folder\\images\\elephant.png"];
3510+
const path = "folder/index.xml";
3511+
const expected: string[] = ["folder\\images\\elephant.png"];
3512+
check(expected, bodyValue, filePaths, path);
3513+
});
3514+
35063515
test(suiteLabel("single_image_lookup_dupe_name"), async () => {
35073516
const bodyValue: string =
35083517
'<ri:attachment ri:filename="elephant.png" ri:version-at-save="1" />';
@@ -3651,39 +3660,5 @@ if (RUN_ALL_TESTS) {
36513660
runUpdateImagePathsForContentBody();
36523661
runCapFirstLetter();
36533662
} else {
3654-
runSpaceCreatesWithNesting();
3663+
runFindAttachments();
36553664
}
3656-
3657-
// WINDOWS
3658-
// publishFiles: {
3659-
// baseDir: "C:\\Users\\Megaport\\Documents\\dev\\quarto-confluence-test-main\\simple-site2\\_site",
3660-
// rootFile: "index.html",
3661-
// files: [
3662-
// "folder\\images\\elephant.png",
3663-
// "folder\\index.xml",
3664-
// "images\\elephant.png",
3665-
// "index.html",
3666-
// "index.xml",
3667-
// "search.json",
3668-
// "site_libs\\bootstrap\\bootstrap-icons.css",
3669-
// "site_libs\\bootstrap\\bootstrap-icons.woff",
3670-
// "site_libs\\bootstrap\\bootstrap.min.css",
3671-
// "site_libs\\bootstrap\\bootstrap.min.js",
3672-
// "site_libs\\clipboard\\clipboard.min.js",
3673-
// "site_libs\\quarto-html\\anchor.min.js",
3674-
// "site_libs\\quarto-html\\popper.min.js",
3675-
// "site_libs\\quarto-html\\quarto-syntax-highlighting.css",
3676-
// "site_libs\\quarto-html\\quarto.js",
3677-
// "site_libs\\quarto-html\\tippy.css",
3678-
// "site_libs\\quarto-html\\tippy.umd.min.js",
3679-
// "site_libs\\quarto-nav\\headroom.min.js",
3680-
// "site_libs\\quarto-nav\\quarto-nav.js",
3681-
// "site_libs\\quarto-search\\autocomplete.umd.js",
3682-
// "site_libs\\quarto-search\\fuse.min.js",
3683-
// "site_libs\\quarto-search\\quarto-search.js"
3684-
// ],
3685-
// metadataByInput: {
3686-
// "folder\\index.qmd": { title: "Page 2", author: undefined, date: undefined },
3687-
// "index.qmd": { title: "Page 1", author: undefined, date: undefined }
3688-
// }
3689-
// },

0 commit comments

Comments
 (0)