-
-
Notifications
You must be signed in to change notification settings - Fork 168
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
adb-commands module refactoring #256
adb-commands module refactoring #256
Conversation
…ting, getSetting]
… of `isScreenLocked()`.
… statement in `getSetting`.
lib/tools/adb-commands.js
Outdated
}; | ||
|
||
methods.setSetting = async function (namespace, setting, value) { | ||
await this.shell(['settings', 'put', namespace, setting, value]); |
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.
return might be also useful here
Linting is broken. Please run |
…`. Bypass lint error for chai `should`.
lib/tools/adb-commands.js
Outdated
}; | ||
|
||
/* |
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.
Please rebase your branch with the recent master to see docstring examples. We do follow JSDoc tutorial while creating docstrings.
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.
oops...
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.
great idea @vmaxim!
@@ -7,7 +7,7 @@ import { apiLevel, platformVersion, MOCHA_TIMEOUT } from './setup'; | |||
import { fs, mkdirp } from 'appium-support'; | |||
import temp from 'temp'; | |||
|
|||
|
|||
const should = chai.should(); |
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.
the way that i usually get around lint errors here is simply:
chai.should();
i.e., you don't need to assign it to a variable for it to do the thing you need it to do.
Thanks! Functional tests pass locally, and fix for Travis is in master, so merging. |
No description provided.