Skip to content

fix(opsworks): add internal sdk_completeness_test without SDK v2 import (go-5uju)#2233

Merged
agbishop merged 4 commits into
mainfrom
polecat/jade/go-5uju@mqbpj2bm
Jun 13, 2026
Merged

fix(opsworks): add internal sdk_completeness_test without SDK v2 import (go-5uju)#2233
agbishop merged 4 commits into
mainfrom
polecat/jade/go-5uju@mqbpj2bm

Conversation

@agbishop

Copy link
Copy Markdown
Collaborator

Fix OpsWorks SDK completeness test for deprecated service.

Problem

  • aws-sdk-go-v2/service/opsworks does not exist (OpsWorks deprecated)
  • sdk_completeness_test.go references non-existent package
  • Build/test failures due to import

Solution

  • Add internal sdk_completeness_test.go for opsworks
  • Validates internal consistency without SDK v2 import
  • Tests work with internal implementation only

Testing

  • go build ./services/opsworks/... ✓
  • go test ./services/opsworks/... ✓

… import

OpsWorks was deprecated and not ported to aws-sdk-go-v2, so there is no
aws-sdk-go-v2/service/opsworks package. Add sdk_completeness_test.go that
verifies internal consistency (no duplicate ops, dispatch table matches
GetSupportedOperations) instead of comparing against an SDK client.

Export HandlerDispatchKeys via export_test.go to enable the comparison.

Closes go-5uju
@agbishop agbishop enabled auto-merge (squash) June 13, 2026 02:05
@agbishop

agbishop commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator Author

📊 Code Coverage Report

Metric Value Status
Total Coverage 0.0%
82.7%
New Code Coverage N/A (0/0 stmts)

Tip

This project maintains a minimum coverage threshold of 85%. Maintain or improve coverage on new code to ensure long-term stability.


Last updated: Sat, 13 Jun 2026 07:20:00 GMT

@agbishop agbishop merged commit 9997b04 into main Jun 13, 2026
26 checks passed
@agbishop agbishop deleted the polecat/jade/go-5uju@mqbpj2bm branch June 13, 2026 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant