Skip to content

Simplify field key assertions using set comparison#151

Open
denini08 wants to merge 1 commit intoAggify:mainfrom
denini08:list-set
Open

Simplify field key assertions using set comparison#151
denini08 wants to merge 1 commit intoAggify:mainfrom
denini08:list-set

Conversation

@denini08
Copy link

This PR updates some assertions in test_aggify.py to:

  • Compare directly with a set instead of converting keys to a list for faster lookups O(1).
  • Make the tests simpler and cleaner.
  • Keep the same test logic and behavior.

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