从https://github.com/sebastianbergmann/phpunit/pull/3335#issuecomment -428570649 移到这里。
也替换#3180。
从assertEquals()
等中删除$maxDepth
参数将破坏所有使用assertEquals()
和$maxDepth
之后的可选参数的测试。
清理assertEquals()
{hopefully|possibly} 导致更少问题的唯一方法是更彻底地清理,不仅删除$maxDepth
,还删除它之后的所有可选参数。 这也不是很吸引人。
所有这些可选参数都不应该被添加,它们构成了一个糟糕的 API。 不过,我不确定通过删除这些可选参数并添加更具体的断言方法来清理assertEquals()
等的好处是否超过了破坏兼容性的痛苦。
我有时使用的唯一参数是忽略案例之一。 没用过其他的。
迁移到一个专门的断言对我来说似乎是一个合理的升级路径。
被#3341 取代。
最有用的评论
从
assertEquals()
等中删除$maxDepth
参数将破坏所有使用assertEquals()
和$maxDepth
之后的可选参数的测试。清理
assertEquals()
{hopefully|possibly} 导致更少问题的唯一方法是更彻底地清理,不仅删除$maxDepth
,还删除它之后的所有可选参数。 这也不是很吸引人。所有这些可选参数都不应该被添加,它们构成了一个糟糕的 API。 不过,我不确定通过删除这些可选参数并添加更具体的断言方法来清理
assertEquals()
等的好处是否超过了破坏兼容性的痛苦。