Skip to content

Conversation

@ilovelinabell
Copy link

Changes: 4 tests cases: oneBidstoplimit, oneAsklimit, twoAskmarketoneBidlimitCROSSED, oneBidoneAskFILL, throw exception on CROSSEDMARKET condition

…BidlimitCROSSED, oneBidoneAskFILL, throw exception on CROSSEDMARKET condition
@ilovelinabell ilovelinabell linked an issue Mar 5, 2023 that may be closed by this pull request
@DanielMistrik DanielMistrik self-requested a review March 5, 2023 18:53
Copy link
Collaborator

@DanielMistrik DanielMistrik left a comment

Choose a reason for hiding this comment

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

I haven't reviewed all of it as it you have some Merge conflcits and I'm not sure whether your iml versions will work on everyone elses set-up (I will test it on my computer and lyk). For the time being try to resolve the merge issues and make sure that everything is working afterwards.

<root url="jar://$MODULE_DIR$/lib/junit-jupiter-params-5.7.0.jar!/" />
<root url="jar://$MODULE_DIR$/lib/junit-jupiter-engine-5.7.0.jar!/" />
<root url="jar://$MODULE_DIR$/lib/junit-platform-engine-1.7.0.jar!/" />
</CLASSES>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure whether this will work on other people's computers. Your unique set-up might make this not work on anyone elses. Also try to use Junit 5.7.0, not Junit 5.8.1

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.

Task 1 - Implement testOrderBook - Howard

3 participants