Flex close polygon

[br648]

Checklist

• [x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
• [x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
• [x] The description lists all applicable issues this PR seeks to resolve
• [x] The description lists any configuration setting(s) that differ from the default settings
• [x] All tests and CI builds passing

Description

Update to validate polygons and to add an additional shape (matching the first shape) if the polygon has not already been closed.

[br648]

@binh-dam-ibigroup and @miles-grant-ibigroup I'm not sure what has happened with the amount of tests in JDBCTableWriterTest. I may have branched from flex-table-additons instead of dev-flex. Having said that the only diff between dev-flex and flex-table-additions is the pattern reconciliation tests, so I've merged this into dev-flex (and then merged dev-flex into flex-close-polygon). The additional tests in JDBCTableWriterTest remain when I would only expect canValidateLocationShapes()!

@miles-grant-ibigroup I'll catch you later to discuss, but I think we can merge all working flex branches into dev-flex.

[br648]

@binh-dam-ibigroup and @miles-grant-ibigroup I'm not sure what has happened with the amount of tests in JDBCTableWriterTest. I may have branched from flex-table-additons instead of dev-flex. Having said that the only diff between dev-flex and flex-table-additions is the pattern reconciliation tests, so I've merged this into dev-flex (and then merged dev-flex into flex-close-polygon). The additional tests in JDBCTableWriterTest remain when I would only expect canValidateLocationShapes()!

@miles-grant-ibigroup I'll catch you later to discuss, but I think we can merge all working flex branches into dev-flex.

JDBCTableWriterTest test are now as expected after a push.