Skip to content

Commit 69a71b3

Browse files
committed
Refactoring of Linq where sub-query and Any() tests
1 parent e0d4d45 commit 69a71b3

File tree

3 files changed

+94
-122
lines changed

3 files changed

+94
-122
lines changed

src/NHibernate.Test/Linq/ByMethod/AnyTests.cs

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -10,21 +10,22 @@ public class AnyTests : LinqTestCase
1010
public void AnySublist()
1111
{
1212
var orders = db.Orders.Where(o => o.OrderLines.Any(ol => ol.Quantity == 5)).ToList();
13-
Assert.AreEqual(61, orders.Count);
13+
Assert.That(orders.Count, Is.EqualTo(61));
1414

1515
orders = db.Orders.Where(o => o.OrderLines.Any(ol => ol.Order == null)).ToList();
16-
Assert.AreEqual(0, orders.Count);
16+
Assert.That(orders.Count, Is.EqualTo(0));
1717
}
1818

1919
[Test]
2020
public void NestedAny()
2121
{
2222
var test = (from c in db.Customers
23-
where c.ContactName == "Bob" &&
24-
(c.CompanyName == "NormalooCorp" ||
25-
c.Orders.Any(o => o.OrderLines.Any(ol => ol.Discount < 20 && ol.Discount >= 10)))
26-
select c).ToList();
27-
Assert.AreEqual(0, test.Count);
23+
where c.ContactName == "Bob" &&
24+
(c.CompanyName == "NormalooCorp" ||
25+
c.Orders.Any(o => o.OrderLines.Any(ol => ol.Discount < 20 && ol.Discount >= 10)))
26+
select c).ToList();
27+
28+
Assert.That(test.Count, Is.EqualTo(0));
2829
}
2930

3031
[Test]
@@ -33,17 +34,16 @@ public void ManyToManyAny()
3334
var test = db.Orders.Where(o => o.Employee.FirstName == "test");
3435
var result = test.Where(o => o.Employee.Territories.Any(t => t.Description == "test")).ToList();
3536

36-
Assert.AreEqual(0, result.Count);
37+
Assert.That(result.Count, Is.EqualTo(0));
3738
}
3839

39-
[Test]
40+
[Test(Description = "NH-2654")]
4041
public void AnyWithCount()
4142
{
42-
//NH-2654
4343
var result = db.Orders
4444
.Any(p => p.OrderLines.Count == 0);
4545

46-
Assert.IsFalse(result);
46+
Assert.That(result, Is.False);
4747
}
4848
}
4949
}

src/NHibernate.Test/Linq/ByMethod/AssertOrderedBy.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,22 +9,22 @@ public static class AssertOrderedBy
99
public static void Ascending<T>(IList<T> result, Func<T, IComparable> keySelector)
1010
{
1111
if (result == null) throw new ArgumentNullException("result");
12-
for (int i = 0; i < result.Count - 1; i++)
12+
for (var i = 0; i < result.Count - 1; i++)
1313
{
1414
var first = result[i];
1515
var second = result[i + 1];
16-
Assert.LessOrEqual(keySelector(first), keySelector(second));
16+
Assert.That(keySelector(first), Is.LessThanOrEqualTo(keySelector(second)));
1717
}
1818
}
1919

2020
public static void Descending<T>(IList<T> result, Func<T, IComparable> keySelector)
2121
{
2222
if (result == null) throw new ArgumentNullException("result");
23-
for (int i = 0; i < result.Count - 1; i++)
23+
for (var i = 0; i < result.Count - 1; i++)
2424
{
2525
var first = result[i];
2626
var second = result[i + 1];
27-
Assert.GreaterOrEqual(keySelector(first), keySelector(second));
27+
Assert.That(keySelector(first), Is.GreaterThanOrEqualTo(keySelector(second)));
2828
}
2929
}
3030
}

0 commit comments

Comments
 (0)