Skip to content
Closed
Show file tree
Hide file tree
Changes from 1 commit
Commits
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
Next Next commit
ci: generate code coverage in flutter integration test
  • Loading branch information
vaind committed Oct 3, 2023
commit 537aa37ac3bfb0ae387e897095ffcefd89e530da
1 change: 0 additions & 1 deletion .github/workflows/flutter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,6 @@ jobs:
with:
name: sentry_flutter
file: ./flutter/coverage/lcov.info
functionalities: "search" # remove after https://github.com/codecov/codecov-action/issues/600

- uses: VeryGoodOpenSource/very_good_coverage@e5c91bc7ce9843e87c800b3bcafdfb86fbe28491 # [email protected]
if: runner.os == 'Linux' && matrix.sdk == 'stable' && matrix.target == 'linux'
Expand Down
16 changes: 14 additions & 2 deletions .github/workflows/flutter_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,13 @@ jobs:
avd-name: macOS-avd-x86_64-31
emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
disable-animations: true
script: flutter test integration_test --verbose
script: flutter test --coverage integration_test --verbose

- uses: codecov/codecov-action@d9f34f8cd5cb3b3eb79b3e4b5dae3a16df499a70 # pin@v3
if: matrix.sdk == 'stable'
with:
name: sentry_flutter_integration_android
file: ./flutter/coverage/lcov.info

cocoa:
name: "${{ matrix.target }} | ${{ matrix.sdk }}"
Expand Down Expand Up @@ -155,11 +161,17 @@ jobs:
- name: run integration test
# Disable flutter integration tests for iOS for now (https://github.com/getsentry/sentry-dart/issues/1605#issuecomment-1695809346)
if: ${{ matrix.target != 'ios' }}
run: flutter test -d "${{ steps.device.outputs.name }}" integration_test --verbose
run: flutter test --coverage -d "${{ steps.device.outputs.name }}" integration_test --verbose

- name: run native test
# We only have the native unit test package in the iOS xcodeproj at the moment.
# Should be OK because it will likely be removed after switching to FFI (see https://github.com/getsentry/sentry-dart/issues/1444).
if: ${{ matrix.target != 'macos' }}
working-directory: ./flutter/example/${{ matrix.target }}
run: xcodebuild test -workspace Runner.xcworkspace -scheme Runner -configuration Debug -destination "platform=${{ steps.device.outputs.platform }}" -allowProvisioningUpdates CODE_SIGNING_ALLOWED=NO

- uses: codecov/codecov-action@d9f34f8cd5cb3b3eb79b3e4b5dae3a16df499a70 # pin@v3
if: matrix.sdk == 'stable'
with:
name: sentry_flutter_integration_cocoa
file: ./flutter/coverage/lcov.info
3 changes: 1 addition & 2 deletions .github/workflows/sqflite.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
access_token: ${{ github.token }}

build:
name: ${{ matrix.target }} | ${{ matrix.os }} | ${{ matrix.sdk }}
name: "${{ matrix.target }} | ${{ matrix.os }} | ${{ matrix.sdk }}"
runs-on: ${{ matrix.os }}
timeout-minutes: 30
defaults:
Expand Down Expand Up @@ -98,7 +98,6 @@ jobs:
with:
name: sentry_sqflite
file: ./sqflite/coverage/lcov.info
functionalities: "search" # remove after https://github.com/codecov/codecov-action/issues/600

- uses: VeryGoodOpenSource/very_good_coverage@e5c91bc7ce9843e87c800b3bcafdfb86fbe28491 # [email protected]
if: runner.os == 'Linux' && matrix.sdk == 'stable' && matrix.target == 'linux'
Expand Down
8 changes: 1 addition & 7 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -14,14 +14,8 @@ build/
.cxx/
.vscode/


.test_coverage.dart
dart/coverage/*
logging/coverage/*
dio/coverage/*
file/coverage/*
flutter/coverage/*
sqflite/coverage/*
**/coverage/*

pubspec.lock
Podfile.lock
Expand Down