-
Notifications
You must be signed in to change notification settings - Fork 53
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
🦺 Restore HackAndSlashTest.cs
#2444
🦺 Restore HackAndSlashTest.cs
#2444
Conversation
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
bf10b2e
to
4468a52
Compare
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
4468a52
to
0e9af4f
Compare
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
See #2424 for context. This is a rather serious issue considering we haven't had a proper test since
release/1.7.0
. Considering the size and scope of changes made inrelease/1.8.0
(libplanet 4.0 update), we should consider ourselves lucky that there wasn't a major problem pertaining to this particular issue.HackAndSlashTest.cs
in this PR was taken fromrelease/1.6.0
version ofHackAndSlash21Test.cs
and modified to fit the currentIWorld
API. See 4468a52 for the actual changes.I have removed some tests concerning old
AvatarState
model for following reasons:AvatarState
is not supported by the API.AvatarState
is legitimate is notHackAndSlash
's concern.