NotificationsYou must be signed in to change notification settings
Fork3.6k
Star5.1k

Commitf54fe93

authored

[pigeon] Validates support for nested collections with generic types. (#7614)

Validates support for nested collections with generic types.Really just adds tests to ensure behavior. small step towardflutter/flutter#116117

1 parentd83d021 commitf54fe93Copy full SHA for f54fe93

File tree

20 files changed

+1549

-415

lines changed

packages/pigeon
- pigeons
  - core_tests.dart
- platform_tests
  - alternate_language_test_plugin
    - android/src
      - main/java/com/example/alternate_language_test_plugin
        CoreTests.java
      - test/java/com/example/alternate_language_test_plugin
        AllDatatypesTest.java
    - example/ios/RunnerTests
      - AllDatatypesTest.m
    - ios/Classes
      - CoreTests.gen.h
      - CoreTests.gen.m
    - macos/Classes
      - CoreTests.gen.h
      - CoreTests.gen.m
  - shared_test_plugin_code/lib
    - src/generated
      - core_tests.gen.dart
    - test_types.dart
  - test_plugin
    - android/src
      - main/kotlin/com/example/test_plugin
        CoreTests.gen.kt
      - test/kotlin/com/example/test_plugin
        AllDatatypesTest.kt
    - example/ios/RunnerTests
      - AllDatatypesTests.swift
    - ios/Classes
      - CoreTests.gen.swift
    - linux
      - pigeon
        core_tests.gen.cc
        core_tests.gen.h
      - test_plugin.cc
    - macos/Classes
      - CoreTests.gen.swift
    - windows/pigeon
      - core_tests.gen.cpp
      - core_tests.gen.h

20 files changed

+1549

-415

lines changed

`‎packages/pigeon/pigeons/core_tests.dart‎`

Lines changed: 18 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -52,13 +52,16 @@ class AllTypes {`
`52`	`52`	`requiredthis.enumList,`
`53`	`53`	`requiredthis.objectList,`
`54`	`54`	`requiredthis.listList,`
	`55`	`+requiredthis.mapList,`
`55`	`56`
`56`	`57`	`// Maps`
`57`	`58`	`requiredthis.map,`
`58`	`59`	`requiredthis.stringMap,`
`59`	`60`	`requiredthis.intMap,`
`60`	`61`	`requiredthis.enumMap,`
`61`	`62`	`requiredthis.objectMap,`
	`63`	`+requiredthis.listMap,`
	`64`	`+requiredthis.mapMap,`
`62`	`65`	`});`
`63`	`66`
`64`	`67`	`bool aBool;`
`@@ -84,6 +87,7 @@ class AllTypes {`
`84`	`87`	`List<AnEnum> enumList;`
`85`	`88`	`List<Object> objectList;`
`86`	`89`	`List<List<Object?>> listList;`
	`90`	`+List<Map<Object?,Object?>> mapList;`
`87`	`91`
`88`	`92`	`// Maps`
`89`	`93`	`// ignore: strict_raw_type, always_specify_types`
`@@ -92,6 +96,8 @@ class AllTypes {`
`92`	`96`	`Map<int,int> intMap;`
`93`	`97`	`Map<AnEnum,AnEnum> enumMap;`
`94`	`98`	`Map<Object,Object> objectMap;`
	`99`	`+Map<int,List<Object?>> listMap;`
	`100`	`+Map<int,Map<Object?,Object?>> mapMap;`
`95`	`101`	`}`
`96`	`102`
`97`	`103`	`/// A class containing all supported nullable types.`
`@@ -123,6 +129,7 @@ class AllNullableTypes {`
`123`	`129`	`this.enumList,`
`124`	`130`	`this.objectList,`
`125`	`131`	`this.listList,`
	`132`	`+this.mapList,`
`126`	`133`	`this.recursiveClassList,`
`127`	`134`
`128`	`135`	`// Maps`
`@@ -131,6 +138,8 @@ class AllNullableTypes {`
`131`	`138`	`this.intMap,`
`132`	`139`	`this.enumMap,`
`133`	`140`	`this.objectMap,`
	`141`	`+this.listMap,`
	`142`	`+this.mapMap,`
`134`	`143`	`this.recursiveClassMap,`
`135`	`144`	`);`
`136`	`145`
`@@ -158,6 +167,7 @@ class AllNullableTypes {`
`158`	`167`	`List<AnEnum?>? enumList;`
`159`	`168`	`List<Object?>? objectList;`
`160`	`169`	`List<List<Object?>?>? listList;`
	`170`	`+List<Map<Object?,Object?>?>? mapList;`
`161`	`171`	`List<AllNullableTypes?>? recursiveClassList;`
`162`	`172`
`163`	`173`	`// Maps`
`@@ -167,6 +177,8 @@ class AllNullableTypes {`
`167`	`177`	`Map<int?,int?>? intMap;`
`168`	`178`	`Map<AnEnum?,AnEnum?>? enumMap;`
`169`	`179`	`Map<Object?,Object?>? objectMap;`
	`180`	`+Map<int?,List<Object?>?>? listMap;`
	`181`	`+Map<int?,Map<Object?,Object?>?>? mapMap;`
`170`	`182`	`Map<int?,AllNullableTypes?>? recursiveClassMap;`
`171`	`183`	`}`
`172`	`184`
`@@ -199,13 +211,16 @@ class AllNullableTypesWithoutRecursion {`
`199`	`211`	`this.enumList,`
`200`	`212`	`this.objectList,`
`201`	`213`	`this.listList,`
	`214`	`+this.mapList,`
`202`	`215`
`203`	`216`	`// Maps`
`204`	`217`	`this.map,`
`205`	`218`	`this.stringMap,`
`206`	`219`	`this.intMap,`
`207`	`220`	`this.enumMap,`
`208`	`221`	`this.objectMap,`
	`222`	`+this.listMap,`
	`223`	`+this.mapMap,`
`209`	`224`	`);`
`210`	`225`
`211`	`226`	`bool? aNullableBool;`
`@@ -231,6 +246,7 @@ class AllNullableTypesWithoutRecursion {`
`231`	`246`	`List<AnEnum?>? enumList;`
`232`	`247`	`List<Object?>? objectList;`
`233`	`248`	`List<List<Object?>?>? listList;`
	`249`	`+List<Map<Object?,Object?>?>? mapList;`
`234`	`250`
`235`	`251`	`// Maps`
`236`	`252`	`// ignore: strict_raw_type, always_specify_types`
`@@ -239,6 +255,8 @@ class AllNullableTypesWithoutRecursion {`
`239`	`255`	`Map<int?,int?>? intMap;`
`240`	`256`	`Map<AnEnum?,AnEnum?>? enumMap;`
`241`	`257`	`Map<Object?,Object?>? objectMap;`
	`258`	`+Map<int?,List<Object?>?>? listMap;`
	`259`	`+Map<int?,Map<Object?,Object?>?>? mapMap;`
`242`	`260`	`}`
`243`	`261`
`244`	`262`	`/// A class for testing nested class handling.`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitf54fe93

File tree

20 files changed

20 files changed

`‎packages/pigeon/pigeons/core_tests.dart‎`

0 commit comments