NPO-Accounting/conservancy_beancount Changeset - e8e713721628

Changeset - e8e713721628

Parent rev.

Child rev.

[Not reviewed]

0 2 0

Brett Smith - 4 years ago 2020-03-25 14:18:09
brettcsmith@brettcsmith.org

rtutil: Fix parsing server URLs that include /REST/.

2 files changed with 7 insertions and 1 deletions:

conservancy_beancount/rtutil.py

tests/test_rtutil.py

0 comments (0 inline, 0 general)

conservancy_beancount/rtutil.py

➞

Show inline comments

@@ ... / @@ -47,13 +47,13 @@ class RT: @@
+    ]
     def __init__(self, rt_client: rt.Rt) -> None:
         self.rt = rt_client
         urlparts = urlparse.urlparse(rt_client.url)
         try:
-            index = urlparts.path.index('/REST/')
+            index = urlparts.path.rindex('/REST/')
         except ValueError:
             base_path = urlparts.path.rstrip('/') + '/'
         else:
             base_path = urlparts.path[:index + 1]
         self.url_base = urlparts._replace(path=base_path)

tests/test_rtutil.py

➞

Show inline comments

@@ ... / @@ -118,6 +118,12 @@ def test_exists_caches(new_client): @@
     ('rt://example.org', None),
     ('rt://example.org/1', None),
     ('https://example.org/rt/Ticket/Display.html?id=123', None),
 ])
 def test_parse(rt, link, expected):
     assert rt.parse(link) == expected
 def test_uncommon_server_url_parsing():
     url = 'https://example.org/REST/1.0/'
     client = testutil.RTClient(url + 'REST/1.0/')
     rt = rtutil.RT(client)
     assert rt.url(1).startswith(url)

0 comments (0 inline, 0 general)