Hi,

I found a mismatch in the execution of DataRow tests between MsTestRunner and NCrunch.
In detail it seems like NCrunch simply ignores the last parameter of the DataRowAttribute if it is null, which leads to a failing test with the following message: "Test parameter count mismatch".

We are currently using MsTest.TestFramework & MsTest.TestFramework in version 1.2.0.

Reproduction:
The TestMethod takes two parameters.
First one is an enum value, the second one is a nullable enum value.

When we set the second parameter to null, the test fails, whereas MsTestRunner runs fine.
If we add a third dummy parameter (e.g. string) everything works fine again, independent of whether it is set to null or not.

If I had to take a guess, NCrunch interprets the null of the second parameter as DisplayName, which seems to be an optional parameter for DataRow, and discards it.

Kind regards,
Erik

Hi,

sorry for resurrecting this old thread.
But as it seems, this bug has even got worse with the latest update.

It looks like every test which looks like the sample below fails to run.
Independent of how many parameters the datarow has or in which order they are passed.

[TestMethod]
[DataRow("abc", CustomEnum.value)]
public void TestMethod(string text, CustomEnum? parameter)
{
...
}