Làm cách nào để kiểm tra “không có ngoại lệ nào xảy ra” trong bài kiểm tra đơn vị MSTest của tôi?

Tôi đang viết một bài kiểm tra đơn vị cho một phương thức này trả về "void". Tôi muốn có một trường hợp mà bài kiểm tra vượt qua khi không có ngoại lệ nào được ném ra. Làm cách nào để viết điều đó trong C #?

Assert.IsTrue(????)

(Tôi đoán đây là cách tôi nên kiểm tra, nhưng những gì đi vào "???")

Tôi hy vọng câu hỏi của tôi đủ rõ ràng.

Bài kiểm tra đơn vị của bạn dù sao cũng sẽ thất bại nếu một ngoại lệ được đưa ra - bạn không cần phải đưa ra một xác nhận đặc biệt.

Đây là một trong số ít các tình huống mà bạn sẽ thấy các bài kiểm tra đơn vị không có xác nhận nào cả - bài kiểm tra sẽ hoàn toàn thất bại nếu một ngoại lệ được đưa ra.

Tuy nhiên, nếu bạn thực sự muốn viết một xác nhận cho điều này - có lẽ để có thể bắt được ngoại lệ và báo cáo "dự kiến ​​không có ngoại lệ nhưng đã nhận được điều này ...", bạn có thể làm điều này:

[Test]
public void TestNoExceptionIsThrownByMethodUnderTest()
{
    var myObject = new MyObject();

    try
    {
        myObject.MethodUnderTest();
    }
    catch (Exception ex)
    {
        Assert.Fail("Expected no exception, but got: " + ex.Message);
    }
}

(ở trên là một ví dụ cho NUnit, nhưng điều này cũng đúng cho MSTest)

Trong NUnit, bạn có thể sử dụng:

Assert.DoesNotThrow(<expression>); 

để khẳng định rằng mã của bạn không có ngoại lệ. Mặc dù thử nghiệm sẽ thất bại nếu một ngoại lệ được ném ra ngay cả khi không có Xác nhận xung quanh nó, giá trị của phương pháp này là sau đó bạn có thể phân biệt giữa các kỳ vọng chưa được đáp ứng và lỗi trong các thử nghiệm của mình và bạn có tùy chọn thêm thông báo tùy chỉnh sẽ được hiển thị trong đầu ra thử nghiệm của bạn. Một kết quả kiểm tra có từ ngữ tốt có thể giúp bạn xác định lỗi trong mã của bạn đã khiến kiểm tra không thành công.

Tôi nghĩ rằng việc thêm các bài kiểm tra để đảm bảo rằng mã của bạn không có ngoại lệ là hợp lệ; ví dụ, hãy tưởng tượng bạn đang xác thực đầu vào và cần chuyển đổi một chuỗi đến thành chuỗi dài. Có thể có những trường hợp chuỗi là null và điều này có thể chấp nhận được, vì vậy bạn muốn đảm bảo rằng việc chuyển đổi chuỗi không có ngoại lệ. Do đó, sẽ có mã để xử lý trường hợp này, và nếu bạn chưa viết một bài kiểm tra cho nó, bạn sẽ thiếu sự bao quát xung quanh một phần logic quan trọng.

Đừng kiểm tra rằng điều gì đó không xảy ra . Nó giống như đảm bảo rằng mã không bị phá vỡ . Điều đó ngụ ý rằng, tất cả chúng ta đều cố gắng tạo ra mã không vi phạm, ít lỗi. Bạn muốn viết các bài kiểm tra cho điều đó? Tại sao chỉ một phương pháp? Bạn không muốn tất cả các phương pháp của bạn được kiểm tra mà chúng không có một số ngoại lệ ? Theo con đường đó, bạn sẽ kết thúc với một thử nghiệm bổ sung, giả, không xác nhận cho mọi phương pháp trong cơ sở mã của bạn. Nó không mang lại giá trị gì.

Tất nhiên, nếu yêu cầu của bạn là xác minh phương thức không bắt được ngoại lệ , bạn sẽ kiểm tra điều đó (hoặc đảo ngược nó một chút; kiểm tra xem nó không ném những gì nó được cho là bắt).

Tuy nhiên, phương pháp / thực hành chung vẫn còn nguyên vẹn - bạn không viết thử nghiệm cho một số yêu cầu giả tạo / mơ hồ nằm ngoài phạm vi của mã được thử nghiệm (và thử nghiệm "nó hoạt động" hoặc "không ném" thường là một ví dụ về như vậy - đặc biệt là trong kịch bản khi trách nhiệm của phương pháp được biết rõ).

Nói một cách đơn giản - hãy tập trung vào những gì mã của bạn phải làm và kiểm tra điều đó.

Lớp người trợ giúp này đã gãi ngứa cho tôi bằng MSTest. Có lẽ nó cũng có thể làm xước của bạn.

[TestMethod]
public void ScheduleItsIneligibilityJob_HasValid_CronSchedule()
{
    // Arrange
    var factory = new StdSchedulerFactory();
    IScheduler scheduler = factory.GetScheduler();

    // Assert
    AssertEx.NoExceptionThrown<FormatException>(() =>
        // Act
        _service.ScheduleJob(scheduler)
    );
}

public sealed class AssertEx
{
    public static void NoExceptionThrown<T>(Action a) where T:Exception
    {
        try
        {
            a();
        }
        catch (T)
        {
            Assert.Fail("Expected no {0} to be thrown", typeof(T).Name);
        }
    }
}

Tôi thích nhìn thấy một Assert.Whateverđiểm ở cuối mỗi bài kiểm tra, chỉ để nhất quán ... nếu không có một bài kiểm tra, tôi thực sự có thể chắc chắn là không có một bài kiểm tra ở đó?

Đối với tôi, điều này đơn giản như đặt Assert.IsTrue(true);

Tôi biết tôi đã không vô tình đặt mã đó vào đó, và do đó tôi nên đủ tự tin khi đọc lướt qua rằng điều này đúng như dự định.

    [TestMethod]
    public void ProjectRejectsGappedVersioningByDefault() {

        var files = new List<ScriptFile>();
        files.Add(ScriptProjectTestMocks.GetVersion1to2());
        files.Add(ScriptProjectTestMocks.GetVersion3to4());

        Assert.Throws<ScriptProject.InvalidProjectFormatException>(() => {
            var sut = new ScriptProject(files);
        });

    }

    [TestMethod]
    public void ProjectAcceptsGappedVersionsExplicitly() {

        var files = new List<ScriptFile>();
        files.Add(ScriptProjectTestMocks.GetVersion1to2());
        files.Add(ScriptProjectTestMocks.GetVersion3to4());

        var sut = new ScriptProject(files, true);

        Assert.IsTrue(true);   // Assert.Pass() would be nicer... build it in if you like

    }

Bạn tôi, Tim đã nói với tôi về mong đợi . Tôi thực sự thích điều này b / c nó ngắn gọn hơn, ít mã hơn và rất rõ ràng rằng bạn đang kiểm tra một ngoại lệ.

[TestMethod()]
[ExpectedException(typeof(System.Exception))]
public void DivideTest()
{
    int numerator = 4;
    int denominator = 0;
    int actual = numerator / denominator;
}

Bạn có thể đọc thêm về nó ở đây: Sử dụng thuộc tính ngoại lệ mong đợi .