-
Notifications
You must be signed in to change notification settings - Fork 73
[native_doc_dartifier] Add initial framework for End-to-End tests #2383
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@HosseinYousefi, I see the error in the CI run is about the GEMINI_API_KEY.
When trying with my API key on my fork, I pass all the tests and can't reproduce that error. |
PR HealthChangelog Entry ❗
Changes to files need to be accounted for in their respective changelogs. This check can be disabled by tagging the PR with API leaks ✔️The following packages contain symbols visible in the public API, but not exported by the library. Export these symbols or remove them from your publicly visible API.
License Headers ✔️
All source files should start with a license header. Unrelated files missing license headers
|
@athomas Could you please double check that the |
Replaces #2379, which was closed due to fork deletion by mistake.
Related to: #2343
End-to-End (E2E) testing workflow that validates Dartified Java code against expected Java behavior.
Java source files located in:
dartify_simple_cases/java/com/...
Java snippet definitions in:
dartify_simple_cases/java_snippets.dart
Tool:
tool/prepare_dartify_test.dart
that needs to be run beforeflutter test
dartify_simple_cases/dartified_snippets/...
A runtime Dart test:
test/runtime_test.dart
This verifies that the Dartified output matches the expected Java result (not running the Java snippet yet, but assuming we know the result).