From e01d77b93a8a52ed5c2bc642cac29973fd578e76 Mon Sep 17 00:00:00 2001 From: Eliot Jones Date: Thu, 5 Dec 2019 13:56:12 +0000 Subject: [PATCH] add negative test case and make tests non-lenient --- .../PigReproductionPowerpointTests.cs | 8 ++++---- .../SinglePageSimpleOpenOfficeTests.cs | 17 +++++++++++++---- 2 files changed, 17 insertions(+), 8 deletions(-) diff --git a/src/UglyToad.PdfPig.Tests/Integration/PigReproductionPowerpointTests.cs b/src/UglyToad.PdfPig.Tests/Integration/PigReproductionPowerpointTests.cs index c8590ef5..2ecfc37e 100644 --- a/src/UglyToad.PdfPig.Tests/Integration/PigReproductionPowerpointTests.cs +++ b/src/UglyToad.PdfPig.Tests/Integration/PigReproductionPowerpointTests.cs @@ -13,7 +13,7 @@ [Fact] public void CanReadContent() { - using (var document = PdfDocument.Open(GetFilename())) + using (var document = PdfDocument.Open(GetFilename(), ParsingOptions.LenientParsingOff)) { var page = document.GetPage(1); @@ -24,7 +24,7 @@ [Fact] public void HasCorrectNumberOfPages() { - using (var document = PdfDocument.Open(GetFilename())) + using (var document = PdfDocument.Open(GetFilename(), ParsingOptions.LenientParsingOff)) { Assert.Equal(35, document.NumberOfPages); } @@ -33,7 +33,7 @@ [Fact] public void CanReadAllPages() { - using (var document = PdfDocument.Open(GetFilename())) + using (var document = PdfDocument.Open(GetFilename(), ParsingOptions.LenientParsingOff)) { for (var i = 0; i < document.NumberOfPages; i++) { @@ -45,7 +45,7 @@ [Fact] public void CanGetBookmarks() { - using (var document = PdfDocument.Open(GetFilename())) + using (var document = PdfDocument.Open(GetFilename(), ParsingOptions.LenientParsingOff)) { var foundBookmarks = document.TryGetBookmarks(out var bookmarks); Assert.True(foundBookmarks); diff --git a/src/UglyToad.PdfPig.Tests/Integration/SinglePageSimpleOpenOfficeTests.cs b/src/UglyToad.PdfPig.Tests/Integration/SinglePageSimpleOpenOfficeTests.cs index 5ed1a82f..2e99a290 100644 --- a/src/UglyToad.PdfPig.Tests/Integration/SinglePageSimpleOpenOfficeTests.cs +++ b/src/UglyToad.PdfPig.Tests/Integration/SinglePageSimpleOpenOfficeTests.cs @@ -17,7 +17,7 @@ { var file = GetFilename(); - using (var document = PdfDocument.Open(File.ReadAllBytes(file))) + using (var document = PdfDocument.Open(File.ReadAllBytes(file), ParsingOptions.LenientParsingOff)) { Assert.Equal(1, document.NumberOfPages); } @@ -26,7 +26,7 @@ [Fact] public void HasCorrectPageSize() { - using (var document = PdfDocument.Open(GetFilename())) + using (var document = PdfDocument.Open(GetFilename(), ParsingOptions.LenientParsingOff)) { var page = document.GetPage(1); @@ -37,7 +37,7 @@ [Fact] public void HasCorrectLetterBoundingBoxes() { - using (var document = PdfDocument.Open(GetFilename())) + using (var document = PdfDocument.Open(GetFilename(), ParsingOptions.LenientParsingOff)) { var page = document.GetPage(1); @@ -72,7 +72,7 @@ [Fact] public void GetsCorrectPageTextIgnoringHiddenCharacters() { - using (var document = PdfDocument.Open(GetFilename())) + using (var document = PdfDocument.Open(GetFilename(), ParsingOptions.LenientParsingOff)) { var page = document.GetPage(1); @@ -81,5 +81,14 @@ Assert.Equal("I am a simple pdf.", text); } } + + [Fact] + public void TryGetBookmarksFalse() + { + using (var document = PdfDocument.Open(GetFilename(), ParsingOptions.LenientParsingOff)) + { + Assert.False(document.TryGetBookmarks(out _)); + } + } } }