C#单元测试验证Activator即时演示的类上的非抽象方法

问题描述 投票:0回答:2

首先,让我向您介绍我的项目。

我们正在开发一个用户可以使用程序的应用程序。作为程序,我指的是保密使用的指令列表。

有不同类型的程序都继承自程序抽象基类。

由于用户可以创建不同类型的程序,我们开发了一个ProgramManager,可以按类型实例化任何类型的程序。我们不需要实例化抽象类,而是所有具体的类(并且它可以工作)但是由于具体的程序具有相同的方法(AddNewChannel,Save,...),我们像程序一样处理它们。

这是一个代码示例:

public Program CreateProgram(Type type)
    {
        Program program = Activator.CreateInstance(type) as Program;
        program.Directory = ProgramsPath;

        int nbChannels = 2; //not 2 but a long line searching the correct number where it is.

        for (int i = 1; i <= nbChannels; i++)
        {
            program.AddNewChannel(i);
        }

        program.Save();
        return program;
    }

我现在要做的就是测试这个函数,我不想复制我已经为不同的Program类创建的unitTests。

举个例子,这是我的一个测试函数(对于Save方法),它的init。我将需要测试的类型存储在xml文件中。

    [TestInitialize]
    public void TestInitialize()
    {
        if (!TestContext.TestName.StartsWith("GetKnownTypes"))
            type = UnitTestsInitialization.applicationHMIAssembly.GetType((string)TestContext.DataRow["Data"]);
    }


    [TestMethod]
    [DataSource("Microsoft.VisualStudio.TestTools.DataSource.XML",
               "|DataDirectory|\\" + DATA_FILE, "Row",
                DataAccessMethod.Sequential)]
    public void SavedProgramCreatesFile()
    {
        Program program = Activator.CreateInstance(type) as Program;
        program.Name = "SavedProgramCreatesFile";
        program.Directory = DIRECTORY;

        program.Save();

        string savedProgramFileName = program.GetFilePath();

        bool result = File.Exists(savedProgramFileName);

        Assert.IsTrue(result);
    }

我所有具体的程序类都已经过单独测试。

因此,我想测试是否调用以下方法program.AddNewChannelprogram.Save

我看了一下Moq,但第一个问题是方法Save不是抽象的。

此外,使用Activator不允许我制作Mock<Program>

我在单元测试中尝试了以下内容,以尝试实例化mock并将其用作程序:

    [TestMethod]
    [DataSource("Microsoft.VisualStudio.TestTools.DataSource.XML",
               "|DataDirectory|\\" + DATA_FILE, "Row",
                DataAccessMethod.Sequential)]
    public void CreateProgram_CallsProgramSaveMethod()
    {
        Mock<Program> mock = new Mock<Program>();
        mock.Setup(p => p.AddNewChannel(It.IsAny<int>()));

        Program program = pm.CreateProgram(mock.Object.GetType());
        mock.Verify(p => p.Save());
        mock.Verify(p => p.GetFilePath(It.IsAny<string>()));
        mock.Verify(p => p.AddNewChannel(It.IsAny<int>()), Times.Exactly(ProgramManager.NB_MACHINE_CHANNELS));
        Assert.IsNotNull(program);
        program.DeleteFile();
    }

这个问题的灵感来源于:How to mock An Abstract Base Class

它一直工作,直到它到达for循环中的program.AddNewChannel(i);行。错误如下:

System.NotImplementedException:'这是一个DynamicProxy2错误:拦截器试图'继续'方法'Void AddNewChannel(Int32)'是抽象的。在调用抽象方法时,没有实现“继续”,拦截器有责任模仿实现(设置返回值,输出参数等)'

似乎设置不起作用,但我可能理解为什么。 (我尝试实例化一个没有实现验证方法的Proxy子类型)

我还试图在我的程序类上使用一个代理,它将实现一个包含我需要的方法的接口,但这里的问题又是激活器。

