Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests: Add unit tests for org.osbuild.mkfs.xfs stage #1678

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

elkoniu
Copy link
Contributor

@elkoniu elkoniu commented Mar 21, 2024

Add stage unit test for mkfs.xfs

@elkoniu elkoniu requested a review from mvo5 March 21, 2024 00:46
@elkoniu elkoniu force-pushed the org.osbuild.mkfs.xfs-unit-tests branch from 4f96959 to e415dcc Compare March 21, 2024 00:47
@elkoniu elkoniu changed the title Org.osbuild.mkfs.xfs unit tests Tests: Add unit tests for org.osbuild.mkfs.xfs stage Mar 21, 2024
@elkoniu elkoniu force-pushed the org.osbuild.mkfs.xfs-unit-tests branch from e415dcc to 32b34cd Compare March 21, 2024 01:10
Copy link
Contributor

@mvo5 mvo5 left a comment

Choose a reason for hiding this comment

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

Thank you, this is very nice! Some small inline suggestions/questions but it feels super close to land.

stages/test/test_mkfs_xfs.py Show resolved Hide resolved
stages/test/test_mkfs_xfs.py Outdated Show resolved Hide resolved
stages/test/test_mkfs_xfs.py Outdated Show resolved Hide resolved
stages/test/test_mkfs_xfs.py Outdated Show resolved Hide resolved
stages/test/test_mkfs_xfs.py Outdated Show resolved Hide resolved
stages/test/test_mkfs_xfs.py Outdated Show resolved Hide resolved
@elkoniu elkoniu force-pushed the org.osbuild.mkfs.xfs-unit-tests branch from 32b34cd to 49bfeb7 Compare March 22, 2024 13:27
@elkoniu
Copy link
Contributor Author

elkoniu commented Mar 22, 2024

@mvo5 thanks a lot for the review. I have fixed some obvious issues and rebased to latest main.

@elkoniu elkoniu force-pushed the org.osbuild.mkfs.xfs-unit-tests branch from 49bfeb7 to 9af55d7 Compare March 22, 2024 13:28
stages/test/test_mkfs_xfs.py Outdated Show resolved Hide resolved
supakeen
supakeen previously approved these changes Mar 27, 2024
Copy link
Member

@supakeen supakeen left a comment

Choose a reason for hiding this comment

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

The minor issues can be resolved or left in, up to your discretion.

mvo5
mvo5 previously approved these changes Apr 2, 2024
Copy link
Contributor

@mvo5 mvo5 left a comment

Choose a reason for hiding this comment

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

Thank you

@supakeen
Copy link
Member

supakeen commented Apr 3, 2024

Has an unused re import. I dont't know what's up with the JSON checks here, cc @achilleas-k.

@elkoniu elkoniu force-pushed the org.osbuild.mkfs.xfs-unit-tests branch from ae148ae to c8956aa Compare April 3, 2024 20:14
@achilleas-k
Copy link
Member

Gitlab tests have been poked to retry.

@achilleas-k
Copy link
Member

I don't know what's up with the JSON checks here, cc @achilleas-k.

Weird. I see some failures in the history of the workflow that seem to be reformatting the devcontainer.json, but that wasn't touched. I can't imagine how a check this simple could be flaky. Unless there's some inconsistency or non-deterministic ordering in jq's output? (unlikely)

@supakeen supakeen merged commit f3a5267 into osbuild:main Apr 9, 2024
65 checks passed
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.

None yet

4 participants