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.

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

feat: added bulk sdk tests and onboarding #97

Merged
merged 14 commits into from
Nov 15, 2022
Merged

feat: added bulk sdk tests and onboarding #97

merged 14 commits into from
Nov 15, 2022

Conversation

mdebarros
Copy link
Member

  • updates for rules files and additional collections

  • updated request with env vars

  • Updated happy path to remove to fspId

  • Updated test cases

  • Added TC for NDC limit error

  • renamed variables to use TTKSIM instead of SIM

  • removed url variable

  • Updates to parameterize parties error test cases

  • Updates with latest async approach

  • updates for async feature

  • Updates for bad request test case

  • Updated parties errors with async callbacks

  • Updates for async test mode

  • Updates for quotes error cases

  • Updates for transfers -ve test case to use async

  • Updated TC based on the updated rule

  • Updates to TC2 of transfers

  • updated TC2

  • Updates for TC4

  • Added env file for onboarding

  • added env file

  • Updated TC5 with async

  • Updated TC for parties error cases

  • Updated test for async

  • Updated BQ-10

  • Updates to TC-4

  • Split the TCs into failures and pass

  • move failures to a different folder

  • changed folder and file names

  • Updated test cases to reflect correct status

  • updates to tests

  • updated TC-4

  • renamed folders and file names

  • updates

  • fix: aligned with rules file in mvp bulk branch

  • fix: changed example env files

Co-authored-by: Vijay [email protected]

Vijay and others added 12 commits October 14, 2022 15:58
* updates for rules files and additional collections

* updated request with env vars

* Updated happy path to remove to fspId

* Updated test cases

* Added TC for NDC limit error

* renamed variables to use TTKSIM instead of SIM

* removed url variable

* Updates to parameterize parties error test cases

* Updates with latest async approach

* updates for async feature

* Updates for bad request test case

* Updated parties errors with async callbacks

* Updates for async test mode

* Updates for quotes error cases

* Updates for transfers -ve test case to use async

* Updated TC based on the updated rule

* Updates to TC2 of transfers

* updated TC2

* Updates for TC4

* Added env file for onboarding

* added env file

* Updated TC5 with async

* Updated TC for parties error cases

* Updated test for async

* Updated BQ-10

* Updates to TC-4

* Split the TCs into failures and pass

* move failures to a different folder

* changed folder and file names

* Updated test cases to reflect correct status

* updates to tests

* updated TC-4

* renamed folders and file names

* updates

* fix: aligned with rules file in mvp bulk branch

* fix: changed example env files

Co-authored-by: Vijay <[email protected]>
* Updates to parties test cases

* updates to assertions

* Added a new test case

* Updates for TC-BQ9

* updates for BQ-9

* refactoring of test cases
* Status updated on BP2

* State update on BP6

* fix: fixed tests and changed some values in sdk bulk tests

Co-authored-by: Sridevi Miriyala <[email protected]>
vijayg10
vijayg10 previously approved these changes Nov 14, 2022
sri-miriyala
sri-miriyala previously approved these changes Nov 14, 2022
Copy link
Contributor

@sri-miriyala sri-miriyala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mdebarros mdebarros dismissed stale reviews from sri-miriyala and vijayg10 via 9b47b92 November 14, 2022 15:30
Copy link
Contributor

@sri-miriyala sri-miriyala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@mdebarros mdebarros merged commit 3e39645 into master Nov 15, 2022
@mdebarros mdebarros deleted the mvp/bulk-sdk branch November 15, 2022 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants