Fix test for NTFS. Use path.resolve() and __test/tmp/*

Author: yammerjp

.gitignore

@@ -2,3 +2,5 @@ node_modules/
 dist/
 ignore/
 coverage/
+/__test__/tmp/**/*
+!/__test__/tmp/.gitkeep

__test__/fileRequest.test.ts

@@ -1,9 +1,11 @@
 import FileRequest from '../src/fileRequest'
 import { promises as fs } from 'fs'
+import path from 'path'
+import testDir from './testDir'
 
 // Initialize Directory
-const entryDir: string = '/tmp/gimonfu/fileRequest.test.ts/entry';
-(async () => await fs.rmdir(entryDir, {recursive: true}));
+const entryDir: string = path.resolve(testDir, 'tmp', 'gimonfu', 'fileRequest.test.ts', 'entry');
+(async () => await fs.rm(entryDir, {recursive: true, force: true}));
 (async () => await fs.mkdir(entryDir, {recursive: true}));
 
 const fileRequest = new FileRequest(entryDir)
@@ -33,25 +35,25 @@ Bye~
 
 test('customUrl2filePath', () => {
   const filePath = fileRequest.customUrl2filePath(article)
-  expect(filePath).toBe('/tmp/gimonfu/fileRequest.test.ts/entry/2020/05/12/today-blog.md')
+  expect(filePath).toBe(path.resolve(testDir, 'tmp', 'gimonfu', 'fileRequest.test.ts', 'entry', '2020', '05', '12', 'today-blog.md'))
 });
 
 test('filePath2customUrl', async () => {
-  const filePath = '/tmp/gimonfu/fileRequest.test.ts/entry/hoge/fuga.md'
+  const filePath = path.resolve(testDir, 'tmp', 'gimonfu', 'fileRequest.test.ts', 'entry', 'hoge', 'fuga.md')
   const customUrl = await (fileRequest as any).filePath2customUrl(filePath)
   expect(customUrl).toBe('hoge/fuga')
 })
 
 test('(invalid) filePath2customUrl', () => {
-  const filePath = '/out-of-entry-dir/tmp/gimonfu/fileRequest.test.ts/entry/hoge/fuga.md'
+  const filePath = path.resolve(testDir, 'out-of-entry-dir', 'tmp', 'gimonfu', 'fileRequest.test.ts', 'entry', 'hoge', 'fuga.md')
   expect(
     (fileRequest as any).filePath2customUrl(filePath)
-  ).rejects.toThrowError('Base directory /tmp/gimonfu/fileRequest.test.ts/entry does not contain markdown file path /out-of-entry-dir/tmp/gimonfu/fileRequest.test.ts/entry/hoge/fuga.md') //toMatch('does not contain markdown file path')
+  ).rejects.toThrowError(`Base directory ${entryDir} does not contain markdown file path ${filePath}`) //toMatch('does not contain markdown file path')
 })
 
 test('write/read', async () => {
   // articleオブジェクトをファイルに一度writeしてのちにreadしたとき、内容が一致する
   await fileRequest.write(article)
-  const readArticle = await fileRequest.read('/tmp/gimonfu/fileRequest.test.ts/entry/2020/05/12/today-blog.md', {})
+  const readArticle = await fileRequest.read(path.resolve(testDir, 'tmp', 'gimonfu', 'fileRequest.test.ts', 'entry', '2020', '05', '12', 'today-blog.md'), {})
   expect(readArticle).toEqual(article)
 })

__test__/loadConfig.test.ts

@@ -1,24 +1,26 @@
 import loadConfig from '../src/loadConfig'
 import { promises as fs } from 'fs'
