Change Record._endpoint_from_url() to remove api.base_url and then determine app and endpoint

Author: markkuleinio

pynetbox/core/response.py

@@ -280,7 +280,11 @@ def list_parser(list_item):
             setattr(self, k, v)
 
     def _endpoint_from_url(self, url):
-        app, name = urlsplit(url).path.split("/")[-4:-2]
+        # Remove the base URL from the beginning
+        url = url[len(self.api.base_url):]
+        if url.startswith("/"):
+            url = url[1:]
+        app, name = urlsplit(url).path.split("/")[:2]
         return getattr(pynetbox.core.app.App(self.api, app), name)
 
     def full_details(self):

tests/unit/test_response.py

@@ -254,13 +254,30 @@ def test_nested_write_with_directory_in_base_url(self):
         )
 
     def test_endpoint_from_url(self):
+        api = Mock()
+        api.base_url = "http://localhost:8080/api"
         test = Record(
             {
                 "id": 123,
                 "name": "test",
                 "url": "http://localhost:8080/api/test-app/test-endpoint/1/",
             },
-            Mock(),
+            api,
+            None,
+        )
+        ret = test._endpoint_from_url(test.url)
+        self.assertEqual(ret.name, "test-endpoint")
+
+    def test_endpoint_from_url_with_directory_in_base_url(self):
+        api = Mock()
+        api.base_url = "http://localhost:8080/testing/api"
+        test = Record(
+            {
+                "id": 123,
+                "name": "test",
+                "url": "http://localhost:8080/testing/api/test-app/test-endpoint/1/",
+            },
+            api,
             None,
         )
         ret = test._endpoint_from_url(test.url)