[Refactor] Added refactored version of JunitParser for review

[unroma]

Issue being resolved: #308 #107

Solution description

I refactored JunitParser for better maintainability.
At first I would like to provide for review and only then if everything is ok, replace the current solution.
Also It should solve the issue with case result statuses mapping.

Changes

Restructured huge parse_file method with small methods.
Replaced and reworked methods from TestRailResult post_init section (they should not be there) into JunitParser,
Still todo: remove them from data class
this part:

    def __post_init__(self):
        if self.junit_result_unparsed is not None:
            self.status_id = self.calculate_status_id_from_junit_element(
                self.junit_result_unparsed
            )
            self.comment = self.get_comment_from_junit_element(
                self.junit_result_unparsed
            )

Now case result statuses {"passed": 1, "skipped": 4,"error": 5, "failure": 5} might be overridden using config.yml

Potential impacts

Result of junit xml parsing

Steps to test

replace from trcli.readers.junit_xml import JunitParser in cmd_parse_junit.py with from trcli.readers.junit_xml_refactored import JunitParser
then run unit tests
also add to config.yaml:

case_result_statuses:
  passed: 1
  skipped: 3
  error: 4
  failure: 5

(your desired mappings)
endpoint to get ids of result statuses for your project /api/v2/get_statuses

UPD:
Replaced current junit parser with refactored after review.

PR Tasks

• PR reference added to issue
• README updated
• Unit tests added/updated

[unroma]

@acuanico-tr-galt Any reaction?

[ecollins666]

Hi @unroma,
We will look into it today. Thank you

[acuanico-tr-galt]

Hi @unroma thank you for your patience. The changes looks solid. Can you now update the PR to change the actual file to update? (junit_xml) So that we can include in the next release? Thank you.

[unroma]

Hi @unroma thank you for your patience. The changes looks solid. Can you now update the PR to change the actual file to update? (junit_xml) So that we can include in the next release? Thank you.

Hi @acuanico-tr-galt @ecollins666 Done. Also I have updated documentation.

[acuanico-tr-galt]

Thanks @unroma can you also point the merge to release/1.10.0 branch? Thanks.

[unroma]

Thanks @unroma can you also point the merge to release/1.10.0 branch? Thanks.

@acuanico-tr-galt Done