Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
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
14 changes: 7 additions & 7 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -288,8 +288,8 @@ jobs:
flags: unittests
fail_ci_if_error: false

test-integration:
name: Test - Integration (Dart ${{ matrix.dart-version }})
test-wif:
name: Test - WIF Auth (Dart ${{ matrix.dart-version }})
runs-on: ubuntu-latest
# Skip for fork PRs — secrets are not available
if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.fork == false
Expand Down Expand Up @@ -343,19 +343,19 @@ jobs:
run: melos bootstrap
working-directory: .

- name: Run integration tests
run: dart test test/app/firebase_app_prod_test.dart --concurrency=1
- name: Run WIF auth test
run: dart test test/integration/app/firebase_app_prod_test.dart --concurrency=1

build:
name: Build verification (Dart ${{ matrix.dart-version }})
runs-on: ubuntu-latest
needs: [lint, test, test-integration]
# Run even if test-integration was skipped (fork PRs), but not if lint or test failed
needs: [lint, test, test-wif]
# Run even if test-wif was skipped (fork PRs), but not if lint or test failed
if: |
always() &&
needs.lint.result == 'success' &&
needs.test.result == 'success' &&
(needs.test-integration.result == 'success' || needs.test-integration.result == 'skipped')
(needs.test-wif.result == 'success' || needs.test-wif.result == 'skipped')

strategy:
fail-fast: false
Expand Down
2 changes: 1 addition & 1 deletion packages/dart_firebase_admin/firebase.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
},
"functions": [
{
"source": "test/functions",
"source": "test/fixtures/task_queue_functions",
"codebase": "default",
"ignore": [
"node_modules",
Expand Down
6 changes: 3 additions & 3 deletions packages/dart_firebase_admin/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,6 @@ dev_dependencies:
uuid: ^4.0.0

false_secrets:
- /test/auth/jwt_test.dart
- /test/client/get_id_token.js
- /test/mock_service_account.dart
- /test/unit/auth/jwt_test.dart
- /test/fixtures/mock_service_account.dart
- /test/unit/app/credential_test.dart
39 changes: 0 additions & 39 deletions packages/dart_firebase_admin/test/client/get_id_token.js

This file was deleted.

14 changes: 0 additions & 14 deletions packages/dart_firebase_admin/test/client/package.json

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import 'package:dart_firebase_admin/auth.dart';
import 'package:dart_firebase_admin/src/app.dart';
import 'package:test/test.dart';

import '../../helpers.dart';
import '../helpers.dart';

Future<void> cleanup(Auth auth) async {
// Only cleanup if we're using the emulator
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import 'package:dart_firebase_admin/src/app.dart';
import 'package:googleapis_auth/auth_io.dart';
import 'package:test/test.dart';

import '../../helpers.dart';
import '../helpers.dart';

/// Validates that Cloud Tasks emulator environment variable is set.
///
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Loading