Failed test cases are skipped from being pushed to polarion

[vboulos]

All test cases with status "failed" are not pushed into polarion due to the following error: Failed due to the following error: Server raised fault: 'java.lang.IllegalArgumentException: projectId is null

I've been using pylero since 2021, and everyhting was working perfectly, all our test cases were pushed as expected to polarion. This issue has started when this PR got merged https://github.com/RedHatQE/pylero/commit/3af53c70febba00c6268ec8c52beac20969c793c

Note that test cases with status "passed" and "blocked" are being pushed normally to Polarion. Only "failed" test cases are affected

[waynesun09]

@leelavg please help check when you are free, I'll be offline until Wednesday next week.

@vboulos thanks for the feedback. 3af53c7 haven't been released into Pypi yet, you could install with Pypi release if this blocks you at the moment. Thanks.

[vboulos]

Thank you @waynesun09 for the reply. Could you please elaborate more about "you could install with Pypi release if this blocks you at the moment" I did not get what do you exactly mean, sorry :)

[waynesun09]

Sure, pylero release could be found on Pypi package page: https://pypi.org/project/pylero/0.0.8/

To install pylero:

$ pip install pylero

or point to the latest version 0.0.8:

$ pip install pylero==0.0.8

The doc: https://github.com/RedHatQE/pylero#install-from-pypi

[vboulos]

Sounds good, thank you @waynesun09

[leelavg]

• below seems to fix the issue, however down the code path we'll hit an issue

  
  diff --git a/src/pylero/test_run.py b/src/pylero/test_run.py
  index 9efd82f..25a2c4e 100644
  --- a/src/pylero/test_run.py
  +++ b/src/pylero/test_run.py
  @@ -930,7 +930,7 @@ class TestRun(BasePolarion):
           suds_object = test_record
       if test_record.result == "failed" and not test_record.defect_case_id:
           test_record.defect_case_id = create_incident_report(

• self, test_record, _WorkItem(work_item_id=test_case_id)
• self, test_record, _WorkItem(project_id=self.project_id, work_item_id=test_case_id) ) self.session.test_management_client.service.addTestRecordToTestRun( self.uri, suds_object @@ -1306,7 +1306,7 @@ class TestRun(BasePolarion): else: if test_record.result == "failed" and not test_record.defect_case_id: test_record.defect_case_id = create_incident_report(
• self, test_record, _WorkItem(work_item_id=test_case_id)
• self, test_record, _WorkItem(project_id=self.project_id, work_item_id=test_case_id) ) index = test_case_ids.index(test_case_id) if isinstance(test_record, TestRecord):

• it'll definitely fail at https://github.com/RedHatQE/pylero/blob/main/src/pylero/test_run.py#L129-L130
• there's a possible failure at https://github.com/RedHatQE/pylero/blob/main/src/pylero/test_run.py#L150 as well
• this function https://github.com/RedHatQE/pylero/blob/main/src/pylero/test_run.py#L139 wants TestCase and maybe #132 isn't a legit fix 🤔

@simzacks any hints on how to proceed? we didn't tag the version yet so no hurries wrt internal usage.