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

Optimise test framework #78

Closed
1 of 2 tasks
OGKevin opened this issue Apr 3, 2018 · 0 comments · Fixed by #81
Closed
1 of 2 tasks

Optimise test framework #78

OGKevin opened this issue Apr 3, 2018 · 0 comments · Fixed by #81
Assignees
@OGKevin
Labels
enhancement

Comments

@OGKevin
Copy link
Contributor

OGKevin commented Apr 3, 2018

Steps to reproduce:

  1. Try and execute tests

What should happen:

  1. It should be easy and fast

What happens:

  1. It is hard to set up and can take up to 10 mins to complete.

Traceback

SDK version and environment

  • Tested on 0.12.4
  • Sandbox
  • Production

Response id

  • Response id:

Extra info:

Reference: bunq/sdk_php#120
@OGKevin OGKevin added this to the 0.13.5 milestone Apr 3, 2018
@OGKevin OGKevin self-assigned this Apr 3, 2018
@OGKevin OGKevin changed the title Optimize test framework Optimise test framework Apr 3, 2018
OGKevin added a commit that referenced this issue Apr 5, 2018
@OGKevin
Added util with automatic install. (#78)
2bef3e6
OGKevin added a commit that referenced this issue Apr 5, 2018
@OGKevin
Added more functionality to base test class. (#78)
e37da36
OGKevin added a commit that referenced this issue Apr 5, 2018
@OGKevin
Moved constatns accordingly and use parent class methods. (#78)
475c50b
OGKevin added a commit that referenced this issue Apr 5, 2018
@OGKevin
Call parent setup class. (#78)
0709090
@OGKevin OGKevin mentioned this issue Apr 5, 2018
Merged
1 task
OGKevin added a commit that referenced this issue Apr 5, 2018
@OGKevin
Removed unneeded empty new line. (#78)
1b6a2ac
OGKevin added a commit that referenced this issue Apr 6, 2018
@OGKevin
Added missing doc and removed unneeded import. (#78)
3beb284
OGKevin added a commit that referenced this issue Apr 6, 2018
@OGKevin
Shorten code lines. (#78)
1ffd10f
OGKevin added a commit that referenced this issue Apr 6, 2018
@OGKevin
Removed unneeded new line. (#78)
84a95e0
OGKevin added a commit that referenced this issue Apr 6, 2018
@OGKevin
Removed magic values. (#78)
2962349
OGKevin added a commit that referenced this issue Apr 6, 2018
@OGKevin
Return api context instead of saving it to file. (#78)
bd2f987
OGKevin added a commit that referenced this issue Apr 7, 2018
@OGKevin
Use local import to prevent import errors. (#78)
4bb5612
OGKevin added a commit that referenced this issue Apr 7, 2018
@OGKevin
Fixed broken reset session method. (#78)
18933db
OGKevin added a commit that referenced this issue Apr 7, 2018
@OGKevin
Removed direct import and use package import. (#78)
cb3d71a
OGKevin added a commit that referenced this issue Apr 7, 2018
@OGKevin
Fixed broken tests due to bad file path. (#78)
8b46486
OGKevin added a commit that referenced this issue Apr 7, 2018
@OGKevin
Split import accordingly. (#78)
56583cd
OGKevin added a commit that referenced this issue Apr 7, 2018
@OGKevin
Removed magic value. (#78)
b23614f
@jeroenmeulenaar jeroenmeulenaar mentioned this issue Apr 7, 2018
Closed
@OGKevin OGKevin mentioned this issue Apr 8, 2018
Closed
OGKevin added a commit that referenced this issue Apr 12, 2018
@OGKevin
Merge pull request #81 from bunq/optimise_test_framework_auto_topup_b…
15237f6 
…unq/sdk_python#78

Optimise test framework auto topup #78
OGKevin added a commit that referenced this issue Sep 18, 2018
@OGKevin
Deleted example config. (#78)
b9a8eed
OGKevin added a commit that referenced this issue Sep 18, 2018
@OGKevin
Removed config imports. (#78)
cc5ef6f
OGKevin added a commit that referenced this issue Sep 18, 2018
@OGKevin
Removed API key retrieval from config. (#78)
c5d502f
@OGKevin OGKevin mentioned this issue Sep 18, 2018
Merged
OGKevin added a commit that referenced this issue Sep 18, 2018
@OGKevin
Removed API key retrieval from config. (#78)
0d54d43
OGKevin added a commit that referenced this issue Sep 18, 2018
@OGKevin
Merge pull request #106 from bunq/#78/remove-config-dep
33d107b 
#78/remove config dep
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OGKevin OGKevin

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Optimise test framework auto topup bunq/sdk_python#78 bunq/sdk_python
1 participant
@OGKevin