Skip to content
This repository has been archived by the owner. It is now read-only.

Updated Jenkinsfile to run tests using Appium #19

Merged
merged 2 commits into from May 9, 2018

Conversation

@sharath-cliqz
Copy link
Contributor

@sharath-cliqz sharath-cliqz commented Apr 10, 2018

  • Updated Jenkinsfile to run tests using Appium
@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:IB2-81 branch 5 times, most recently from 97de1d9 to b2b666e Apr 10, 2018
@sharath-cliqz sharath-cliqz changed the title [WIP] Updated Jenkinsfile to run tests using Appium Updated Jenkinsfile to run tests using Appium Apr 11, 2018
@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:IB2-81 branch from b2b666e to 0daad13 Apr 13, 2018
Copy link
Contributor

@naira-cliqz naira-cliqz left a comment

@sharath-cliqz Are these scheme changes to disable FF tests?

@sharath-cliqz
Copy link
Contributor Author

@sharath-cliqz sharath-cliqz commented Apr 16, 2018

@naira-cliqz Yes. I Disabled the FF Tests and added a basic Test Sceme for Appium.

Jenkinsfile Outdated
chmod 600 ~/.ssh/id_rsa
echo $CLIQZ_CI_SSH_KEY
ssh-keyscan -t rsa github.com >> ~/.ssh/known_hosts
git clone -b version2.0 --single-branch --depth=1 git@github.com:cliqz/autobots.git

This comment has been minimized.

@chrmod

chrmod Apr 16, 2018
Member

wonder if we couldn't use checkout helper instead?

This comment has been minimized.

@sharath-cliqz

sharath-cliqz Apr 17, 2018
Author Contributor

Writing a Helper as we use something similar to Android and iOS.

Jenkinsfile Outdated
@@ -67,18 +81,65 @@ node('mac-mini-ios') {
timeout(10) {
sh '''#!/bin/bash -l
set -e
xcodebuild -workspace Client.xcworkspace -scheme "Fennec" -sdk iphonesimulator -destination "platform=iOS Simulator,OS=11.2,id=185B34BB-DCB8-4A17-BDCA-843086B67193" ONLY_ACTIVE_ARCH=NO -derivedDataPath clean build
xcodebuild -workspace Client.xcworkspace -scheme "Fennec" -sdk iphonesimulator -destination "platform=iOS Simulator,OS=11.2,id=185B34BB-DCB8-4A17-BDCA-843086B67193" ONLY_ACTIVE_ARCH=NO -derivedDataPath clean build test

This comment has been minimized.

@chrmod

chrmod Apr 16, 2018
Member

please split arguments into subsequent lines

This comment has been minimized.

@sharath-cliqz

sharath-cliqz Apr 17, 2018
Author Contributor

Done.

Jenkinsfile Outdated
junit "autobots/test-reports/*.xml"
zip archive: true, dir: 'autobots/screenshots', glob: '', zipFile: 'autobots/screenshots.zip'
} catch (e) {
// no screenshots, no problem

This comment has been minimized.

@chrmod

chrmod Apr 16, 2018
Member

have seen this somewhere before? can we extract something to our jenkins-pipeline-library ? /cc @sharath-cliqz

This comment has been minimized.

@sharath-cliqz

sharath-cliqz Apr 17, 2018
Author Contributor

Done. Put it into the same file for now. Will move it to the jenkins-pipeline-library.

Jenkinsfile Outdated
stage('Setup Test Environment'){
sh '''#!/bin/bash -l
set -e
npm install -g appium

This comment has been minimized.

@chrmod

chrmod Apr 16, 2018
Member

we could add those to package.json and run with npm run - this will add a better version control and it would pollute a global namespace

This comment has been minimized.

@sharath-cliqz

sharath-cliqz May 7, 2018
Author Contributor

Done.

@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:IB2-81 branch 9 times, most recently from 8ea330d to eb7df0d Apr 17, 2018
@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:IB2-81 branch 4 times, most recently from a897c05 to 8740cfd Apr 18, 2018
@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:IB2-81 branch 5 times, most recently from f5d56f9 to 994e816 May 7, 2018
@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:IB2-81 branch 4 times, most recently from 5b4d033 to da20877 May 8, 2018
@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:IB2-81 branch from da20877 to 87b6437 May 8, 2018
@naira-cliqz naira-cliqz merged commit d500e92 into ghostery:master May 9, 2018
1 check passed
1 check passed
@cliqz-oss-ci
continuous-integration/jenkins/pr-merge This commit looks good
Details
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants