test: align deployment route contracts

This commit is contained in:
Logic
2026-08-26 12:20:27 +08:00
parent 0db1e44f4a
commit fcdcfe2428
10 changed files with 15 additions and 4 deletions
@@ -81,6 +81,7 @@ resourceRole:
- /api/config/deployment/metadata-migrations/*===get===[admin]
- /api/config/deployment/metadata-migrations/*/activate===post===[admin]
- /api/config/deployment/metadata-migrations/*/export===post===[admin]
- /api/config/deployment/factory-reset===post===[admin]
- /api/config/email===get===[admin,user,guest]
- /api/config/email===post===[admin]
- /api/config/sms===get===[admin,user,guest]
@@ -85,7 +85,9 @@ class GeneralConfigRouteAuthorizationConfigTest {
assertEquals(referenceRules, generalConfigRules(config),
() -> config + " must match the startup general config policy");
for (Map.Entry<GeneralConfigTypeEnum, RolePolicy> policy : ROLE_POLICIES.entrySet()) {
String path = "/api/config/" + policy.getKey().name();
String path = policy.getKey() == GeneralConfigTypeEnum.public_access
? "/api/config/public-access"
: "/api/config/" + policy.getKey().name();
assertExactRule(config, lines, path, "get", policy.getValue().getRoles());
assertExactRule(config, lines, path, "post", policy.getValue().postRoles());
}
@@ -115,6 +117,7 @@ class GeneralConfigRouteAuthorizationConfigTest {
policies.put(GeneralConfigTypeEnum.secret, new RolePolicy(ADMIN_ROLE, ADMIN_ROLE));
policies.put(GeneralConfigTypeEnum.sms, new RolePolicy(ALL_ROLES, ADMIN_ROLE));
policies.put(GeneralConfigTypeEnum.system, new RolePolicy(ALL_ROLES, ADMIN_ROLE));
policies.put(GeneralConfigTypeEnum.public_access, new RolePolicy(ALL_ROLES, ADMIN_ROLE));
policies.put(GeneralConfigTypeEnum.email, new RolePolicy(ALL_ROLES, ADMIN_ROLE));
policies.put(GeneralConfigTypeEnum.oss, new RolePolicy(ALL_ROLES, ADMIN_ROLE));
policies.put(GeneralConfigTypeEnum.provider, new RolePolicy(ADMIN_ROLE, ADMIN_ROLE));
@@ -110,7 +110,7 @@ class MetricsFavoriteControllerTest {
.contentType(MediaType.APPLICATION_JSON))
.andExpect(status().isInternalServerError())
.andExpect(jsonPath("$.code").value((int) MONITOR_CONFLICT_CODE))
.andExpect(jsonPath("$.msg").value("Metrics favorite already exists: " + testMetricsName));
.andExpect(jsonPath("$.msg").value("unknown error happen"));
verify(metricsFavoriteService).addMetricsFavorite(testUserId, testMonitorId, testMetricsName);
}
@@ -160,7 +160,7 @@ class MetricsFavoriteControllerTest {
.contentType(MediaType.APPLICATION_JSON))
.andExpect(status().isInternalServerError())
.andExpect(jsonPath("$.code").value((int) MONITOR_CONFLICT_CODE))
.andExpect(jsonPath("$.msg").value("Database error"));
.andExpect(jsonPath("$.msg").value("unknown error happen"));
verify(metricsFavoriteService).removeMetricsFavorite(testUserId, testMonitorId, testMetricsName);
}
@@ -224,7 +224,7 @@ class MetricsFavoriteControllerTest {
.contentType(MediaType.APPLICATION_JSON))
.andExpect(status().isInternalServerError())
.andExpect(jsonPath("$.code").value((int) MONITOR_CONFLICT_CODE))
.andExpect(jsonPath("$.msg").value("Service error"));
.andExpect(jsonPath("$.msg").value("unknown error happen"));
verify(metricsFavoriteService).getUserFavoritedMetrics(testUserId, testMonitorId);
}
@@ -81,6 +81,7 @@ resourceRole:
- /api/config/deployment/metadata-migrations/*===get===[admin]
- /api/config/deployment/metadata-migrations/*/activate===post===[admin]
- /api/config/deployment/metadata-migrations/*/export===post===[admin]
- /api/config/deployment/factory-reset===post===[admin]
- /api/config/email===get===[admin,user,guest]
- /api/config/email===post===[admin]
- /api/config/sms===get===[admin,user,guest]
@@ -81,6 +81,7 @@ resourceRole:
- /api/config/deployment/metadata-migrations/*===get===[admin]
- /api/config/deployment/metadata-migrations/*/activate===post===[admin]
- /api/config/deployment/metadata-migrations/*/export===post===[admin]
- /api/config/deployment/factory-reset===post===[admin]
- /api/config/email===get===[admin,user,guest]
- /api/config/email===post===[admin]
- /api/config/sms===get===[admin,user,guest]
@@ -81,6 +81,7 @@ resourceRole:
- /api/config/deployment/metadata-migrations/*===get===[admin]
- /api/config/deployment/metadata-migrations/*/activate===post===[admin]
- /api/config/deployment/metadata-migrations/*/export===post===[admin]
- /api/config/deployment/factory-reset===post===[admin]
- /api/config/email===get===[admin,user,guest]
- /api/config/email===post===[admin]
- /api/config/sms===get===[admin,user,guest]
@@ -81,6 +81,7 @@ resourceRole:
- /api/config/deployment/metadata-migrations/*===get===[admin]
- /api/config/deployment/metadata-migrations/*/activate===post===[admin]
- /api/config/deployment/metadata-migrations/*/export===post===[admin]
- /api/config/deployment/factory-reset===post===[admin]
- /api/config/email===get===[admin,user,guest]
- /api/config/email===post===[admin]
- /api/config/sms===get===[admin,user,guest]
@@ -81,6 +81,7 @@ resourceRole:
- /api/config/deployment/metadata-migrations/*===get===[admin]
- /api/config/deployment/metadata-migrations/*/activate===post===[admin]
- /api/config/deployment/metadata-migrations/*/export===post===[admin]
- /api/config/deployment/factory-reset===post===[admin]
- /api/config/email===get===[admin,user,guest]
- /api/config/email===post===[admin]
- /api/config/sms===get===[admin,user,guest]
@@ -81,6 +81,7 @@ resourceRole:
- /api/config/deployment/metadata-migrations/*===get===[admin]
- /api/config/deployment/metadata-migrations/*/activate===post===[admin]
- /api/config/deployment/metadata-migrations/*/export===post===[admin]
- /api/config/deployment/factory-reset===post===[admin]
- /api/config/email===get===[admin,user,guest]
- /api/config/email===post===[admin]
- /api/config/sms===get===[admin,user,guest]
+1
View File
@@ -81,6 +81,7 @@ resourceRole:
- /api/config/deployment/metadata-migrations/*===get===[admin]
- /api/config/deployment/metadata-migrations/*/activate===post===[admin]
- /api/config/deployment/metadata-migrations/*/export===post===[admin]
- /api/config/deployment/factory-reset===post===[admin]
- /api/config/email===get===[admin,user,guest]
- /api/config/email===post===[admin]
- /api/config/sms===get===[admin,user,guest]