Merge pull request #39 from marp-team/remove-ts-ignore

Remove ts-ignore magic comments from test

CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Fixed
 
 - Use `Buffer.from()` instead of deprecated constructor ([#37](https://github.com/marp-team/marp-cli/pull/37))
+- Remove `@ts-ignore` magic comment from test ([#39](https://github.com/marp-team/marp-cli/pull/39))
 
 ## v0.0.13 - 2018-11-10
 

test/converter.ts

@@ -199,8 +199,7 @@ describe('Converter', () => {
         it('converts into PDF by using temporally file', async () => {
           const file = new File(onePath)
 
-          // @ts-ignore to check cleanup tmpfile
-          const fileCleanup = jest.spyOn(File.prototype, 'cleanup')
+          const fileCleanup = jest.spyOn(<any>File.prototype, 'cleanup')
           const fileSave = jest
             .spyOn(File.prototype, 'save')
             .mockImplementation()

test/engine.ts

@@ -17,8 +17,7 @@ describe('Engine', () => {
     })
 
     it("loads browser script from defined in module's marpBrowser section", async () => {
-      // @ts-ignore: #findClassPath is mocked
-      const finder = <jest.Mock>ResolvedEngine.prototype.findClassPath
+      const finder = <jest.Mock>(<any>ResolvedEngine.prototype).findClassPath
 
       // Core (defined marpBrowser)
       finder.mockImplementation(() => require.resolve(coreModule))

test/marp-cli.ts

@@ -530,8 +530,8 @@ describe('Marp CLI', () => {
         .spyOn(getStdin, 'buffer')
         .mockResolvedValue(Buffer.from('# markdown'))
 
-      // @ts-ignore: reset cached stdin buffer
-      File.stdinBuffer = undefined
+      // reset cached stdin buffer
+      ;(<any>File).stdinBuffer = undefined
 
       expect(await marpCli()).toBe(0)
       expect(cliInfo).toHaveBeenCalledWith(