Skip to content

Add a possibility to retrieve battery info from real devices#164

Merged
mykola-mokhnach merged 1 commit intoappium:masterfrom
mykola-mokhnach:battery_info
May 7, 2018
Merged

Add a possibility to retrieve battery info from real devices#164
mykola-mokhnach merged 1 commit intoappium:masterfrom
mykola-mokhnach:battery_info

Conversation

@mykola-mokhnach
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

@imurchie imurchie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this work for emulated devices, too?

@mykola-mokhnach
Copy link
Copy Markdown
Contributor Author

The documentation says it should

@mykola-mokhnach
Copy link
Copy Markdown
Contributor Author

I'd like to add an integration test first after this functionality is present on the server side

Comment thread lib/commands/battery.js
return await this.uiautomator2.jwproxy.command('/appium/device/battery_info', 'GET', {});
const result = await this.uiautomator2.jwproxy.command('/appium/device/battery_info', 'GET', {});
// Give it the same name as in iOS
result.state = result.status;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the change

@mykola-mokhnach mykola-mokhnach merged commit 95e51a9 into appium:master May 7, 2018
@mykola-mokhnach mykola-mokhnach deleted the battery_info branch May 7, 2018 07:08
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.

3 participants