Add constraints format-clause with support for UNIQUE and PRIMARY KEY

[drew-patel]

Related issue - #15

• new constraints defhelper which adds a top-level :constraints key to the complete-sql-map
• moved format-clause for :with-columns into its own function so that it can be reused in the constraints format-clause method as well
• when constraints aren't present, the formatter uses :with-columns to construct the SQL string
• when constraints are present, the formatter uses :constraints to construct the columns and constraints string wrapped in the same set of parens

[scimetfoo]

@drew-patel thanks for submitting this PR!

A couple of suggestions:

1. Use defhelper only to update the query map with the constraint clauses and not to construct the SQL function call. We want to utilise the format-clause multimethod to construct the function call so that it works with honeysql's priority-based ordering rule system. Also, add a priority to the constraint clause, which I think would be a number slightly larger than the priority assigned to with-columns. I just noticed that with-columns doesn't have a priority assigned to it either, but it will need one now. Could you please add one for it as well?
2. Pluralise constraint so that going forward we're able to use this function for defining multiple constraints. Eg:

(constraints [:unique [:product_no :product_sku]]
             [:primary-key [:name]])

3. Add a top-level key called :constraints to the query map instead of associng the constraints to the collection of columns. It otherwise ends up treating each constraint as a column.
4. Support multiple constraints? It could also just be multiple unique constraints to being with.

[drew-patel]

@scimetfoo Thanks for the feedback and suggestions!

All the points are clear. I'll ask questions if I'm unsure about anything while working on them. Will get back to you in a few days with the changes.

[drew-patel]

@scimetfoo I have addressed comments 1-3 as described above. I've only implemented the solution for UNIQUE and PRIMARY KEY so far since they behave similarly.

• I have no idea why I'm getting an extra space after the table name. See one of the tests.

• Also, I'm not sure if the priority numbers chosen are ok. Happy to fix those if they aren't.

Thanks!

[scimetfoo]

Looks good. Please add examples to the readme.

[drew-patel]

@scimetfoo Please let me know if the examples are sufficient. Thanks again for the review!