fixing params bug

Author: danielwelch

tests/test_xmlstats.py

@@ -88,7 +88,7 @@ def test_teams(self):
         self.assertEqual(result[0].team_id, "atlanta-hawks")
 
     def test_team_results(self):
-        result = self.s.team_results(sport="nba", team_id="atlanta-hawks")
+        result = self.s.team_results(sport="nba", team_id="atlanta-hawks", event_status="completed", season="2016")
         self.assertEqual(result[0].team.team_id, "atlanta-hawks")
 
     def test_blog_example(self):

xmlstats/xmlstats.py

@@ -33,7 +33,8 @@ def _format_result(self, name, data_str):
                 object_hook=lambda d: namedtuple(name, d.keys())(*d.values())
         )
 
-    def _build_url(self, method, sport=None, date=None, id=None, format="json"):
+    def _build_url(self, method, sport=None,
+                   date=None, id=None, format="json"):
         host = "https://erikberg.com/"
         path = "/".join(filter(None, (sport, method, date, id)))
         url = host + path + "." + format
@@ -99,15 +100,15 @@ def team_results(self, sport, team_id, season=None, opponent=None,
         data = self._http_get(
                 self._build_url(method="results", sport=sport, id=team_id),
                 params={
-                    season: season,
-                    opponent: opponent,
-                    location_type: location_type,
-                    event_status: event_status,
-                    since: since,
-                    until: until,
-                    last: last,
-                    next: next,
-                    order: order
+                    "season": season,
+                    "opponent": opponent,
+                    "location_type": location_type,
+                    "event_status": event_status,
+                    "since": since,
+                    "until": until,
+                    "last": last,
+                    "next": next,
+                    "order": order
                 }
         )
         return self._format_result("TeamResults", data)
@@ -126,8 +127,8 @@ def nba_draft(self, season=None, team_id=None):
         data = self._http_get(
                 self._build_url(sport="nba", method="draft"),
                 params={
-                    season: season,
-                    team_id: team_id
+                    "season": season,
+                    "team_id": team_id
                 }
         )
         return self._format_result("NBADraft", data)
@@ -137,9 +138,11 @@ def nba_draft(self, season=None, team_id=None):
 
     def nba_daily_leaders(self, date=None, sort=None):
         data = self._http_get(
-                self._build_url(sport="nba", method="daily-leaders", date=date),
+                self._build_url(
+                    sport="nba", method="daily-leaders", date=date
+                ),
                 params={
-                    sort: sort
+                    "sort": sort
                 }
         )
         return self._format_result("NBADailyLeaders", data)
@@ -148,8 +151,8 @@ def nba_team_stats(self, date=None, team_id=None, season_type=None):
         data = self._http_get(
                 self._build_url(sport="nba", method="team-stats", date=date),
                 params={
-                    season_type: season_type,
-                    team_id: team_id
+                    "season_type": season_type,
+                    "team_id": team_id
                 }
         )
         return self._format_result("NBATeamStats", data)