feat: allow *.xml (beside *.log) files as sample input files

[hsekowski-splunk]

No description provided.

[hsekowski-splunk]

This PR is to address #549

[ghost]

I think it is a good one, @nandinivij WDYT?

[ghost]

What about this line

pytest-splunk-addon/pytest_splunk_addon/standard_lib/event_ingestors/requirement_event_ingester.py

Line 111 in 23deab5

if filename.endswith(".log"):

? Do we need to change this?

And I would like to see a docker-based test that confirms that this feature works.

[hsekowski-splunk]

@arys-splunk ,
Good catch and valid question - yes, better to have the data ingested :)

[ghost]

@hsekowski-splunk Can you please also change the target branch to develop?

[nandinivij]

We will need changes unit test side too. Currently, we generate errors for anything other than .log and do not run prerequisite tests for that.

[codecov-commenter]

Codecov Report

Merging #550 (6fb6702) into develop (4dc506e) will decrease coverage by 0.41%.
The diff coverage is 57.14%.

@@             Coverage Diff             @@
##           develop     #550      +/-   ##
===========================================
- Coverage    95.13%   94.72%   -0.42%     
===========================================
  Files           49       42       -7     
  Lines         2613     2557      -56     
===========================================
- Hits          2486     2422      -64     
- Misses         127      135       +8     

Impacted Files	Coverage Δ
...unk_addon/standard_lib/addon_requirements_basic.py	0.00% <0.00%> (ø)
...n/standard_lib/requirement_tests/test_generator.py	70.98% <75.00%> (-0.09%)	⬇️
..._lib/event_ingestors/requirement_event_ingester.py	54.25% <100.00%> (ø)
..._tests/requirement_test_datamodel_tag_constants.py	0.00% <0.00%> (-100.00%)	⬇️
...on/standard_lib/sample_generation/sample_stanza.py	97.22% <0.00%> (-1.40%)	⬇️
...ddon/standard_lib/addon_parser/eventtype_parser.py	96.15% <0.00%> (-0.28%)	⬇️
.../standard_lib/addon_parser/savedsearches_parser.py	96.87% <0.00%> (-0.19%)	⬇️
...don/standard_lib/addon_parser/transforms_parser.py	97.10% <0.00%> (-0.09%)	⬇️
...nk_addon/standard_lib/addon_parser/props_parser.py	99.20% <0.00%> (-0.02%)	⬇️
...plunk_addon/standard_lib/sample_generation/rule.py	99.34% <0.00%> (-0.01%)	⬇️
... and 34 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23deab5...6fb6702. Read the comment docs.

[github-actions]

Expected release notes (by @hsekowski-splunk)

features:
remove splunk-appinspect dependency (#499) (a56e9d1)
allow *.xml (beside *.log) files as sample input files (b2b3596)

fixes:
ingestion extended by *.xml files (0147b5e)
fake_path corrected for xml sample (85226b4)
unit test addopted to the change (85717f6)

others (will not be included in Semantic-Release notes):
assertion updated (6fb6702)
assertion changed to reflect second source of the data that causes additional successful pass (e2c3f68)
add *.xml file to requirement tests (c01b8eb)
update development instructions (#564) (5071ddb)
reorganize how doc are being handled in CI (#563) (330a0cb)
remove pytest-mock as not used (#560) (04cb76c)
fix semgrep issues (#558) (6c8bb63)
remove Ederson from owners list [ci skip] (#559) (278bfdd)
make tests run in parallel (#553) (acfddc8)
updated error msg (5ffe040)
move test_cim_report.py to unit tests (#554) (43365bf)
add pyupgrade and isort to pre-commit (#552) (5acb649)
clean up helmut after initial refactoring (#544) (eb940bc)
3.0.0-beta.1 (d17c293)
to test the project unit tests (3ccf59c)
remove test_events_with_untokenised_values from RequirementBasic (#543) (23deab5)
remove matrix option from test-splunk-matrix step (#545) (5464510)

• I hereby acknowledge these release notes are 🥙 AWESOME 🥙