Skip to content

Commit

Permalink
No longer tests Act phases for throwing if there's something more int…
Browse files Browse the repository at this point in the history
…eresting to test
  • Loading branch information
svengeance committed Dec 24, 2023
1 parent 9873e3b commit e061bcb
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions tests/QPdfSharp.Tests/QPdfTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,9 @@ public void Can_read_qpdf_version()
{
// Arrange
// Act
var getVersion = () => QPdf.Version;
var version = QPdf.Version;

// Assert
var version = getVersion.Should().NotThrow().Subject;
Version.TryParse(version, out _).Should().BeTrue();
}

Expand Down Expand Up @@ -77,11 +76,10 @@ public void Can_write_pdf_bytes()
// Arrange
// Act
using var qpdf = new QPdf(TestAssets.Grug);
// ReSharper disable once AccessToDisposedClosure
var getPdfBytes = () => qpdf.WriteBytes().ToArray();
var getPdfBytes = qpdf.WriteBytes().ToArray();

// Assert
getPdfBytes.Should().NotThrow().Subject.Should().NotBeEmpty();
getPdfBytes.Should().NotBeEmpty();
}

[Fact]
Expand All @@ -90,10 +88,9 @@ public void Can_write_pdf_stream()
// Arrange
// Act
using var qpdf = new QPdf(TestAssets.Grug);
var getPdfStream = qpdf.WriteStream;
var pdfStream = qpdf.WriteStream();

// Assert
var pdfStream = getPdfStream.Should().NotThrow().Subject;
pdfStream.Should().NotHaveLength(0);
pdfStream.Should().BeReadOnly();
}
Expand Down

0 comments on commit e061bcb

Please sign in to comment.