Gmtool open

[stm2]

There is now working shortcut for "loaddata" (I have no KEY_OPEN). I think 'O' is appropriate.

[stm2]

Shitshitshit, I commited too much. Therefore the force push. I genuinely don't understand why the checks failed, though.

[ennorehling]

Ich kann den Testfehler auch nicht reproduzieren, dafür habe ich ein anderes Problem:

Loaded testsuite with 392 tests in 40 testcases.

ERROR: current turn 1 is before first 184
ERROR: current turn 0 is before first 184
ERROR: current turn 1 is before first 184
ERROR: current turn 0 is before first 184
    F...........................................................
    ............................................................
    ............................................................
    ............................................................
    ............................................................
    ............................................................
    ................................

1196 Assertions checked.

  1) Failure (tests.e2.xmas.test_xmastree):
../scripts/tests/e2/xmas.lua:84: expected 0 but was nil

Das ist nicht das erste Mal, und ich stehe vor einem Rätsel, weil das eben ein Heisenbug ist, der schon lange immer wieder mal auftritt, und beim nächtsten Testlauf verschwindet.

Heisenbugs sind furchtbar, und so lange wir nicht erklären können, was da mit "error 22 during cr report" passiert ist, würde ich das lieber nicht mergen, auch wenn die Änderung ja wirklich trivial ist. Hast Du eine Idee, welcher Test das sein könnte?

[ennorehling]

Was wir wissen:

1. Die Meldung kommt aus write_reports (in reports.c)
2. Das wird in den Tests nur einmal aufgerufen, von test_write_reports in scripts/tests/shared/process.lua
3. Da kriegt die Partei aber vorher explizit eine neue Nummer: "NUMMER PARTEI 777"
4. Von einer Partei h72y ist nirgendwo die Rede.

Seltsam.

[ennorehling]

Ist auf jeden Fall unrelated zu dieser Änderung, tritt jetzt auch anderswo auf, siehe issue #1026