puffyCid / artemis

A cross platform forensic parser written in Rust!
MIT License
59 stars 6 forks source link

Some code cleanup related to artifact merging #120

Closed puffyCid closed 4 months ago

puffyCid commented 4 months ago

Minor PR that cleans up the last remaining parts of the artifact merging

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 76.75277% with 63 lines in your changes are missing coverage. Please review.

Project coverage is 84.59%. Comparing base (fd92e45) to head (816ff60).

Files Patch % Lines
core/src/artifacts/os/processes/artifact.rs 84.61% 10 Missing :warning:
core/src/artifacts/collection.rs 65.21% 8 Missing :warning:
core/src/artifacts/os/systeminfo/error.rs 0.00% 8 Missing :warning:
core/src/artifacts/error.rs 0.00% 7 Missing :warning:
core/src/artifacts/os/systeminfo/artifact.rs 86.04% 6 Missing :warning:
core/src/artifacts/applications/artifacts.rs 40.00% 3 Missing :warning:
core/src/artifacts/os/files/artifact.rs 95.89% 3 Missing :warning:
core/src/artifacts/os/files/filelisting.rs 40.00% 3 Missing :warning:
core/src/artifacts/os/linux/artifacts.rs 50.00% 3 Missing :warning:
core/src/artifacts/os/macos/artifacts.rs 62.50% 3 Missing :warning:
... and 4 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #120 +/- ## ========================================== + Coverage 84.44% 84.59% +0.15% ========================================== Files 468 474 +6 Lines 53345 53370 +25 ========================================== + Hits 45046 45151 +105 + Misses 8299 8219 -80 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.