Skip to content

Conversation

@griffith-h
Copy link
Contributor

请不要在没有先创建Issue的情况下创建Pull Request。

变更的目的是什么

测试部分代码集成testcontainers(#870)

简短的更新日志

  1. 删除core中test依赖
  2. 排除persistence中junit依赖
  3. 添加testcontainers依赖
  4. 实现testcontainers基础框架
  5. 调整现有各个测试方法,使其可以测试通过

验证这一变化

1.(建议) 先拉取对应镜像,不然会执行特别慢
2. 执行mvn test提示BUILD SUCCESS

请遵循此清单,以帮助我们快速轻松地整合您的贡献:

  • 一个 PR(Pull Request的简写)只解决一个问题,禁止一个 PR 解决多个问题;
  • 确保 PR 有对应的 Issue(通常在您开始处理之前创建),除非是书写错误之类的琐碎更改不需要 Issue ;
  • 格式化 PR 及 Commit-Log 的标题及内容,例如 [Bugfix]修复用户信息修改后,用户列表会抛出空指针异常的问题(#860) #861 。PS:Commit-Log 需要在 Git Commit 代码时进行填写,在 GitHub 上修改不了;
  • 编写足够详细的 PR 描述,以了解 PR 的作用、方式和原因;
  • 编写必要的单元测试来验证您的逻辑更正。如果提交了新功能或重大更改,请记住在 test 模块中添加 integration-test;
  • 确保编译通过,集成测试通过;

@griffith-h
Copy link
Contributor Author

没加mock,太麻烦了~~,先看看这个版本

@ZQKC
Copy link
Collaborator

ZQKC commented Feb 7, 2023

没加mock,太麻烦了~~,先看看这个版本

我在 https://github.com/haoqi123/KnowStreaming/tree/Support_running_tests_in_composite_env.(container) 这个分支上,直接执行了 mvn test,发现有一些报错,是否还需要配置docker什么的环境啊?

错误信息:
image

错误信息:
image

@ZQKC ZQKC merged commit 256f770 into didi:master Feb 8, 2023
@ZQKC ZQKC linked an issue Feb 8, 2023 that may be closed by this pull request
2 tasks
@griffith-h griffith-h deleted the Support_running_tests_in_composite_env.(container) branch February 8, 2023 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

测试部分代码集成testcontainers

2 participants