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

@@ ... / @@ -52,3 +52,3 @@ class RT: @@
         try:
-            index = urlparts.path.index('/REST/')
+            index = urlparts.path.rindex('/REST/')
         except ValueError:

tests/test_rtutil.py

➞

Show inline comments

@@ ... / @@ -123 +123,7 @@ 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)