-
Notifications
You must be signed in to change notification settings - Fork 1
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
[EGD-3646] Add starting and switching back apps #4
base: master
Are you sure you want to change the base?
Conversation
44ca80b
to
9f865b4
Compare
@@ -120,3 +120,16 @@ def set_tethering_state(self, enabled: bool): | |||
body = {"tethering": state} | |||
log.info(f"Set tethering state to: {state}") | |||
return self.endpoint_request("developerMode", "put", body) | |||
|
|||
def get_application_stack(self): | |||
body = {"applicationManager": True, "applicationStack": True } |
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.
Won't it be more readable if you put it in two separate lines? I Mean something like that:
body = {"applicationManager": True,
"applicationSTack": True}
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.
Agree, will do.
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.
is this already implemented in the OS part or it's in progress?
do you want to replace the keystroke application opening with this?
@SP2FET there is a separate review for the OF part: mudita/MuditaOS#1770 Ideally, this would replace the key combos for app opening (more reliable IMO). |
yep, it should be more reliable but imho it should be discussed - harness had to simulate user's input :) |
Well, we can use both for different kinds of testing. Up to the test design I suppose. |
No description provided.