Skip to content

Revert "Run self-hosted and cloud tests only when not a PR (#37)" #39

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
alexgomezlf merged 2 commits into from
Mar 9, 2026
Merged

Revert "Run self-hosted and cloud tests only when not a PR (#37)" #39

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
16e0a55
Revert "Run self-hosted and cloud tests only when not a PR (#37)"
alexgomezlf Mar 5, 2026
9e43aab
Update GitHub Actions workflow configuration
alexgomezlf Mar 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 8 additions & 17 deletions .github/workflows/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -142,27 +142,25 @@ jobs:

- name: Test Report
uses: dorny/test-reporter@v2
if: always() # run this step even if previous step failed
if: success() || failure() # run this step even if previous step failed
with:
name: JS Utils Jest Test Results
path: ./packages/lf-js-utils/*.xml
reporter: jest-junit
only-summary: 'false'
list-tests: 'failed'
fail-on-error: 'false'
use-actions-summary: 'true'

- name: pnpm test lf-api-client-core on cloud
id: test-cloud
if: github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name
env:
ACCESS_KEY: ${{ secrets.DEV_CA_PUBLIC_USE_INTEGRATION_TEST_ACCESS_KEY }}
ACCESS_KEY_JSON: ${{ secrets.DEV_CA_PUBLIC_USE_INTEGRATION_TEST_ACCESS_KEY_JSON }}
SERVICE_PRINCIPAL_KEY: ${{ secrets.DEV_CA_PUBLIC_USE_TESTOAUTHSERVICEPRINCIPAL_SERVICE_PRINCIPAL_KEY }}
run: pnpm --filter @laserfiche/lf-api-client-core run test:Cloud

- name: pnpm test lf-api-client-core on self-hosted
if: always() && (steps.test-cloud.outcome == 'success' || steps.test-cloud.outcome == 'failure') && (github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name)
if: always() && (steps.test-cloud.outcome == 'success' || steps.test-cloud.outcome == 'failure')
env:
REPOSITORY_ID: ${{ secrets.APISERVER_REPOSITORY_ID }}
APISERVER_USERNAME: ${{ secrets.APISERVER_USERNAME }}
Expand All @@ -172,18 +170,16 @@ jobs:

- name: Test Report
uses: dorny/test-reporter@v2
if: always() && (github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name)
if: success() || failure() # run this step even if previous step failed
with:
name: Client Core Jest Test Results
path: ./packages/lf-api-client-core-js/*.xml
reporter: jest-junit
only-summary: 'false'
list-tests: 'failed'
fail-on-error: 'false'
use-actions-summary: 'true'

- name: pnpm test @laserfiche/lf-repository-api-client run all cloud tests
if: github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name
env:
ACCESS_KEY: ${{ secrets.DEV_US_PUBLIC_USE_INTEGRATION_TEST_ACCESS_KEY }}
SERVICE_PRINCIPAL_KEY: ${{ secrets.DEV_US_PUBLIC_USE_TESTOAUTHSERVICEPRINCIPAL_SERVICE_PRINCIPAL_KEY }}
Expand All @@ -193,7 +189,6 @@ jobs:
run: pnpm --filter @laserfiche/lf-repository-api-client run test:all

- name: pnpm test @laserfiche/lf-repository-api-client run all self-hosted tests
if: github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name
env:
ACCESS_KEY: ${{ secrets.DEV_US_PUBLIC_USE_INTEGRATION_TEST_ACCESS_KEY }}
SERVICE_PRINCIPAL_KEY: ${{ secrets.DEV_US_PUBLIC_USE_TESTOAUTHSERVICEPRINCIPAL_SERVICE_PRINCIPAL_KEY }}
Expand All @@ -207,19 +202,17 @@ jobs:

- name: Test Report
uses: dorny/test-reporter@v2
if: always() && (github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name)
if: success() || failure() # run this step even if previous step failed
with:
name: Api Client v1 Jest Test Results
path: ./packages/lf-repository-api-client-v1/*.xml
reporter: jest-junit
only-summary: 'false'
list-tests: 'failed'
fail-on-error: 'false'
use-actions-summary: 'true'

- name: pnpm test lf-repository-api-client-v2 on cloud (node environment)
id: test-cloud-node
if: github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name
env:
ACCESS_KEY: ${{ secrets.DEV_CA_PUBLIC_USE_INTEGRATION_TEST_ACCESS_KEY }}
SERVICE_PRINCIPAL_KEY: ${{ secrets.DEV_CA_PUBLIC_USE_TESTOAUTHSERVICEPRINCIPAL_SERVICE_PRINCIPAL_KEY }}
Expand All @@ -230,7 +223,6 @@ jobs:

- name: pnpm test lf-repository-api-client-v2 on cloud (browser environment)
id: test-cloud-browser
if: github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name
env:
ACCESS_KEY: ${{ secrets.DEV_CA_PUBLIC_USE_INTEGRATION_TEST_ACCESS_KEY }}
SERVICE_PRINCIPAL_KEY: ${{ secrets.DEV_CA_PUBLIC_USE_TESTOAUTHSERVICEPRINCIPAL_SERVICE_PRINCIPAL_KEY }}
Expand All @@ -241,31 +233,29 @@ jobs:

- name: Test report
uses: dorny/test-reporter@v2
if: always() && (github.event_name != 'pull_request' || github.repository == github.event.pull_request.head.repo.full_name)
if: success() || failure() # run this step even if previous step failed
with:
name: Api Client v2 Jest Test Results
path: ./packages/lf-repository-api-client-v2/*.xml
reporter: jest-junit
only-summary: 'false'
list-tests: 'failed'
fail-on-error: 'false'
use-actions-summary: 'true'

- name: pnpm test @laserfiche/lf-api-js run all tests
run: pnpm --filter @laserfiche/lf-api-js run test

- name: Test Report
uses: dorny/test-reporter@v2
if: always() # run this step even if previous step failed
if: success() || failure() # run this step even if previous step failed
with:
name: Api JS Jest Test Results
path: ./packages/lf-api-js/*.xml
reporter: jest-junit
only-summary: 'false'
list-tests: 'failed'
fail-on-error: 'false'
use-actions-summary: 'true'


build_documentation:
runs-on: ubuntu-latest
needs: [build_libraries, test_libraries]
Expand Down Expand Up @@ -543,3 +533,4 @@ jobs:
uses: actions/deploy-pages@v4
with:
artifact_name: github-pages