Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

playground/client: API Error wrapping #38

Open
jxson opened this issue Apr 22, 2015 · 0 comments
Open

playground/client: API Error wrapping #38

jxson opened this issue Apr 22, 2015 · 0 comments
Assignees
@jxson
Labels
Area: Playground

Comments

@jxson
Copy link
Member

jxson commented Apr 22, 2015

Error cases that should be covered:

  • Not ok responses (HTTP error response wrapping)
  • Timeouts
  • Connection error
  • JSON parsing errors
@jxson jxson self-assigned this Apr 22, 2015
@jxson jxson mentioned this issue Apr 22, 2015
Open
11 tasks
jxson added a commit to vanadium-archive/playground that referenced this issue May 6, 2015
@jxson
TBR playground/client: add JSON stream test coverage
7e49e0a 
* Add tests for error and edge cases for the JSONStream
* api.run() returns stream directly instead of having the extra callback.
* Add initial message to let the user know that the run has started.

Change-Id: I9056da42389c4b4e3dd62e00b9e0b2d69175800b
Related: vanadium/issues#38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jxson jxson

Labels
Area: Playground
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jxson