Skip to content

Conversation

@AmodChaubey1110
Copy link

@AmodChaubey1110 AmodChaubey1110 commented Jul 7, 2023

This PR is created for testing purposes only, No code changes are done. Testing is done to check Exception Handling in placeOrder, modifyOrder, and getOrderBook APIs for the error " Cannot read property 'status' of undefined ".

Copy link

@anurag-amx anurag-amx left a comment

Choose a reason for hiding this comment

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

  1. provide description .
  2. Unable to understand the PR. Is sss3Test.js added only? why so many commented lines?

@AmodChaubey1110
Copy link
Author

  1. provide description .
  2. Unable to understand the PR. Is sss3Test.js added only? why so many commented lines?

In sss3Test.js testing is done to check exception handling, there are no code changes done in this PR. Exception Handling issue was already resolved in the previous PR and merged also.

anurag-amx
anurag-amx previously approved these changes Jul 20, 2023
@AmodChaubey1110 AmodChaubey1110 dismissed anurag-amx’s stale review July 20, 2023 05:53

The merge-base changed after approval.

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.

2 participants