Page MenuHomePhabricator

[lib] Bind output validators to endpoints on lib
ClosedPublic

Authored by inka on May 14 2024, 5:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 6:04 PM
Unknown Object (File)
Mon, Dec 16, 6:04 PM
Unknown Object (File)
Mon, Dec 16, 6:04 PM
Unknown Object (File)
Mon, Dec 16, 6:04 PM
Unknown Object (File)
Oct 31 2024, 6:37 AM
Unknown Object (File)
Oct 26 2024, 5:06 AM
Unknown Object (File)
Oct 26 2024, 5:06 AM
Unknown Object (File)
Oct 26 2024, 5:05 AM
Subscribers

Details

Summary

issue: ENG-8080
As I explained in the issue, I wanted to simply use those objects instead of objects in lib/types/endpoints.js. But this causes very complicated import cycles. I know how to solve them, but it would requre refactoring a lot of files, hundreds possibly.
So instead I am creating a separate file with this mapping endpoint-output validator. The dev will not be able to forget about adding the validator here, because that would cause flow issues in the next diff.

Test Plan

Tested with next diffs - checked a couple of endpoints, and verified that no errors are thrown if outputs are validated with validators specified by endpointValidators for those endpoints.

Diff Detail

Repository
rCOMM Comm
Branch
inka/userStore
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka requested review of this revision.May 14 2024, 5:50 AM
tomek added inline comments.
lib/types/validators/endpoint-validators.js
65–110

Why do we split these into multiple objects but export a single value?

This revision is now accepted and ready to land.May 14 2024, 8:14 AM
lib/types/validators/endpoint-validators.js
65–110

I kept them split the way they are split in lib/types/endpoints.js. No reason I suppose, I just found it to be easier to navigate