Skip to content

test(plugins/cgroups/oar): Updates to add 80% test code coverage for CGroups/oar plugin#388

Open
victoryeagle77 wants to merge 1 commit intoalumet-dev:mainfrom
victoryeagle77:plugins/cgroups/oar
Open

test(plugins/cgroups/oar): Updates to add 80% test code coverage for CGroups/oar plugin#388
victoryeagle77 wants to merge 1 commit intoalumet-dev:mainfrom
victoryeagle77:plugins/cgroups/oar

Conversation

@victoryeagle77
Copy link
Copy Markdown
Contributor

No description provided.

@victoryeagle77 victoryeagle77 self-assigned this Apr 20, 2026
@victoryeagle77 victoryeagle77 requested a review from AngeCyp April 20, 2026 12:17
@victoryeagle77 victoryeagle77 force-pushed the plugins/cgroups/oar branch 4 times, most recently from 4735ebf to 9bfec08 Compare April 20, 2026 13:34
Copy link
Copy Markdown
Member

@TheElectronWill TheElectronWill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the right approach to the problem.
util_cgroups::Cgroup exists for a good reason, and we want to keep it.

To create mock cgroups in your tests, use CgroupHierarchy::manually_unchecked.
See plugins/cgroups/util-cgroups/tests/cgroupv1.rs

Comment thread plugins/cgroups/oar/src/attr.rs Outdated
Comment thread plugins/cgroups/oar/src/attr.rs Outdated
@victoryeagle77 victoryeagle77 force-pushed the plugins/cgroups/oar branch 3 times, most recently from dab7f2a to 572d488 Compare April 21, 2026 10:32
@victoryeagle77 victoryeagle77 force-pushed the plugins/cgroups/oar branch 4 times, most recently from e148f91 to b85e74d Compare April 22, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants