mirror of https://github.com/vector-im/riot-web
Use throw error instead of try-catch
parent
edfc8af6cf
commit
9cbdc4a613
|
@ -168,22 +168,15 @@ describe('export', function() {
|
||||||
expect(textForFormat('HTML')).toBeTruthy();
|
expect(textForFormat('HTML')).toBeTruthy();
|
||||||
expect(textForFormat('JSON')).toBeTruthy();
|
expect(textForFormat('JSON')).toBeTruthy();
|
||||||
expect(textForFormat('PLAIN_TEXT')).toBeTruthy();
|
expect(textForFormat('PLAIN_TEXT')).toBeTruthy();
|
||||||
try {
|
expect(() => textForFormat('PDF')).toThrowError("Unknown format");
|
||||||
textForFormat('PDF');
|
|
||||||
throw new Error("Expected to throw an error");
|
|
||||||
} catch (e) {
|
|
||||||
expect(e.message).toBe("Unknown format");
|
|
||||||
}
|
|
||||||
});
|
});
|
||||||
|
|
||||||
it('checks if the export options are valid', function() {
|
it('checks if the export options are valid', function() {
|
||||||
for (const exportOption of invalidExportOptions) {
|
for (const exportOption of invalidExportOptions) {
|
||||||
try {
|
expect(
|
||||||
new PlainTextExporter(mockRoom, ExportType.Beginning, exportOption, null);
|
() =>
|
||||||
throw new Error("Expected to throw an error");
|
new PlainTextExporter(mockRoom, ExportType.Beginning, exportOption, null),
|
||||||
} catch (e) {
|
).toThrowError("Invalid export options");
|
||||||
expect(e.message).toBe("Invalid export options");
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue