chore(docgen): remove constructorName property

build/gulp/plugins/util/getComponentInfo.ts

@@ -46,10 +46,6 @@ const getComponentInfo = (filepath: string, checksum?: string) => {
   // add checksum
   info.checksum = checksum
 
-  // add exported Component info
-  const Component = require(absPath).default
-  info.constructorName = _.get(Component, 'prototype.constructor.name', null)
-
   // add component type
   info.type = componentType
 

build/gulp/tasks/generate/templates/test-specs-components/$DisplayName/$DisplayName-test.tsx

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import $DisplayName from 'src/components/$DisplayName/$DisplayName'
 
 describe('$DisplayName', () => {
-  isConformant($DisplayName)
+  isConformant($DisplayName, '$DisplayName')
 })

docs/src/utils/componentInfoShape.ts

@@ -17,7 +17,6 @@ const componentInfoShape = PropTypes.shape({
       name: PropTypes.string,
     }),
   ),
-  constructorName: PropTypes.string.isRequired,
   type: PropTypes.string.isRequired,
   isParent: PropTypes.bool.isRequired,
   isChild: PropTypes.bool.isRequired,

test/specs/commonTests/isConformant.tsx

@@ -1,7 +1,6 @@
 import * as _ from 'lodash'
 import * as React from 'react'
 import { ReactWrapper } from 'enzyme'
