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.

Sign up for GitHub

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

Jump to bottom

[1.16] Add a unit test for [move_unit]location_id with an occupied hex #6571

Merged
merged 1 commit into from Mar 29, 2022
Merged

[1.16] Add a unit test for [move_unit]location_id with an occupied hex #6571

merged 1 commit into from Mar 29, 2022

Conversation

stevecotton
Copy link
Contributor

No description provided.

@stevecotton stevecotton self-assigned this Mar 22, 2022
@github-actions github-actions bot added the Unit Tests Issues involving Wesnoth's unit test suite. label Mar 22, 2022
@stevecotton stevecotton changed the title Add a unit test for [move_unit]location_id with an occupied hex [1.16] Add a unit test for [move_unit]location_id with an occupied hex Mar 22, 2022
@stevecotton stevecotton added the Fwdport A reminder of a bugfix that was added to the stable branch that needs to be duplicated on master. label Mar 22, 2022
Pentarctagon
Pentarctagon reviewed Mar 28, 2022
View reviewed changes
data/test/scenarios/test_move_unit_to_occupied_hex.cfg

[move_unit]
id=alice
to_location=2
Copy link
Member

Choose a reason for hiding this comment

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

This is a location that has already been defined, I assume?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Numerical location ids 1, 2, etc correspond to the starting location for that side.

@stevecotton stevecotton merged commit ba1767f into wesnoth:1.16 Mar 29, 2022
@stevecotton stevecotton deleted the test_move_to_occupied branch March 29, 2022 07:24
@stevecotton stevecotton mentioned this pull request Mar 31, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pentarctagon Pentarctagon Pentarctagon approved these changes

Assignees

@stevecotton stevecotton

Labels
Fwdport A reminder of a bugfix that was added to the stable branch that needs to be duplicated on master. Unit Tests Issues involving Wesnoth's unit test suite.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@stevecotton @Pentarctagon