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

Fix Land Scramble Error by preventing a route to/from the same territory #4494

Merged
merged 1 commit into from
Jan 3, 2019

Conversation

ron-murhammer
Copy link
Member

Overview

Functional Changes

  • Add check to avoid a route from/to same territory which causes a crash

Manual Testing Performed

  • Tested with this save game by scrambling UK fighters which then lose their starting territory:
    test.zip

@ron-murhammer ron-murhammer changed the title Add check to prevent having a route to/from the same territory Fix Land Scramble Error by preventing a route to/from the same territory Jan 3, 2019
@codecov-io
Copy link

Codecov Report

Merging #4494 into master will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #4494      +/-   ##
============================================
+ Coverage     22.96%   22.96%   +<.01%     
- Complexity     6209     6211       +2     
============================================
  Files           880      880              
  Lines         71236    71236              
  Branches      11335    11335              
============================================
+ Hits          16357    16360       +3     
+ Misses        52847    52846       -1     
+ Partials       2032     2030       -2
Impacted Files Coverage Δ Complexity Δ
...ames/strategy/triplea/delegate/BattleDelegate.java 24.42% <0%> (ø) 58 <0> (ø) ⬇️
.../strategy/triplea/odds/calculator/DummyPlayer.java 35.95% <0%> (+1.12%) 10% <0%> (+1%) ⬆️
...rategy/triplea/attachments/UnitTypeComparator.java 53.84% <0%> (+7.69%) 13% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bea06d8...a63763d. Read the comment docs.

Copy link
Member

@RoiEXLab RoiEXLab left a comment

Choose a reason for hiding this comment

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

👍

@RoiEXLab RoiEXLab merged commit 513cb9e into master Jan 3, 2019
@RoiEXLab RoiEXLab deleted the Fix_Land_Scrambling_Bug branch January 3, 2019 22:08
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.

None yet

3 participants