fix(Aggregate Uniq Count):

[simpers]

Added a test to demonstrate the aggregate issue

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

[simpers]

Ignore this for now, as I had forgotten that the SQL stuff is (probably) in AshSql. The unit test shows the issue though, but the fix is probably not in this repo.

Whether it is useful to have these tests here anyway, I'm not sure, but until I redo this in the AshSql repo I'll just keep this here for my own memory 🤓

[zachdaniel]

@simpers we actually don't really have tests in ash_sql so generally we commit failing tests here and then fix it in ash_sql. 😄

[zachdaniel]

If you're ready to merge this then lets mark it ready and I'll merge 🙇

[simpers]

@simpers we actually don't really have tests in ash_sql so generally we commit failing tests here and then fix it in ash_sql. 😄

Haha I noticed that after posting 😅 I will try to wrap it up tonight or tomorrow, bit swamped at the moment.