Skip to content

Commit

Permalink
Feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
jerone committed Sep 9, 2024
1 parent ac8e4d6 commit 48421d5
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,24 +16,24 @@ public void CheatSheetComponent_HasColumns()
const int expectedCount = 3;

// Act.
var elements = CheatSheetComponent.Columns;
var data = CheatSheetComponent.Columns;

// Assert.
Assert.Equal(expectedCount, elements.Length);
Assert.Equal(expectedCount, data.Length);
}

[Fact]
[Description("Verify that the cheat sheet component has rows.")]
public async Task CheatSheetComponent_HasRows()
{
// Arrange.
var expectedCount = 18;
const int expectedCount = 18;

// Act.
var elements = CheatSheetComponent.Rows;
var data = CheatSheetComponent.Rows;

// Assert.
Assert.Equal(expectedCount, elements.Length);
await Verify(elements);
Assert.Equal(expectedCount, data.Length);
await Verify(data);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,10 @@ public void Read_WithNoStartToken_ThrowsJsonException()
var options = new JsonSerializerOptions();

// Act.
[ExcludeFromCodeCoverage(Justification = "Used in unit-tests only.")]
[ExcludeFromCodeCoverage(Justification = "Used in unit-test only.")]
void Result()
{
var reader = new Utf8JsonReader(Array.Empty<byte>());
var reader = new Utf8JsonReader();
converter.Read(ref reader, typeof(List<string>), options);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ public class SettingsTests
public async Task Settings_HasHttpAgreementClosed()
{
// Act.
var defaultValue = Settings.HttpAgreementClosed;
var setting = Settings.HttpAgreementClosed;

// Assert.
await Verify(defaultValue);
await Verify(setting);
}
}

0 comments on commit 48421d5

Please sign in to comment.