Skip to content

Commit

Permalink
Restore ability to execute tests in custom OpenCB builds (#394)
Browse files Browse the repository at this point in the history
* Configs update
* Restore ability to execute tests
* Disable failing tests
  • Loading branch information
WojciechMazur authored Aug 26, 2024
1 parent d0e1d10 commit 76f8852
Show file tree
Hide file tree
Showing 17 changed files with 140 additions and 30 deletions.
6 changes: 5 additions & 1 deletion .github/actions/build-project/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,9 @@ inputs:
extra-library-dependencies:
description: "List of library dependencies which should be injected when building projects, in format org:artifact:version or org::artifact:version for Scala cross version. Multiple entires should be seperated by a single semicolon character `;`"
default: ""
execute-tests:
descriptions: "Should tests of the projects be executed"
default: false
custom-build-id:
description: "Custom buildId to use instead of autogenerated github job id"
default: ""
Expand Down Expand Up @@ -144,7 +147,8 @@ runs:
echo 'failure' > build-status.txt
echo 'unknown' > build-tool.txt
/build/build-revision.sh \
OPENCB_EXECUTE_TESTS="${{ inputs.execute-tests }}" \
/build/build-revision.sh \
"$(config .project)" \
"$(config .repoUrl)" \
"$(config .revision)" \
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/buildExecuteCustom-A.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@ on:
type: string
description: "List of library dependencies which should be injected when building projects, in format org:artifact:version, or org::artifact:version for Scala cross version. Multiple entires should be seperated by a single semicolon character `;`. (Best effort)"
default: ""
execute-tests:
type: boolean
description: "Should the workflow execute tests of the build projects (compile only by default)"
default: false
push-to-gh-pages:
type: boolean
description: "Should the workflow push the generated raport to gh-pages branch"
Expand All @@ -46,6 +50,7 @@ jobs:
extra-scalac-options: ${{ inputs.extra-scalac-options }}
disabled-scalac-options: ${{ inputs.disabled-scalac-options }}
extra-library-dependencies: ${{ inputs.extra-library-dependencies }}
execute-tests: ${{ inputs.execute-tests }}
custom-build-id: ${{ inputs.build-name }}
secrets: inherit

Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/buildExecuteCustom-B.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@ on:
type: string
description: "List of library dependencies which should be injected when building projects, in format org:artifact:version, or org::artifact:version for Scala cross version. Multiple entires should be seperated by a single semicolon character `;`. (Best effort)"
default: ""
execute-tests:
type: boolean
description: "Should the workflow execute tests of the build projects (compile only by default)"
default: false
push-to-gh-pages:
type: boolean
description: "Should the workflow push the generated raport to gh-pages branch"
Expand All @@ -46,6 +50,7 @@ jobs:
extra-scalac-options: ${{ inputs.extra-scalac-options }}
disabled-scalac-options: ${{ inputs.disabled-scalac-options }}
extra-library-dependencies: ${{ inputs.extra-library-dependencies }}
execute-tests: ${{ inputs.execute-tests }}
custom-build-id: ${{ inputs.build-name }}
secrets: inherit

Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/buildSingle.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@ on:
type: string
description: "List of scalacOptions which should be filtered out when building projects."
default: ""
execute-tests:
type: boolean
description: "Should the workflow execute tests of the build projects (compile only by default)"
default: false
jobs:
setup-build:
runs-on: ubuntu-22.04
Expand Down Expand Up @@ -64,6 +68,7 @@ jobs:
custom-build-id: ${{ inputs.build-name }}
extra-scalac-options: ${{ inputs.extra-scalac-options }}
disabled-scalac-options: ${{ inputs.disabled-scalac-options }}
execute-tests: ${{ inputs.execute-tests }}
scala-version: ${{ needs.setup-build.outputs.scala-version }}
maven-repo-url: ${{ needs.setup-build.outputs.maven-repo-url }}
elastic-user: ${{ secrets.OPENCB_ELASTIC_USER }}
Expand Down
7 changes: 4 additions & 3 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,11 @@ jobs:
- name: Build up base docker images
env:
BUILD_ONLY_DEFAULT_JDK: true
# Stopped to resolve DNS for archives.ubuntu.com when bulding after minikube docker-env
# eval $(minikube -p minikube docker-env)
run: |
eval $(minikube -p minikube docker-env)
scripts/build-all.sh test
- name: Test build
run: scripts/test-build.sh
# - name: Test build
# run: scripts/test-build.sh

1 change: 1 addition & 0 deletions coordinator/configs/filtered-projects.txt
Original file line number Diff line number Diff line change
Expand Up @@ -222,6 +222,7 @@ vigoo:zio-http-rust:.*
# gradle projects (unsupprted)
xebia-functional:xef:.*
mbari-org:commons:.*
newrelic:newrelic-java-agent:.*

# scala-compose (unsupprted)
virtuslab:besom:.*
Expand Down
Loading

0 comments on commit 76f8852

Please sign in to comment.