Improve the API unit-tests by checking that PDFPageProxy.render returns a RenderTask-instance

This is similar to existing unit-tests, which checks for `PDFDocumentProxy`- and `PDFPageProxy`-instances.
This commit is contained in:
Jonas Jenwald 2021-09-13 13:34:37 +02:00
parent fa7a607d33
commit d854352cd5
2 changed files with 18 additions and 1 deletions

View File

@ -3365,6 +3365,7 @@ export {
PDFDocumentProxy, PDFDocumentProxy,
PDFPageProxy, PDFPageProxy,
PDFWorker, PDFWorker,
RenderTask,
setPDFNetworkStreamFactory, setPDFNetworkStreamFactory,
version, version,
}; };

View File

@ -39,6 +39,7 @@ import {
PDFDocumentProxy, PDFDocumentProxy,
PDFPageProxy, PDFPageProxy,
PDFWorker, PDFWorker,
RenderTask,
} from "../../src/display/api.js"; } from "../../src/display/api.js";
import { import {
RenderingCancelledException, RenderingCancelledException,
@ -1874,11 +1875,14 @@ describe("api", function () {
viewport.width, viewport.width,
viewport.height viewport.height
); );
const renderTask = pdfPage.render({ const renderTask = pdfPage.render({
canvasContext: canvasAndCtx.context, canvasContext: canvasAndCtx.context,
canvasFactory: CanvasFactory, canvasFactory: CanvasFactory,
viewport, viewport,
}); });
expect(renderTask instanceof RenderTask).toEqual(true);
await renderTask.promise; await renderTask.promise;
const stats = pdfPage.stats; const stats = pdfPage.stats;
@ -1911,6 +1915,8 @@ describe("api", function () {
canvasFactory: CanvasFactory, canvasFactory: CanvasFactory,
viewport, viewport,
}); });
expect(renderTask instanceof RenderTask).toEqual(true);
renderTask.cancel(); renderTask.cancel();
try { try {
@ -1939,6 +1945,8 @@ describe("api", function () {
canvasFactory: CanvasFactory, canvasFactory: CanvasFactory,
viewport, viewport,
}); });
expect(renderTask instanceof RenderTask).toEqual(true);
renderTask.cancel(); renderTask.cancel();
try { try {
@ -1955,6 +1963,8 @@ describe("api", function () {
canvasFactory: CanvasFactory, canvasFactory: CanvasFactory,
viewport, viewport,
}); });
expect(reRenderTask instanceof RenderTask).toEqual(true);
await reRenderTask.promise; await reRenderTask.promise;
CanvasFactory.destroy(canvasAndCtx); CanvasFactory.destroy(canvasAndCtx);
@ -1976,12 +1986,15 @@ describe("api", function () {
viewport, viewport,
optionalContentConfigPromise, optionalContentConfigPromise,
}); });
expect(renderTask1 instanceof RenderTask).toEqual(true);
const renderTask2 = page.render({ const renderTask2 = page.render({
canvasContext: canvasAndCtx.context, canvasContext: canvasAndCtx.context,
canvasFactory: CanvasFactory, canvasFactory: CanvasFactory,
viewport, viewport,
optionalContentConfigPromise, optionalContentConfigPromise,
}); });
expect(renderTask2 instanceof RenderTask).toEqual(true);
await Promise.all([ await Promise.all([
renderTask1.promise, renderTask1.promise,
@ -2014,8 +2027,9 @@ describe("api", function () {
canvasFactory: CanvasFactory, canvasFactory: CanvasFactory,
viewport, viewport,
}); });
await renderTask.promise; expect(renderTask instanceof RenderTask).toEqual(true);
await renderTask.promise;
await pdfDoc.cleanup(); await pdfDoc.cleanup();
expect(true).toEqual(true); expect(true).toEqual(true);
@ -2042,6 +2056,8 @@ describe("api", function () {
canvasFactory: CanvasFactory, canvasFactory: CanvasFactory,
viewport, viewport,
}); });
expect(renderTask instanceof RenderTask).toEqual(true);
// Ensure that clean-up runs during rendering. // Ensure that clean-up runs during rendering.
renderTask.onContinue = function (cont) { renderTask.onContinue = function (cont) {
waitSome(cont); waitSome(cont);