Fix route set by mapper.get()

Author: Anthony Ruhier

netboxapi/mapper.py

@@ -60,7 +60,7 @@ def get(self, *args, **kwargs):
         try:
             for d in new_mappers_dict:
                 yield self._build_new_mapper_from(
-                    d, route + "{}/".format(d["id"])
+                    d, self._route + "{}/".format(d["id"])
                 )
         except KeyError:
             # Result objects have no id, cannot build a mapper from them,

tests/test_mapper.py

@@ -57,6 +57,16 @@ def test_get_submodel(self, mapper):
         for key, val in expected_attr["results"][0].items():
             assert getattr(submodel_mapper, key) == val
 
+    def test_get_id(self, mapper):
+        url = self.get_mapper_url(mapper) + "1/"
+        expected_attr = {"id": 1, "name": "test"}
+        with requests_mock.Mocker() as m:
+            m.register_uri("get", url, json=expected_attr)
+            obj = next(mapper.get(1))
+
+        assert obj.name == "test"
+        assert obj.id == 1
+
     def test_get_query(self, mapper):
         url = self.get_mapper_url(mapper) + "?name=test"
         expected_attr = {
@@ -416,7 +426,7 @@ def test_delete_from_child(self, mapper):
             obj_mapper = next(mapper.get(1))
 
         with requests_mock.Mocker() as m:
-            m.register_uri("delete", url + "1/")
+            m.register_uri("delete", url)
             response = obj_mapper.delete()
 
         assert response.status_code == 200