Skip to content

Commit

Permalink
Fixes ribbon plot tests
Browse files Browse the repository at this point in the history
In c590e0c we bundled a default font with AP.exe and started using it by default in the ribbon plot tests. However I forgot to update the unit tests to deal with the new width output by the different font.

Fixes the tests

Work done for #196
  • Loading branch information
atruskie committed Mar 24, 2020
1 parent 310b38a commit 628d95c
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,8 @@ public async Task RendersRibbonPlotsCorrectly()
var plot3 = this.TestOutputDirectory.CombineFile($"{this.TestOutputDirectory.Name}__RibbonPlot_ENT-CVR-OSC.png");
Assert.That.FileExists(plot3);

const int Left = 223;
// label width + padding left + padding right
const int Left = 200 + 10 + 10;
const int Height = ((32 + 2) * 12) + 2;

// whole image that goes all through the data
Expand Down Expand Up @@ -176,7 +177,8 @@ public async Task RendersRibbonPlotsWithDifferentMidnightCorrectly()
var plot3 = this.TestOutputDirectory.CombineFile($"{this.TestOutputDirectory.Name}__RibbonPlot_ENT-CVR-OSC_Midnight=1200.png");
Assert.That.FileExists(plot3);

const int Left = 223;
// label width + padding left + padding right
const int Left = 200 + 10 + 10;
const int Height = ((32 + 2) * (12 + 1)) + 2;

// whole image that goes all through the data (but it is two rows longer because of midnight config)
Expand Down

0 comments on commit 628d95c

Please sign in to comment.