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

@@ ... / @@ -50,7 +50,7 @@ class RT: @@
         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:

tests/test_rtutil.py

➞

Show inline comments

@@ ... / @@ -121,3 +121,9 @@ def test_exists_caches(new_client): @@
 ])
 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)