Add tests for generated getPropKey implementation

Workiva · Dec 5, 2023 · 481269f · 481269f
1 parent 0840751
commit 481269f
Show file tree

Hide file tree

Showing 5 changed files with 39 additions and 3 deletions.
diff --git a/...eclaration/builder_integration_tests/backwards_compatible/component_integration_test.dart b/...eclaration/builder_integration_tests/backwards_compatible/component_integration_test.dart
@@ -92,7 +92,15 @@ main() {
         expect(ComponentTest()..customKeyAndNamespaceProp = 'test',
             containsPair('custom namespace~~custom key!', 'test'));
       });
+    });
+
+    test('generates a functional getPropKey implementation', () {
+      expect(ComponentTest().getPropKey((p) => p.stringProp), 'ComponentTestProps.stringProp');
+      expect(ComponentTest().getPropKey((p) => p.customKeyAndNamespaceProp),
+          'custom namespace~~custom key!');
+    });
 
+    group('generates componentDefaultProps with', () {
       test('default props', () {
         expect(ComponentTest().componentDefaultProps, equals({'id':'testId'}));
       });

diff --git a/...omponent_declaration/builder_integration_tests/component2/component_integration_test.dart b/...omponent_declaration/builder_integration_tests/component2/component_integration_test.dart
@@ -128,7 +128,15 @@ main() {
           expect(ComponentTest()..customKeyAndNamespaceProp = 'test',
               containsPair('custom namespace~~custom key!', 'test'));
         });
+      });
+
+      test('generates a functional getPropKey implementation', () {
+        expect(ComponentTest().getPropKey((p) => p.stringProp), 'ComponentTestProps.stringProp');
+        expect(ComponentTest().getPropKey((p) => p.customKeyAndNamespaceProp),
+            'custom namespace~~custom key!');
+      });
 
+      group('generates componentDefaultProps with', () {
         test('default props', () {
           expect(ComponentTest().componentDefaultProps, equals
             ({'id':'testId',

diff --git a/...ent_declaration/builder_integration_tests/new_boilerplate/component_integration_test.dart b/...ent_declaration/builder_integration_tests/new_boilerplate/component_integration_test.dart
@@ -149,7 +149,15 @@ main() {
           expect(ComponentTest()..customKeyAndNamespaceProp = 'test',
               containsPair('custom namespace~~custom key!', 'test'));
         });
+      });
+
+      test('generates a functional getPropKey implementation', () {
+        expect(ComponentTest().getPropKey((p) => p.stringProp), 'ComponentTestProps.stringProp');
+        expect(ComponentTest().getPropKey((p) => p.customKeyAndNamespaceProp),
+            'custom namespace~~custom key!');
+      });
 
+      group('generates componentDefaultProps with', () {
         test('default props', () {
           expect(ComponentTest().componentDefaultProps, equals
             ({'id':'testId',

diff --git a/...ponent_declaration/builder_integration_tests/new_boilerplate/function_component_test.dart b/...ponent_declaration/builder_integration_tests/new_boilerplate/function_component_test.dart
@@ -255,6 +255,12 @@ void functionComponentTestHelper(UiFactory<TestProps> factory,
     });
   });
 
+  test('generates a functional getPropKey implementation', () {
+    expect(factory().getPropKey((p) => p.stringProp), 'TestPropsMixin.stringProp');
+    expect(factory().getPropKey((p) => p.customKeyAndNamespaceProp),
+        'custom namespace~~custom key!');
+  });
+
   group('can pass along unconsumed props', () {
     const stringProp = 'a string';
     const anotherProp = 'this should be filtered';

diff --git a/.../component_declaration/builder_integration_tests/new_boilerplate/props_map_view_test.dart b/.../component_declaration/builder_integration_tests/new_boilerplate/props_map_view_test.dart
@@ -68,10 +68,16 @@ main() {
         expect(Test()..customKeyAndNamespaceProp = 'test',
             containsPair('custom namespace~~custom key!', 'test'));
       });
+    });
 
-      test('returns an empty map for componentDefaultProps', () {
-        expect(Test().componentDefaultProps, equals({}));
-      });
+    test('generates a functional getPropKey implementation', () {
+      expect(Test().getPropKey((p) => p.stringProp), 'TestProps.stringProp');
+      expect(
+          Test().getPropKey((p) => p.customKeyAndNamespaceProp), 'custom namespace~~custom key!');
+    });
+
+    test('generates componentDefaultProps that returns an empty map', () {
+      expect(Test().componentDefaultProps, equals({}));
     });
 
     test('raises an assertion when invoked via call/build, since it has no componentFactory', () {