提交 30f097ed authored 作者: YunaiV's avatar YunaiV

完善 DictDataServiceImpl 单元测试

上级 6d770bba
...@@ -224,7 +224,7 @@ public class BpmTaskAssignRuleServiceImpl implements BpmTaskAssignRuleService { ...@@ -224,7 +224,7 @@ public class BpmTaskAssignRuleServiceImpl implements BpmTaskAssignRuleService {
} else if (Objects.equals(type, BpmTaskAssignRuleTypeEnum.USER_GROUP.getType())) { } else if (Objects.equals(type, BpmTaskAssignRuleTypeEnum.USER_GROUP.getType())) {
userGroupService.validUserGroups(options); userGroupService.validUserGroups(options);
} else if (Objects.equals(type, BpmTaskAssignRuleTypeEnum.SCRIPT.getType())) { } else if (Objects.equals(type, BpmTaskAssignRuleTypeEnum.SCRIPT.getType())) {
dictDataApi.validDictDatas(DictTypeConstants.TASK_ASSIGN_SCRIPT, dictDataApi.validateDictDatas(DictTypeConstants.TASK_ASSIGN_SCRIPT,
CollectionUtils.convertSet(options, String::valueOf)); CollectionUtils.convertSet(options, String::valueOf));
} else { } else {
throw new IllegalArgumentException(format("未知的规则类型({})", type)); throw new IllegalArgumentException(format("未知的规则类型({})", type));
......
...@@ -25,7 +25,7 @@ public interface DictDataApi { ...@@ -25,7 +25,7 @@ public interface DictDataApi {
@ApiImplicitParam(name = "dictType", value = "字典类型", example = "SEX", required = true, dataTypeClass = String.class), @ApiImplicitParam(name = "dictType", value = "字典类型", example = "SEX", required = true, dataTypeClass = String.class),
@ApiImplicitParam(name = "values", value = "字典数据值的数组", example = "1,2", required = true, allowMultiple = true) @ApiImplicitParam(name = "values", value = "字典数据值的数组", example = "1,2", required = true, allowMultiple = true)
}) })
CommonResult<Boolean> validDictDatas(@RequestParam("dictType") String dictType, CommonResult<Boolean> validateDictDatas(@RequestParam("dictType") String dictType,
@RequestParam("values") Collection<String> values); @RequestParam("values") Collection<String> values);
@GetMapping(PREFIX + "/get") @GetMapping(PREFIX + "/get")
......
...@@ -6,7 +6,6 @@ import cn.iocoder.yudao.module.system.convert.dict.DictDataConvert; ...@@ -6,7 +6,6 @@ import cn.iocoder.yudao.module.system.convert.dict.DictDataConvert;
import cn.iocoder.yudao.module.system.dal.dataobject.dict.DictDataDO; import cn.iocoder.yudao.module.system.dal.dataobject.dict.DictDataDO;
import cn.iocoder.yudao.module.system.service.dict.DictDataService; import cn.iocoder.yudao.module.system.service.dict.DictDataService;
import org.apache.dubbo.config.annotation.DubboService; import org.apache.dubbo.config.annotation.DubboService;
import org.springframework.stereotype.Service;
import org.springframework.validation.annotation.Validated; import org.springframework.validation.annotation.Validated;
import org.springframework.web.bind.annotation.RestController; import org.springframework.web.bind.annotation.RestController;
...@@ -25,8 +24,8 @@ public class DictDataApiImpl implements DictDataApi { ...@@ -25,8 +24,8 @@ public class DictDataApiImpl implements DictDataApi {
private DictDataService dictDataService; private DictDataService dictDataService;
@Override @Override
public CommonResult<Boolean> validDictDatas(String dictType, Collection<String> values) { public CommonResult<Boolean> validateDictDatas(String dictType, Collection<String> values) {
dictDataService.validDictDatas(dictType, values); dictDataService.validateDictDataList(dictType, values);
return success(true); return success(true);
} }
......
...@@ -62,7 +62,7 @@ public class DictDataController { ...@@ -62,7 +62,7 @@ public class DictDataController {
@ApiOperation(value = "获得全部字典数据列表", notes = "一般用于管理后台缓存字典数据在本地") @ApiOperation(value = "获得全部字典数据列表", notes = "一般用于管理后台缓存字典数据在本地")
// 无需添加权限认证,因为前端全局都需要 // 无需添加权限认证,因为前端全局都需要
public CommonResult<List<DictDataSimpleRespVO>> getSimpleDictDatas() { public CommonResult<List<DictDataSimpleRespVO>> getSimpleDictDatas() {
List<DictDataDO> list = dictDataService.getDictDatas(); List<DictDataDO> list = dictDataService.getDictDataList();
return success(DictDataConvert.INSTANCE.convertList(list)); return success(DictDataConvert.INSTANCE.convertList(list));
} }
...@@ -86,7 +86,7 @@ public class DictDataController { ...@@ -86,7 +86,7 @@ public class DictDataController {
@PreAuthorize("@ss.hasPermission('system:dict:export')") @PreAuthorize("@ss.hasPermission('system:dict:export')")
@OperateLog(type = EXPORT) @OperateLog(type = EXPORT)
public void export(HttpServletResponse response, @Valid DictDataExportReqVO reqVO) throws IOException { public void export(HttpServletResponse response, @Valid DictDataExportReqVO reqVO) throws IOException {
List<DictDataDO> list = dictDataService.getDictDatas(reqVO); List<DictDataDO> list = dictDataService.getDictDataList(reqVO);
List<DictDataExcelVO> data = DictDataConvert.INSTANCE.convertList02(list); List<DictDataExcelVO> data = DictDataConvert.INSTANCE.convertList02(list);
// 输出 // 输出
ExcelUtils.write(response, "字典数据.xls", "数据列表", DictDataExcelVO.class, data); ExcelUtils.write(response, "字典数据.xls", "数据列表", DictDataExcelVO.class, data);
......
...@@ -44,7 +44,7 @@ public interface DictDataService { ...@@ -44,7 +44,7 @@ public interface DictDataService {
* *
* @return 字典数据全列表 * @return 字典数据全列表
*/ */
List<DictDataDO> getDictDatas(); List<DictDataDO> getDictDataList();
/** /**
* 获得字典数据分页列表 * 获得字典数据分页列表
...@@ -60,7 +60,7 @@ public interface DictDataService { ...@@ -60,7 +60,7 @@ public interface DictDataService {
* @param reqVO 列表请求 * @param reqVO 列表请求
* @return 字典数据列表 * @return 字典数据列表
*/ */
List<DictDataDO> getDictDatas(DictDataExportReqVO reqVO); List<DictDataDO> getDictDataList(DictDataExportReqVO reqVO);
/** /**
* 获得字典数据详情 * 获得字典数据详情
...@@ -86,7 +86,7 @@ public interface DictDataService { ...@@ -86,7 +86,7 @@ public interface DictDataService {
* @param dictType 字典类型 * @param dictType 字典类型
* @param values 字典数据值的数组 * @param values 字典数据值的数组
*/ */
void validDictDatas(String dictType, Collection<String> values); void validateDictDataList(String dictType, Collection<String> values);
/** /**
* 获得指定的字典数据 * 获得指定的字典数据
......
...@@ -48,7 +48,7 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -48,7 +48,7 @@ public class DictDataServiceImpl implements DictDataService {
private DictDataMapper dictDataMapper; private DictDataMapper dictDataMapper;
@Override @Override
public List<DictDataDO> getDictDatas() { public List<DictDataDO> getDictDataList() {
List<DictDataDO> list = dictDataMapper.selectList(); List<DictDataDO> list = dictDataMapper.selectList();
list.sort(COMPARATOR_TYPE_AND_SORT); list.sort(COMPARATOR_TYPE_AND_SORT);
return list; return list;
...@@ -60,7 +60,7 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -60,7 +60,7 @@ public class DictDataServiceImpl implements DictDataService {
} }
@Override @Override
public List<DictDataDO> getDictDatas(DictDataExportReqVO reqVO) { public List<DictDataDO> getDictDataList(DictDataExportReqVO reqVO) {
List<DictDataDO> list = dictDataMapper.selectList(reqVO); List<DictDataDO> list = dictDataMapper.selectList(reqVO);
list.sort(COMPARATOR_TYPE_AND_SORT); list.sort(COMPARATOR_TYPE_AND_SORT);
return list; return list;
...@@ -74,7 +74,7 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -74,7 +74,7 @@ public class DictDataServiceImpl implements DictDataService {
@Override @Override
public Long createDictData(DictDataCreateReqVO reqVO) { public Long createDictData(DictDataCreateReqVO reqVO) {
// 校验正确性 // 校验正确性
checkCreateOrUpdate(null, reqVO.getValue(), reqVO.getDictType()); validateDictDataForCreateOrUpdate(null, reqVO.getValue(), reqVO.getDictType());
// 插入字典类型 // 插入字典类型
DictDataDO dictData = DictDataConvert.INSTANCE.convert(reqVO); DictDataDO dictData = DictDataConvert.INSTANCE.convert(reqVO);
...@@ -85,7 +85,7 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -85,7 +85,7 @@ public class DictDataServiceImpl implements DictDataService {
@Override @Override
public void updateDictData(DictDataUpdateReqVO reqVO) { public void updateDictData(DictDataUpdateReqVO reqVO) {
// 校验正确性 // 校验正确性
checkCreateOrUpdate(reqVO.getId(), reqVO.getValue(), reqVO.getDictType()); validateDictDataForCreateOrUpdate(reqVO.getId(), reqVO.getValue(), reqVO.getDictType());
// 更新字典类型 // 更新字典类型
DictDataDO updateObj = DictDataConvert.INSTANCE.convert(reqVO); DictDataDO updateObj = DictDataConvert.INSTANCE.convert(reqVO);
...@@ -95,7 +95,7 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -95,7 +95,7 @@ public class DictDataServiceImpl implements DictDataService {
@Override @Override
public void deleteDictData(Long id) { public void deleteDictData(Long id) {
// 校验是否存在 // 校验是否存在
checkDictDataExists(id); validateDictDataExists(id);
// 删除字典数据 // 删除字典数据
dictDataMapper.deleteById(id); dictDataMapper.deleteById(id);
...@@ -106,18 +106,17 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -106,18 +106,17 @@ public class DictDataServiceImpl implements DictDataService {
return dictDataMapper.selectCountByDictType(dictType); return dictDataMapper.selectCountByDictType(dictType);
} }
private void validateDictDataForCreateOrUpdate(Long id, String value, String dictType) {
private void checkCreateOrUpdate(Long id, String value, String dictType) {
// 校验自己存在 // 校验自己存在
checkDictDataExists(id); validateDictDataExists(id);
// 校验字典类型有效 // 校验字典类型有效
checkDictTypeValid(dictType); validateDictTypeExists(dictType);
// 校验字典数据的值的唯一性 // 校验字典数据的值的唯一性
checkDictDataValueUnique(id, dictType, value); validateDictDataValueUnique(id, dictType, value);
} }
@VisibleForTesting @VisibleForTesting
public void checkDictDataValueUnique(Long id, String dictType, String value) { public void validateDictDataValueUnique(Long id, String dictType, String value) {
DictDataDO dictData = dictDataMapper.selectByDictTypeAndValue(dictType, value); DictDataDO dictData = dictDataMapper.selectByDictTypeAndValue(dictType, value);
if (dictData == null) { if (dictData == null) {
return; return;
...@@ -132,7 +131,7 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -132,7 +131,7 @@ public class DictDataServiceImpl implements DictDataService {
} }
@VisibleForTesting @VisibleForTesting
public void checkDictDataExists(Long id) { public void validateDictDataExists(Long id) {
if (id == null) { if (id == null) {
return; return;
} }
...@@ -143,7 +142,7 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -143,7 +142,7 @@ public class DictDataServiceImpl implements DictDataService {
} }
@VisibleForTesting @VisibleForTesting
public void checkDictTypeValid(String type) { public void validateDictTypeExists(String type) {
DictTypeDO dictType = dictTypeService.getDictType(type); DictTypeDO dictType = dictTypeService.getDictType(type);
if (dictType == null) { if (dictType == null) {
throw exception(DICT_TYPE_NOT_EXISTS); throw exception(DICT_TYPE_NOT_EXISTS);
...@@ -154,7 +153,7 @@ public class DictDataServiceImpl implements DictDataService { ...@@ -154,7 +153,7 @@ public class DictDataServiceImpl implements DictDataService {
} }
@Override @Override
public void validDictDatas(String dictType, Collection<String> values) { public void validateDictDataList(String dictType, Collection<String> values) {
if (CollUtil.isEmpty(values)) { if (CollUtil.isEmpty(values)) {
return; return;
} }
......
Markdown 格式
0%
您添加了 0 到此讨论。请谨慎行事。
请先完成此评论的编辑!
注册 或者 后发表评论