fix(file): return controlled missing package errors
This commit is contained in:
+1
-1
@@ -87,7 +87,7 @@ public class PlatformQualificationFileController {
|
||||
public ApiResponse<QualificationPackageService.PackageUploadResult> uploadRequiredPackage(
|
||||
@AuthenticationPrincipal AuthenticatedUser user,
|
||||
@PathVariable long tenantId,
|
||||
@RequestPart("businessLicense") MultipartFile businessLicense,
|
||||
@RequestPart(value = "businessLicense", required = false) MultipartFile businessLicense,
|
||||
@RequestPart(value = "legalPersonId", required = false) MultipartFile legalPersonId,
|
||||
@RequestParam String unifiedSocialCreditCode,
|
||||
@RequestParam(required = false) LocalDate businessLicenseValidTo,
|
||||
|
||||
@@ -78,7 +78,7 @@ public class QualificationFileController {
|
||||
@PreAuthorize("hasAuthority('merchant:qualification:upload')")
|
||||
public ApiResponse<QualificationPackageService.PackageUploadResult> uploadRequiredPackage(
|
||||
@AuthenticationPrincipal AuthenticatedUser user,
|
||||
@RequestPart("businessLicense") MultipartFile businessLicense,
|
||||
@RequestPart(value = "businessLicense", required = false) MultipartFile businessLicense,
|
||||
@RequestPart(value = "legalPersonId", required = false) MultipartFile legalPersonId,
|
||||
@RequestParam String unifiedSocialCreditCode,
|
||||
@RequestParam(required = false) LocalDate businessLicenseValidTo,
|
||||
|
||||
@@ -83,7 +83,7 @@ class QualificationFileContractTest {
|
||||
}
|
||||
|
||||
@Test
|
||||
void requiredPackageKeepsBusinessLicenseRequiredAndMakesLegalPersonIdOptional() {
|
||||
void requiredPackageLetsServiceReturnControlledValidationForMissingParts() {
|
||||
assertRequiredPackageParts(QualificationFileController.class);
|
||||
assertRequiredPackageParts(PlatformQualificationFileController.class);
|
||||
}
|
||||
@@ -316,7 +316,7 @@ class QualificationFileContractTest {
|
||||
.map(parameter -> parameter.getAnnotation(RequestPart.class))
|
||||
.filter(java.util.Objects::nonNull)
|
||||
.collect(java.util.stream.Collectors.toMap(RequestPart::value, RequestPart::required));
|
||||
assertEquals(Boolean.TRUE, parts.get("businessLicense"));
|
||||
assertEquals(Boolean.FALSE, parts.get("businessLicense"));
|
||||
assertEquals(Boolean.FALSE, parts.get("legalPersonId"));
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user