Added start_activity function and tests

[0x1mason]

No description provided.

[0x1mason]

@bootstraponline I can't get any tests to pass. Can you take a look at this and see if there's anything I may have messed up?

[bootstraponline]

we're using minitest, not test unit.

current_activity.include?('Node').must_equal true

[bootstraponline]

wait { current_activity.include?('Node').must_equal true }

[bootstraponline]

I think a hash is cleaner.

# example usage
start_activity app_package: 'io.appium.android.apis', app_activity: '.accessibility.AccessibilityNodeProviderActivity'

# implementation
def start_activity(opts)
  app_package = opts[:app_package]
  raise 'app_package is required' unless app_package
  app_activity = opts[:app_activity]
  raise 'app_activity is required' unless opts[:app_activity]
  app_wait_package  = opts.fetch(:app_wait_package, '')
  app_wait_activity = opts.fetch(:app_wait_activity, '')

  unknown_opts = opts.keys - [:app_package, :app_activity, :app_wait_package, :app_wait_activity]
  raise "Unknown options #{unknown_opts}" unless unknown_opts.empty?

  execute :start_activity, {}, { appPackage:     app_package,      appActivity: app_activity,
                                 appWaitPackage: app_wait_package, appWaitActivity: app_wait_activity }
end

[0x1mason]

Thanks for the pointers!

[0x1mason]

@bootstraponline Updated

[bootstraponline]

the opts should be validated as well

raise 'opts must be a hash' unless opts.is_a? Hash

[0x1mason]

Updated with opts validation.

[bootstraponline]

merged, thanks!

[0x1mason]

yay! \o/

On Fri, Sep 12, 2014 at 1:42 PM, bootstraponline notifications@github.com
wrote:

merged, thanks!

—
Reply to this email directly or view it on GitHub
#267 (comment).