+import path from 'path'
+import testDir from './testDir'
 
 test('loadConfig', async () => {
-  await fs.rmdir('/tmp/gimonfu/loadConfig.test.ts', { recursive: true })
-  await fs.mkdir('/tmp/gimonfu/loadConfig.test.ts/entry/2020/05/12', { recursive: true })
+   await fs.rm(path.resolve(testDir, 'tmp', 'gimonfu', 'loadConfig.test.ts'), { recursive: true, force: true })
+  await fs.mkdir(path.resolve(testDir, 'tmp', 'gimonfu', 'loadConfig.test.ts', 'entry', '2020', '05', '12'), { recursive: true })
 
   await fs.writeFile(
-    '/tmp/gimonfu/loadConfig.test.ts/.gimonfu.json',
+    path.resolve(testDir, 'tmp', 'gimonfu', 'loadConfig.test.ts', '.gimonfu.json'),
     '{"user_id":"uid", "blog_id": "example.hatenablog.com", "api_key": "api-key-string"}'
   )
 
   // 実行時のカレントディレクトリをモック
   const spy = jest.spyOn(process, 'cwd')
-  spy.mockReturnValue('/tmp/gimonfu/loadConfig.test.ts/entry/2020/05/12')
+  spy.mockReturnValue(path.resolve(testDir, 'tmp', 'gimonfu', 'loadConfig.test.ts', 'entry', '2020', '05', '12'))
 
   expect( await loadConfig() ).toEqual({
     user_id: "uid",
     blog_id: "example.hatenablog.com",
     api_key: "api-key-string",
-    baseDir: "/tmp/gimonfu/loadConfig.test.ts"
+    baseDir: path.resolve(testDir, 'tmp', 'gimonfu', 'loadConfig.test.ts')
   })
 })
 

__test__/pull.test.ts

@@ -1,22 +1,23 @@
 import pull from '../src/pull'
 import { promises as fs } from 'fs'
 import path from 'path'
+import testDir from './testDir'
 
 // TODO: 全てdownloadするとき以外の test case も追加する。
 // TODO: ファイルを書き換えたらスキップされるか
 // TODO: ファイルを書き換えてmtimeを古くしたら上書きされるか
 
-const entryDir: string = '/tmp/gimonfu/pull.test.ts/entry';
+const entryDir: string = path.resolve(testDir, 'tmp', 'gimonfu', 'pull.test.ts', 'entry');
 beforeEach( async () => {
-  await fs.rmdir(entryDir, {recursive: true})
+  await fs.rm(entryDir, {recursive: true, force: true})
   await fs.mkdir(entryDir, {recursive: true})
 })
 
 jest.mock('../src/loadConfig', () => () => ({
   user_id: 'user',
   blog_id: 'blogId',
   api_key: '',
-  baseDir: '/tmp/gimonfu/pull.test.ts'
+  baseDir: path.resolve(testDir, 'tmp', 'gimonfu', 'pull.test.ts')
 }))
 
 // Suppress console.log

__test__/push.test.ts

@@ -2,15 +2,16 @@ import pull from '../src/pull'
 import push from '../src/push'
 import { promises as fs } from 'fs'
 import path from 'path'
+import testDir from './testDir'
 
 // TODO: 新規ファイルをpostすることを確認
 // TODO: ファイルを書き換えたらputされることを確認
 // TODO: ファイルを書き換えてmtimeを古くしたらスキップされることを確認
 
-const entryDir: string = '/tmp/gimonfu/push.test.ts/entry';
+const entryDir: string = path.resolve(testDir, 'tmp',' gimonfu', 'push.test.ts', 'entry');
 
 beforeEach( async () => {
-  await fs.rmdir(entryDir, {recursive: true});
+  await fs.rm(entryDir, {recursive: true, force: true});
   await fs.mkdir(entryDir, {recursive: true});
 })
 
@@ -21,7 +22,7 @@ jest.mock('../src/loadConfig', () => () => ({
   user_id: 'user',
   blog_id: 'blogId',
   api_key: '',
-  baseDir: '/tmp/gimonfu/push.test.ts'
+  baseDir: path.resolve(testDir, 'tmp', 'gimonfu', 'push.test.ts')
 }))
 
 jest.mock('request-promise-native', () => ( (req: any) => {

__test__/testDir.ts

@@ -0,0 +1 @@
+export default __dirname

__test__/tmp/.gitkeep

@@ -4,7 +4,7 @@ module.exports = {
 //    "<rootDir>/src"
 //  ],
   "testMatch": [
-    "**/__test__/**/*.+(ts|tsx|js)",
+    "**/__test__/**/*.test.+(ts|tsx|js)",
 //    "**/?(*.)+(spec|test).+(ts|tsx|js)"
   ],
   "transform": {