Support continuable and fatal exceptions.

Implemented support for new feature in RF 2.8.4 [1]. Fixes #11.

[1] https://code.google.com/p/robotframework/issues/detail?id=1628

Older Remote libraries will simply ignore the new values in the result
dictionary.

This version always sends continuable/fatal info. Issue #12 covers sending
only information that is actually needed.

Author: spooning

atest/libs/failing.py

@@ -22,6 +22,21 @@ def failure_deeper(self, rounds=10):
             raise RuntimeError('Finally failing')
         self.failure_deeper(rounds-1)
 
+    def continuable(self, message):
+        self._raise_special(message, continuable=True)
+
+    def fatal(self, message):
+        self._raise_special(message, fatal='yes')
+
+    def not_special(self, message):
+        self._raise_special(message)
+
+    def _raise_special(self, message, continuable=False, fatal=False):
+        special = AssertionError(message)
+        special.ROBOT_CONTINUE_ON_FAILURE = continuable
+        special.ROBOT_EXIT_ON_FAILURE = fatal
+        raise special
+
 
 class MyException(Exception):
     pass

atest/tests/Z_LAST__special_failures.robot

@@ -0,0 +1,28 @@
+*** Settings ***
+Resource          resource.robot
+Suite Setup       Start And Import Remote Library    failing.py
+Suite Teardown    Stop Remote Library
+
+*** Test Cases ***
+Not special
+    [Documentation]  FAIL message
+    Not special    message
+    Fail    This should not be executed
+
+Continuable
+    [Documentation]  FAIL Several failures occurred:\n\n
+    ...    1) message\n\n
+    ...    2) second message\n\n
+    ...    3) third message
+    Continuable    message
+    Continuable    second message
+    Continuable    third message
+
+Fatal
+    [Documentation]  FAIL Execution ends here
+    Fatal    Execution ends here
+    Fail    This should not be executed
+
+Fails due to earlier fatal error
+    [Documentation]  FAIL Test execution stopped due to a fatal error.
+    Fail    This should not be executed

atest/tests/basic_communication.robot

@@ -13,6 +13,7 @@ Passing
 Failing
     [Documentation]    FAIL This is the error we get
     Failing    This is the error we get
+    Fail    This should not be executed
 
 Logging
     [Documentation]    LOG 1 INFO Hello, world! LOG 2 WARN Warning, warning!!

src/robotremoteserver.py

@@ -112,12 +112,17 @@ def run_keyword(self, name, args, kwargs=None):
         try:
             return_value = self._get_keyword(name)(*args, **kwargs)
         except:
-            result['error'], result['traceback'] = self._get_error_details()
+            exc_type, exc_value, exc_tb = sys.exc_info()
+            result['error'] = self._get_error_message(exc_type, exc_value)
+            result['traceback'] = self._get_error_traceback(exc_tb)
+            result['continuable'] = self._get_error_attribute(exc_value, 'CONTINUE')
+            result['fatal'] = self._get_error_attribute(exc_value, 'EXIT')
         else:
             try:
                 result['return'] = self._handle_return_value(return_value)
             except:
-                result['error'] = self._get_error_message()
+                exc_type, exc_value, _ = sys.exc_info()
+                result['error'] = self._get_error_message(exc_type, exc_value)
             else:
                 result['status'] = 'PASS'
         result['output'] = self._restore_std_streams()
@@ -165,17 +170,10 @@ def _get_keyword(self, name):
             return kw
         return None
 
-    def _get_error_details(self):
-        exc_type, exc_value, exc_tb = sys.exc_info()
+    def _get_error_message(self, exc_type, exc_value):
         if exc_type in self._fatal_exceptions:
             self._restore_std_streams()
             raise
-        return (self._get_error_message(exc_type, exc_value),
-                self._get_error_traceback(exc_tb))
-
-    def _get_error_message(self, exc_type=None, exc_value=None):
-        if exc_type is None:
-            exc_type, exc_value = sys.exc_info()[:2]
         name = exc_type.__name__
         message = self._get_message_from_exception(exc_value)
         if not message:
@@ -199,6 +197,9 @@ def _get_error_traceback(self, exc_tb):
         trace = ''.join(traceback.format_list(entries))
         return 'Traceback (most recent call last):\n' + trace
 
+    def _get_error_attribute(self, exc_value, name):
+        return bool(getattr(exc_value, 'ROBOT_%s_ON_FAILURE' % name, False))
+
     def _handle_return_value(self, ret):
         if isinstance(ret, basestring):
             return self._handle_binary_result(ret)