fix build order bug, and tests that would have caught it

Author: dsjoerg

sc2reader/resources.py

@@ -945,10 +945,10 @@ def load_player_stats(self):
             #   }
             stat_id = item[0][1]
             if stat_id in translation:
+                stat_name = translation[stat_id]
                 # Assume anything under 1 million is a normal score screen item
                 # Build order ids are generally 16 million+
                 if stat_id < 1000000:
-                    stat_name = translation[stat_id]
                     for pid, value in enumerate(item[1]):
                         if not value: continue
 

test_s2gs/test_all_s2gs.py

@@ -13,7 +13,9 @@
 def test_a_WoL_s2gs():
     summary = sc2reader.load_game_summary("test_s2gs/s2gs1.s2gs")
     assert summary.players[0].resource_collection_rate == 1276
+    assert summary.players[0].build_order[0].order == 'Probe'
 
 def test_a_HotS_s2gs():
     summary = sc2reader.load_game_summary("test_s2gs/hots1.s2gs")
     assert summary.players[0].resource_collection_rate == 1599
+    assert summary.players[0].build_order[0].order == 'SCV'