Skip to content

Tests for ‎dev/integration_tests/hook_user_defines don't run on CI (and currently fail) #180575

Closed

Description

@goderbauer

As far as I can tell, the tests for dev/integration_tests/hook_user_defines were never added to a CI build and currently don't run.

They also fail when running them locally with flutter test:

Unhandled exception:
Invalid argument(s): User-define `magic_value` must be an integer, found: null.
#0      main.<anonymous closure> (file:///Users/goderbauer/dev/flutter/dev/integration_tests/hook_user_defines/hook/build.dart:13:7)
#1      build (package:hooks/src/api/build_and_link.dart:158:18)
#2      main (file:///Users/goderbauer/dev/flutter/dev/integration_tests/hook_user_defines/hook/build.dart:10:9)
#3      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:312:33)
#4      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:193:12)

This failure is probably due to resolution: workspace in the package's pubspec.yaml. The workspace pubspec does not define a value for the magic_value user define. Only the package-local pubspec does. We probably don't want random test user defines in the root pubspec. Maybe this packages has to be removed from the workspace?

Steps necessary to close this issue:

  • Fix the tests of dev/integration_tests/hook_user_defines
  • Make these tests run on CI somewhere (maybe as parts of the tools integration test shard?) so they don't break again.

Activity

  1. goderbauer commented on Jan 6, 2026

    @goderbauer
    MemberAuthor
  2. goderbauer commented on Jan 6, 2026

    @goderbauer
    MemberAuthor

    FWIW, the tests of dev/integration_tests/link_hooks are also not running on CI. I am trying to fix that as part of #180507.

  3. dcharkes commented on Jan 6, 2026

    @dcharkes
    Contributor

    Maybe this packages has to be removed from the workspace?

    Yes, that would make sense. 👍 Unless there's a workspace for the integration tests as a group. (I would think integration test projects get copied to a temp directory, and then workspace resolution wouldn't work at all. How are the other integration tests run? Are they run in place?)

  4. bkonyi commented on Jan 6, 2026

    @bkonyi
    Contributor

    They're run in place as they're part of the _flutter_packages workspace at the root of the repository.

  5. added
    in triagePresently being triaged by the triage team
    a: tests"flutter test", flutter_test, or one of our tests
    toolAffects the "flutter" command-line tool. See also t: labels.
    a: error messageError messages from the Flutter framework
    team-toolOwned by Flutter Tool team
    f: integration_testThe flutter/packages/integration_test plugin
    and removed
    in triagePresently being triaged by the triage team
    on Jan 7, 2026
  6. added
    r: fixedIssue is closed as already fixed in a newer version
    on Jan 8, 2026
  7. github-actions commented on Jan 22, 2026

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  8. locked as resolved and limited conversation to collaborators on Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    a: error messageError messages from the Flutter frameworka: tests"flutter test", flutter_test, or one of our testsf: integration_testThe flutter/packages/integration_test pluginr: fixedIssue is closed as already fixed in a newer versionteam-toolOwned by Flutter Tool teamtoolAffects the "flutter" command-line tool. See also t: labels.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions