大家好!
MeiliSearch 服务器的当前模拟让我有点担心我们的测试。 我宁愿在真正的 MeiliSearch 服务器上运行测试。 目标是确保这个包在生产中与 MeiliSearch 的最新版本兼容。
实际上,MeiliSearch 和 meilisearch-php 还不稳定,MeiliSearch 服务器中的每次修改都会强制我们更新模拟。 否则,我们的测试不是最新的 MeiliSearch 的最新修改,我们的包可能包含错误而不会注意到它们。
我不是说我们应该删除模拟部分,但我们至少应该有针对真实 MeiliSearch 服务器的集成测试部分,那么为什么不保留两者呢? 除非你觉得这样维护太多了,那么让我们删除模拟部分。
我们在 Symfony bundler 包中做了同样的事情: https :
在 meilisearch-php SDK 仓库中: https :
你怎么看@shokme? (只是你的意见,当然我不是问你有没有时间去做😁)
还有大家,你对此有何看法?
在这个@curquiza 上,我完全站在你这边。 在 CI 测试中添加 meilisearch 服务将是一个很好的补充。
同时,我们可以将测试拆分为Unit
测试和Feature
测试之类的结构。 让当前的模拟测试继续存在。
我在 #89 中的更改添加了测试覆盖输出,其中显示了目前未经测试的部分。 那部分可能是新的Feature
测试的开始
好的,那我们就这样做吧! 这将使我们的回购更安全💪
@curquiza这可以关闭吗? 你怎么看
是的,抱歉,PR 应该自动关闭它! 😇
最有用的评论
在这个@curquiza 上,我完全站在你这边。 在 CI 测试中添加 meilisearch 服务将是一个很好的补充。
同时,我们可以将测试拆分为
Unit
测试和Feature
测试之类的结构。 让当前的模拟测试继续存在。我在 #89 中的更改添加了测试覆盖输出,其中显示了目前未经测试的部分。 那部分可能是新的
Feature
测试的开始