Skip to content

test(s3): expand service and dashboard coverage#15

Merged
simota merged 1 commit intomainfrom
feat/s3-test-coverage
May 6, 2026
Merged

test(s3): expand service and dashboard coverage#15
simota merged 1 commit intomainfrom
feat/s3-test-coverage

Conversation

@simota
Copy link
Copy Markdown
Owner

@simota simota commented May 6, 2026

Summary

  • Add targeted S3 service edge/error tests for bucket, object, metadata, copy, multipart, and presigned URL behavior.
  • Add dashboard /api/s3 coverage for validation, disabled state, metadata propagation, mutation safety, and destructive paths.
  • Add S3 test coverage autoloop with service, dashboard API, E2E/docs, and coverage threshold gates.

Verification

  • VERIFY_STAGE=full-test-coverage bash scripts/s3-test-coverage-autoloop/verify.sh
    • passed=7 failed=0
  • go test -cover ./internal/services/s3
    • coverage: 72.6% of statements
  • go test -cover ./internal/dashboard
    • coverage: 69.8% of statements

Notes

  • Test-focused change. Product behavior should remain unchanged except for defects a new test would expose.
  • Tests avoid external AWS calls and must not log credentials, Authorization headers, SigV4 material, presigned secrets, or object payloads.

@simota simota merged commit d2782d5 into main May 6, 2026
0 of 2 checks passed
@simota simota deleted the feat/s3-test-coverage branch May 6, 2026 03:14
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