diff --git a/test/unit/api_spec.js b/test/unit/api_spec.js index 380b951a3caa3..ebdc3d6d5fc3b 100644 --- a/test/unit/api_spec.js +++ b/test/unit/api_spec.js @@ -85,9 +85,7 @@ describe('api', function() { expect(data[1] instanceof PDFDocumentProxy).toEqual(true); expect(loadingTask).toEqual(data[1].loadingTask); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('creates pdf doc from URL and aborts before worker initialized', function(done) { @@ -112,9 +110,7 @@ describe('api', function() { destroyed.then(function (data) { expect(true).toEqual(true); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('creates pdf doc from typed array', function(done) { var typedArrayPdf; @@ -150,9 +146,7 @@ describe('api', function() { loadingTask.promise.then(function(data) { expect(data instanceof PDFDocumentProxy).toEqual(true); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('creates pdf doc from invalid PDF file', function(done) { // A severely corrupt PDF file (even Adobe Reader fails to open it). @@ -214,9 +208,7 @@ describe('api', function() { Promise.all(promises).then(function (data) { expect(data[2] instanceof PDFDocumentProxy).toEqual(true); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('creates pdf doc from PDF file protected with only a user password', function (done) { @@ -258,9 +250,7 @@ describe('api', function() { }); Promise.all([result1, result2, result3]).then(function () { done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('creates pdf doc from password protected PDF file and aborts/throws ' + @@ -310,9 +300,7 @@ describe('api', function() { Promise.all([result1, result2]).then(function () { done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); }); @@ -334,9 +322,7 @@ describe('api', function() { expect(!!worker.port).toEqual(false); expect(worker.destroyed).toEqual(true); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('worker created or destroyed by getDocument', function (done) { if (isNodeJS()) { @@ -358,9 +344,7 @@ describe('api', function() { expect(!!destroyedWorker).toEqual(false); expect(worker.destroyed).toEqual(true); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('worker created and can be used in getDocument', function (done) { if (isNodeJS()) { @@ -387,9 +371,7 @@ describe('api', function() { expect(worker.destroyed).toEqual(false); worker.destroy(); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('creates more than one worker', function (done) { if (isNodeJS()) { @@ -409,9 +391,7 @@ describe('api', function() { worker2.destroy(); worker3.destroy(); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets current workerSrc', function() { if (isNodeJS()) { @@ -453,9 +433,7 @@ describe('api', function() { expect(data instanceof PDFPageProxy).toEqual(true); expect(data.pageIndex).toEqual(0); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets non-existent page', function(done) { var outOfRangePromise = doc.getPage(100); @@ -481,9 +459,7 @@ describe('api', function() { Promise.all([outOfRangePromise, nonIntegerPromise, nonNumberPromise]). then(function () { done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets page index', function(done) { // reference to second page @@ -492,9 +468,7 @@ describe('api', function() { promise.then(function(pageIndex) { expect(pageIndex).toEqual(1); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets invalid page index', function (done) { var ref = { num: 3, gen: 0, }; // Reference to a font dictionary. @@ -514,9 +488,7 @@ describe('api', function() { chapter1: [{ gen: 0, num: 17, }, { name: 'XYZ', }, 0, 841.89, null], }); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets a destination, from /Dests dictionary', function(done) { var promise = doc.getDestination('chapter1'); @@ -524,9 +496,7 @@ describe('api', function() { expect(data).toEqual([{ gen: 0, num: 17, }, { name: 'XYZ', }, 0, 841.89, null]); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets a non-existent destination, from /Dests dictionary', function(done) { @@ -534,9 +504,7 @@ describe('api', function() { promise.then(function(data) { expect(data).toEqual(null); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets destinations, from /Names (NameTree) dictionary', function(done) { @@ -551,9 +519,7 @@ describe('api', function() { }); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets a destination, from /Names (NameTree) dictionary', function(done) { var loadingTask = getDocument(buildGetDocumentParams('issue6204.pdf')); @@ -565,9 +531,7 @@ describe('api', function() { 0, 375, null]); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets a non-existent destination, from /Names (NameTree) dictionary', function(done) { @@ -579,9 +543,7 @@ describe('api', function() { expect(destination).toEqual(null); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets non-string destination', function(done) { @@ -615,9 +577,7 @@ describe('api', function() { promise.then(function (data) { expect(data).toEqual(null); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets page labels', function (done) { // PageLabels with Roman/Arabic numerals. @@ -658,9 +618,7 @@ describe('api', function() { loadingTask2.destroy(), loadingTask3.destroy() ]).then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets default page mode', function(done) { @@ -672,17 +630,13 @@ describe('api', function() { expect(mode).toEqual('UseNone'); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets non-default page mode', function(done) { doc.getPageMode().then(function(mode) { expect(mode).toEqual('UseOutlines'); done(); - }).catch(function(reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets non-existent attachments', function(done) { @@ -690,9 +644,7 @@ describe('api', function() { promise.then(function (data) { expect(data).toEqual(null); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets attachments', function(done) { if (isNodeJS()) { // The PDF file used is a linked test-case. @@ -709,9 +661,7 @@ describe('api', function() { expect(attachment.content.length).toEqual(30098); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets javascript', function(done) { @@ -719,9 +669,7 @@ describe('api', function() { promise.then(function (data) { expect(data).toEqual(null); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); // Keep this in sync with the pattern in viewer.js. The pattern is used to // detect whether or not to automatically start printing. @@ -737,9 +685,7 @@ describe('api', function() { expect(data).toEqual(['print({});']); expect(data[0]).toMatch(viewerPrintRegExp); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets javascript with printing instructions (JS action)', function(done) { @@ -753,9 +699,7 @@ describe('api', function() { ['this.print({bUI:true,bSilent:false,bShrinkToFit:true});']); expect(data[0]).toMatch(viewerPrintRegExp); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets non-existent outline', function(done) { var loadingTask = getDocument(buildGetDocumentParams('tracemonkey.pdf')); @@ -767,9 +711,7 @@ describe('api', function() { expect(outline).toEqual(null); loadingTask.destroy().then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets outline', function(done) { var promise = doc.getOutline(); @@ -792,9 +734,7 @@ describe('api', function() { expect(outlineItem.items.length).toEqual(1); expect(outlineItem.items[0].title).toEqual('Paragraph 1.1'); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets outline containing a url', function(done) { var loadingTask = getDocument(buildGetDocumentParams('issue3214.pdf')); @@ -819,9 +759,7 @@ describe('api', function() { loadingTask.destroy().then(done); }); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets non-existent permissions', function(done) { @@ -829,9 +767,7 @@ describe('api', function() { expect(permissions).toEqual(null); done(); - }).catch(function(reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets permissions', function (done) { @@ -875,9 +811,7 @@ describe('api', function() { loadingTask1.destroy(), loadingTask2.destroy(), ]).then(done); - }).catch(function(reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets metadata', function(done) { @@ -895,9 +829,7 @@ describe('api', function() { expect(contentDispositionFilename).toEqual(null); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets data', function(done) { var promise = doc.getData(); @@ -905,27 +837,21 @@ describe('api', function() { expect(data instanceof Uint8Array).toEqual(true); expect(data.length).toEqual(basicApiFileLength); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets download info', function(done) { var promise = doc.getDownloadInfo(); promise.then(function (data) { expect(data).toEqual({ length: basicApiFileLength, }); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets document stats', function(done) { var promise = doc.getStats(); promise.then(function (stats) { expect(stats).toEqual({ streamTypes: [], fontTypes: [], }); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('checks that fingerprints are unique', function(done) { @@ -950,9 +876,7 @@ describe('api', function() { loadingTask1.destroy(), loadingTask2.destroy() ]).then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); describe('Cross-origin', function() { @@ -1046,9 +970,7 @@ describe('api', function() { page = data; done(); }); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); afterAll(function(done) { @@ -1108,9 +1030,7 @@ describe('api', function() { Promise.all([defaultPromise, displayPromise, printPromise]).then( function () { done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets annotations containing relative URLs (bug 766086)', @@ -1170,9 +1090,7 @@ describe('api', function() { docBaseUrlLoadingTask.destroy(), invalidDocBaseUrlLoadingTask.destroy() ]).then(done); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets text content', function (done) { @@ -1194,9 +1112,7 @@ describe('api', function() { // A simple check that ensures the two `textContent` object match. expect(JSON.stringify(data[0])).toEqual(JSON.stringify(data[1])); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets operator list', function(done) { var promise = page.getOperatorList(); @@ -1205,9 +1121,7 @@ describe('api', function() { expect(!!oplist.argsArray).toEqual(true); expect(oplist.lastChunk).toEqual(true); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets operatorList with JPEG image (issue 4888)', function(done) { let loadingTask = getDocument(buildGetDocumentParams('cmykjpeg.pdf')); @@ -1224,9 +1138,7 @@ describe('api', function() { done(); }); }); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets document stats after parsing page', function(done) { var promise = page.getOperatorList().then(function () { @@ -1242,9 +1154,7 @@ describe('api', function() { expect(stats).toEqual({ streamTypes: expectedStreamTypes, fontTypes: expectedFontTypes, }); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('gets page stats after parsing page, without `pdfBug` set', @@ -1478,9 +1388,7 @@ describe('api', function() { }); promiseDone.then(function() { done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); }); describe('PDFDataRangeTransport', function () { @@ -1537,9 +1445,7 @@ describe('api', function() { expect(page.rotate).toEqual(0); expect(fetches).toBeGreaterThan(2); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); it('should fetch document info and page using range and streaming', function (done) { @@ -1578,9 +1484,7 @@ describe('api', function() { expect(page.rotate).toEqual(0); expect(fetches).toEqual(1); done(); - }).catch(function (reason) { - done.fail(reason); - }); + }).catch(done.fail); }); }); });