Merge pull request nashby#9 from edchiou/master

*Finally fixed

Author: nashby

lib/dota/match.rb

@@ -233,6 +233,27 @@ def gpm
       def items
         (0..5).map { |i| Constants::Items[raw_player["item_#{i}"]] }
       end
+
+      # List of items on secondary unit (e.g. Syllabear's bear)
+      #
+      # @return [Array<String>] an array of item names
+      def additional_unit_items
+        (0..5).map { |i| Items[raw_player['additional_units'][0]["item_#{i}"]] }
+      end
+
+      # Names of secondary units
+      #
+      # @return [Array<String>] an array of secondary unit names
+      def additional_unit_names
+        raw_player['additional_units'][0]['unitname']   
+      end
+
+      # List of ability upgrades, their timings, and level taken at
+      #
+      # @return [Array<Hash>] an array of ability upgrades
+      def upgrades
+        (0..raw_player['ability_upgrades'].length).map { |i| raw_player['ability_upgrades'][i] }
+      end
     end
   end
 end

spec/fixtures/match_player.json

@@ -21,5 +21,54 @@
   "hero_damage": 8270,
   "tower_damage": 597,
   "hero_healing": 39,
-  "level": 13
+  "level": 13,
+  "ability_upgrades": [
+					{
+						"ability": 5412,
+						"time": 97,
+						"level": 1
+					},
+					{
+						"ability": 5414,
+						"time": 277,
+						"level": 2
+					},
+					{
+						"ability": 5412,
+						"time": 329,
+						"level": 3
+					},
+					{
+						"ability": 5414,
+						"time": 460,
+						"level": 4
+					},
+					{
+						"ability": 5412,
+						"time": 580,
+						"level": 5
+					},
+					{
+						"ability": 5415,
+						"time": 794,
+						"level": 6
+					},
+					{
+						"ability": 5412,
+						"time": 1075,
+						"level": 7
+					}
+				]
+				,
+				"additional_units": [
+					{
+						"unitname": "spirit_bear",
+						"item_0": 181,
+						"item_1": 182,
+						"item_2": 0,
+						"item_3": 0,
+						"item_4": 0,
+						"item_5": 0
+					}
+				]
 }

spec/match_player_spec.rb

@@ -41,7 +41,7 @@ def player.raw_player; { 'leaver_status' => 2 }; end
     player.last_hits.must_equal 98
   end
 
-  it 'returns the number of times a player denied a creep' do
+  it 'returns denies' do
     player.denies.must_equal 2
   end
 
@@ -76,4 +76,16 @@ def player.raw_player; { 'leaver_status' => 2 }; end
   it 'returns xpm' do
     player.xpm.must_equal 304
   end
+
+  it 'returns additional unit names' do
+    player.additional_unit_names.must_equal 'spirit_bear'
+  end
+
+  it 'returns additional unit items' do
+    player.additional_unit_items.must_equal ['orb_of_venom', 'stout_shield', 'emptyitembg', 'emptyitembg', 'emptyitembg', 'emptyitembg']
+  end
+
+  it 'returns ability upgrades' do
+    player.upgrades[0].must_equal ({"ability" => 5412, "time" => 97, "level" => 1})
+  end
 end