agoda-com / agoda-com/Shouldly.FromAssert

Issues after Big rework

Offen
#10 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug
Vorherrschende Sprache
C#
Sterne
11
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

```csharp
Assert.AreEqual(isRateValid, actualData.RateAdjustments != null && actualData.RateAdjustments.Any());
// became
actualData.RateAdjustments != null && actualData.RateAdjustments.Any().ShouldBe(isRateValid);
// should be
(actualData.RateAdjustments != null && actualData.RateAdjustments.Any()).ShouldBe(isRateValid);
```

```csharp
Assert.IsTrue(propertyId == 0 ? actual.Items.Count == 0 : actual.Items.Count > 0);
//became
propertyId == 0 ? actual.Items.Count == 0 : actual.Items.Count > 0.ShouldBeTrue();
// should be
(propertyId == 0 ? actual.Items.Count == 0 : actual.Items.Count > 0).ShouldBeTrue();
```

HOWEVER this one worked

```csharp
Assert.IsTrue(actual.Items.Count > 0);
// became
(actual.Items.Count > 0).ShouldBeTrue();
```

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.