在单元测试时我应该重用封装的常量吗？

Question

所以，我正在编写一个函数的单元测试。技术细节并不重要，但为了清晰起见，我想分享一些代码。 这是功能：

export function getSalutations(lang: Salutation = Salutation.default): string[] {
    return SALUTATIONS[lang] || SALUTATIONS.default;
};

export const enum Salutation {
    de,
    es,
    default
};
const SALUTATIONS = {
    de: ['Hr.', 'Fr.'],
    es: ['Sr.', 'Sra.'],
    default: ['Mr.', 'Mrs.', 'Ms.']
};

单元测试的一个例子：

QUnit.module('getSalutations', (hooks) => {
    QUnit.test('null / default salutations', (assert) => {
        assert.ok(DW.Utils.isEqual(DW.Utils.getSalutations(null), ['Mr.', 'Mrs.', 'Ms.']));
    });
});

我不想像这样复制数据，以便测试方法。我想重复使用SALUTATIONS进行测试，因此没有冗余代码。问题是我无法使用它，因为它不是从它所在的范围导出的，所以我无法访问它。

所以我提出以下选择：

1. 我必须将代码复制为模拟数据。
2. 我必须公开封装代码只是为了单元测试

我对这两者都不满意，所以我希望有人可以告诉我什么是最佳解决方案。也许还有其他选择，我无法弄清楚？

谢谢！

Answer 1

出于主观原因，这个问题可能会被关闭，但无论如何我都会回答。

您的代码是抽象的地方。

您的测试是结核的地方。

这意味着如果您正在测试应用程序将默认编写Hello Mr Smith，而Hallo Herr Smith用于德语 - 您应该在测试中使用这些字符串，因为它们是预期输出的具体示例。

如果使用SALUTATIONS对象构造测试的字符串，我可以在不破坏任何测试的情况下执行此操作：

const SALUTATIONS = {
    de: ['Hr.', 'Fr.'],
    es: ['Sr.', 'Sra.'],
    default: ['Whatever.', 'I quit.', 'I hate my job.']
};

Answer 2

有些人可能认为这是一个基于意见的问题（正如@Fenton建议的那样），但我看到了你的困境，JS世界中的许多人都有这种困境。我的建议是重复测试中的值。您需要确定您正在测试的内容...而且您正在测试的是，这些值将从getSalutations方法返回。如果是这种情况，那么你可以测试三件事：

1. 一个数组实际上已返回，
2. 在该数组中至少有一个条目（或2或20），
3. 并且该数组中有特定值。

所以现在你必须决定你要测试哪一个，如果它是第三个（在这种情况下它实际上是三个），那么你应该重复测试中的值。如果有人添加了新值，那么您的测试应该更新，因为它可能是无效值。

只需2美分。