Set artifacts_links in final results in testing farm workflow

[velezd]

This fixes missing links in Polarion reporting.

[velezd]

@jstodola this will change the "Comment" content in results pushed to Polarion. I don't know if you have any automation that is reading it.

[PLukas2018]

Claude thinks that there could be a bug:

Commit 9cedefb — Set artifacts_links in final results

Bug: self.weblink can be None at result-reporting time.

self.weblink starts as None and is only set if status['run']['artifacts'] is reachable (lines 151–155, wrapped in a try/except that silently swallows failures). If
Testing Farm errors or cancels before that field is populated, self.weblink is still None.

The new code then passes artifacts_links=[None] to Result(). In descriptionOf() (base.py:244–248):

', '.join([link for link in crc.result.extra_fields.get('artifacts_links', ['None'])])

The default ['None'] (string) only applies when the key is absent. When it's present as [None] (Python None), str.join raises TypeError: sequence item 0: expected str
instance, NoneType found.

Fix:
artifacts_links=[self.weblink] if self.weblink else []

Or filter in base.py:
', '.join([str(link) for link in crc.result.extra_fields.get('artifacts_links', []) if link is not None]) or 'None'

Otherwise LGTM.

[PLukas2018]

LGTM