Skip to content

Commit 210bccb

Browse files
author
Al Manning
committed
improve replace extension tests, remove duplication
1 parent 89666f8 commit 210bccb

File tree

2 files changed

+5
-7
lines changed

2 files changed

+5
-7
lines changed

src/publish/confluence/confluence-helper.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -480,16 +480,15 @@ export const replaceExtension = (
480480
oldExtension: string,
481481
newExtension: string
482482
) => {
483-
const base = parse(fileName).name;
484-
return `${base}${newExtension}`;
483+
return fileName.replace(oldExtension, newExtension);
485484
};
486485

487486
export const getTitle = (
488487
fileName: string,
489488
metadataByInput: Record<string, InputMetadata>
490489
): string => {
491490
const qmdFileName = replaceExtension(fileName, ".xml", ".qmd");
492-
console.log("qmdFileName", qmdFileName);
491+
493492
const metadataTitle = metadataByInput[qmdFileName]?.title;
494493

495494
const titleFromFilename = capitalizeWord(fileName.split(".")[0] ?? fileName);

tests/unit/confluence.test.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -645,22 +645,22 @@ const runGetTitle = () => {
645645
author: "fake-author",
646646
date: "fake-date",
647647
},
648-
"fake-file2.qmd": {
648+
"folder/fake-file2.qmd": {
649649
title: "fake-title2",
650650
author: "fake-author2",
651651
date: "fake-date2",
652652
},
653653
};
654654

655-
otest(suiteLabel("valid"), async () => {
655+
test(suiteLabel("valid"), async () => {
656656
const fileName = "fake-file.xml";
657657
const expected = "fake-title1";
658658
const result = getTitle(fileName, fakeInputMetadata);
659659
assertEquals(expected, result);
660660
});
661661

662662
test(suiteLabel("valid2"), async () => {
663-
const fileName = "fake-file2.xml";
663+
const fileName = "folder/fake-file2.xml";
664664
const expected = "fake-title2";
665665
const result = getTitle(fileName, fakeInputMetadata);
666666
assertEquals(expected, result);
@@ -3772,7 +3772,6 @@ const runConvertForSecondPass = () => {
37723772
server,
37733773
parent
37743774
);
3775-
console.log("result", result);
37763775
assertEquals(expected, result);
37773776
};
37783777

0 commit comments

Comments
 (0)