Skip to content

Conversation

@yuly3
Copy link
Collaborator

@yuly3 yuly3 commented Aug 21, 2025

Summary

This PR enhances SnowflakeColumn and SnowflakeColumnType structs to make them easier to mock and test in user code. The primary goal is to improve the developer experience when writing tests that involve these column metadata structures.

Changes

  • Add new() constructors for both SnowflakeColumn and SnowflakeColumnType
  • Implement PartialEq and Eq traits to enable equality comparisons
  • Add comprehensive documentation with usage examples
  • Expand integration tests to cover various Snowflake data types

Benefits

Users can now easily create mock instances of these structs in their tests without relying on internal implementation details. The equality traits also enable straightforward assertions in test scenarios.

yuly3 added 2 commits August 21, 2025 14:36
- Add `new` method for SnowflakeColumn and SnowflakeColumnType
- Implement PartialEq and Eq traits for both structs
- Add comprehensive documentation with doctests and examples
- Expand integration tests to validate column_types() with multiple data types
- Test coverage includes fixed, text, boolean, date, and timestamp_ntz types
@yuly3 yuly3 self-assigned this Aug 21, 2025
Reverts to the original test behavior that inserts two rows of data
@yuly3 yuly3 marked this pull request as ready for review August 21, 2025 06:06
@yuly3 yuly3 requested a review from kenkoooo August 21, 2025 06:11
Copy link
Member

@kenkoooo kenkoooo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@yuly3 yuly3 merged commit f655c0f into main Aug 21, 2025
6 checks passed
@yuly3 yuly3 deleted the feat/add-column-constructor-and-equality branch August 21, 2025 06:48
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