You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Tests in main_test.go has been expanded to cover new cases as features has been added or changed. Several of the tested features are hosed in endpoint.go or request.go in the api folder. Maybe it would be more natural to move some of these tests to request_tests.go. It could also be considered to create endpoint_test.go if this is structurally beneficial.
It should also be considered if NormalizeConnection can be tested directly and not via an endpoint.
What is the reason for "utils_main_test.go"?
If we deviate from the pattern where "filename_test.go" test functions in "filename.go" it would be beneficial to add a line or two describing the motivation.
The text was updated successfully, but these errors were encountered:
Tests in main_test.go has been expanded to cover new cases as features has been added or changed. Several of the tested features are hosed in endpoint.go or request.go in the api folder. Maybe it would be more natural to move some of these tests to request_tests.go. It could also be considered to create endpoint_test.go if this is structurally beneficial.
It should also be considered if NormalizeConnection can be tested directly and not via an endpoint.
What is the reason for "utils_main_test.go"?
If we deviate from the pattern where "filename_test.go" test functions in "filename.go" it would be beneficial to add a line or two describing the motivation.
The text was updated successfully, but these errors were encountered: