widelands

Merge lp://staging/~widelands-dev/widelands/bug-1825486-inputqueue-test2 into lp://staging/widelands

  1. bug-1825486-inputqueue-test2
  2. Merge into trunk
Proposed by GunChleoc
Status: Merged
Merged at revision: 9159
Proposed branch: lp://staging/~widelands-dev/widelands/bug-1825486-inputqueue-test2
Merge into: lp://staging/widelands
Diff against target: 12 lines (+2/-1)
1 file modified
test/maps/plain.wmf/scripting/test_inputqueues.lua (+2/-1)
To merge this branch: bzr merge lp://staging/~widelands-dev/widelands/bug-1825486-inputqueue-test2
Reviewer Review Type Date Requested Status
Widelands Developers Pending
Review via email: mp+369238@code.staging.launchpad.net

Commit message

Wait longer in test_inputqueues.lua after stable_save

To post a comment you must log in.
Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 5232. State: failed. Details: https://travis-ci.org/widelands/widelands/builds/549670864.
Appveyor build 5011. State: failed. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1825486_inputqueue_test2-5011.

Revision history for this message
GunChleoc (gunchleoc) wrote :

This version of my fix attempts has run successfully twice, so I'll go ahead and merge it.

I tried implementing an indicator into the save handler, but that does not work, because the test suite will reset the state before loading.

Travis failure is a temporary package downloading network issue for Mac

@bunnybot merge force

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: