Skip to content

Conversation

@jsw800
Copy link
Contributor

@jsw800 jsw800 commented Nov 20, 2025

My team uses the Tsvector type, and we're trying to start using generators in tests, but tsvector currently doesn't have a generator so trying to generate something with it raises. This adds one that just always creates an empty list. I'm open to suggestions if you think the generator should be doing something more complex here—I assume this is a scenario where, if someone wants to be generating something specific for it in a test, they probably have a clear idea of what that is.

Contributor checklist

Leave anything that you believe does not apply unchecked.

  • I accept the AI Policy, or AI was not used in the creation of this PR.
  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

Copy link
Contributor

@zachdaniel zachdaniel left a comment

Choose a reason for hiding this comment

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

This is reasonable for now, doesn't hurt, but we may eventually want to figure out some way to do it in a more realistic way etc.

@zachdaniel zachdaniel merged commit d19d4ce into ash-project:main Nov 21, 2025
64 of 67 checks passed
@jsw800 jsw800 deleted the add-generators-to-types branch November 21, 2025 16:28
rgraff added a commit that referenced this pull request Dec 10, 2025
…tributes

* main: (23 commits)
  fix: do not build query filter with or: [] when all records are successfully inserted, for return_skipped_upsert?: true (#659)
  test: load aggregate with select and sort on from_many relationship (#661)
  test: add failing test for expr(has_one.function_calc) nil poisoning in batch (#660)
  test: aggregate with parent ref in relationship filter and sorting on relationship field (#658)
  test: binding error when loading aggregate (#657)
  chore(deps): bump ash_sql in the production-dependencies group (#656)
  test: multi relationship problems (#654)
  chore: release version v2.6.26
  chore: update ash_sql, use new select binding
  test: add complex calculation tests with filtered aggregates (#652)
  improvement: add generator to tsvector type (#655)
  test: add failing test for aggregate filtering on nested first aggregate (#653)
  chore: update ash_sql and add test
  docs: Mention :define_ecto_repo? option of use AshPostgres.Repo macro in getting started guide (#648)
  improvement: verify check constraint attributes at compile time
  chore: update ash
  chore: reuse compliance
  improvement: update deps for bug fixes
  chore: release version v2.6.25
  test: Add failing test for aggregate with parent() + select() + limit() (#644)
  ...
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