单元测试是否应该依赖于应用程序定义的常量?

Mat*_*hew 6 language-agnostic testing unit-testing constants

考虑以下伪代码.它旨在确定成绩是否为及格分数.

class Student:
    int grade

    boolean IsStudentPassing():
        return grade >= MIN_PASSING_GRADE


...

// In another file
constant int MIN_PASSING_GRADE = 70
Run Code Online (Sandbox Code Playgroud)

如果我们正在编写单元测试IsStudentPassing,我们可以使用常量值:

ensure that IsStudentPassing is false when grade is MIN_PASSING_GRADE - 1
ensure that IsStudentPassing is true when grade is MIN_PASSING_GRADE
Run Code Online (Sandbox Code Playgroud)

或者,我们可以亲自挑选价值观:

ensure that IsStudentPassing is false when grade is 69
ensure that IsStudentPassing is true when grade is 70
Run Code Online (Sandbox Code Playgroud)

对于第二种方法,如果MIN_PASSING_GRADE更改,则必须重新编写测试.第一种方法更灵活,但依赖于MIN_PASSING_GRADE具有正确的价值.

我不完全确定选择哪种方法,并根据具体情况进行选择.一方面,确保MIN_PASSING_GRADE理智是应该通过不同的测试来处理.另一方面,我担心所谓的"单元"测试会触及代码库中太多其他地方.

这是一个人为的例子,但类似的情况经常出现在真实的程序中.解决这些问题的最佳方法是什么?

dar*_*rch 1

根据偏好,您可以通过自己的设计注入“常量”值,这样您的单元测试就可以与实际上构成及格分数的变幻莫测的因素隔离开来。做到这一点的容易程度因编程语言而异。考虑一下这种语言的代码,使其变得简单:

use MooseX::Declare;
class Student {
    has grade => (
        is => 'ro', isa => 'Num', required => 1,
    );

    method min_passing_grade {
        return MIN_PASSING_GRADE;
    )

    method is_student_passing () {
        return $self->grade >= $self->min_passing_grade
    }
}

class t::Student {
    use Test::Sweet;
    use Test::MockObject::Extends;

    test correctly_determines_student_is_passing () {
        my $student = $self->_make_student($self->_passing_grade);

        ok($student->is_student_passing);
    }

    method _make_student (Num $grade) {
        my $student = Test::MockObject::Extends->new(
            $student->new(grade => $grade)
        );
        # Here's the important line:
        $student->set_always(
            min_passing_grade => $self->_passing_grade
        );
        return $student;
    }

    method _passing_grade () { 40 }

    test correctly_determines_student_is_failing () {
        my $student = $self->_make_student($self->_passing_grade - 1);

        ok(not $student->is_student_passing);
    }
}
Run Code Online (Sandbox Code Playgroud)

Student::min_passing_grade现在,这就是 Perl,它使猴子修补变得非常简单(上面的“重要行”替换了运行时的实现)。您还可以将 questino 中的值作为默认为常量的属性,甚至提供常量文件的特殊版本以供单元测试使用。

如果没有真正强大的性能要求,我将选择上述方法,而不是让该值成为一个真正的常数。只有当我无法找到一种方法来从单元测试中注入我想要的值时,我才会使用通用定义的常量。我认为在几乎任何情况下你都不应该做的是重复这个测试中的常量,这毕竟是为了确保的逻辑Student是正确的。