Skip to content

Added Jenkinsfile to automatically build benchmarks and run tests.#27

Merged
geoffreyblake merged 1 commit intoARM-software:masterfrom
rpgolshan:master
Jun 7, 2018
Merged

Added Jenkinsfile to automatically build benchmarks and run tests.#27
geoffreyblake merged 1 commit intoARM-software:masterfrom
rpgolshan:master

Conversation

@rpgolshan
Copy link
Copy Markdown
Contributor

Jenkinsfile is a pipeline flow for a multibranch pipeline project. This
commit adds stages to build every main directory (e.g. lockhammer) and
run all tests starting with directory/scripts/test*

Jenkins will update any pull requests statuses with either a 'success'
or a 'failure' while listing what failed.

@geoffreyblake
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

Jenkinsfile is a pipeline flow for a multibranch pipeline project. This
commit adds stages to build every main directory (e.g. lockhammer) and
run all tests starting with directory/scripts/test*

Jenkins will update any pull requests statuses with either a 'success'
or a 'failure' while listing what failed.
@geoffreyblake
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

Comment thread Jenkinsfile
def scr = new File("${it}/scripts/")
scr.traverse(type: FILES, filter: ~/.*\/test[^\/]*/) {
try {
sh it
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this expecting tests are shell scripts, or does this work for an executable python script as well?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It should run "it" in a bourne shell. So if the test script is executable and it has a shebang at the top, it should run correctly.

@geoffreyblake geoffreyblake merged commit e59d728 into ARM-software:master Jun 7, 2018
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.

2 participants