Skip to content

Commit 22648e0

Browse files
committed
Add comment regarding if statements in tests
1 parent 84165e5 commit 22648e0

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

homeassistant/components/jewish_calendar/quality_scale.yaml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@ rules:
2121
test-before-setup:
2222
status: exempt
2323
comment: Local calculation does not require setup.
24-
unique-config-entry: done
24+
unique-config-entry:
25+
status: done
2526
comment: >- The multiple config entry was removed due to
2627
multiple bugs in the integration and low ROI.
2728
We might consider revisiting this as an additional feature in the future
@@ -44,7 +45,10 @@ rules:
4445
reauthentication-flow:
4546
status: exempt
4647
comment: This integration does not require reauthentication, since it is a local calculation.
47-
test-coverage: done
48+
test-coverage:
49+
status: todo
50+
comment: |- The following points should be addressed:
51+
* Don't use if-statements in tests (test_jewish_calendar_sensor, test_shabbat_times_sensor)
4852
4953
# Gold
5054
devices:

0 commit comments

Comments
 (0)