added keyword retry listener

[MarvKler]

No description provided.

[MarvKler]

@Snooz82 : PR contains keyword retry listener

[Snooz82]

@MarvKler

I did some refactoring, that you should merge into your branch.
https://github.com/MarketSquare/robotframework-retryfailed/tree/pr-14

This includes:

• switching from setup.py to pyproject.toml,
• introducing ruff and mypy
• introducing pre-commit hooks
• added typing to your code
• added some other test cases with [Setup] and [Teardown] which do not work as expected.
• added python keyword that can be retried 3 times and has a configurable failure index.

Please also see the comments in your code.
But fix them after you merge the branch to your repo

[Snooz82]

this whole start_keyword basically does add a keyword to the list of keyword that can be retried.
Couldn't that just be done in the end_keyword?

[Snooz82]

shouldn't the log level only be decreased by the keyword that did increase it?
e.g. if a userkeyword has to be retried, all its called keywords should be executed on the elevated log level. This here seems to switch back to the original, after the first library keyword.

[MarvKler]

added code to reset log level only on failed / passed state by the keyword which has initially changed the log level

[Snooz82]

that whole section seems to check if the keyword can be retried. But isn't that only relevant if the status is fail the first time?
Because on the second time, we should be sure already that we are currently retrying a keyword.
Also here, in the case we fail the first time, it would be the part, where we do all the code from start_keyword?!

[MarvKler]

New strategy:

RetryKeyword:
  keyword: RunningKeyword
  remaining_retries: int

End Keyword 
if not _retries := _check_if_retry(keyword):
  return

If status != FAIL
  if retry_stack and retry_stack[-1].keyword == keyword) :
    keyword.doc += Passed on {retries - remaining} retries
    retry_stack.pop()
    if not retry_stack and not self.test_retries:
      Build().log_level = self._original_log_level
      self._original_log_level = None
  Return

if keyword.type in ("SETUP", "TEARDOWN"):
  keyword.msg += "SETUP and TEARDOWN can not be retried"
  return

if retry_stack and retry_stack[-1].keyword == keyword) : 
# wir sind im Stack an letzter Stelle
  if not retry_stack[-1].remaining_retries:
    retry_stack().pop()
    keyword.doc += Final Retry Failed message
    return
  retry_stack[-1].remaining_retries -= 1
else: 
#das ist die initiale detection eines Fehlers
  retry_stack.append(RetryKeyword(keyword, _retries)) 
  result.status = "NOT RUN"

keyword.doc  += Funky Message
keyword.parent.body.inser(keyword.parent.body.index(keyword), keyword)
if self.log_level:
  original = BuiltIn().log_level = self.log_level
  if not self._original_log_level:
    self._original_log_level = original

[MarvKler]

@Snooz82 Please check the latest state of this PR - i have applied the new strategy discussed last week. Additionally, i have changed the initial loglevel handling a bit, but from my point of view this should match our expectation now.

PS: The RetryFailed listener seems to be working also after removing the end_suite listener function...