[backlinq commit] r270 - trunk/test/BackLINQ.Tests

[codesite...@google.com]

Author: azizatif
Date: Sun Jan 25 05:17:05 2009
New Revision: 270

Modified:
trunk/test/BackLINQ.Tests/EnumerableFixture.cs

Log:
Addressed BUGBUG items in GroupBy operator tests.

Modified: trunk/test/BackLINQ.Tests/EnumerableFixture.cs
==============================================================================
--- trunk/test/BackLINQ.Tests/EnumerableFixture.cs (original)
+++ trunk/test/BackLINQ.Tests/EnumerableFixture.cs Sun Jan 25 05:17:05 2009
@@ -684,14 +684,14 @@

var result = persons.GroupBy(
p => p.LastName,
- (key, group) => { // BUGBUG Use the key in
some way otherwise its untested!
- var ages = 0;
+ (key, group) => {
+ var total = 0;
foreach (var p in group)
- ages += p.Age;
- return ages;
+ total += p.Age;
+ return key + ":" + total;
});

- result.AssertEquals(43, 47);
+ result.AssertEquals("M\u00FCller:43", "Meier:47");
}

[Test]
@@ -720,16 +720,17 @@
{
var persons = Read(Person.CreatePersons());

- var result = persons.GroupBy(p => p.LastName, p => p.Age,
- (key, ages) => // BUGBUG Use the
key in some way otherwise its untested!
- {
- var total = 0;
- foreach (var age in ages)
- total += age;
- return total;
- });
-
- result.AssertEquals(43, 47);
+ var result = persons.GroupBy(
+ p => p.LastName,
+ p => p.Age,
+ (key, ages) => {
+ var total = 0;
+ foreach (var age in ages)
+ total += age;
+ return key + ":" + total;
+ });
+
+ result.AssertEquals("M\u00FCller:43", "Meier:47");
}

[Test]
@@ -737,17 +738,17 @@
{
var persons =
Read(Person.CreatePersonsWithNamesUsingMixedCase());

- var result = persons.GroupBy(p => p.LastName,
- (key, values) => // BUGBUG Use
the key in some way otherwise its untested!
- {
- var total = 0;
- foreach (var person in values)
- total += person.Age;
- return total;
- },
-
StringComparer.CurrentCultureIgnoreCase);
-
- result.AssertEquals(43, 47);
+ var result = persons.GroupBy(
+ p => p.LastName,
+ (key, values) => {
+ var total = 0;
+ foreach (var person in values)
+ total += person.Age;
+ return key + ":" + total;
+ },
+ StringComparer.CurrentCultureIgnoreCase);
+
+ result.AssertEquals("M\u00FCller:43", "Meier:47");
}

[Test]
@@ -755,17 +756,18 @@
{
var persons =
Read(Person.CreatePersonsWithNamesUsingMixedCase());

- var result = persons.GroupBy(p => p.LastName, p => p.Age,
- (key, ages) => // BUGBUG Use the
key in some way otherwise its untested!
- {
- var total = 0;
- foreach (var i in ages)
- total += i;
- return total;
- },
-
StringComparer.CurrentCultureIgnoreCase);
+ var result = persons.GroupBy(
+ p => p.LastName,
+ p => p.Age,
+ (key, ages) => {
+ var total = 0;
+ foreach (var age in ages)
+ total += age;
+ return key + ":" + total;
+ },
+ StringComparer.CurrentCultureIgnoreCase);

- result.AssertEquals(43, 47);
+ result.AssertEquals("M\u00FCller:43", "Meier:47");
}

class Pet