test: fix unit test fixtures

Author: jGleitz

test/fixtures/commentListing.ts

@@ -125,7 +125,7 @@ export const deeplyNested = {
 									website: null,
 									created: 138733525.572392,
 									modified: null,
-									total_replies: 1,
+									total_replies: 0,
 									hidden_replies: 0,
 									likes: 0,
 									dislikes: 0,
@@ -145,7 +145,7 @@ export const deeplyNested = {
 							website: null,
 							created: 138733125.572392,
 							modified: null,
-							total_replies: 1,
+							total_replies: 0,
 							hidden_replies: 0,
 							likes: 0,
 							dislikes: 0,

test/unit/CommentList.test.ts

@@ -267,8 +267,8 @@ describe('CommentList', () => {
 	it('can be sorted by multiple criteria', () => {
 		return pageWithCommentList(SERVER_FIXTURES.forSorting).then(page => {
 			page.comments.sortBys(
-				{criterion: SortCriterion.LIKES, mode: SortMode.DESCENDING},
-				{criterion: SortCriterion.MODIFICATION, mode: SortMode.ASCENDING}
+				{ criterion: SortCriterion.LIKES, mode: SortMode.DESCENDING },
+				{ criterion: SortCriterion.MODIFICATION, mode: SortMode.ASCENDING }
 			);
 			expect(page.comments.map(toId)).toEqual([4, 6, 5, 2, 1, 3]);
 		});

test/util/FakeIssoServer.ts

@@ -22,7 +22,7 @@ export default class FakeIssoServer extends IssoServer {
 		delete: stubRegistry;
 	};
 
-	private setStubsToEmpty() {
+	private setStubsToEmpty(): void {
 		this.stubs = {
 			get: {},
 			post: {},
@@ -122,7 +122,7 @@ export default class FakeIssoServer extends IssoServer {
 		const stub = jest
 			.spyOn(request, 'end')
 			.mockName('request.end')
-			.mockImplementation(function (this: Http.Request, callback?: RequestCallback): Http.Request {
+			.mockImplementation(function(this: Http.Request, callback?: RequestCallback): Http.Request {
 				endStub.call(this, callback || (() => null));
 				return request;
 			});

test/util/IssoManagement.ts

@@ -1,4 +1,4 @@
-/// <reference path="./hasbin.d.ts" />
+// / <reference path="./hasbin.d.ts" />
 
 /* eslint-env node */
 
@@ -171,8 +171,7 @@ export default class IssoManagement {
 					}
 				});
 		} else {
-			imageInstall = findDocker()
-				.then(docker => execAndPrint(`${docker} pull ${ISSO_IMAGE}`));
+			imageInstall = findDocker().then(docker => execAndPrint(`${docker} pull ${ISSO_IMAGE}`));
 		}
 		return imageInstall
 			.then(() => tmp.dir())
@@ -381,8 +380,7 @@ class Isso {
 	/**
 	 * Creates a new isso server instance with the given instance id (but does not start it)
 	 */
-	public constructor(public readonly id: number) {
-	}
+	public constructor(public readonly id: number) {}
 
 	/**
 	 * Starts this instance if it’s not already running.
@@ -393,43 +391,48 @@ class Isso {
 		if (this.process !== undefined) {
 			return Promise.resolve();
 		}
-		return findDocker()
-			.then(docker => new Promise((resolve, reject) => {
-				this.process = execFile(
-					docker,
-					[
-						'run',
-						'--rm',
-						'--network', 'host',
-						'-v', `${this.configDir}:/config`,
-						'-v', `${this.dbDir}:/db`,
-						ISSO_IMAGE
-					],
-					error => {
-						if (error && !error.killed) {
-							this.stop().finally(() => reject(error));
+		return findDocker().then(
+			docker =>
+				new Promise((resolve, reject) => {
+					this.process = execFile(
+						docker,
+						[
+							'run',
+							'--rm',
+							'--network',
+							'host',
+							'-v',
+							`${this.configDir}:/config`,
+							'-v',
+							`${this.dbDir}:/db`,
+							ISSO_IMAGE
+						],
+						error => {
+							if (error && !error.killed) {
+								this.stop().finally(() => reject(error));
+							}
 						}
-					}
-				);
-				const startTimeout = setTimeout(
-					() => this.stop().finally(() => reject('Isso did not start within 20 seconds!')),
-					20000
-				);
-
-				const startListener = (data: string): void => {
-					clearTimeout(startTimeout);
-					this.process?.stderr?.removeListener('data', startListener);
-					if (!INFO_REGEX.test(data)) {
-						console.error(data);
-						this.stop().finally(() => reject(data));
-					} else if (STARTED_REGEX.test(data)) {
-						resolve();
-					}
-				};
+					);
+					const startTimeout = setTimeout(
+						() => this.stop().finally(() => reject('Isso did not start within 20 seconds!')),
+						20000
+					);
+
+					const startListener = (data: string): void => {
+						clearTimeout(startTimeout);
+						this.process?.stderr?.removeListener('data', startListener);
+						if (!INFO_REGEX.test(data)) {
+							console.error(data);
+							this.stop().finally(() => reject(data));
+						} else if (STARTED_REGEX.test(data)) {
+							resolve();
+						}
+					};
 
-				this.process.stderr?.on('data', startListener);
-				printSpawned(this.process);
-			}));
+					this.process.stderr?.on('data', startListener);
+					printSpawned(this.process);
+				})
+		);
 	}
 
 	/**
@@ -454,7 +457,7 @@ class Isso {
 	 */
 	public removeDatabase(): Promise<void> {
 		return rmrf(this.dbDir).then(() =>
-			fs.promises.mkdir(this.dbDir, {mode: 0o777, recursive: true})
+			fs.promises.mkdir(this.dbDir, { mode: 0o777, recursive: true })
 		);
 	}
 
@@ -482,8 +485,8 @@ class Isso {
 			enabled = false
 		`.replace(/\t/g, '');
 		return rmrf(this.configDir)
-			.then(() => fs.promises.mkdir(this.configDir, {mode: 0o777, recursive: true}))
-			.then(() => writeFile(`${this.configDir}/isso.conf`, config, {mode: 0o777}));
+			.then(() => fs.promises.mkdir(this.configDir, { mode: 0o777, recursive: true }))
+			.then(() => writeFile(`${this.configDir}/isso.conf`, config, { mode: 0o777 }));
 	}
 
 	private get configDir(): string {

test/util/clone.d.ts

@@ -1,5 +1,5 @@
 declare module 'clone' {
-	function clone<T>(o: T): T
+	function clone<T>(o: T): T;
 
-	export default clone
+	export default clone;
 }