Merge remote-tracking branch 'graylin/hots'

Author: dsjoerg

sc2reader/resources.py

@@ -987,7 +987,7 @@ def load_players(self):
             if not struct[0][1]: continue # Slot is closed
 
             player = PlayerSummary(struct[0][0])
-            stats = self.player_stats[index]
+            stats = self.player_stats.get(index, dict())
             settings = self.player_settings[index]
             player.is_ai = not isinstance(struct[0][1], dict)
             if not player.is_ai:

sc2reader/scripts/sc2parse.py

@@ -43,7 +43,7 @@ def main():
                     replay = sc2reader.load_replay(path, debug=True, load_level=1)
                     if not args.one_each or replay.is_ladder:
                         replay = sc2reader.load_replay(path, debug=True)
-                        print 'No problems with {build} - {real_type} on {map_name} - Played {start_time}'.format(**replay.__dict__)
+                        print 'No problems with {path}: {build} - {real_type} on {map_name} - Played {start_time}'.format(path=path, **replay.__dict__)
             except sc2reader.exceptions.ReadError as e:
                 if args.ladder_only and not e.replay.is_ladder: continue
 

test_s2gs/hots1.s2gs

@@ -0,0 +1,17 @@
+# Encoding: UTF-8
+
+# Run tests with "py.test" in the project root dir
+import os, sys
+import pytest
+import datetime
+
+sys.path.insert(0, os.path.normpath(os.path.join(os.path.dirname(os.path.abspath(__file__)),"../")))
+
+import sc2reader
+from sc2reader.exceptions import ParseError
+
+def test_a_WoL_s2gs():
+    summary = sc2reader.load_game_summary("test_s2gs/s2gs1.s2gs")
+
+def test_a_HotS_s2gs():
+    summary = sc2reader.load_game_summary("test_s2gs/hots1.s2gs")