Genserver timeout #8
Assignees
Comments
pkrawat1
added
severity: low
severity: medium
hotlist: core-team
effort1: easy(hours)
and removed
severity: medium
labels
oyeb
added a commit
that referenced
this issue
Jan 30, 2018
* Integration tests no longer use the worker as a workaround for #8 * Added more test cases, can possibly be improved using describe blocks with local setup. * There are almost no assertions and it is expected that errors will bubble up to the pattern matches.
Merged
oyeb
added a commit
that referenced
this issue
Feb 15, 2018
oyeb
added a commit
that referenced
this issue
Mar 13, 2018
* Integration tests no longer use the worker as a workaround for #8 * Added more test cases, can possibly be improved using describe blocks with local setup. * There are almost no assertions and it is expected that errors will bubble up to the pattern matches.
ashish173
pushed a commit
that referenced
this issue
Mar 15, 2018
* Ignore some optional params for RF, RV, CP Some optional params like billing, customer, merchant must not be expanded in case of capture, refund and void. * Improved mock tests, fixes #98 Mock tests now mostly check if the request is correctly built. Since most requests have common parameters, they are not checked everywhere. * Improve integration tests (more cases), fixes #108 * Integration tests no longer use the worker as a workaround for #8 * Added more test cases, can possibly be improved using describe blocks with local setup. * There are almost no assertions and it is expected that errors will bubble up to the pattern matches.
ashish173
pushed a commit
that referenced
this issue
Apr 22, 2018
ashish173
pushed a commit
that referenced
this issue
Apr 22, 2018
* Ignore some optional params for RF, RV, CP Some optional params like billing, customer, merchant must not be expanded in case of capture, refund and void. * Improved mock tests, fixes #98 Mock tests now mostly check if the request is correctly built. Since most requests have common parameters, they are not checked everywhere. * Improve integration tests (more cases), fixes #108 * Integration tests no longer use the worker as a workaround for #8 * Added more test cases, can possibly be improved using describe blocks with local setup. * There are almost no assertions and it is expected that errors will bubble up to the pattern matches.
ashish173
pushed a commit
that referenced
this issue
Apr 22, 2018
ashish173
pushed a commit
that referenced
this issue
Apr 22, 2018
* Ignore some optional params for RF, RV, CP Some optional params like billing, customer, merchant must not be expanded in case of capture, refund and void. * Improved mock tests, fixes #98 Mock tests now mostly check if the request is correctly built. Since most requests have common parameters, they are not checked everywhere. * Improve integration tests (more cases), fixes #108 * Integration tests no longer use the worker as a workaround for #8 * Added more test cases, can possibly be improved using describe blocks with local setup. * There are almost no assertions and it is expected that errors will bubble up to the pattern matches.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When the api takes more time to respond, the Genserver timeouts.
One options is to wait for the request to complete using
call(worker, {:authorize, gateway, amount, card, opts}, :infinity)
There is third filed which whcih specifies time in milliseconds for the handle_call of genserver. Default is 5s
The text was updated successfully, but these errors were encountered: