Skip to content

Commit

Permalink
Merge pull request #40 from mainmatter/passing-ci
Browse files Browse the repository at this point in the history
chore(ci): make CI passing
  • Loading branch information
BobrImperator authored Sep 10, 2024
2 parents 35964de + 0274540 commit 0d01911
Show file tree
Hide file tree
Showing 2 changed files with 32 additions and 5 deletions.
35 changes: 31 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ on:
pull_request: {}

jobs:
test-ember-memory-leaks:
name: Test Emberfest2024
test-detect-leaking-classes:
name: Test detectLeakingClasses
runs-on: ubuntu-latest
timeout-minutes: 10

Expand All @@ -28,5 +28,32 @@ jobs:
- name: install dependencies
run: pnpm install --frozen-lockfile

- name: tests
run: pnpm test:memory-leak-ember
- run: |
set +e
pnpm -F ember-test-app start:memory-leak-detector & pid=$!
pnpm -F ember-test-app test:memory-leak
wait $pid
if [ $? -eq 1 ]; then exit 0; else exit $?; fi
test-detect-memory-leak:
name: Test detectMemoryLeak
runs-on: ubuntu-latest
timeout-minutes: 10

steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node_version: 20
- uses: pnpm/action-setup@v4
- uses: actions/cache@v4
with:
path: "**/node_modules"
key: ${{ runner.os }}-pnpm-${{ hashFiles('**/pnpm-lock.yaml') }}
restore-keys: |
${{ runner.os }}-pnpm-
- name: install dependencies
run: pnpm install --frozen-lockfile

- run: pnpm -F ember-test-app start:memory-leak-detector & pid=$!; pnpm -F ember-test-app test; wait $pid
2 changes: 1 addition & 1 deletion packages/test-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
"start:memory-leak-detector": "memory-leak-detector ./app/",
"test:memory-leak": "ember test --filter MEMORY_LEAK",
"test:users": "ember test --filter Users",
"test": "ember test"
"test": "ember test --filter !MEMORY_LEAK"
},
"devDependencies": {
"@babel/core": "7.25.2",
Expand Down

0 comments on commit 0d01911

Please sign in to comment.