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.

Sign up for GitHub

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

Jump to bottom

feature: OpenApi 自动化测试 #2520 #2561

Merged
merged 3 commits into from
Oct 30, 2023
Merged

feature: OpenApi 自动化测试 #2520 #2561

merged 3 commits into from
Oct 30, 2023

Conversation

wangyu096
Copy link
Collaborator

  1. 基于 junit5 + rest-assured 实现 API 的自动化测试
  2. job 网关改造 ，open api 认证逻辑变更：如果spring profile=openApiTestEnv, 并且请求 HEADER 中包含 "X-JOB-OPENAPI-TEST"，按照 Job jwt token 的方式认证（不使用 ESB JWT，避免测试过程与 ESB 依赖）
  3. 一些自动化测试 Demo 的简单实现，后续由各 API 负责人完善
  4. 测试数据可配置 - 支持通过系统变量加载外部的 properties 文件 （TestProps.java 中实现）
  5. 支持生成测试报告

jsonwan
jsonwan reviewed Oct 30, 2023
View reviewed changes
tests/openapi/src/test/java/com/tencent/bk/job/api/v3/model/EsbFileDestinationV3DTO.java Outdated

/**
* @author citruswang
* @since 17/11/2020 21:47
Copy link
Collaborator

Choose a reason for hiding this comment

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

author信息不正确

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

jsonwan
jsonwan requested changes Oct 30, 2023
View reviewed changes
Copy link
Collaborator

@jsonwan jsonwan left a comment

Choose a reason for hiding this comment

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

1个小问题需处理

@wangyu096 wangyu096 merged commit 3a9cad8 into TencentBlueKing:master Oct 30, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsonwan jsonwan jsonwan approved these changes

@liuliaozhong liuliaozhong Awaiting requested review from liuliaozhong

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wangyu096 @jsonwan