Skip to content

Conversation

ki4070ma
Copy link
Collaborator

For #377

@ki4070ma
Copy link
Collaborator Author

TODO in the next PR or later

  • Publish test report
  • Run Android tests

@KazuCocoa @mykola-mokhnach
Can I merge this PR and update along to review comments in the next PR?
Because I can't setup azure pipeline before azure-pipelines.yml exists in repository.
Or I'll check the behavior on my forked repository with changes along to review comments.

I've checked this PR behavior on forked repository.
https://dev.azure.com/ki4070ma/ki4070ma.python-client/_build/results?buildId=49

displayName: Run Appium in background
- script: brew install ffmpeg
displayName: Resolve dependencies (Appium server)
- script: python setup.py install
Copy link
Member

Choose a reason for hiding this comment

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

What about calling only pipenv like install CI related dependencies by ci-requirements.txt?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Used existing Pipfile, it's too enough for ci, though.
Pipfile seems impossible to have another section for CI like package/dev-package.

- task: UsePythonVersion@0
inputs:
versionSpec: '3.x'
- script: appium --relaxed-security &
Copy link
Member

Choose a reason for hiding this comment

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

Could you move running appium after all check?

Copy link
Member

Choose a reason for hiding this comment

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

You can also run appium server via Python script

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You can also run appium server via Python script

Let me have another PR for this. I will think about how to use it and it(Appium_service) doesn't work well when I tried on my local..

@ki4070ma
Copy link
Collaborator Author

jobs:
- job: func_test_ios1
pool:
vmImage: 'macOS-10.14'
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

(notice) I'd like to have parameters section, but can't have it in azure-pipelines.yml.

[ref]
https://github.com/appium/ruby_lib_core/blob/master/ci-jobs/functional_test.yml

@ki4070ma ki4070ma merged commit a8bdf22 into appium:master May 27, 2019
@ki4070ma ki4070ma deleted the run-ios-func-test branch May 27, 2019 06:14
@ki4070ma
Copy link
Collaborator Author

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