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

Add test asserting that time to throw is independent of size of thrown item stack size #39092

Merged
merged 1 commit into from
Mar 29, 2020

Conversation

kevingranade
Copy link
Member

Summary

SUMMARY: None

Purpose of change

Simple regression test for #39090
Cost to throw an item unexpectedly depended on wielded stack size, prevent that from accidentally recurring.

Describe the solution

Have a player wield a stack of items and check time to throw while decreasing stack size.

Testing

Revert a0fe8c1 and run this test, you can see it fail.
Un-revert a0fe8c1 and run the test, and it passes.

Additional context

Easy to extend this to multiple item types, but I don't see a reason at the moment.

@ZhilkinSerg ZhilkinSerg added [C++] Changes (can be) made in C++. Previously named `Code` Code: Tests Measurement, self-control, statistics, balancing. labels Mar 29, 2020
@ZhilkinSerg ZhilkinSerg merged commit 2c3f6c3 into CleverRaven:master Mar 29, 2020
@kevingranade kevingranade deleted the time-to-throw-test branch June 28, 2020 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Code: Tests Measurement, self-control, statistics, balancing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants