HHS / OPRE-OPS

ACF's OPRE OPS product. Code name Unicorn.
Other
12 stars 3 forks source link

change table name user to ops_user #2435

Closed johndeange closed 3 weeks ago

johndeange commented 3 weeks ago

What changed

We found that our development Postgres DB in Azure is enforcing a stricter constraint than the Official Postgres Docker image, i.e. the Azure DB does not allow tables to be named user (since it is a keyword).

This PR renames the user table to ops_user

How to test

Automated tests pass.

Definition of Done Checklist

johndeange commented 3 weeks ago

@jonnalley Yes - there was a query that was returning results in an inconsistent order (did not have an order by) that was making the tests flaky. Also it was using the old Query API so I updated it to use the new SQLAlchemy v2 syntax and removed the flaky part of the test.