Add basic test for pptxEditable converter option

.circleci/config.yml

@@ -24,6 +24,10 @@ commands:
         type: steps
         default: []
     steps:
+      - run:
+          name: Set up LibreOffice
+          command: sudo apt-get install -y libreoffice-common
+
       - restore_cache:
           keys:
             - v3-dependencies-{{ .Environment.CIRCLE_JOB }}-{{ checksum "package-lock.json" }}-{{ .Branch }}

.github/workflows/test-win.yml

@@ -37,6 +37,9 @@ jobs:
           npm ci
           npx patch-package
 
+      - name: Set up LibreOffice
+        run: sudo apt-get install -y libreoffice-common
+
       # Retry tests up to 3 times due to flaky tests on Windows CI
       # https://stackoverflow.com/a/59365905
       - name: Jest

test/converter.ts

@@ -1060,6 +1060,54 @@ describe('Converter', () => {
           timeout
         )
       })
+
+      describe('with pptxEditable option', () => {
+        beforeEach(() => {
+          // Don't mock writeFile to use actually saved tmp file for conversion
+          writeFileSpy.mockRestore()
+        })
+
+        it(
+          'converts markdown file into PDF -> PPTX through soffice',
+          async () => {
+            const cvt = converter({ pptxEditable: true })
+            const editablePptxSpy = jest.spyOn(
+              cvt as any,
+              'convertFileToEditablePPTX'
+            )
+            const unlink = jest.spyOn(fs.promises, 'unlink')
+            const fileSave = jest
+              .spyOn(File.prototype, 'save')
+              .mockImplementation()
+            const fileTmp = jest.spyOn(File.prototype, 'saveTmpFile')
+            const warn = jest.spyOn(console, 'warn').mockImplementation()
+
+            await cvt.convertFile(new File(onePath))
+            expect(editablePptxSpy).toHaveBeenCalled()
+            expect(warn).toHaveBeenCalledWith(
+              expect.stringContaining(
+                'Converting to editable PPTX is experimental feature'
+              )
+            )
+            expect(fileTmp).toHaveBeenCalledWith(
+              expect.objectContaining({ extension: '.pdf' })
+            )
+            expect(unlink).toHaveBeenCalledWith(
+              expect.stringContaining(os.tmpdir())
+            )
+            expect(fileSave).toHaveBeenCalled()
+
+            const savedFile = fileSave.mock.instances[0] as unknown as File
+            expect(savedFile).toBeInstanceOf(File)
+            expect(savedFile.path).toBe('test.pptx')
+
+            // ZIP PK header for Office Open XML
+            expect(savedFile.buffer?.toString('ascii', 0, 2)).toBe('PK')
+            expect(savedFile.buffer?.toString('hex', 2, 4)).toBe('0304')
+          },
+          timeoutLarge
+        )
+      })
     })
 
     describe('when convert type is PNG', () => {

test/marp-cli.ts

@@ -1501,6 +1501,20 @@ describe('Marp CLI', () => {
       })
     })
 
+    describe('with --pptx-editable options', () => {
+      it('prefers PPTX than HTML if not specified conversion type', async () => {
+        const cmd = [onePath, '--pptx-editable']
+        expect((await conversion(...cmd)).options.type).toBe(ConvertType.pptx)
+
+        // This option is actually not for defining conversion type so other
+        // options to set conversion type are always prioritized.
+        const cmdPptx = [onePath, '--pptx-editable', '--pdf']
+        expect((await conversion(...cmdPptx)).options.type).toBe(
+          ConvertType.pdf
+        )
+      })
+    })
+
     describe('with PUPPETEER_TIMEOUT env', () => {
       beforeEach(() => {
         process.env.PUPPETEER_TIMEOUT = '12345'