Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

自动化测试用例 #130

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

自动化测试用例 #130

wants to merge 10 commits into from

Conversation

yimfeng
Copy link
Contributor

@yimfeng yimfeng commented Dec 8, 2021

No description provided.


// 测试getBackupByCaseId接口 get方法
backupController.getBackupByCaseId(2481L, "beginTime", "endTime");
MvcResult mvcResultGet = mockMvc.perform(MockMvcRequestBuilders.get("/api/backup/getBackupByCaseId")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1.这里重复测试了,调用了2次
2.校验点太少,仅校验了返回结果ok
3.这里依赖了内部数据库的内容,顺序应该是先上传一个用例,利用返回的id进行查询接口的测试

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.

None yet

2 participants