-
Notifications
You must be signed in to change notification settings - Fork 7
Implement snapshot testing #906
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
Merged
Changes from 50 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
1ebe4f7
Rough idea, not working though
CoMPaTech ee014cc
Change to generic approach
CoMPaTech 75bb31c
Add binary_sensor snapshot
CoMPaTech d04fac7
Smaller approach
CoMPaTech afb8ede
Remove replaced binary_sensor testcase
bouwew 7615062
Remove replaced climate testcase
bouwew 2c8e0b7
Remove replaced sensor testcase
bouwew 8774a02
Add switch snapshot testing
bouwew 11c1b6c
Save added switch snapshot
bouwew d31dcad
Remove replaced switch testcase
bouwew 1307227
Replace switch stretch testcase by snapshot testing
bouwew 9169b8e
Save updated switch snapshot
bouwew 6c02d6f
Add adam snapshot testcase
bouwew a1d67c4
Save updated sensor snapshot
bouwew eaf1d7f
Replace two more sensor testcases
bouwew 576fd48
Save updated sensor snapshot
bouwew 3648b8d
Remove replaced testcase
bouwew c4f6489
More sensor test updates
bouwew f1e1b39
Save updates/removals
bouwew 865d9af
Docstring improvements
bouwew 224f7b8
Shorten switch testcase names
bouwew c9ff831
Improve sensor testcase name
bouwew 6f1f828
Add select snapshot test
bouwew cbc8518
Save updates
bouwew 002ffa0
Add-replace a 2nd select snapshot testcase
bouwew b12931b
Improve docstring
bouwew c7a8ab5
Avoid testcase name collision
bouwew 1a6548f
Save updates
bouwew ebad3df
Add/update number snapshot tests
bouwew f02bf76
Save new snapshot
bouwew 48bf4fd
Add/improve climate snapshot tests
bouwew ba71aac
Save updated climate snapshot
bouwew 6623567
Add/update binary_sensor snapshot testing
bouwew d5a10d2
Save updated binary_sensor snapshot, cleanup
bouwew b74fea4
Revert deletion of binary_sensor testcase
bouwew eccf41d
Add/update button snapshot testing
bouwew 5379196
Save updates
bouwew 648d3cb
Update script as suggested
bouwew f961de2
Remove async as suggested
bouwew 665ae66
Fix typo
bouwew b76506b
Revert reordering
bouwew 583f670
More fixes
bouwew f3c97ef
Test-button: try without block_till_done()
bouwew a4b76db
Update binary_sensor tests
bouwew a9d8916
Save updated snapshot
bouwew a875c9b
Formatting
bouwew f18e1ae
Add comment, update docstring
bouwew db10282
Line up with Core tests
bouwew 991b89d
Save updated snapshot files, ruff fixes
bouwew 9df73bc
Re-ruffed via precommit
bouwew 2f72b01
CRAI improvement
bouwew 452190f
Revert delete of notification testcase
bouwew 31872fa
Update CHANGELOG
bouwew File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah I skipped those 'CRAI wants handbrakes on everything' :) But good enough - mind you the script is flawed when translations fails (which you'll find when bringing the script to other parts of the universe)