Support get_keyword_tags added by robotframework/robotframework#2538

Author: pekkaklarck

src/robotlibcore.py

@@ -54,6 +54,7 @@ def get_keyword_names(self):
 
 
 class DynamicCore(HybridCore):
+    _get_keyword_tags_supported = False  # get_keyword_tags is new in RF 3.0.2
 
     def run_keyword(self, name, args, kwargs):
         return self.keywords[name](*args, **kwargs)
@@ -77,17 +78,18 @@ def _get_arg_spec(self, kw):
         defaults = zip(args[nargs:], defaults)
         return mandatory, defaults, spec.varargs, spec.keywords
 
-    def get_keyword_documentation(self, name):
-        doc, tags = self._get_doc_and_tags(name)
-        doc = doc or ''
-        tags = 'Tags: {}'.format(', '.join(tags)) if tags else ''
-        sep = '\n\n' if doc and tags else ''
-        return doc + sep + tags
+    def get_keyword_tags(self, name):
+        self._get_keyword_tags_supported = True
+        return self.keywords[name].robot_tags
 
-    def _get_doc_and_tags(self, name):
+    def get_keyword_documentation(self, name):
         if name == '__intro__':
-            return inspect.getdoc(self), None
+            return inspect.getdoc(self) or ''
         if name == '__init__':
-            return inspect.getdoc(self.__init__), None
+            return inspect.getdoc(self.__init__) or ''
         kw = self.keywords[name]
-        return inspect.getdoc(kw), kw.robot_tags
+        doc = inspect.getdoc(kw) or ''
+        if kw.robot_tags and not self._get_keyword_tags_supported:
+            tags = 'Tags: {}'.format(', '.join(kw.robot_tags))
+            doc = '{}\n\n{}'.format(doc, tags) if doc else tags
+        return doc

utest/test_robotlibcore.py

@@ -33,6 +33,7 @@ def test_dir():
                 'get_keyword_arguments',
                 'get_keyword_documentation',
                 'get_keyword_names',
+                'get_keyword_tags',
                 'instance_attribute',
                 'keyword_in_main',
                 'keywords',
@@ -47,6 +48,7 @@ def test_dir():
     assert [a for a in dir(DynamicLibrary()) if a[0] != '_'] == expected
     expected = [e for e in expected if e not in ('get_keyword_arguments',
                                                  'get_keyword_documentation',
+                                                 'get_keyword_tags',
                                                  'run_keyword')]
     assert [a for a in dir(HybridLibrary()) if a[0] != '_'] == expected
 
@@ -83,7 +85,17 @@ def test_get_keyword_documentation():
     assert doc('__init__') == 'Library init doc.'
 
 
-def test_tags():
+def test_embed_tags_to_doc_when_get_keyword_tags_is_not_called():
     doc = DynamicLibrary().get_keyword_documentation
     assert doc('tags') == 'Tags: tag, another tag'
     assert doc('doc_and_tags') == 'I got doc!\n\nTags: tag'
+
+
+def test_get_keyword_tags():
+    lib = DynamicLibrary()
+    tags = lib.get_keyword_tags
+    doc = lib.get_keyword_documentation
+    assert tags('tags') == ['tag', 'another tag']
+    assert tags('doc_and_tags') == ['tag']
+    assert doc('tags') == ''
+    assert doc('doc_and_tags') == 'I got doc!'