From e1355139458f443c501b708f5b09397c2addea0b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B8rgen=20Sverre=20Lien=20Sell=C3=A6g?= Date: Sun, 8 Nov 2020 01:56:57 +0100 Subject: [PATCH] remove irrelevant tests --- tests/create_images_directory.test.js | 50 --------------------------- tests/parse_args.test.js | 21 ----------- 2 files changed, 71 deletions(-) delete mode 100644 tests/create_images_directory.test.js diff --git a/tests/create_images_directory.test.js b/tests/create_images_directory.test.js deleted file mode 100644 index 2e69868..0000000 --- a/tests/create_images_directory.test.js +++ /dev/null @@ -1,50 +0,0 @@ -import { create_images_directory } from '../src/logic'; - -beforeEach(jest.restoreAllMocks); - -const fs = require('fs').promises; -const filesystem = require('fs'); - -describe('create_images_directory', () => { - it('creates images directory', () => { - const mock = jest - .spyOn(fs, 'mkdir') - .mockImplementation(() => Promise.resolve()); - jest.spyOn(filesystem, 'existsSync').mockImplementation(() => false); - const dir = 'new directory'; - expect.assertions(2); - return create_images_directory(dir).then(() => { - expect(mock).toHaveBeenCalledTimes(1); - expect(mock).toHaveBeenCalledWith(dir, { recursive: true }); - }); - }); - - it('it skips creating images directory', () => { - const fs_mock = jest - .spyOn(fs, 'mkdir') - .mockImplementation(() => Promise.resolve()); - const exist_mock = jest - .spyOn(filesystem, 'existsSync') - .mockImplementation(() => true); - const dir = 'new directory'; - expect.assertions(3); - return create_images_directory(dir).then(() => { - expect(fs_mock).toHaveBeenCalledTimes(0); - expect(exist_mock).toHaveBeenCalledTimes(1); - expect(exist_mock).toHaveBeenCalledWith(dir); - }); - }); - - [undefined, null].forEach((dir) => - it('throws if directory is not set', () => { - const mock = jest - .spyOn(filesystem, 'existsSync') - .mockImplementation(() => true); - expect.assertions(2); - return create_images_directory(dir).catch((error) => { - expect(mock).toHaveBeenCalledTimes(0); - expect(error).toEqual('Image path was not set'); - }); - }), - ); -}); diff --git a/tests/parse_args.test.js b/tests/parse_args.test.js index 576fce9..674d08f 100644 --- a/tests/parse_args.test.js +++ b/tests/parse_args.test.js @@ -71,27 +71,6 @@ describe('test parse args', () => { expect(res.output).toEqual('jacobi'); }); - ['--images', '-i'].forEach((param) => { - it('parses image options', () => { - const res = parse_args([param]); - expect(res.images).toEqual(true); - }); - }); - - [ - ['--image-directory=img'], - ['--image-directory="img"'], - ["--image-directory='img'"], - ['--image-directory', 'img'], - ['--image-directory', '"img"'], - ['--image-directory', "'img'"], - ].forEach((param) => { - it('parses image_directory options', () => { - const res = parse_args(param); - expect(res.image_directory).toEqual('img'); - }); - }); - it('parses skip upcoming events option', () => { const res = parse_args(['--skip-upcoming-events']); expect(res.get_upcoming_events).toEqual(false);