Meilisearch-laravel-scout: 添加未模拟的测试

创建于 2021-02-01  ·  4评论  ·  资料来源: meilisearch/meilisearch-laravel-scout

大家好!

MeiliSearch 服务器的当前模拟让我有点担心我们的测试。 我宁愿在真正的 Meil​​iSearch 服务器上运行测试。 目标是确保这个包在生产中与 Meil​​iSearch 的最新版本兼容。
实际上,MeiliSearch 和 meilisearch-php 还不稳定,MeiliSearch 服务器中的每次修改都会强制我们更新模拟。 否则,我们的测试不是最新的 Meil​​iSearch 的最新修改,我们的包可能包含错误而不会注意到它们。

我不是说我们应该删除模拟部分,但我们至少应该有针对真实 Meil​​iSearch 服务器的集成测试部分,那么为什么不保留两者呢? 除非你觉得这样维护太多了,那么让我们删除模拟部分。

我们在 Symfony bundler 包中做了同样的事情: https :
在 meilisearch-php SDK 仓库中: https :

你怎么看@shokme? (只是你的意见,当然我不是问你有没有时间去做😁)
还有大家,你对此有何看法?

Laravel integration

最有用的评论

在这个@curquiza 上,我完全站在你这边。 在 CI 测试中添加 meilisearch 服务将是一个很好的补充。

同时,我们可以将测试拆分为Unit测试和Feature测试之类的结构。 让当前的模拟测试继续存在。

我在 #89 中的更改添加了测试覆盖输出,其中显示了目前未经测试的部分。 那部分可能是新的Feature测试的开始

所有4条评论

在这个@curquiza 上,我完全站在你这边。 在 CI 测试中添加 meilisearch 服务将是一个很好的补充。

同时,我们可以将测试拆分为Unit测试和Feature测试之类的结构。 让当前的模拟测试继续存在。

我在 #89 中的更改添加了测试覆盖输出,其中显示了目前未经测试的部分。 那部分可能是新的Feature测试的开始

好的,那我们就这样做吧! 这将使我们的回购更安全💪

@curquiza这可以关闭吗? 你怎么看

是的,抱歉,PR 应该自动关闭它! 😇

此页面是否有帮助?
0 / 5 - 0 等级

相关问题

marky291 picture marky291  ·  6评论

dreamcog picture dreamcog  ·  14评论

ctf0 picture ctf0  ·  7评论

desaintflorent picture desaintflorent  ·  8评论

JexPY picture JexPY  ·  8评论