PIMS-421 Routes - Properties

[dbarkowsky]

🎯 Summary

PIMS-421

These were all search routes. There was one, properties/search/names that I don't think is actually used currently (at least I can't find an example), so I omitted it.

Additionally, I changed the routes with wfs to geo. These are the ones that retrieve the lists of properties for map marker population right now. wfs isn't a very descriptive name, so I'm hoping that geo will make it a little more obvious what it's used for.

Some other considerations when we start implementing controllers...
It feels like the filter routes could just be part of the GET routes, but with some query parameters. There doesn't appear to be anything confidential transmitted, so the use of a POST might not be necessary.

Changes

• Added stub controllers for properties
• Added routes that expose these controllers
• Added stub integration and unit tests.

🔰 Checklist

• I have read and agree with the following checklist and am following the guidelines in our Code of Conduct document.

• I have performed a self-review of my code.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation where required.
• I have tested my changes to the best of my ability.
• My changes generate no new warnings.

[codeclimate]

Code Climate has analyzed commit 9932be3 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 93.7%.

View more on Code Climate.

[TaylorFries]

While testing the routes I get the following response:

"error": "User must have all of the following roles: [Admin]"

The following log is printed in the express-api:

{"level":"http","message":{"method":"GET","responseTimeMs":194.385,"route":"/api/v2/properties/search","status":403},"timestamp":"2024-01-03 11:14:06.934 AM"}

[TaylorFries]

All property routes return expected results. tested a few other routes too and they all act as expected.