任何人都可以建议我测试这些方法调用的方法吗? (即使我需要改变我的方法CreateProgram

我看了一下:How to mock non virtual methods?,但我不确定这是否适用于我的问题。

我使用MS测试进行单元测试。

注意

其他一切都很好。我的所有其他测试都没有遇到麻烦,我的代码似乎工作(手动测试)。

提前致谢。

c# unit-testing abstract activator
2个回答
1
投票

Activator周围创建一个包装器接口和类,然后将类型传递给:

public interface IActivatorWrapper
{
    object CreateInstance(Type type);
}

public class ActivatorWrapper : IActivatorWrapper
{
    public object CreateInstance(Type type)
    {
        return Activator.CreateInstance(type);
    }
}

直接用它代替Activator,然后模拟IActivatorWrapper返回你想要的任何模拟对象。


帮助解决问题的另一个想法是在你的抽象IProgram类中添加一个Program接口,然后使用它来引用你的具体Program实例。如果您想要编写具有不同基类的具体Program,这也可能对您有所帮助。


2
投票

问题的根本原因是您使用类型作为参数,然后使用该参数创建此类型的实例。但是,您传递的是抽象类的类型,它特别不用于实例化。您需要直接使用具体类。


因此,我想测试是否调用了以下方法program.AddNewChannel和program.Save。

这不足以作为测试。您想测试这些方法是否按预期工作,而不仅仅是调用它们然后假设它们有效。

您所描述的是(非常基本的)集成测试,而不是单元测试。


我不想复制我已经为不同的Program类创建的unitTests

这是一个非常危险的决定。单元测试背后的部分想法是为不同的(具体)对象创建单独的测试。测试需要尽可能地隔离。您正在尝试重用测试逻辑,这是一件好事,但它需要以不损害您的测试隔离的方式完成。

但是有一些方法可以做到这一点而不会影响您的测试隔离。我只有NUnit的测试经验,但我认为类似的方法也适用于其他框架。

假设如下:

public abstract class Program 
{
    public bool BaseMethod() {}
}

public class Foo : Program 
{
    public bool CustomFooMethod() {}
}

public class Bar : Program 
{
    public bool CustomBarMethod() {}
}

创建一个抽象类测试方法:

[TestFixture]
[Ignore]
public class ProgramTests
{
     public virtual Program GetConcrete()
     {
         throw new NotImplementedException();
     }

    [Test]
    public void BaseMethodTestReturnsFalse()
    {
        var result = GetConcrete().BaseMethod();

        Assert.IsFalse(result);
    }
}

[Ignore]确保ProgramTests类不会自行测试。

然后从这个类继承,测试具体类:

[TestFixture]
public class FooTests
{
    private readonly Foo Foo;

    public FooTests()  
    {
        this.Foo = new Foo();
    }

    public overrides Program GetConcrete()
    {
        return this.Foo;
    }

    [Test]
    public void CustomFooMethodTestReturnsFalse()
    {
        var result = this.Foo.CustomFooMethod();

        Assert.IsFalse(result);
    }
}

BarTests同样实施。

NUnit(可能是其他测试框架)将发现所有继承的测试,并将为派生类运行这些测试。因此,源自ProgramTests的每个类都将包括BaseMethodTestReturnsTrue测试。

这样,您的基类测试可以重复使用,但每个具体类仍将单独测试。这样可以保持测试分离,同时还可以防止必须为每个具体类复制/粘贴测试逻辑。


我也注意到了这一点:

Mock<Program> mock = new Mock<Program>();
mock.Setup(p => p.AddNewChannel(It.IsAny<int>()));
Program program = pm.CreateProgram(mock.Object.GetType());

我不明白这段代码的目的。与简单的做法有什么不同:

Program program = pm.CreateProgram(typeof(Program).GetType());

据我所知,模拟和它的设置都是无关紧要的,因为你只是看它的类型,然后让CreateProgram()为你创建一个新的对象。

其次,这回到我测试具体类的例子,你不应该直接测试Program,你应该测试你的派生程序类(FooBar)。

这是问题的根本原因。您正在使用类型作为参数,然后使用该参数创建此类型的实例。但是,您传递的是抽象类的类型,它特别不用于实例化。您需要直接使用具体类。

© www.soinside.com 2019 - 2024. All rights reserved.