[Dan-Knott]

I've rebased all of the Enums for the functions and triggers on FromStrings and added __post_init__ to normalise them back to their enum values.

I decided to do the from_string conversion in __post_init__ so bad definitions fail at import instead of only coming up when being compared against the db.

Also added the missing with_volatility and with_parallel methods to the Postgres Function class.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.26%. Comparing base (111029c) to head (4080b4c).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #153      +/-   ##
==========================================
+ Coverage   89.93%   90.26%   +0.33%     
==========================================
  Files          86       86              
  Lines        4151     4179      +28     
  Branches      861      861              
==========================================
+ Hits         3733     3772      +39     
+ Misses        338      329       -9     
+ Partials       80       78       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[DanCardin]

A drawback of using __post_init__ is that the api (and internal impl broadly) is very heavily dataclasses.replace based, so it'll get normalized a bunch of times repeatedly. It's not like its the worst thing in the world, but it's somewhat unfortunate.

[DanCardin]

with that said, 1. i'm not too worried about it and 2. it could probably be made more efficient (or at least not to unnecessarily perform the assignment) without breaking anything if it turns out to be a problem.