Skip to content

Conversation

pawiecz
Copy link
Contributor

@pawiecz pawiecz commented Jan 16, 2025

Changes (kernelci/kernelci-core#2746 and kernelci/kernelci-core#2927) to the model caused unit tests to get out-of-sync with its new state.

Tests adjusted in this patch should be kept in sync with Node class from "kernelci/api/models.py" in "kernelci/kernelci-core" repo

…cidb_bridge field)

Changes [0] to the model caused unit tests to get out-of-sync with its
new state.

Tests adjusted in this patch should be kept in sync with Node class from
"kernelci/api/models.py" in "kernelci/kernelci-core" repo [1].

[0] kernelci/kernelci-core#2746
[1] https://github.com/kernelci/kernelci-core/blob/ee64d7e4cdaeef2176d9755c9112ff97c8ad8e3a/kernelci/api/models.py#L184

Signed-off-by: Paweł Wieczorek <[email protected]>
…field)

Changes [0] to the model caused unit tests to get out-of-sync with its
new state.

Tests adjusted in this patch should be kept in sync with Node class from
"kernelci/api/models.py" in "kernelci/kernelci-core" repo [1].

[0] kernelci/kernelci-core#2927
[1] https://github.com/kernelci/kernelci-core/blob/ee64d7e4cdaeef2176d9755c9112ff97c8ad8e3a/kernelci/api/models.py#L184

Signed-off-by: Paweł Wieczorek <[email protected]>
@pawiecz pawiecz force-pushed the update-unit-tests-processed_by_kcidb_bridge branch from 0433b61 to a72965f Compare August 13, 2025 09:24
@pawiecz pawiecz requested a review from JenySadadia August 13, 2025 09:26
@pawiecz pawiecz marked this pull request as ready for review August 13, 2025 09:26
@pawiecz pawiecz changed the title tests/unit_tests: sync unit tests after model changes (processed_by_kcidb_bridge field) tests/unit_tests: sync unit tests after model changes Aug 13, 2025
Copy link
Collaborator

@JenySadadia JenySadadia 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!

@JenySadadia JenySadadia added this pull request to the merge queue Aug 13, 2025
Merged via the queue into kernelci:main with commit 7eaf5a8 Aug 13, 2025
4 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.

2 participants