KubeArmor icon indicating copy to clipboard operation
KubeArmor copied to clipboard

tests(non-k8s): Add HSP test suite for non-k8s mode

Open navin772 opened this issue 1 year ago • 5 comments

Purpose of PR?:

Partially fixes #1625 Adds the HSP test suite for non-k8s mode

Does this PR introduce a breaking change? No

If the changes in this PR are manually verified, list down the scenarios covered::

Additional information for reviewer? : Mention if this PR is part of any design or a continuation of previous PRs

Checklist:

  • [ ] Bug fix. Fixes #
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update
  • [x] PR Title follows the convention of <type>(<scope>): <subject>
  • [ ] Commit has unit tests
  • [x] Commit has integration tests

navin772 avatar Aug 29 '24 18:08 navin772

Can we include https://github.com/kubearmor/KubeArmor/pull/1879 here?

daemon1024 avatar Oct 18 '24 05:10 daemon1024

@navin772 why does the systemd test fail here? It cannot find the cov out file.

daemon1024 avatar Oct 18 '24 05:10 daemon1024

Some of the tests are failing, so the Kill KubeArmor Process step was skipped, hence no coverage file was generated.

navin772 avatar Oct 18 '24 06:10 navin772

@navin772 what is failing? how can I help?

daemon1024 avatar Oct 18 '24 07:10 daemon1024

@daemon1024 4 out of the new 8 non-k8s HSP tests are failing which were passing earlier, was there any changes to non-k8s recently? I will take a detailed look and try to run the tests locally this weekend.

navin772 avatar Oct 18 '24 08:10 navin772