[improvement] ContactCard: functional (#2696)

packages/contact-card/index.js

@@ -4,6 +4,8 @@ import Cell from '../cell';
 const [sfc, bem, t] = use('contact-card');
 
 export default sfc({
+  functional: true,
+
   props: {
     tel: String,
     name: String,
@@ -18,28 +20,27 @@ export default sfc({
     }
   },
 
-  methods: {
+  render(h, context, inherit) {
-    onClick(event) {
+    const { props, listeners } = context;
-      if (this.editable) {
+    const { type, editable } = props;
-        this.$emit('click', event);
-      }
-    }
-  },
 
-  render(h) {
-    const { type } = this;
     return (
       <Cell
         center
         border={false}
         class={bem([type])}
-        isLink={this.editable}
+        isLink={editable}
         icon={type === 'edit' ? 'contact' : 'add-square'}
-        onClick={this.onClick}
+        onClick={event => {
+          if (editable && listeners.click) {
+            listeners.click(event);
+          }
+        }}
+        {...inherit}
       >
         {type === 'add'
-          ? this.addText || t('addText')
+          ? props.addText || t('addText')
-          : [<div>{`${t('name')}：${this.name}`}</div>, <div>{`${t('tel')}：${this.tel}`}</div>]}
+          : [<div>{`${t('name')}：${props.name}`}</div>, <div>{`${t('tel')}：${props.tel}`}</div>]}
       </Cell>
     );
   }

packages/contact-card/test/index.spec.js

@@ -10,18 +10,34 @@ const contactInfo = {
 
 describe('ContactCard', () => {
   test('click event', () => {
+    const click = jest.fn();
     const wrapper = mount(ContactCard, {
-      propsData: {
+      context: {
-        editable: false
+        on: {
+          click
+        }
       }
     });
 
     wrapper.trigger('click');
-    expect(wrapper.emitted('click')).toBeFalsy();
+    expect(click.mock.calls.length).toEqual(1);
+  });
+
+  test('not editable', () => {
+    const click = jest.fn();
+    const wrapper = mount(ContactCard, {
+      propsData: {
+        editable: false
+      },
+      context: {
+        on: {
+          click
+        }
+      }
+    });
 
-    wrapper.setProps({ editable: true });
     wrapper.trigger('click');
-    expect(wrapper.emitted('click')).toBeTruthy();
+    expect(click.mock.calls.length).toEqual(0);
   });
 });