-import * as ReactDOMServer from 'react-dom/server'
 
 import isExportedAtTopLevel from './isExportedAtTopLevel'
 import {
@@ -28,14 +27,15 @@ export interface Conformant {
 /**
  * Assert Component conforms to guidelines that are applicable to all components.
  * @param {React.Component|Function} Component A component that should conform.
+ * @param {String} componentName A name of a component that should conform.
  * @param {Object} [options={}]
  * @param {Object} [options.eventTargets={}] Map of events and the child component to target.
  * @param {boolean} [options.exportedAtTopLevel=false] Is this component exported as top level API?
  * @param {boolean} [options.rendersPortal=false] Does this component render a Portal powered component?
  * @param {Object} [options.requiredProps={}] Props required to render Component without errors or warnings.
  * @param {boolean} [options.usesWrapperSlot=false] This component uses wrapper slot to wrap the 'meaningful' element.
  */
-export default (Component, options: Conformant = {}) => {
+export default (Component, componentName: string, options: Conformant = {}) => {
   const {
     eventTargets = {},
     exportedAtTopLevel = true,
@@ -77,24 +77,13 @@ export default (Component, options: Conformant = {}) => {
     throwError(`Components should export a class or function, got: ${componentType}.`)
   }
 
-  // tests depend on Component constructor names, enforce them
-  const constructorName = Component.prototype.constructor.name
-  if (!constructorName) {
-    throwError(
-      [
-        'Component is not a named function. This should help identify it:\n\n',
-        `${ReactDOMServer.renderToStaticMarkup(<Component />)}`,
-      ].join(''),
-    )
-  }
-
   // ----------------------------------------
   // Component info
   // ----------------------------------------
   // This is pretty ugly because:
   // - jest doesn't support custom error messages
   // - jest will run all test
-  const infoJSONPath = `docs/src/componentInfo/${constructorName}.info.json`
+  const infoJSONPath = `docs/src/componentInfo/${componentName}.info.json`
 
   let info
 
@@ -137,16 +126,16 @@ export default (Component, options: Conformant = {}) => {
   }
 
   // ----------------------------------------
-  // Class and file name
+  // Component and file name
   // ----------------------------------------
-  test(`constructor name matches filename "${constructorName}"`, () => {
-    expect(constructorName).toEqual(info.filenameWithoutExt)
+  test(`component name matches filename "${componentName}"`, () => {
+    expect(componentName).toEqual(info.filenameWithoutExt)
   })
 
   // find the apiPath in the stardust object
   const foundAsSubcomponent = _.isFunction(_.get(stardust, info.apiPath))
 
-  exportedAtTopLevel && isExportedAtTopLevel(constructorName, info.displayName)
+  exportedAtTopLevel && isExportedAtTopLevel(componentName, info.displayName)
   if (info.isChild) {
     test('is a static component on its parent', () => {
       const message =
@@ -489,7 +478,7 @@ export default (Component, options: Conformant = {}) => {
   // ----------------------------------------
   describe('static displayName (common)', () => {
     test('matches constructor name', () => {
-      expect(Component.displayName).toEqual(info.constructorName)
+      expect(Component.displayName).toEqual(componentName)
     })
   })
 

test/specs/commonTests/isExportedAtTopLevel.tsx

@@ -5,8 +5,8 @@ import * as stardust from 'src/'
 // Is exported or private
 // ----------------------------------------
 // detect components like: stardust.H1
-export default (constructorName: string, displayName: string) => {
-  const isTopLevelAPIProp = _.has(stardust, constructorName)
+export default (componentName: string, displayName: string) => {
+  const isTopLevelAPIProp = _.has(stardust, componentName)
 
   // require all components to be exported at the top level
   test('is exported at the top level', () => {

test/specs/components/Accordion/Accordion-test.ts

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Accordion from 'src/components/Accordion/Accordion'
 
 describe('Accordion', () => {
-  isConformant(Accordion)
+  isConformant(Accordion, 'Accordion')
 })

test/specs/components/Animation/Animation-test.tsx

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Animation from 'src/components/Animation/Animation'
 
 describe('Animation', () => {
-  isConformant(Animation)
+  isConformant(Animation, 'Animation')
 })

test/specs/components/Attachment/Attachment-test.tsx

@@ -8,7 +8,7 @@ import Button from 'src/components/Button/Button'
 const attachmentImplementsShorthandProp = implementsShorthandProp(Attachment)
 
 describe('Attachment', () => {
-  isConformant(Attachment)
+  isConformant(Attachment, 'Attachment')
   attachmentImplementsShorthandProp('header', Text)
   attachmentImplementsShorthandProp('description', Text)
   attachmentImplementsShorthandProp('icon', Icon, { mapsValueToProp: 'name' })

test/specs/components/Avatar/Avatar-test.tsx

@@ -7,7 +7,7 @@ import Image from 'src/components/Image/Image'
 const avatarImplementsShorthandProp = implementsShorthandProp(Avatar)
 
 describe('Avatar', () => {
-  isConformant(Avatar)
+  isConformant(Avatar, 'Avatar')
   avatarImplementsShorthandProp('label', Label)
   avatarImplementsShorthandProp('image', Image, { mapsValueToProp: 'src' })
 

test/specs/components/Button/Button-test.tsx

@@ -16,7 +16,7 @@ import Icon from 'src/components/Icon/Icon'
 const buttonImplementsShorthandProp = implementsShorthandProp(Button)
 
 describe('Button', () => {
-  isConformant(Button)
+  isConformant(Button, 'Button')
   buttonImplementsShorthandProp('icon', Icon, { mapsValueToProp: 'name' })
 
   describe('accessibility', () => {

test/specs/components/Button/ButtonGroup-test.tsx

@@ -8,7 +8,7 @@ import { AccessibilityDefinition } from 'src/lib/accessibility/types'
 const buttonGroupImplementsCollectionShorthandProp = implementsCollectionShorthandProp(ButtonGroup)
 
 describe('ButtonGroup', () => {
-  isConformant(ButtonGroup)
+  isConformant(ButtonGroup, 'ButtonGroup')
   buttonGroupImplementsCollectionShorthandProp('buttons', Button)
 
   describe('accessibility', () => {

test/specs/components/Chat/Chat-test.ts

@@ -9,7 +9,7 @@ import { AccessibilityDefinition } from 'src/lib/accessibility/types'
 const chatImplementsCollectionShorthandProp = implementsCollectionShorthandProp(Chat)
 
 describe('Chat', () => {
-  isConformant(Chat)
+  isConformant(Chat, 'Chat')
   chatImplementsCollectionShorthandProp('items', ChatItem)
 
   describe('accessibility', () => {

test/specs/components/Chat/ChatItem-test.tsx

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import ChatItem from 'src/components/Chat/ChatItem'
 
 describe('ChatItem', () => {
-  isConformant(ChatItem)
+  isConformant(ChatItem, 'ChatItem')
 })

test/specs/components/Chat/ChatMessage-test.tsx

@@ -9,7 +9,7 @@ import { AccessibilityDefinition } from 'src/lib/accessibility/types'
 import Text from 'src/components/Text/Text'
 
 describe('ChatMessage', () => {
-  isConformant(ChatMessage)
+  isConformant(ChatMessage, 'ChatMessage')
   implementsShorthandProp(ChatMessage)('avatar', Avatar, { mapsValueToProp: 'name' })
   implementsShorthandProp(ChatMessage)('author', Text)
   implementsShorthandProp(ChatMessage)('timestamp', Text)

test/specs/components/Divider/Divider-test.tsx

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Divider from 'src/components/Divider/Divider'
 
 describe('Divider', () => {
-  isConformant(Divider)
+  isConformant(Divider, 'Divider')
 })

test/specs/components/Dropdown/Dropdown-test.tsx

@@ -2,5 +2,5 @@ import { isConformant } from 'test/specs/commonTests'
 
 import Dropdown from 'src/components/Dropdown/Dropdown'
 describe('Dropdown', () => {
-  isConformant(Dropdown)
+  isConformant(Dropdown, 'Dropdown')
 })

test/specs/components/Form/Form-test.tsx

@@ -7,6 +7,6 @@ import FormField from 'src/components/Form/FormField'
 const formImplementsCollectionShorthandProp = implementsCollectionShorthandProp(Form)
 
 describe('Form', () => {
-  isConformant(Form)
+  isConformant(Form, 'Form')
   formImplementsCollectionShorthandProp('fields', FormField, { mapsValueToProp: 'label' })
 })

test/specs/components/Form/FormField-test.tsx

@@ -12,7 +12,7 @@ const getFormField = (control: ComponentClass<any> | string) =>
   mountWithProvider(<FormField control={{ as: control }} name="firstName" />).find('FormField')
 
 describe('FormField', () => {
-  isConformant(FormField)
+  isConformant(FormField, 'FormField')
   formFieldImplementsShorthandProp('label', Text)
   formFieldImplementsShorthandProp('message', Text)
   formFieldImplementsShorthandProp('control', Slot, { mapsValueToProp: 'children' })

test/specs/components/Grid/Grid-test.tsx

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Grid from 'src/components/Grid/Grid'
 
 describe('Grid', () => {
-  isConformant(Grid)
+  isConformant(Grid, 'Grid')
 })

test/specs/components/Header/Header-test.ts

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Header from 'src/components/Header/Header'
 
 describe('Header', () => {
-  isConformant(Header)
+  isConformant(Header, 'Header')
 })

test/specs/components/Header/HeaderDescription-test.ts

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import HeaderDescription from 'src/components/Header/HeaderDescription'
 
 describe('HeaderDescription', () => {
-  isConformant(HeaderDescription)
+  isConformant(HeaderDescription, 'HeaderDescription')
 })

test/specs/components/Icon/Icon-test.tsx

@@ -6,7 +6,7 @@ import { mountWithProviderAndGetComponent } from 'test/utils'
 import { ThemeInput } from 'src/themes/types'
 
 describe('Icon', () => {
-  isConformant(Icon)
+  isConformant(Icon, 'Icon')
 
   describe('accessibility', () => {
     handlesAccessibility(Icon, {

test/specs/components/Image/Image-test.tsx

@@ -5,7 +5,7 @@ import Image from 'src/components/Image/Image'
 import { mountWithProviderAndGetComponent } from 'test/utils'
 
 describe('Image', () => {
-  isConformant(Image)
+  isConformant(Image, 'Image')
 
   describe('accessibility', () => {
     handlesAccessibility(Image, {

test/specs/components/Input/Input-test.tsx

@@ -30,7 +30,7 @@ const setUserInputValue = (inputComp: ReactWrapper, value: string) => {
 
 describe('Input', () => {
   describe('conformance', () => {
-    isConformant(Input, {
+    isConformant(Input, 'Input', {
       eventTargets: { onChange: 'input' },
     })
   })

test/specs/components/ItemLayout/ItemLayout-test.ts

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import ItemLayout from 'src/components/ItemLayout/ItemLayout'
 
 describe('ItemLayout', () => {
-  isConformant(ItemLayout)
+  isConformant(ItemLayout, 'ItemLayout')
 })

test/specs/components/Label/Label-test.tsx

@@ -10,7 +10,7 @@ import { implementsShorthandProp } from '../../commonTests'
 const labelImplementsShorthandProp = implementsShorthandProp(Label)
 
 describe('Label', () => {
-  isConformant(Label)
+  isConformant(Label, 'Label')
   labelImplementsShorthandProp('icon', Icon, { mapsValueToProp: 'name' })
   labelImplementsShorthandProp('image', Image, { mapsValueToProp: 'src' })
 

test/specs/components/Layout/Layout-test.ts

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Layout from 'src/components/Layout/Layout'
 
 describe('Layout', () => {
-  isConformant(Layout)
+  isConformant(Layout, 'Layout')
 })

test/specs/components/List/List-test.ts

@@ -7,7 +7,7 @@ import ListItem from 'src/components/List/ListItem'
 const listImplementsCollectionShorthandProp = implementsCollectionShorthandProp(List)
 
 describe('List', () => {
-  isConformant(List)
+  isConformant(List, 'List')
   handlesAccessibility(List, { defaultRootRole: 'list' })
   listImplementsCollectionShorthandProp('items', ListItem, { mapsValueToProp: 'main' })
 })

test/specs/components/List/ListItem-test.ts

@@ -3,6 +3,6 @@ import { isConformant, handlesAccessibility } from 'test/specs/commonTests'
 import ListItem from 'src/components/List/ListItem'
 
 describe('ListItem', () => {
-  isConformant(ListItem)
+  isConformant(ListItem, 'ListItem')
   handlesAccessibility(ListItem, { defaultRootRole: 'listitem' })
 })

test/specs/components/Menu/Menu-test.tsx

@@ -12,7 +12,7 @@ import { AccessibilityDefinition } from 'src/lib/accessibility/types'
 const menuImplementsCollectionShorthandProp = implementsCollectionShorthandProp(Menu)
 
 describe('Menu', () => {
-  isConformant(Menu)
+  isConformant(Menu, 'Menu')
   menuImplementsCollectionShorthandProp('items', MenuItem)
 
   const getItems = () => [

test/specs/components/Menu/MenuItem-test.tsx

@@ -6,7 +6,7 @@ import MenuItem from 'src/components/Menu/MenuItem'
 import { toolbarButtonBehavior, tabBehavior } from '../../../../src/lib/accessibility'
 
 describe('MenuItem', () => {
-  isConformant(MenuItem, {
+  isConformant(MenuItem, 'MenuItem', {
     eventTargets: {
       onClick: 'a',
     },

test/specs/components/Popup/PopupContent-test.ts

@@ -2,5 +2,5 @@ import { isConformant } from 'test/specs/commonTests'
 import PopupContent from 'src/components/Popup/PopupContent'
 
 describe('PopupContent', () => {
-  isConformant(PopupContent, { rendersPortal: true })
+  isConformant(PopupContent, 'PopupContent', { rendersPortal: true })
 })

test/specs/components/RadioGroup/RadioGroup-test.tsx

@@ -42,7 +42,7 @@ const getShorthandItems = (props?: { disabledItem?: number }) => [
 ]
 
 describe('RadioGroup', () => {
-  isConformant(RadioGroup)
+  isConformant(RadioGroup, 'RadioGroup')
 
   describe('accessibility', () => {
     handlesAccessibility(RadioGroup, {

test/specs/components/RadioGroup/RadioGroupItem-test.ts

@@ -3,7 +3,7 @@ import { isConformant, handlesAccessibility } from 'test/specs/commonTests'
 import RadioGroupItem from 'src/components/RadioGroup/RadioGroupItem'
 
 describe('RadioGroupItem', () => {
-  isConformant(RadioGroupItem)
+  isConformant(RadioGroupItem, 'RadioGroupItem')
 
   describe('accessibility', () => {
     handlesAccessibility(RadioGroupItem, {

test/specs/components/Segment/Segment-test.ts

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Segment from 'src/components/Segment/Segment'
 
 describe('Segment', () => {
-  isConformant(Segment)
+  isConformant(Segment, 'Segment')
 })

test/specs/components/Slot/Slot-test.ts

@@ -8,7 +8,7 @@ describe('Slot', () => {
     mount(factoryFn(val, options)).find(Slot)
 
   xdescribe('is conformant', () => {
-    isConformant(Slot, { exportedAtTopLevel: false })
+    isConformant(Slot, 'Slot', { exportedAtTopLevel: false })
   })
 
   describe(`create`, () => {

test/specs/components/Status/Status-test.tsx

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Status from 'src/components/Status/Status'
 
 describe('Status', () => {
-  isConformant(Status)
+  isConformant(Status, 'Status')
 })

test/specs/components/Text/Text-test.tsx

@@ -6,7 +6,7 @@ import { mountWithProvider } from 'test/utils'
 import Text from 'src/components/Text/Text'
 
 describe('Text', () => {
-  isConformant(Text)
+  isConformant(Text, 'Text')
 
   test('renders children', () => {
     expect(mountWithProvider(<Text>children</Text>).text()).toEqual('children')

test/specs/components/Tree/Tree-test.tsx

@@ -3,5 +3,5 @@ import { isConformant } from 'test/specs/commonTests'
 import Tree from 'src/components/Tree/Tree'
 
 describe('Tree', () => {
-  isConformant(Tree)
+  isConformant(Tree, 'Tree')
 })