Unit test code coverage now at almost 100%

src/__tests__/.eslintrc.js

@@ -1,5 +1,8 @@
 module.exports = {
 	env: {
 		jest: true
+	},
+	rules: {
+		'jest/no-hooks': 'off',
 	}
 };

src/__tests__/dom.ts

@@ -46,6 +46,8 @@ describe('createElement', () => {
 	it('should create an element in the DOM', () => {
 		expect.hasAssertions();
 
+		createElement(document.body, 'div');
+
 		createElement(document.body, 'div', 'big', {color: 'red'}, 'text', {id: '1'});
 		expect(document.getElementById('1')).not.toBeNull();
 

src/__tests__/index.ts

@@ -1,7 +1,18 @@
+/* eslint-disable @typescript-eslint/ban-ts-comment, @typescript-eslint/no-explicit-any */
+
 import {GanttChart} from '../index';
-//import {getElementByClassName} from '../dom/dom';
+import {getElementByClassName} from '../dom/dom';
 
 describe('the class GanttChart', () => {
+	beforeEach(() => {
+		//@ts-ignore
+		jest.spyOn(window, 'requestAnimationFrame').mockImplementation(cb => cb());
+	});
+
+	afterEach(() => {
+		(window.requestAnimationFrame as any).mockRestore();
+	});
+
 	it('should render a gantt chart', () => {
 		expect.hasAssertions();
 
@@ -26,10 +37,8 @@ describe('the class GanttChart', () => {
 		expect(container.innerHTML).toMatch(/^<div id=".*" class="gc_root_container_view".*>.*<\/div>$/);
 
 		// let's test scrolling
-		/*
 		const scrollbarElement = getElementByClassName('gc_timeline_scrollbar_cell');
-		console.log('scrollbarElement', scrollbarElement, scrollbarElement.innerHTML);
-		scrollbarElement.scrollTo(10, 10);
-		*/
+		scrollbarElement.scrollLeft = 10;
+		scrollbarElement.dispatchEvent(new Event('scroll'));
 	});
 });

src/__tests__/tags.ts

@@ -11,6 +11,7 @@ describe('tags', () => {
 	it('getContainerOpen', () => {
 		expect.hasAssertions();
 
+		expect(getContainerOpen([])).toStrictEqual('<div>');
 		expect(getContainerOpen([], {})).toStrictEqual('<div>');
 		expect(getContainerOpen('big', {color: 'red', top: 0}, {id: 'content'})).toStrictEqual('<div id="content" class="big" style="color:red;top:0;">');
 		expect(getContainerOpen(['big', 'small'], {color: 'red', top: 0}, {id: 'content'})).toStrictEqual('<div id="content" class="big small" style="color:red;top:0;">');
@@ -26,6 +27,7 @@ describe('tags', () => {
 	it('getContainer', () => {
 		expect.hasAssertions();
 
+		expect(getContainer([])).toStrictEqual('<div></div>');
 		expect(getContainer(['big', 'small'], {color: 'red', top: 0}, 'text', {id: 'content'})).toStrictEqual('<div id="content" class="big small" style="color:red;top:0;">text</div>');
 	});
 });

src/components/Link.ts

@@ -140,6 +140,7 @@ function getArrowMarkup(point: pointType, direction: Direction, lineHeight: numb
 			break;
 
 		default:
+			/* istanbul ignore next */
 			throw new Error(`Invalid direction "${direction}"`);
 	}
 
@@ -174,6 +175,7 @@ function getLineSize(vector: vectorType, lineWidth: number, containerWidth: numb
 			};
 
 		default:
+			/* istanbul ignore next */
 			throw new Error(`Invalid direction "${vector.direction}"`);
 	}
 }
@@ -219,6 +221,7 @@ function getLineContainerSize(vector: vectorType, cellHeight: number, wrapperWid
 			};
 
 		default:
+			/* istanbul ignore next */
 			throw new TypeError(`Invalid direction "${vector.direction}"`);
 	}
 }

src/components/TimelineScrollbar.ts

@@ -78,6 +78,7 @@ export class TimelineScrollbar {
 }
 
 function getScrollLeft(event: Event): number|null {
+	/* istanbul ignore if */
 	if (event === null || event.target === null) {
 		return null;
 	} else {

src/dom/HtmlStyle.ts

@@ -34,7 +34,7 @@ export class HtmlStyle {
 	 *	@param {stylesType} styles - A object where each property represents a style component.
 	 *	@example style.merge({'color': black, 'background-color': 'white'});
 	 */
-	public merge(styles: stylesType = {}): void {
+	public merge(styles: stylesType): void {
 		for (const e in styles) {
 			const name = e.toLocaleLowerCase();
 			this._styles[name] = styles[e];

src/dom/dom.ts

@@ -98,14 +98,14 @@ export function addCustomEventListener(selector: string, event: string, handler:
  *
  *	@param {Node} container - The parent node.
  *	@param {string} tag - The tag name.
- *	@param {string|Array<string>} classes - The string in the class attribute.
+ *	@param {string|Array<string>} [classes=[]] - The string in the class attribute.
  *	@param {HtmlStyle|stylesType} [styles={}] - The styles in the style attribute.
  *	@param {string} [content] - The content of the div.
  *	@param {HtmlAttributes|attributesType} [attributes={}] - The optional attributes.
  *	@returns {HTMLElement} - The html markup.
  *	@example const element = createElement(document, 'div', 'cell', {width: 0}, 'hello world', {id: 'id'});
  */
-export function createElement(container: Node, tag: string, classes: string|Array<string>, styles: HtmlStyle|stylesType = {}, content = '', attributes: HtmlAttributes|attributesType = {}): HTMLElement {
+export function createElement(container: Node, tag: string, classes: string|Array<string> = [], styles: HtmlStyle|stylesType = {}, content = '', attributes: HtmlAttributes|attributesType = {}): HTMLElement {
 	const htmlAttributes = new HtmlAttributes(attributes);
 	const htmlStyle = new HtmlStyle(styles).toString();