[Pkg-javascript-commits] [pdf.js] 31/414: Change test-case `issue4914` to an `annotations` test

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository pdf.js.

commit ae4ec2e347e0100bd9ce6623d4ba9f549ad3336a
Author: Jonas Jenwald <jonas.jenwald at gmail.com>
Date:   Mon Dec 21 15:15:26 2015 +0100

    Change test-case `issue4914` to an `annotations` test
    
    This PDF file (see issue 4914) originally regressed in PR 4318, and was subsequently fixed in PR 4915.
    
    I added the PDF file as a (linked) test-case in PR 6481, in an effort to prevent regressions. Since we at that time didn't have the necessary framework in place, in order to correctly test annotations, this almost regressed *again* in PR https://github.com/mozilla/pdf.js/pull/6672#issuecomment-158689392.
    
    In that PDF file, some of the annotations are both printable and hidden, and should definitely *not* be visible on normal display. Hence this patch, which adds the `annotations` flag to the manifest in order to ensure that those annotations won't be rendered when `intent === 'display'`.
---
 test/test_manifest.json | 1 +
 1 file changed, 1 insertion(+)

diff --git a/test/test_manifest.json b/test/test_manifest.json
index 32b74c9..0538c17 100644
--- a/test/test_manifest.json
+++ b/test/test_manifest.json
@@ -498,6 +498,7 @@
        "link": true,
        "lastPage": 1,
        "type": "eq",
+       "annotations": true,
        "about": "PDF with annotations, some of which have the Hidden flag set."
     },
     {  "id": "issue4665-text",

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/pdf.js.git