feat: remove string component type (#193)

aws-amplify · Nov 8, 2021 · 986fc5f · 986fc5f
1 parent b5e1f98
commit 986fc5f
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 63 deletions.
diff --git a/...ui-codegen-react/lib/__tests__/amplify-ui-renderers/__snapshots__/primitives.test.ts.snap b/...ui-codegen-react/lib/__tests__/amplify-ui-renderers/__snapshots__/primitives.test.ts.snap
@@ -62,10 +62,6 @@ exports[`Primitives SelectField 1`] = `"<SelectField {...rest} {...getOverridePr
 
 exports[`Primitives StepperField 1`] = `"<StepperField {...rest} {...getOverrideProps(overrides, \\"StepperField\\")}></StepperField>"`;
 
-exports[`Primitives StringRenderer 1`] = `"<>{\\"test\\"}</>"`;
-
-exports[`Primitives StringRenderer throws on missing props 1`] = `"Failed to render String - Unexpected component structure"`;
-
 exports[`Primitives SwitchField 1`] = `"<SwitchField {...rest} {...getOverrideProps(overrides, \\"SwitchField\\")}></SwitchField>"`;
 
 exports[`Primitives Tabs 1`] = `"<Tabs {...rest} {...getOverrideProps(overrides, \\"Tabs\\")}></Tabs>"`;

diff --git a/packages/studio-ui-codegen-react/lib/__tests__/amplify-ui-renderers/primitives.test.ts b/packages/studio-ui-codegen-react/lib/__tests__/amplify-ui-renderers/primitives.test.ts
@@ -16,7 +16,6 @@
 import { StudioComponent } from '@amzn/studio-ui-codegen';
 import { assertASTMatchesSnapshot } from '../__utils__/snapshot-helpers';
 
-import renderString from '../../amplify-ui-renderers/string';
 import { AmplifyRenderer } from '../../amplify-ui-renderers/amplify-renderer';
 import Primitives from '../../primitives';
 
@@ -36,28 +35,4 @@ describe('Primitives', () => {
       });
     });
   });
-
-  test('StringRenderer', () => {
-    const component = {
-      componentType: 'String',
-      name: 'MyString',
-      properties: {
-        value: {
-          value: 'test',
-        },
-      },
-      bindingProperties: {},
-    };
-    testPrimitive(component);
-  });
-
-  test('StringRenderer throws on missing props', () => {
-    const component = {
-      componentType: 'String',
-      name: 'MyString',
-      properties: {},
-      bindingProperties: {},
-    };
-    expect(() => renderString(component)).toThrowErrorMatchingSnapshot();
-  });
 });
diff --git a/packages/studio-ui-codegen-react/lib/amplify-ui-renderers/amplify-renderer.ts b/packages/studio-ui-codegen-react/lib/amplify-ui-renderers/amplify-renderer.ts
@@ -56,7 +56,6 @@ import {
 import Primitives from '../primitives';
 import { ReactStudioTemplateRenderer } from '../react-studio-template-renderer';
 import TextRenderer from './text';
-import renderString from './string';
 import CustomComponentRenderer from './customComponent';
 import CollectionRenderer from './collection';
 import { ReactComponentWithChildrenRenderer } from '../react-component-with-children-renderer';
@@ -329,10 +328,6 @@ export class AmplifyRenderer extends ReactStudioTemplateRenderer {
           parent,
         ).renderElement(renderChildren);
 
-      // to be removed
-      case 'String':
-        return renderString(component as StudioComponentChild);
-
       default:
         return new CustomComponentRenderer(component, this.importCollection, parent).renderElement(renderChildren);
     }

diff --git a/packages/studio-ui-codegen-react/lib/amplify-ui-renderers/string.ts b/packages/studio-ui-codegen-react/lib/amplify-ui-renderers/string.ts