Support @keyword decorator #35

Activity

1. pekkaklarck commented on Dec 22, 2016

   

   pekkaklarck

   on Dec 22, 2016

   Member

   More actions

   This is a good idea. Remote server doesn't really need to care about @keyword decorator, though, it only needs to check robot_name and robot_tags the decorator sets (and which can also be set otherwise). Implementation should basically contain two things:

   • If robot_name is set to any meaningful value (i.e. non-empty string), keyword name reported to Remote library should be set based on it.

   • Possible tags set with robot_tags should be added at the end of the keyword documentation using Tags: t1, t2, t2 syntax. This is needed because the dynamic API and the remote interface lack possibility to report keyword tags directly. Adding new get_keyword_tags method would be nice, but that is a separate task.

   To learn more about the remote library interface implementation, please take a look at the docs in the User Guide:
   http://robotframework.org/robotframework/latest/RobotFrameworkUserGuide.html#remote-library-interface
2. EddyPronk commented on Dec 26, 2016

   

   EddyPronk

   on Dec 26, 2016

   Last edited by EddyPronk

   ContributorAuthor

   More actions

   I implemented both now.

   When a library imports robot.api.deco then test/libs/logging.py conflicts with the logging module of the standard library.

   Follow the progress here:
   https://github.com/epronk/PythonRemoteServer

   👍React with 👍1Reacted by Stefan Zimmermann
3. EddyPronk

   added 4 commits that reference this issue on Dec 26, 2016

   Implemented support for @Keyword decorator

   ...

   6381d86

   Implemented support for @Keyword decorator

   ...

   4df3f49

   Implemented support for @Keyword decorator

   ...

   e7f5ec4

   Implemented support for @Keyword decorator

   ...

   dfb54d9
4. pekkaklarck

   added this to the 1.1This milestone has been deleted milestone on Jan 9, 2017
5. pekkaklarck

   added

   enhancement

   on Jan 9, 2017
6. EddyPronk

   added a commit that references this issue on Jan 11, 2017

   Implemented support for @Keyword decorator

   ...

   b430137
7. pekkaklarck

   added 2 commits that reference this issue on Jan 13, 2017

   Cleanup and fixes to @Keyword support (issue #35, PR #37)

   ...

   ce976e0

   Test keyword tags set by @Keyword. Issue #35, PR #37.

   e7a575f
8. pekkaklarck

   self-assigned this

   on Jan 13, 2017
9. pekkaklarck commented on Jan 13, 2017

   

   pekkaklarck

   on Jan 13, 2017

   Member

   More actions

   PR #37 merged few fixes, cleanups done afterwards. I consider this issue done.

   Thanks @epronk for the PR! Comment the commits above or add a note here if you see something suspicious in my changes.
10. pekkaklarck

    closed this as completedon Jan 13, 2017
11. pekkaklarck

    added

    prio-high

    on Sep 27, 2017
12. pekkaklarck

    changed the title [-]Please support the @keyword decorator[/-] [+]Support `@keyword` decorator[/+] on Sep 27, 2017