fix(测试计划): 生成自定义报告日志内容名称错误
--bug=1044155 --user=宋昌昌 【测试计划】计划-生成自定义报告,查看日志信息,名称显示自动报告格式 https://www.tapd.cn/55049933/s/1550257
This commit is contained in:
parent
a53f332650
commit
5a5d20a0e5
|
@ -0,0 +1,16 @@
|
||||||
|
package io.metersphere.plan.dto;
|
||||||
|
|
||||||
|
import io.swagger.v3.oas.annotations.media.Schema;
|
||||||
|
import lombok.Builder;
|
||||||
|
import lombok.Data;
|
||||||
|
|
||||||
|
@Data
|
||||||
|
@Builder
|
||||||
|
public class TestPlanReportManualParam {
|
||||||
|
|
||||||
|
@Schema(description = "目标计划ID")
|
||||||
|
private String targetId;
|
||||||
|
|
||||||
|
@Schema(description = "手动生成的报告名称")
|
||||||
|
private String manualName;
|
||||||
|
}
|
|
@ -234,7 +234,7 @@ public class TestPlanReportService {
|
||||||
* 2. 保存报告布局组件 (只对当前生成的计划/组有效, 不会对下面的子计划报告生效)
|
* 2. 保存报告布局组件 (只对当前生成的计划/组有效, 不会对下面的子计划报告生效)
|
||||||
* 3. 处理富文本图片
|
* 3. 处理富文本图片
|
||||||
*/
|
*/
|
||||||
Map<String, String> reportMap = genReport(IDGenerator.nextStr(), request, true, currentUser);
|
Map<String, String> reportMap = genReport(IDGenerator.nextStr(), request, true, currentUser, request.getReportName());
|
||||||
String genReportId = reportMap.get(request.getTestPlanId());
|
String genReportId = reportMap.get(request.getTestPlanId());
|
||||||
List<TestPlanReportComponentSaveRequest> components = request.getComponents();
|
List<TestPlanReportComponentSaveRequest> components = request.getComponents();
|
||||||
if (CollectionUtils.isNotEmpty(components)) {
|
if (CollectionUtils.isNotEmpty(components)) {
|
||||||
|
@ -256,7 +256,6 @@ public class TestPlanReportService {
|
||||||
TestPlanReport record = new TestPlanReport();
|
TestPlanReport record = new TestPlanReport();
|
||||||
record.setId(genReportId);
|
record.setId(genReportId);
|
||||||
record.setDefaultLayout(false);
|
record.setDefaultLayout(false);
|
||||||
record.setName(request.getReportName());
|
|
||||||
testPlanReportMapper.updateByPrimaryKeySelective(record);
|
testPlanReportMapper.updateByPrimaryKeySelective(record);
|
||||||
// 处理富文本文件
|
// 处理富文本文件
|
||||||
transferRichTextTmpFile(genReportId, request.getProjectId(), request.getRichTextTmpFileIds(), currentUser, TestPlanReportAttachmentSourceType.RICH_TEXT.name());
|
transferRichTextTmpFile(genReportId, request.getProjectId(), request.getRichTextTmpFileIds(), currentUser, TestPlanReportAttachmentSourceType.RICH_TEXT.name());
|
||||||
|
@ -269,7 +268,7 @@ public class TestPlanReportService {
|
||||||
* @param currentUser 当前用户
|
* @param currentUser 当前用户
|
||||||
*/
|
*/
|
||||||
public String genReportByAuto(TestPlanReportGenRequest request, String currentUser) {
|
public String genReportByAuto(TestPlanReportGenRequest request, String currentUser) {
|
||||||
Map<String, String> reportMap = genReport(IDGenerator.nextStr(), request, true, currentUser);
|
Map<String, String> reportMap = genReport(IDGenerator.nextStr(), request, true, currentUser, null);
|
||||||
return reportMap.get(request.getTestPlanId());
|
return reportMap.get(request.getTestPlanId());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -282,11 +281,12 @@ public class TestPlanReportService {
|
||||||
*/
|
*/
|
||||||
@Transactional(rollbackFor = Exception.class, propagation = Propagation.REQUIRES_NEW)
|
@Transactional(rollbackFor = Exception.class, propagation = Propagation.REQUIRES_NEW)
|
||||||
public Map<String, String> genReportByExecution(String prepareReportId, TestPlanReportGenRequest request, String currentUser) {
|
public Map<String, String> genReportByExecution(String prepareReportId, TestPlanReportGenRequest request, String currentUser) {
|
||||||
return genReport(prepareReportId, request, false, currentUser);
|
return genReport(prepareReportId, request, false, currentUser, null);
|
||||||
}
|
}
|
||||||
|
|
||||||
public Map<String, String> genReport(String prepareReportId, TestPlanReportGenRequest request, boolean manual, String currentUser) {
|
public Map<String, String> genReport(String prepareReportId, TestPlanReportGenRequest request, boolean manual, String currentUser, String manualReportName) {
|
||||||
Map<String, String> preReportMap = new HashMap<>();
|
Map<String, String> preReportMap = new HashMap<>();
|
||||||
|
TestPlanReportManualParam reportManualParam = TestPlanReportManualParam.builder().manualName(manualReportName).targetId(request.getTestPlanId()).build();
|
||||||
try {
|
try {
|
||||||
// 所有计划
|
// 所有计划
|
||||||
List<TestPlan> plans = getPlans(request.getTestPlanId());
|
List<TestPlan> plans = getPlans(request.getTestPlanId());
|
||||||
|
@ -312,7 +312,7 @@ public class TestPlanReportService {
|
||||||
genPreParam.setUseManual(manual);
|
genPreParam.setUseManual(manual);
|
||||||
//如果是测试计划的独立报告,使用参数中的预生成的报告id。否则只有测试计划组报告使用该id
|
//如果是测试计划的独立报告,使用参数中的预生成的报告id。否则只有测试计划组报告使用该id
|
||||||
String prepareItemReportId = isGroupReports ? IDGenerator.nextStr() : prepareReportId;
|
String prepareItemReportId = isGroupReports ? IDGenerator.nextStr() : prepareReportId;
|
||||||
TestPlanReport preReport = preGenReport(prepareItemReportId, genPreParam, currentUser, moduleParam, childPlanIds);
|
TestPlanReport preReport = preGenReport(prepareItemReportId, genPreParam, currentUser, moduleParam, childPlanIds, reportManualParam);
|
||||||
if (manual) {
|
if (manual) {
|
||||||
// 汇总
|
// 汇总
|
||||||
if (genPreParam.getIntegrated()) {
|
if (genPreParam.getIntegrated()) {
|
||||||
|
@ -343,7 +343,8 @@ public class TestPlanReportService {
|
||||||
*
|
*
|
||||||
* @return 报告
|
* @return 报告
|
||||||
*/
|
*/
|
||||||
public TestPlanReport preGenReport(String prepareId, TestPlanReportGenPreParam genParam, String currentUser, TestPlanReportModuleParam moduleParam, List<String> childPlanIds) {
|
public TestPlanReport preGenReport(String prepareId, TestPlanReportGenPreParam genParam, String currentUser, TestPlanReportModuleParam moduleParam,
|
||||||
|
List<String> childPlanIds, TestPlanReportManualParam reportManualParam) {
|
||||||
// 计划配置
|
// 计划配置
|
||||||
TestPlanConfig config = testPlanConfigMapper.selectByPrimaryKey(genParam.getTestPlanId());
|
TestPlanConfig config = testPlanConfigMapper.selectByPrimaryKey(genParam.getTestPlanId());
|
||||||
|
|
||||||
|
@ -355,7 +356,8 @@ public class TestPlanReportService {
|
||||||
TestPlanReport report = new TestPlanReport();
|
TestPlanReport report = new TestPlanReport();
|
||||||
BeanUtils.copyBean(report, genParam);
|
BeanUtils.copyBean(report, genParam);
|
||||||
report.setId(genParam.getIntegrated() ? genParam.getGroupReportId() : prepareId);
|
report.setId(genParam.getIntegrated() ? genParam.getGroupReportId() : prepareId);
|
||||||
report.setName(genParam.getTestPlanName() + "-" + DateUtils.getTimeStr(System.currentTimeMillis()));
|
report.setName((StringUtils.equals(genParam.getTestPlanId(), reportManualParam.getTargetId()) && StringUtils.isNotBlank(reportManualParam.getManualName())) ?
|
||||||
|
reportManualParam.getManualName() : genParam.getTestPlanName() + "-" + DateUtils.getTimeStr(System.currentTimeMillis()));
|
||||||
report.setCreateUser(currentUser);
|
report.setCreateUser(currentUser);
|
||||||
report.setCreateTime(System.currentTimeMillis());
|
report.setCreateTime(System.currentTimeMillis());
|
||||||
report.setDeleted(false);
|
report.setDeleted(false);
|
||||||
|
|
Loading…
Reference in New Issue