Merge branch 'feature/plugin_url' into develop

Author: vmalloc

backslash/contrib/slash_plugin.py

@@ -73,6 +73,18 @@ def __init__(self, url=None, keepalive_interval=None, runtoken=None, propagate_e
         self._runtoken = runtoken
         self._propagate_exceptions = propagate_exceptions
 
+    @property
+    def rest_url(self):
+        return URL(self._url).add_path('rest')
+
+    @property
+    def webapp_url(self):
+        returned = str(self._url)
+        if not returned.endswith('/'):
+            returned += '/'
+        returned += '#/'
+        return returned
+
     def _handle_exception(self, exc_info):
         pass
 

docs/changelog.rst

@@ -2,6 +2,7 @@ Changelog
 =========
 
 * :bug:`31 major` Be more resilient to I/O errors when compressing tracebacks
+* :feature:`39` Added ``webapp_url`` and ``rest_url`` to the official Slash plugin
 * :feature:`37` Support reporting fatal exceptions
 * :feature:`36` Added ``get_parent`` to test objects to retrieve the parent session
 * :release:`2.28.0 <15-05-2017>`

tests/test_slash_plugin.py

@@ -1,43 +1,80 @@
 import pytest
+from urlobject import URLObject
 import slash
 import slash.loader
 
 from backslash.contrib.utils import distill_slash_traceback
 
 # pylint: disable=redefined-outer-name
 
+
 def test_importing_slash_plugin():
     from backslash.contrib import slash_plugin  # pylint: disable=unused-variable
 
+
 def test_exception_distilling(traceback):
     assert isinstance(traceback, list)
     assert traceback
     for f in traceback:
         assert f['code_line']
 
+
 def test_exception_distilling_surrounding_code(traceback):
     frame = traceback[-1]
-    assert 'def test_failing():' in [line.strip() for line in frame['code_lines_before']]
-    assert '1/0' in frame['code_line']
+    assert 'def test_failing():' in [line.strip()
+                                     for line in frame['code_lines_before']]
+    assert '1 / 0' in frame['code_line']
     assert frame['code_lines_before']
-    assert not any('1/0' in l for l in frame['code_lines_before'])
+    assert not any('1 / 0' in l for l in frame['code_lines_before'])
     assert frame['code_lines_after']
-    assert not any('1/0' in l for l in frame['code_lines_after'])
+    assert not any('1 / 0' in l for l in frame['code_lines_after'])
+
+
+def test_rest_url(installed_plugin, server_url):
+    assert installed_plugin.rest_url == server_url.add_path('rest')
+
+def test_webapp_url(installed_plugin, server_url):
+    expected = server_url
+    if not expected.endswith('/'):
+        expected += '/'
+    expected += '#/'
+    assert installed_plugin.webapp_url == expected
+
 
 
 @pytest.fixture
 def traceback(error_result):
     [e] = error_result.get_errors()
     return distill_slash_traceback(e)
 
+
 @pytest.fixture
 def error_result():
 
     def test_failing():
-        1/0  # pylint: disable=pointless-statement
+        1 / 0  # pylint: disable=pointless-statement
 
     with slash.Session() as s:
         with s.get_started_context():
-            slash.runner.run_tests(slash.loader.Loader().get_runnables([test_failing]))
+            slash.runner.run_tests(
+                slash.loader.Loader().get_runnables([test_failing]))
             [res] = s.results.iter_test_results()
     return res
+
+
+@pytest.fixture
+def installed_plugin(request, server_url):
+    from backslash.contrib import slash_plugin
+    plugin = slash_plugin.BackslashPlugin(url=str(server_url))
+
+    @request.addfinalizer
+    def cleanup():              # pylint: disable=unused-variable
+        slash.plugins.manager.uninstall(plugin)
+
+    slash.plugins.manager.install(plugin)
+    return plugin
+
+
+@pytest.fixture
+def server_url():
+    return URLObject('http://some.backslash.server')