Support also string NONE in run on failure disabling

Author: aaltat

atest/acceptance/keywords/run_on_failure.robot

@@ -26,17 +26,29 @@ Log Title On Failure
     ...    ${FAILURE MESSAGE}
     ...    Page Should Not Contain    needle    loglevel=None
 
-Disable Run on Failure
+Disable Run on Failure With Nothing
     [Documentation]
     ...    LOG 1 No keyword will be run on failure.
     ...    LOG 2.1:2 NONE
-    ...    LOG 3 No keyword will be run on failure.
-    ...    LOG 4.1:2 NONE
-    Register Keyword to Run On Failure    Nothing
+    SeleniumLibrary.Register Keyword to Run On Failure    Nothing
     Run Keyword And Expect Error
     ...    ${FAILURE MESSAGE}
     ...    Page Should Not Contain    needle    loglevel=None
-    Register Keyword to Run On Failure    ${NONE}
+
+Disable Run on Failure With Python None
+    [Documentation]
+    ...    LOG 1 No keyword will be run on failure.
+    ...    LOG 2.1:2 NONE
+    SeleniumLibrary.Register Keyword to Run On Failure    ${NONE}
+    Run Keyword And Expect Error
+    ...    ${FAILURE MESSAGE}
+    ...    Page Should Not Contain    needle    loglevel=None
+
+Disable Run on Failure With String NONE
+    [Documentation]
+    ...    LOG 1 No keyword will be run on failure.
+    ...    LOG 2.1:2 NONE
+    SeleniumLibrary.Register Keyword to Run On Failure    NONE
     Run Keyword And Expect Error
     ...    ${FAILURE MESSAGE}
     ...    Page Should Not Contain    needle    loglevel=None

src/SeleniumLibrary/keywords/runonfailure.py

@@ -13,13 +13,14 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+from typing import Optional
 
 from SeleniumLibrary.base import LibraryComponent, keyword
 
 
 class RunOnFailureKeywords(LibraryComponent):
     @keyword
-    def register_keyword_to_run_on_failure(self, keyword: str) -> str:
+    def register_keyword_to_run_on_failure(self, keyword: Optional[str]) -> str:
         """Sets the keyword to execute, when a SeleniumLibrary keyword fails.
 
         ``keyword`` is the name of a keyword that will be executed if a
@@ -61,6 +62,8 @@ def register_keyword_to_run_on_failure(self, keyword: str) -> str:
 
     @staticmethod
     def resolve_keyword(name):
-        if name is None or isinstance(name, str) and name.upper() == "NOTHING":
+        if name is None:
+            return None
+        if isinstance(name, str) and name.upper() == "NOTHING" or name.upper() == "NONE":
             return None
         return name