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

适配openubmc接口,增加DT测试用例 #278

Merged
merged 1 commit into from
Dec 10, 2024
Merged

适配openubmc接口,增加DT测试用例 #278

merged 1 commit into from
Dec 10, 2024

Conversation

Kaede10
Copy link
Collaborator

@Kaede10 Kaede10 commented Dec 10, 2024

No description provided.

@shishupei
Copy link

开始门禁检查,等门禁检查通过后可以合入

@shishupei
Copy link

检查项 状态 日志
dt覆盖率 查看日志
Check代码检查 查看日志
敏感信息扫描 查看日志
安全编码扫描 查看日志
流水线链接 点击跳转

@Kaede10 Kaede10 closed this Dec 10, 2024
@Kaede10 Kaede10 reopened this Dec 10, 2024
@shishupei
Copy link

开始门禁检查,等门禁检查通过后可以合入

@shishupei
Copy link

检查项 状态 日志
DT覆盖率 2.12281% 查看日志
Check代码检查 查看日志
敏感信息扫描 查看日志
安全编码扫描 查看日志
流水线链接 点击跳转

@@ -0,0 +1,105 @@
package com.datastat.ds.unit;
Copy link
Contributor

Choose a reason for hiding this comment

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

请加上copy right


import org.asynchttpclient.ListenableFuture;
import org.asynchttpclient.Response;
import org.junit.jupiter.api.BeforeEach;
Copy link
Contributor

Choose a reason for hiding this comment

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

import引入排序建议按照规则进行排序
1、外部引入
2、自由
3、公共引入

@@ -3,6 +3,8 @@
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;

import java.util.Arrays;
Copy link
Contributor

Choose a reason for hiding this comment

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

import引入归类

when(queryDaoContext.getQueryDao("queryDao")).thenReturn(queryDao);
when(queryConfContext.getQueryConfig("queryConf")).thenReturn(queryConfig);
when(queryDao.queryUserOwnerType(queryConfig, user)).thenReturn(result);
when(redisDao.set(key, result, 1l)).thenReturn(true);
Copy link
Contributor

Choose a reason for hiding this comment

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

reuslt 判断 11具体是什么意思呢

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

redis打桩,设置缓存成功(key-value-time)

@zhongjun2 zhongjun2 merged commit 107c85a into main Dec 10, 2024
5 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants