Short story: writing a unit test for an image processing function, I had the following key parameters for each test:
enum Algorithm { A, B, C };
enum ImageFormat { Gray, Color };
enum ImageSize { Small, Medium, Large };
I wrote a core test function that worked on just one combination of the 3 parameters. E.g.
void Test(
Algorithm algorithm,
ImageFormat imageFormat,
ImageSize imageSize)
{
Console.WriteLine($"Testing algorithm {algorithm} for " +
{imageFormat} image with {imageSize} size");
// the test...
}
Next, I wrote a utility to make iterating the values of an enumeration a little easier.. still not sure why this isn’t part of .Net yet:
class EnumHelper<T>
{
static public T[] Values
{
get { return (T[])Enum.GetValues(typeof(T)); }
}
}
Then, I wrote the nested loops that built each combination and sent them for testing:
void TestAllV1()
{
foreach (var algorithm in EnumHelper<Algorithm>.Values)
{
foreach (var imageFormat in EnumHelper<ImageFormat>.Values)
{
foreach (var imageSize in EnumHelper<ImageSize>.Values)
{
Test(algorithm, imageFormat, imageSize);
}
}
}
}
Now there’s nothing really wrong with the above, but it looked like something that should be able to be written more simply. So I came up with this:
void TestAllV2()
{
var tests =
from algorithm in EnumHelper<Algorithm>.Values
from imageFormat in EnumHelper<ImageFormat>.Values
from imageSize in EnumHelper<ImageSize>.Values
select (algorithm, imageFormat, imageSize);
foreach (var test in tests)
{
Test(test.algorithm, test.imageFormat, test.imageSize);
}
}
The use of the from clause seems better, mainly due to the reduced level of nesting. The Visual Studio 2019 code analysis metrics are interesting:
| Member | MI | CycC | ClsC |
| TestAllV1() | 76 | 4 | 7 |
| TestAllV2() | 69 | 2 | 18 |
Where:
- MI: Maintainability Index
- CycC: Cyclomatic Complexity
- ClsC: Class coupling
So the foreach approach is (allegedly!) more maintainable, while the from clause method has a lower cyclomatic complexity. This latter metric reinforces the idea that this is slightly simpler than the foreach technique.
It’s also quite easy to add specific filtering inside the tests generator. For example, to quickly stop testing the B algorithm:
var tests =
from algorithm in EnumHelper<Algorithm>.Values
where algorithm != Algorithm.B
from imageFormat in EnumHelper<ImageFormat>.Values
from imageSize in EnumHelper<ImageSize>.Values
select (algorithm, imageFormat, imageSize);
Food for thought 🙂
Edit: actually found another way to do this using the Linq SelectMany method, but I’m not keen on this:
void TestAllV3()
{
var tests =
EnumHelper<Algorithm>.Values.SelectMany(
algorithm => EnumHelper<ImageFormat>.Values.SelectMany(
imageFormat => EnumHelper<ImageSize>.Values.Select(
imageSize => (algorithm, imageFormat, imageSize))));
foreach(var test in tests)
{
Test(test.algorithm, test.imageFormat, test.imageSize);
}
}
