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
TIMOB-9279: BlackBerry: Add --blackberry-device and --blackberry-password to drillbit #78
TIMOB-9279: BlackBerry: Add --blackberry-device and --blackberry-password to drillbit #78
Conversation
…word to drillbit [Issues Fixed] TIMOB-9279: BlackBerry: Add --blackberry-device and --blackberry-password to drillbit [Changes] drillbit.py - added usage blackberry.js - function BlackBerrySimulator - store the blackberry args - function createDeviceManagementProcess - replace the unused args with the used ones - functions isTestHarnessRunning, pushTestJS, launchTestHarness, killTestHarness - use the builder process instead of devicemanagement builder.py - functions __init__, __name__ - add useLogFile param to direct output to log file or stdout based on the command - function run - move the package name to a reusable function getPackage - added functions terminateApp, isAppRunning, printExitCode, getFile, putFile, appLog - function __name__ - added usage, options, and handling for the new commands devicemanagement.py - use optparse instead of argparse - move package-related commands to builder.py [Tests] Test 1: Run drillbit 1) Run scons to build the mobile sdk 2) Run the following command: drillbit\drillbit.py --platforms=blackberry --blackberry-device=<ip address> 3) Click "run tests" 4) Verify the test_harness app is built, deployed, and run 5) Verify the logs are printed Note: The test will fail because some things aren't implemented
parser.add_argument('-t', '--type', choices=['simulator', 'device', 'deploy'], required=True) | ||
parser.add_argument('-d', '--project_path', help='project directory path', required=True) | ||
parser.add_argument('-p', '--ndk_path', help='blackberry ndk path') | ||
parser = OptionParser() |
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.
Don't we need usage string here to provide more descriptive message rather than "devicemanagement.py [options]"
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.
Done
Reviewed |
…word to drillbit fix usage and required
Updated patch |
Approved |
1 similar comment
Approved |
TIMOB-9279: BlackBerry: Add --blackberry-device and --blackberry-password to drillbit [Issues Fixed] TIMOB-9279: BlackBerry: Add --blackberry-device and --blackberry-password to drillbit [Changes] drillbit.py - added usage blackberry.js - function BlackBerrySimulator - store the blackberry args - function createDeviceManagementProcess - replace the unused args with the used ones - functions isTestHarnessRunning, pushTestJS, launchTestHarness, killTestHarness - use the builder process instead of devicemanagement builder.py - functions __init__, __name__ - add useLogFile param to direct output to log file or stdout based on the command - function run - move the package name to a reusable function getPackage - added functions terminateApp, isAppRunning, printExitCode, getFile, putFile, appLog - function __name__ - added usage, options, and handling for the new commands devicemanagement.py - use optparse instead of argparse - move package-related commands to builder.py [Tests] Test 1: Run drillbit 1) Run scons to build the mobile sdk 2) Run the following command: drillbit\drillbit.py --platforms=blackberry --blackberry-device=<ip address> 3) Click "run tests" 4) Verify the test_harness app is built, deployed, and run 5) Verify the logs are printed Note: The test will fail because some things aren't implemented
Reviewers: Alex, Harut
Note: devicemanagement.py has been gutted in preparation for TIMOB-9378