Random bug fixes (#132)

* fix bugs getting test of a run by name and id

* convert maps to lists for py3

Author: travispavek

testrail/client.py

@@ -67,7 +67,7 @@ def _project_by_id(self, project_id):
 
     # User Methods
     def users(self):
-        return map(User, self.api.users())
+        return list(map(User, self.api.users()))
 
     @methdispatch
     def user(self):
@@ -95,7 +95,7 @@ def inactive_users(self):
 
     # Suite Methods
     def suites(self):
-        return map(Suite, self.api.suites(self._project_id))
+        return list(map(Suite, self.api.suites(self._project_id)))
 
     @methdispatch
     def suite(self):
@@ -133,7 +133,7 @@ def _delete_suite(self, obj):
 
     # Milestone Methods
     def milestones(self):
-        return map(Milestone, self.api.milestones(self._project_id))
+        return list(map(Milestone, self.api.milestones(self._project_id)))
 
     @methdispatch
     def milestone(self):
@@ -266,7 +266,7 @@ def _delete_run(self, obj):
 
     # Case Methods
     def cases(self, suite):
-        return map(Case, self.api.cases(self._project_id, suite.id))
+        return list(map(Case, self.api.cases(self._project_id, suite.id)))
 
     @methdispatch
     def case(self):
@@ -293,7 +293,7 @@ def _add_case(self, obj):
 
     # Test Methods
     def tests(self, run):
-        return map(Test, self.api.tests(run.id))
+        return list(map(Test, self.api.tests(run.id)))
 
     @methdispatch
     def test(self):
@@ -302,14 +302,14 @@ def test(self):
     @test.register(str)
     @test.register(unicode)
     @singleresult
-    def _test_by_name(self, name):
-        return filter(lambda t: t.name.lower() == name.lower(), self.tests())
+    def _test_by_name(self, name, run):
+        return filter(lambda t: t.title.lower() == name.lower(), self.tests(run))
 
     @test.register(int)
     @singleresult
     def _test_by_id(self, test_id, run):
         return filter(
-            lambda t: t.raw_data()['case_id'] == test_id, self.tests(run))
+            lambda t: t.raw_data()['id'] == test_id, self.tests(run))
 
     # Result Methods
     @methdispatch
@@ -336,11 +336,11 @@ def _add_result(self, obj):
     def _add_results(self, results):
         obj, value = results
         if isinstance(obj, Run):
-            self.api.add_results(map(lambda x: x.raw_data(), value), obj.id)
+            self.api.add_results(list(map(lambda x: x.raw_data(), value), obj.id))
 
     # Section Methods
     def sections(self, suite=None):
-        return map(Section, self.api.sections(suite_id=suite.id))
+        return list(map(Section, self.api.sections(suite_id=suite.id)))
 
     @methdispatch
     def section(self):
@@ -362,7 +362,7 @@ def _add_section(self, section):
 
     # Status Methods
     def statuses(self):
-        return map(Status, self.api.statuses())
+        return list(map(Status, self.api.statuses()))
 
     @methdispatch
     def status(self):
@@ -380,4 +380,4 @@ def _status_by_id(self, status_id):
         return filter(lambda s: s.id == status_id, self.statuses())
 
     def configs(self):
-        return ConfigContainer(map(Config, self.api.configs()))
+        return ConfigContainer(list(map(Config, self.api.configs())))

testrail/entry.py

@@ -32,7 +32,7 @@ def name(self):
 
     @property
     def runs(self):
-        return map(EntryRun, self._content.get('runs'))
+        return list(map(EntryRun, self._content.get('runs')))
 
     @property
     def suite(self):