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
{{ message }}
This repository has been archived by the owner on Jun 4, 2021. It is now read-only.
There are already tests for bad responses from SSA in the retirement_api.utils package, but there do not appear to be any tests further up the stack. How to the views handle it? What's that look like client-side?
This could simply be a "they are" and close the issue, but I just want to be sure. The screen scraping is scary (and I know it's scary to you too, @higs4281).
The text was updated successfully, but these errors were encountered:
@willbarton i don't think tests are needed further up the stack. the views don't call SSA. they call our internal endpoint, which calls SSA, and that's the first place we'll encounter any issue.
@higs4281 Alright. At a glance, it wasn't clear to me what happens if there's an error. It looks like the error just propagates up through the view via the JSON past the view (the estimator endpoint doesn't seem to check for it). Would browser tests to make sure the error JSON is handled make sense though?
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
There are already tests for bad responses from SSA in the
retirement_api.utils
package, but there do not appear to be any tests further up the stack. How to the views handle it? What's that look like client-side?This could simply be a "they are" and close the issue, but I just want to be sure. The screen scraping is scary (and I know it's scary to you too, @higs4281).
The text was updated successfully, but these errors were encountered: