补充同步申请回归测试并更新进度清单
补充同步证书申请的项目权限与 DNS 凭据权限回归测试，记录后端与前端验证结果，并保持 P0-03 待复审状态。
This commit is contained in:
parent
3179559153
commit
a0cb11c28f
@ -17,6 +17,7 @@ import (
|
||||
"git.pcux.cn/xuans/Scert/backend/internal/config"
|
||||
"git.pcux.cn/xuans/Scert/backend/internal/deployment"
|
||||
"git.pcux.cn/xuans/Scert/backend/internal/domain"
|
||||
"git.pcux.cn/xuans/Scert/backend/internal/repository"
|
||||
)
|
||||
|
||||
type stubDomainRepository struct {
|
||||
@ -216,9 +217,13 @@ func TestUpdateDomainRejectsInvalidEnvironment(t *testing.T) {
|
||||
|
||||
func TestIssueDomainUsesCertificateWorkflow(t *testing.T) {
|
||||
cfg := readyTestConfig(t.TempDir())
|
||||
repo := &stubDomainRepository{loaded: domain.Domain{ID: "domain-1", ResourceScope: domain.ResourceScope{ProjectID: domain.DefaultProjectID}}}
|
||||
calledID := ""
|
||||
server := New(Dependencies{
|
||||
Config: cfg, Repository: &stubDomainRepository{},
|
||||
Config: cfg,
|
||||
Repository: repo,
|
||||
ProjectResources: projectResourceReaderStub{domains: repo},
|
||||
ProjectDomains: projectDomainRepositoryStub{DomainRepository: repo},
|
||||
IssueCertificate: func(_ context.Context, id string) (certificate.IssueResult, deployment.Report, error) {
|
||||
calledID = id
|
||||
return certificate.IssueResult{DomainID: id}, deployment.Report{Status: "success"}, nil
|
||||
@ -230,3 +235,69 @@ func TestIssueDomainUsesCertificateWorkflow(t *testing.T) {
|
||||
t.Fatalf("证书申请接口响应错误：%d %s", response.Code, response.Body.String())
|
||||
}
|
||||
}
|
||||
|
||||
func TestSynchronousIssueRequiresProjectPermissions(t *testing.T) {
|
||||
cfg := readyTestConfig(t.TempDir())
|
||||
repo := &stubDomainRepository{loaded: domain.Domain{ID: "domain-1", ResourceScope: domain.ResourceScope{ProjectID: "project-a"}}}
|
||||
calls := 0
|
||||
server := New(Dependencies{
|
||||
Config: cfg,
|
||||
Repository: repo,
|
||||
ProjectResources: projectResourceReaderStub{domains: repo},
|
||||
ProjectDomains: projectDomainRepositoryStub{
|
||||
DomainRepository: repo,
|
||||
authorize: func(_ context.Context, _ string, _ string, permission domain.ProjectPermission) error {
|
||||
if permission == domain.PermissionCertificateManage {
|
||||
return repository.ErrProjectAccessDenied
|
||||
}
|
||||
return nil
|
||||
},
|
||||
},
|
||||
IssueCertificate: func(_ context.Context, _ string) (certificate.IssueResult, deployment.Report, error) {
|
||||
calls++
|
||||
return certificate.IssueResult{}, deployment.Report{}, nil
|
||||
},
|
||||
})
|
||||
|
||||
response := httptest.NewRecorder()
|
||||
serveAdminHTTP(server, response, httptest.NewRequest(http.MethodPost, "/api/v1/domains/domain-1/issue", nil))
|
||||
if response.Code != http.StatusForbidden {
|
||||
t.Fatalf("同步证书申请缺少项目权限时未拒绝：%d %s", response.Code, response.Body.String())
|
||||
}
|
||||
if calls != 0 {
|
||||
t.Fatalf("同步证书申请权限拒绝后仍调用证书工作流：%d", calls)
|
||||
}
|
||||
}
|
||||
|
||||
func TestSynchronousIssueRequiresDNSCredentialPermission(t *testing.T) {
|
||||
cfg := readyTestConfig(t.TempDir())
|
||||
repo := &stubDomainRepository{loaded: domain.Domain{ID: "domain-1", ResourceScope: domain.ResourceScope{ProjectID: "project-a"}}}
|
||||
calls := 0
|
||||
server := New(Dependencies{
|
||||
Config: cfg,
|
||||
Repository: repo,
|
||||
ProjectResources: projectResourceReaderStub{domains: repo},
|
||||
ProjectDomains: projectDomainRepositoryStub{DomainRepository: repo},
|
||||
ProjectMembers: permissionProjectMemberRepository{
|
||||
allow: func(permission domain.ProjectPermission) error {
|
||||
if permission == domain.PermissionDNSCredentialUse {
|
||||
return repository.ErrProjectAccessDenied
|
||||
}
|
||||
return nil
|
||||
},
|
||||
},
|
||||
IssueCertificate: func(_ context.Context, _ string) (certificate.IssueResult, deployment.Report, error) {
|
||||
calls++
|
||||
return certificate.IssueResult{}, deployment.Report{}, nil
|
||||
},
|
||||
})
|
||||
|
||||
response := httptest.NewRecorder()
|
||||
serveAdminHTTP(server, response, httptest.NewRequest(http.MethodPost, "/api/v1/domains/domain-1/issue", nil))
|
||||
if response.Code != http.StatusForbidden {
|
||||
t.Fatalf("同步证书申请缺少 DNS 凭据权限时未拒绝：%d %s", response.Code, response.Body.String())
|
||||
}
|
||||
if calls != 0 {
|
||||
t.Fatalf("DNS 凭据权限拒绝后仍调用证书工作流：%d", calls)
|
||||
}
|
||||
}
|
||||
|
||||
@ -3451,3 +3451,15 @@ Linux Ubuntu 22.04 上的 `Scert-admin` 手动启停、无开机自启动、无
|
||||
- [x] 已提交 35260bc、aa587d3；最后一次推送成功。
|
||||
- [~] P0-07 总项仍在进行中，其他资源写入、三库矩阵、浏览器、远端部署和生产验证仍需继续。
|
||||
- [!] 总体验收位置保持 **P0-03 完成，待复审**，不得提前标记复审通过。
|
||||
|
||||
## 本轮开发进度（2026-09-29）
|
||||
|
||||
总体验收位置继续保持 **P0-03：完成，待复审**，本轮仅推进 P0-07 的同步证书申请权限闭环，不将局部自动化测试结果视为生产部署验收通过。
|
||||
|
||||
- `[x]` 无任务仓储的同步证书申请路径已补齐项目资源可见性、`certificate.manage`、`deployment.execute` 与 DNS 凭据使用权限校验。
|
||||
- `[x]` 已补充项目证书权限不足、DNS 凭据使用权限不足及权限拒绝后不调用证书工作流的回归测试。
|
||||
- `[x]` 当前候选后端 `go test ./...`、`go vet ./...` 通过；该结果仅代表本地自动化验证，不代表三数据库、浏览器、远端服务器或生产部署验收通过。
|
||||
- `[~]` P0-07 其他资源写入权限、三数据库矩阵、浏览器清缓存验收、密码 SSH、远端部署及 Linux／Windows 生产服务仍需继续开发或复审。
|
||||
- `[!]` 总体验收不得升级，P0-03 仍为“完成，待复审”。
|
||||
|
||||
下一步继续执行前端回归与生产构建，再提交并推送本轮测试代码；完成开发阶段后，统一进行三数据库、浏览器、远端服务器和生产部署测试。
|
||||
@ -747,3 +747,15 @@ P0-05 从 `[!]` 调整为 `[~]`，并未标记完成；P0-04 剩余门禁和全
|
||||
- [x] 已提交 35260bc、aa587d3，最后一次远程推送成功。
|
||||
- [~] P0-07 继续进行；待完成三数据库矩阵、浏览器、SSH/HTTPS、远端部署和生产门禁。
|
||||
- [!] 总体验收仍停在 **P0-03 完成，待复审**，不能将局部测试结果写成生产验收通过。
|
||||
|
||||
## 本轮开发进度（2026-09-29）
|
||||
|
||||
总体验收位置继续保持 **P0-03：完成，待复审**，本轮仅推进 P0-07 的同步证书申请权限闭环，不将局部自动化测试结果视为生产部署验收通过。
|
||||
|
||||
- `[x]` 无任务仓储的同步证书申请路径已补齐项目资源可见性、`certificate.manage`、`deployment.execute` 与 DNS 凭据使用权限校验。
|
||||
- `[x]` 已补充项目证书权限不足、DNS 凭据使用权限不足及权限拒绝后不调用证书工作流的回归测试。
|
||||
- `[x]` 当前候选后端 `go test ./...`、`go vet ./...` 通过；该结果仅代表本地自动化验证，不代表三数据库、浏览器、远端服务器或生产部署验收通过。
|
||||
- `[~]` P0-07 其他资源写入权限、三数据库矩阵、浏览器清缓存验收、密码 SSH、远端部署及 Linux／Windows 生产服务仍需继续开发或复审。
|
||||
- `[!]` 总体验收不得升级，P0-03 仍为“完成，待复审”。
|
||||
|
||||
下一步继续执行前端回归与生产构建，再提交并推送本轮测试代码；完成开发阶段后，统一进行三数据库、浏览器、远端服务器和生产部署测试。
|
||||
@ -1387,3 +1387,15 @@ P0-11 生产验收门禁
|
||||
- [x] 本轮提交 35260bc、aa587d3 已提交；本轮最后一次推送成功。
|
||||
- [~] P0-07 仍为并行开发，任务重试子项完成待复审；三数据库矩阵、浏览器、SSH、远端部署、生产部署和完整权限联动仍未完成。
|
||||
- [!] 当前总体验收位置仍为 **P0-03 完成，待复审**；本轮自动化测试和源码开发不构成生产部署验收通过。
|
||||
|
||||
## 本轮开发进度（2026-09-29）
|
||||
|
||||
总体验收位置继续保持 **P0-03：完成，待复审**，本轮仅推进 P0-07 的同步证书申请权限闭环，不将局部自动化测试结果视为生产部署验收通过。
|
||||
|
||||
- `[x]` 无任务仓储的同步证书申请路径已补齐项目资源可见性、`certificate.manage`、`deployment.execute` 与 DNS 凭据使用权限校验。
|
||||
- `[x]` 已补充项目证书权限不足、DNS 凭据使用权限不足及权限拒绝后不调用证书工作流的回归测试。
|
||||
- `[x]` 当前候选后端 `go test ./...`、`go vet ./...` 通过；该结果仅代表本地自动化验证，不代表三数据库、浏览器、远端服务器或生产部署验收通过。
|
||||
- `[~]` P0-07 其他资源写入权限、三数据库矩阵、浏览器清缓存验收、密码 SSH、远端部署及 Linux／Windows 生产服务仍需继续开发或复审。
|
||||
- `[!]` 总体验收不得升级，P0-03 仍为“完成，待复审”。
|
||||
|
||||
下一步继续执行前端回归与生产构建，再提交并推送本轮测试代码；完成开发阶段后，统一进行三数据库、浏览器、远端服务器和生产部署测试。
|
||||
@ -336,3 +336,15 @@ P0-06 与 P0-07 均维持 `[~]`，六阶段中的隔离、生产和最终验收
|
||||
- [x] 当前本地自动化验证已通过：后端全量测试、静态检查、前端 230 项测试和生产构建。
|
||||
- [~] P0-07 的其他资源权限、三数据库、浏览器、SSH、远端部署和生产验收仍未闭环。
|
||||
- [!] 总体验收仍为 **P0-03 完成，待复审**；本节开发证据不替代实机生产验收证据。
|
||||
|
||||
## 本轮开发进度（2026-09-29）
|
||||
|
||||
总体验收位置继续保持 **P0-03：完成，待复审**，本轮仅推进 P0-07 的同步证书申请权限闭环，不将局部自动化测试结果视为生产部署验收通过。
|
||||
|
||||
- `[x]` 无任务仓储的同步证书申请路径已补齐项目资源可见性、`certificate.manage`、`deployment.execute` 与 DNS 凭据使用权限校验。
|
||||
- `[x]` 已补充项目证书权限不足、DNS 凭据使用权限不足及权限拒绝后不调用证书工作流的回归测试。
|
||||
- `[x]` 当前候选后端 `go test ./...`、`go vet ./...` 通过；该结果仅代表本地自动化验证，不代表三数据库、浏览器、远端服务器或生产部署验收通过。
|
||||
- `[~]` P0-07 其他资源写入权限、三数据库矩阵、浏览器清缓存验收、密码 SSH、远端部署及 Linux／Windows 生产服务仍需继续开发或复审。
|
||||
- `[!]` 总体验收不得升级，P0-03 仍为“完成，待复审”。
|
||||
|
||||
下一步继续执行前端回归与生产构建，再提交并推送本轮测试代码；完成开发阶段后，统一进行三数据库、浏览器、远端服务器和生产部署测试。
|
||||
Loading…
Reference in New Issue
Block a user