Skip to content

Commit 1b3ee04

Browse files
Fix more tests
1 parent a8eb5aa commit 1b3ee04

File tree

7 files changed

+3
-98
lines changed

7 files changed

+3
-98
lines changed

pkgs/jnigen/cmake/CMakeLists.txt.tmpl

Lines changed: 0 additions & 32 deletions
This file was deleted.

pkgs/jnigen/cmake/README.md

Lines changed: 0 additions & 3 deletions
This file was deleted.

pkgs/jnigen/example/pdfbox_plugin/lib/src/third_party/_init.dart

Lines changed: 0 additions & 25 deletions
This file was deleted.

pkgs/jnigen/example/pdfbox_plugin/linux/CMakeLists.txt

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,3 @@ cmake_minimum_required(VERSION 3.10)
66
# Project-level configuration.
77
set(PROJECT_NAME "pdfbox_plugin")
88
project(${PROJECT_NAME} LANGUAGES CXX)
9-
10-
# Invoke the build for native code shared with the other target platforms.
11-
# This can be changed to accomodate different builds.
12-
add_subdirectory("${CMAKE_CURRENT_SOURCE_DIR}/../src" "${CMAKE_CURRENT_BINARY_DIR}/shared")
13-
14-
# List of absolute paths to libraries that should be bundled with the plugin.
15-
# This list could contain prebuilt libraries, or libraries created by an
16-
# external build triggered from this build file.
17-
set(pdfbox_plugin_bundled_libraries
18-
# Defined in ../src/CMakeLists.txt.
19-
# This can be changed to accomodate different builds.
20-
$<TARGET_FILE:pdfbox_plugin>
21-
PARENT_SCOPE
22-
)

pkgs/jnigen/example/pdfbox_plugin/windows/CMakeLists.txt

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -7,17 +7,3 @@ cmake_minimum_required(VERSION 3.14)
77
# Project-level configuration.
88
set(PROJECT_NAME "pdfbox_plugin")
99
project(${PROJECT_NAME} LANGUAGES CXX)
10-
11-
# Invoke the build for native code shared with the other target platforms.
12-
# This can be changed to accomodate different builds.
13-
add_subdirectory("${CMAKE_CURRENT_SOURCE_DIR}/../src" "${CMAKE_CURRENT_BINARY_DIR}/shared")
14-
15-
# List of absolute paths to libraries that should be bundled with the plugin.
16-
# This list could contain prebuilt libraries, or libraries created by an
17-
# external build triggered from this build file.
18-
set(pdfbox_plugin_bundled_libraries
19-
# Defined in ../src/CMakeLists.txt.
20-
# This can be changed to accomodate different builds.
21-
$<TARGET_FILE:pdfbox_plugin>
22-
PARENT_SCOPE
23-
)

pkgs/jnigen/test/config_test.dart

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,8 @@ import 'test_util/test_util.dart';
1515
const packageTests = 'test';
1616
final jacksonCoreTests = absolute(packageTests, 'jackson_core_test');
1717
final thirdParty = absolute(jacksonCoreTests, 'third_party');
18-
final lib = absolute(thirdParty, 'c_based', 'dart_bindings');
19-
final src = absolute(thirdParty, 'c_based', 'c_bindings');
20-
final testLib = absolute(thirdParty, 'test_', 'c_based', 'dart_bindings');
21-
final testSrc = absolute(thirdParty, 'test_', 'c_based', 'c_bindings');
18+
final lib = absolute(thirdParty, 'bindings');
19+
final testLib = absolute(thirdParty, 'test_', 'bindings');
2220

2321
/// Compares 2 [Config] objects using [expect] to give useful errors when
2422
/// two fields are not equal.
@@ -97,7 +95,6 @@ void main() async {
9795
final config = Config.parseArgs([
9896
'--config',
9997
jnigenYaml,
100-
'-Doutput.c.path=$testSrc${Platform.pathSeparator}',
10198
'-Doutput.dart.path=$testLib${Platform.pathSeparator}',
10299
]);
103100

@@ -111,10 +108,6 @@ void main() async {
111108
});
112109

113110
group('Test for config error checking', () {
114-
testForErrorChecking<ConfigException>(
115-
name: 'Invalid bindings type',
116-
overrides: ['-Doutput.bindings_type=c_base'],
117-
);
118111
testForErrorChecking<ConfigException>(
119112
name: 'Invalid output structure',
120113
overrides: ['-Doutput.dart.structure=singl_file'],

pkgs/jnigen/test/jackson_core_test/jnigen.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ maven_downloads:
66

77
output:
88
dart:
9-
path: third_party/c_based/dart_bindings/
9+
path: third_party/bindings/
1010

1111
classes:
1212
- 'com.fasterxml.jackson.core.JsonFactory'

0 commit comments

Comments
 (0)