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

Test HA core v2023.6 until v2024.2 #942

Merged
merged 18 commits into from
Apr 6, 2024
Merged

Test HA core v2023.6 until v2024.2 #942

merged 18 commits into from
Apr 6, 2024

Conversation

basnijholt
Copy link
Owner

No description provided.

@basnijholt
Copy link
Owner Author

@protyposis, I reverted back to the old way of testing where we hack the adaptive_lighting folder into core/components. I did this because I tried to identify why a test started breaking after ≥2024.1.

With the pytest plugin it was impossible to run git bisect on core

(for my own sake documenting this)
After reverting and removing `

git bisect start
git bisect good 2024.1.6
git bisect bad 2024.2.5
git bisect run ./test.sh
❯ cat test.sh
#!/bin/bash
python3 -b -X dev -m pytest \
            -qq \
            --timeout=9 \
            --durations=10 \
            -o console_output_style=count \
            --dburl=postgresql://postgres:password@127.0.0.1/homeassistant-test \
            -p no:sugar tests/components/adaptive_lighting/test_switch.py::test_light_switch_in_specific_area

I found the problem:

commit 329eca49185034a722f91881568de53b9e8a1971
Author: Erik Montnemery <erik@montnemery.com>
Date:   Tue Jan 23 08:14:28 2024 +0100

    Store area registry entries in a UserDict (#108656)

    * Store area registry entries in a UserDict

    * Address review comments

 homeassistant/helpers/area_registry.py | 92 ++++++++++++++++++++++------------
 1 file changed, 61 insertions(+), 31 deletions(-)
bisect found first bad commit

@basnijholt basnijholt merged commit 9eae150 into main Apr 6, 2024
21 of 24 checks passed
@basnijholt basnijholt deleted the test-more-ha-versions branch April 6, 2024 08:14
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

1 participant