Commit 25564ae2 by DaiJiezhang

test: 补齐手机号身份字段测试

parent 97ff58ad
......@@ -36,7 +36,7 @@ class PhoneAssetControllerTest {
MockMvc mockMvc = MockMvcBuilders.standaloneSetup(new PhoneAssetController(new PhoneAssetService(mapper, mock(AssetDeviceMapper.class)), mock(PagePermissionService.class))).build();
mockMvc.perform(post("/api/phone-assets").contentType(MediaType.APPLICATION_JSON)
.content("{\"phoneNumber\":\"13812345678\",\"realNameOwner\":\"张三\"}"))
.content("{\"phoneNumber\":\"13812345678\",\"cardType\":\"移动\",\"iccid\":\"测试 ICCID\",\"realNameOwner\":\"张三\"}"))
.andExpect(status().isOk());
}
......@@ -55,7 +55,7 @@ class PhoneAssetControllerTest {
MockMvc mockMvc = MockMvcBuilders.standaloneSetup(new PhoneAssetController(new PhoneAssetService(mapper, mock(AssetDeviceMapper.class)), mock(PagePermissionService.class))).build();
mockMvc.perform(put("/api/phone-assets/1").contentType(MediaType.APPLICATION_JSON)
.content("{\"phoneNumber\":\"13912345678\",\"realNameOwner\":\"李四\"}"))
.content("{\"phoneNumber\":\"13912345678\",\"cardType\":\"移动\",\"iccid\":\"测试 ICCID\",\"realNameOwner\":\"李四\"}"))
.andExpect(status().isOk());
}
......
......@@ -60,7 +60,7 @@ class WecomAccountControllerTest {
.when(permissions).require(PagePermissionService.WECOM, "EDIT");
WecomAccountController controller = new WecomAccountController(service, permissions);
org.junit.jupiter.api.Assertions.assertThrows(org.springframework.security.access.AccessDeniedException.class,
() -> controller.update(1L, new com.xyw.console.asset.dto.WecomAccountSaveRequest("名称", null, null, null, "13812345678", null, null, null, null, null)));
() -> controller.update(1L, new com.xyw.console.asset.dto.WecomAccountSaveRequest("名称", null, null, null, "13812345678", null, null, null, null, null, null, null, null)));
org.junit.jupiter.api.Assertions.assertThrows(org.springframework.security.access.AccessDeniedException.class, () -> controller.delete(1L));
org.mockito.Mockito.verifyNoInteractions(service);
}
......
......@@ -193,7 +193,7 @@ class WecomAccountServiceTest {
void createsWithAnExistingPhoneAsset() {
WecomAccountMapper wecomMapper = mock(WecomAccountMapper.class); PhoneAssetMapper phoneMapper = mock(PhoneAssetMapper.class);
PhoneAssetEntity phone = phoneAsset(); when(phoneMapper.selectOne(any())).thenReturn(phone); when(wecomMapper.insert(any(WecomAccountEntity.class))).thenReturn(1);
WecomAccountResponse result = service(wecomMapper, mock(CompanyProfileMapper.class), phoneMapper, mock(AssetDeviceMapper.class), mock(CompanyPersonMapper.class)).create(new WecomAccountSaveRequest("测试企微", "别名", null, null, "13812345678", null, null, null, null, null));
WecomAccountResponse result = service(wecomMapper, mock(CompanyProfileMapper.class), phoneMapper, mock(AssetDeviceMapper.class), mock(CompanyPersonMapper.class)).create(new WecomAccountSaveRequest("测试企微", "别名", null, null, "13812345678", null, null, null, null, null, null, null, null));
assertEquals(20L, result.phoneAssetId()); assertEquals("EXISTING", result.phoneLinkMode());
}
......@@ -202,7 +202,7 @@ class WecomAccountServiceTest {
void createsAnExternalPhoneAssetForANewNumber() {
WecomAccountMapper wecomMapper = mock(WecomAccountMapper.class); PhoneAssetMapper phoneMapper = mock(PhoneAssetMapper.class);
when(phoneMapper.selectOne(any())).thenReturn(null); when(phoneMapper.insert(any(PhoneAssetEntity.class))).thenAnswer(call -> { ((PhoneAssetEntity) call.getArgument(0)).setId(70L); return 1; }); when(wecomMapper.insert(any(WecomAccountEntity.class))).thenAnswer(call -> { ((WecomAccountEntity) call.getArgument(0)).setId(80L); return 1; });
WecomAccountResponse result = service(wecomMapper, mock(CompanyProfileMapper.class), phoneMapper, mock(AssetDeviceMapper.class), mock(CompanyPersonMapper.class)).create(new WecomAccountSaveRequest("测试企微", null, null, null, "13912345678", null, null, null, null, null));
WecomAccountResponse result = service(wecomMapper, mock(CompanyProfileMapper.class), phoneMapper, mock(AssetDeviceMapper.class), mock(CompanyPersonMapper.class)).create(new WecomAccountSaveRequest("测试企微", null, null, null, "13912345678", null, null, null, null, null, "移动", "测试 ICCID", "测试实名人"));
assertEquals("CREATED", result.phoneLinkMode()); verify(phoneMapper).updateById(any(PhoneAssetEntity.class));
}
......@@ -221,7 +221,7 @@ class WecomAccountServiceTest {
WecomAccountService service = service(wecomMapper, companyProfileMapper, phoneMapper, mock(AssetDeviceMapper.class), mock(CompanyPersonMapper.class));
IllegalArgumentException error = assertThrows(IllegalArgumentException.class, () -> service.create(
new WecomAccountSaveRequest("测试企微", null, null, 99L, "13812345678", null, null, null, null, null)));
new WecomAccountSaveRequest("测试企微", null, null, 99L, "13812345678", null, null, null, null, null, null, null, null)));
assertEquals("注册主体不存在或已删除", error.getMessage());
}
......@@ -363,6 +363,6 @@ class WecomAccountServiceTest {
/** 代码作用(白话):构造一份只关心手机号的编辑表单。关联文件:WecomAccountSaveRequest.java。关联逻辑(调用链/数据流):测试 -> DTO -> Service.update。 */
private WecomAccountSaveRequest saveRequest(String phoneNumber) {
return new WecomAccountSaveRequest("编辑后的企微", "别名", "account", null, phoneNumber, "张三", "在职", "男", null, null);
return new WecomAccountSaveRequest("编辑后的企微", "别名", "account", null, phoneNumber, "张三", "在职", "男", null, null, "移动", "测试 ICCID", "测试实名人");
}
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment