-
Notifications
You must be signed in to change notification settings - Fork 15
Split washer and dryer class #96
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
Changes from 2 commits
e59e153
a68c952
5b6b7cd
71390d6
f3f5038
432d137
0dc1749
d5b450e
21e9834
6d153ff
bfd3479
22facc9
19a9544
4b338d1
899da5e
c929d22
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,41 +8,47 @@ | |
from whirlpool.appliancesmanager import AppliancesManager | ||
from whirlpool.auth import Auth | ||
from whirlpool.backendselector import BackendSelector | ||
from whirlpool.dryer import MachineState | ||
from whirlpool.dryer import ( | ||
CycleSelect, | ||
Dryness, | ||
MachineState, | ||
Temperature, | ||
WrinkleShield, | ||
) | ||
|
||
|
||
async def test_attributes(appliances_manager: AppliancesManager): | ||
dryer1 = appliances_manager.dryers[0] | ||
assert dryer1.get_machine_state() == MachineState.Standby | ||
assert dryer1.get_door_open() is False | ||
assert dryer1.get_est_time_remaining() == 1800 | ||
assert dryer1.get_drum_light_on() == 0 | ||
assert dryer1.get_status_extra_steam_changeable() == 1 | ||
assert dryer1.get_status_cycle_select() == 0 | ||
assert dryer1.get_status_dryness() == 1 | ||
assert dryer1.get_status_manual_dry_time() == 1 | ||
assert dryer1.get_status_temperature() == 1 | ||
assert dryer1.get_status_wrinkle_shield() == 1 | ||
assert dryer1.get_dryness() == 10 | ||
assert dryer1.get_manual_dry_time() == 1800 | ||
assert dryer1.get_cycle_select() == 11 | ||
assert dryer1.get_airflow_status() == 0 | ||
assert dryer1.get_cool_down() == 0 | ||
assert dryer1.get_damp() == 0 | ||
assert dryer1.get_drying() == 0 | ||
assert dryer1.get_limited_cycle() == 0 | ||
assert dryer1.get_sensing() == 0 | ||
assert dryer1.get_static_reduce() == 0 | ||
assert dryer1.get_steaming() == 0 | ||
assert dryer1.get_wet() == 0 | ||
assert dryer1.get_cycle_count() == 195 | ||
assert dryer1.get_running_hours() == 148 | ||
assert dryer1.get_total_hours() == 6302 | ||
assert dryer1.get_rssi_antenna_diversity() == -51 | ||
assert dryer1.get_damp_notification_tone_volume() == 0 | ||
assert dryer1.get_alert_tone_volume() == 0 | ||
assert dryer1.get_temperature() == 2 | ||
assert dryer1.get_wrinkle_shield() == 0 | ||
dryer = appliances_manager.dryers[0] | ||
assert dryer.get_machine_state() == MachineState.Standby | ||
assert dryer.get_door_open() is False | ||
assert dryer.get_est_time_remaining() == 1800 | ||
assert dryer.get_drum_light_on() == 0 | ||
lllucius marked this conversation as resolved.
Show resolved
Hide resolved
|
||
assert dryer.get_status_extra_steam_changeable() == 1 | ||
assert dryer.get_status_cycle_select() == 0 | ||
assert dryer.get_status_dryness() == 1 | ||
assert dryer.get_status_manual_dry_time() == 1 | ||
assert dryer.get_status_temperature() == 1 | ||
assert dryer.get_status_wrinkle_shield() == 1 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. lets remove the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not really sure about these. It "seems" that they are telling the caller whether that specific attribute can be changed or not. Maybe they should be more like:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ah! they inform the app if it should allow the user to change those settings trough the app? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. these where not updated. btw, if I got it right, these tell if the user can change the respective settings on the app? |
||
assert dryer.get_dryness() == Dryness.High | ||
assert dryer.get_manual_dry_time() == 1800 | ||
assert dryer.get_cycle_select() == CycleSelect.Timed_Dry | ||
assert dryer.get_airflow_status() == 0 | ||
assert dryer.get_cool_down() == 0 | ||
assert dryer.get_damp() == 0 | ||
assert dryer.get_drying() == 0 | ||
assert dryer.get_limited_cycle() == 0 | ||
assert dryer.get_sensing() == 0 | ||
assert dryer.get_static_reduce() == 0 | ||
assert dryer.get_steaming() == 0 | ||
assert dryer.get_wet() == 0 | ||
assert dryer.get_cycle_count() == 195 | ||
assert dryer.get_running_hours() == 148 | ||
assert dryer.get_total_hours() == 6302 | ||
assert dryer.get_rssi_antenna_diversity() == -51 | ||
assert dryer.get_damp_notification_tone_volume() == 0 | ||
assert dryer.get_alert_tone_volume() == 0 | ||
assert dryer.get_temperature() == Temperature.Cool | ||
assert dryer.get_wrinkle_shield() == WrinkleShield.Off | ||
|
||
|
||
@pytest.mark.parametrize( | ||
|
lllucius marked this conversation as resolved.
Show resolved
Hide resolved
|
Uh oh!
There was an error while loading. Please reload this page.