Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[Windows] Support native functions in test fixtures #35357

Merged
merged 1 commit into from
Aug 12, 2022

Conversation

cbracken
Copy link
Member

Adds the ability to register native functions for use in test fixtures.
This allows registering native C++ functions that can be invoked from
Dart code to perform the following common actions:

  • Signal a waiting latch in the C++ part of the test.
  • Pass data back to the C++ part of the test.
  • Allow the C++ part of the test to pass data to the test.

Fixes: flutter/flutter#109242
Fixes: flutter/flutter#87299

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See testing the engine for instructions on
    writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@cbracken cbracken force-pushed the win-native-functions branch 2 times, most recently from e11ce62 to 0b1b05b Compare August 11, 2022 22:34
Copy link
Member

@loic-sharma loic-sharma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@cbracken cbracken force-pushed the win-native-functions branch from 0b1b05b to 7a5c051 Compare August 11, 2022 22:40
@cbracken cbracken force-pushed the win-native-functions branch from 7a5c051 to a7bb37e Compare August 11, 2022 22:55
Adds the ability to register native functions for use in test fixtures.
This allows registering native C++ functions that can be invoked from
Dart code to perform the following common actions:

* Signal a waiting latch in the C++ part of the test.
* Pass data back to the C++ part of the test.
* Allow the C++ part of the test to pass data to the test.

Fixes: flutter/flutter#109242
Fixes: flutter/flutter#87299
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Windows] Support registering native functions in WindowsTestContext Add Windows test helper API and placeholder test
3 participants