Skip to content

test: add unsupported-type Scan case and clarify examples#5

Merged
min0625 merged 1 commit into
mainfrom
feat/improve-tests-and-examples
Jun 23, 2026
Merged

test: add unsupported-type Scan case and clarify examples#5
min0625 merged 1 commit into
mainfrom
feat/improve-tests-and-examples

Conversation

@min0625

@min0625 min0625 commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add a table-driven test entry in TestTimeZone_Scan that verifies Scan rejects an unsupported input type ([]int) while leaving the receiver unchanged
  • Split a chained one-liner in ExampleTimeZone into a named variable for readability
  • Add inline comments to ExampleTimeZone_zeroValue and ExampleTimeZone_Value_zeroValue to surface the UTC-as-zero-value and no-NULL invariants directly in the runnable docs

Test plan

  • make test passes (new test case covered by table-driven runner)
  • make lint passes (no new linting issues)

🤖 Generated with Claude Code

Add a table-driven test entry that verifies Scan rejects an unsupported
input type ([]int) while leaving the receiver unchanged. Also split a
chained one-liner in ExampleTimeZone and add inline comments to the
zero-value and SQL Value examples to surface the UTC-as-zero-value and
no-NULL invariants directly in the runnable docs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@min0625 min0625 merged commit f9745e1 into main Jun 23, 2026
5 checks passed
@min0625 min0625 deleted the feat/improve-tests-and-examples branch June 23, 2026 04:16
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.

2 participants