Fix data stream component parsing

[jan-cerny]

SCAP source data streams can reference multiple check components usually OCIL, OVAL and CPE OVAL. This commit fixes the situation when there is both OVAL and CPE OVAL needed for an XCCDF Benchmark.

We improve the _find_all_component_contents and make it more generic and also refactor the code.

Fixes: #10408

[github-actions]

Start a new ephemeral environment with changes proposed in this pull request:

Fedora Environment

Oracle Linux 8 Environment

[ggbecker]

Thank for this improvement. Now it loads correctly all the correspondent components even if there is the CPE OVAL content as well.

[ggbecker]

@jan-cerny do you know if we can do something about the codeclimate/diff-coverage — 7% (50% threshold) check. I don't know what to suggest here to make it green.

[jan-cerny]

@ggbecker We could mitigate it by writing concise unit test for the XMLElement class.

[ggbecker]

@ggbecker We could mitigate it by writing concise unit test for the XMLElement class.

Are you willing to create it in this PR? Otherwise it should be fine to merge as it is IMO.

[jan-cerny]

@ggbecker I will take a look and try to create some test

[jan-cerny]

I have dd a simple unit test for the XMLContent class

[codeclimate]

Code Climate has analyzed commit e6dc679 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 85.7% (50% is the threshold).

This pull request will bring the total coverage in the repository to 52.4% (0.6% change).

View more on Code Climate.

[ggbecker]

I have dd a simple unit test for the XMLContent class

It's perfect. Thank you