Some functionalities were broken after the refactor. This fixes it.
We need to add emulator tests to catch this in the future, I see that flutter samples and package:http have this setup in their CI. Tracking this in #1155.
Details
| File | Coverage |
| :--- | :--- |
|pkgs/jni/example/lib/main.dart| :broken_heart: Not covered |
This check for [test coverage](https://github.com/dart-lang/ecosystem/wiki/Test-Coverage) is informational (issues shown here will not fail the PR).
This check can be disabled by tagging the PR with `skip-coverage-check`.
License Headers :heavy_check_mark:
Details
```
// Copyright (c) 2024, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
```
| Files |
| :--- |
| _no missing headers_ |
All source files should start with a [license header](https://github.com/dart-lang/ecosystem/wiki/License-Header).
Unrelated files missing license headers
| Files |
| :--- |
|pkgs/ffi/example/main.dart|
|pkgs/ffigen/example/libclang-example/generated_bindings.dart|
|pkgs/ffigen/example/shared_bindings/generate.dart|
|pkgs/ffigen/example/shared_bindings/lib/generated/a_gen.dart|
|pkgs/ffigen/example/shared_bindings/lib/generated/a_shared_b_gen.dart|
|pkgs/ffigen/example/shared_bindings/lib/generated/base_gen.dart|
|pkgs/ffigen/example/simple/generated_bindings.dart|
|pkgs/ffigen/lib/src/config_provider/config_spec.dart|
|pkgs/ffigen/lib/src/header_parser/clang_bindings/clang_bindings.dart|
|pkgs/ffigen/test/collision_tests/expected_bindings/_expected_decl_decl_collision_bindings.dart|
|pkgs/ffigen/test/collision_tests/expected_bindings/_expected_decl_symbol_address_collision_bindings.dart|
|pkgs/ffigen/test/collision_tests/expected_bindings/_expected_decl_type_name_collision_bindings.dart|
|pkgs/ffigen/test/collision_tests/expected_bindings/_expected_reserved_keyword_collision_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_comment_markup_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_dart_handle_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_forward_decl_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_functions_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_imported_types_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_native_func_typedef_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_opaque_dependencies_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_packed_structs_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_regress_384_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_struct_fptr_fields_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_typedef_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_unions_bindings.dart|
|pkgs/ffigen/test/header_parser_tests/expected_bindings/_expected_varargs_bindings.dart|
|pkgs/ffigen/test/large_integration_tests/_expected_cjson_bindings.dart|
|pkgs/ffigen/test/large_integration_tests/_expected_libclang_bindings.dart|
|pkgs/ffigen/test/large_integration_tests/_expected_sqlite_bindings.dart|
|pkgs/ffigen/test/native_test/_expected_native_test_bindings.dart|
|pkgs/jni/lib/src/lang/jcharacter.dart|
|pkgs/jni/lib/src/third_party/generated_bindings.dart|
|pkgs/jni/lib/src/third_party/global_env_extensions.dart|
|pkgs/jni/lib/src/third_party/jni_bindings_generated.dart|
|pkgs/jnigen/android_test_runner/lib/main.dart|
|pkgs/jnigen/example/in_app_java/lib/android_utils.dart|
|pkgs/jnigen/example/kotlin_plugin/example/lib/main.dart|
|pkgs/jnigen/example/kotlin_plugin/lib/kotlin_bindings.dart|
|pkgs/jnigen/example/kotlin_plugin/lib/kotlin_plugin.dart|
|pkgs/jnigen/example/pdfbox_plugin/lib/pdfbox_plugin.dart|
|pkgs/jnigen/example/pdfbox_plugin/lib/src/third_party/org/apache/pdfbox/pdmodel/PDDocument.dart|
|pkgs/jnigen/example/pdfbox_plugin/lib/src/third_party/org/apache/pdfbox/pdmodel/PDDocumentInformation.dart|
|pkgs/jnigen/example/pdfbox_plugin/lib/src/third_party/org/apache/pdfbox/pdmodel/_package.dart|
|pkgs/jnigen/example/pdfbox_plugin/lib/src/third_party/org/apache/pdfbox/text/PDFTextStripper.dart|
|pkgs/jnigen/example/pdfbox_plugin/lib/src/third_party/org/apache/pdfbox/text/_package.dart|
|pkgs/jnigen/lib/src/bindings/descriptor.dart|
|pkgs/jnigen/lib/src/elements/elements.g.dart|
|pkgs/jnigen/test/jackson_core_test/third_party/bindings/com/fasterxml/jackson/core/_package.dart|
|pkgs/jnigen/tool/command_runner.dart|
|pkgs/native_assets_cli/test/model/checksum_test.dart|
Package publish validation :heavy_check_mark:
Details
| Package | Version | Status |
| :--- | ---: | :--- |
| package:ffi | 2.1.2 | already published at pub.dev |
| package:ffigen | 13.0.0-wip | WIP (no publish necessary) |
| package:jni | 0.9.2 | already published at pub.dev |
| package:jnigen | 0.9.1 | already published at pub.dev |
| package:native_assets_cli | 0.6.0-wip | WIP (no publish necessary) |
| package:objective_c | 1.0.1 | already published at pub.dev |
| package:swiftgen | 0.0.1-wip | WIP (no publish necessary) |
Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.
coverage: 77.224% (+8.2%) from 69.047%
when pulling ed998f510011b69b4591f6e32ba8ab34e7842a64 on fix-jni-android-example
into 102b66213438103ad12af600a005140402d21083 on main.
Some functionalities were broken after the refactor. This fixes it.
We need to add emulator tests to catch this in the future, I see that flutter samples and
package:http
have this setup in their CI. Tracking this in #1155.