resolve eslint error

Author: dev-javascript

example/basic/App.js

@@ -1,3 +1,4 @@
+import React from 'react';
 import './App.css';
 import Basic from './com/basic/index.js';
 import 'react-dyn-tabs/style/react-dyn-tabs.css';

example/basic/com/basic/com/eager/eagerPanel1.js

@@ -1,4 +1,4 @@
 import React from 'react';
-export default function (props) {
+export default function EagerPanel1() {
   return <p>{'eager tab content 1'}</p>;
 }

example/basic/com/basic/com/eager/eagerPanel2.js

@@ -1,4 +1,4 @@
 import React from 'react';
-export default function (props) {
+export default function EagerPanel2() {
   return <p>{'eager tab content 2'}</p>;
 }

example/basic/com/basic/index.js

@@ -3,9 +3,9 @@ import React from 'react';
 import useDynTabs from 'react-dyn-tabs';
 import service from './service/index.js';
 import './index.css';
-export default function (props) {
+export default function Basic() {
   const options = service.getLocalData();
-  const [Tablist, Panellist, api] = useDynTabs(options);
+  const [Tablist, Panellist, ready] = useDynTabs(options);
   return (
     <>
       <Tablist></Tablist>

src/index.test.js

@@ -1,4 +1,4 @@
-import React, {useState} from 'react';
+import React from 'react';
 import {render, unmountComponentAtNode} from 'react-dom';
 import useDynTabs from './index.js';
 import {act} from 'react-dom/test-utils';
@@ -35,7 +35,7 @@ beforeEach(() => {
     beforeSelect: jest.fn(function () {}),
     onDestroy: jest.fn(function () {}),
   };
-  App = function (props) {
+  App = function App() {
     const [Tablist, Panellist, readyFunction] = useDynTabs(op);
     ready = readyFunction;
     readyFunction((instanceParam) => {
@@ -72,8 +72,8 @@ describe('apply multiple actions : ', () => {
     expect.assertions(1);
     renderApp();
     return act(() => {
-      return instance.select('2').then((result) => {
-        return instance.close('2').then((result) => {
+      return instance.select('2').then(() => {
+        return instance.close('2').then(() => {
           expect(document.querySelector('[tab-id="1"]').className.includes('rc-dyn-tabs-selected')).toBe(true);
         });
       });
@@ -82,7 +82,7 @@ describe('apply multiple actions : ', () => {
   test('opening new tab and closing selected tab', () => {
     let _api;
     act(() => {
-      const App = function (props) {
+      const App = function () {
         const [Tablist, Panellist, api] = useDynTabs(op);
         _api = api;
         return (
@@ -122,13 +122,13 @@ describe('apply multiple actions : ', () => {
           return (
             <a {...props.tabProps}>
               {props.children}
-              {props.hasOwnProperty('iconProps') && <span {...props.iconProps}></span>}
+              {Object.prototype.hasOwnProperty.call(props, 'iconProps') && <span {...props.iconProps}></span>}
             </a>
           );
         });
         instance.one('onInit', function () {
           this.setTab('1', {
-            panelComponent: function (props) {
+            panelComponent: function panelComponent() {
               return <div id="updatedPanel1"></div>;
             },
           }).refresh();
@@ -215,7 +215,7 @@ describe('select method : ', () => {
     expect.assertions(2);
     renderApp();
     return act(() => {
-      return instance.select('1').then((result) => {
+      return instance.select('1').then(() => {
         expect(op.onChange.mock.calls.length === 0).toBe(true);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
       });
@@ -225,7 +225,7 @@ describe('select method : ', () => {
     expect.assertions(4);
     renderApp();
     return act(() => {
-      return instance.select('3').then((result) => {
+      return instance.select('3').then(() => {
         expect(document.querySelector('.rc-dyn-tabs-selected') == null).toBe(true);
         expect(instance.getCopyData().selectedTabID === '3').toBe(true);
         expect(op.onChange.mock.calls.length === 1).toBe(true);
@@ -268,7 +268,7 @@ describe('close method : ', () => {
     expect.assertions(2);
     renderApp();
     return act(() => {
-      return instance.close('4').then((result) => {
+      return instance.close('4').then(() => {
         expect(op.onChange.mock.calls.length === 0).toBe(true);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
       });
@@ -320,7 +320,7 @@ describe('open method : ', () => {
     expect.assertions(3);
     renderApp();
     return act(() => {
-      return instance.open(op.tabs[0]).then((result) => {
+      return instance.open(op.tabs[0]).then(() => {
         expect(op.onChange.mock.calls.length === 0).toBe(true);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
         expect(instance.getCopyData().openTabIDs.length === 2).toBe(true);
@@ -362,7 +362,7 @@ describe('ready function : ', () => {
       ready((instance) => {
         instance.one('onLoad', onLoad).one('onInit', onInit);
       });
-      return instance.refresh().then((result) => {
+      return instance.refresh().then(() => {
         expect(op.onLoad.mock.calls.length === 1).toBe(true);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
         expect(onInit.mock.calls.length === 1).toBe(true);

src/panel/panel.test.js

@@ -7,22 +7,30 @@ let container = document.createElement('div'),
   realUseContext;
 const getDefaultApi = () => ({
   tabs: [
-    {id: '1', title: 'tab1', panelComponent: (props) => <p>{props.api.getTab(props.id).title + ' panel'}</p>},
+    {
+      id: '1',
+      title: 'tab1',
+      panelComponent: function panelComponent1(props) {
+        return <p>{props.api.getTab(props.id).title + ' panel'}</p>;
+      },
+    },
     {
       id: '2',
       title: 'tab2',
       disable: true,
-      panelComponent: (props) => <p>{props.api.getTab(props.id).title + ' panel'}</p>,
+      panelComponent: function panelComponent3(props) {
+        return <p>{props.api.getTab(props.id).title + ' panel'}</p>;
+      },
     },
     {id: '3', title: 'tab3'},
     {
       id: '4',
       title: 'tab4',
-      panelComponent: (props) => {
+      panelComponent: function panelComponent4(props) {
         let panelContent = 'panel 4 is ';
         props.isSelected || (panelContent = panelContent + 'not ');
         panelContent = panelContent + 'selected';
-        return <div class="custom-panel">{panelContent}</div>;
+        return <div className="custom-panel">{panelContent}</div>;
       },
     },
     {id: '5', title: 'tab5', panelComponent: <span>panel 6</span>},
@@ -84,7 +92,7 @@ describe('panel structure : ', () => {
         {id: '1', title: 'tab1'},
         {id: '2', title: 'tab2'},
       ],
-      defaultPanelComponent: (props) => {
+      defaultPanelComponent: function defaultPanelComponent(props) {
         const panelContent = props.api.getTab(props.id).title + ' panel';
         const className = props.isSelected ? 'custom-panel-selected' : 'custom-panel-none-selected';
         return <span className={className}>{panelContent}</span>;
@@ -108,7 +116,7 @@ describe('panel structure : ', () => {
       ],
       direction: 'rtl',
       accessibility: false,
-      defaultPanelComponent: (props) => {
+      defaultPanelComponent: function defaultPanelComponent(props) {
         const panelContent = props.api.getTab(props.id).title + ' panel';
         const className = props.isSelected ? 'custom-panel-selected' : 'custom-panel-none-selected';
         return <span className={className}>{panelContent}</span>;

src/panelList/panelList.js

@@ -2,7 +2,7 @@ import React, {memo} from 'react';
 import Panel from '../panel/panel.js';
 import {ApiContext, StateContext} from '../utils/context.js';
 const PanelList = memo(
-  function PanelList(props) {
+  function PanelList() {
     const {openTabIDs, selectedTabID} = React.useContext(StateContext),
       api = React.useContext(ApiContext),
       setting = api.optionsManager.setting,

src/tab/defaulTabInner.js

@@ -3,7 +3,7 @@ const DefaulTabInner = function (props) {
   return (
     <button {...props.tabProps}>
       {props.children}
-      {props.hasOwnProperty('iconProps') && <span {...props.iconProps}></span>}
+      {Object.prototype.hasOwnProperty.call(props, 'iconProps') && <span {...props.iconProps}></span>}
     </button>
   );
 };

src/tab/tab.test.js

@@ -1,7 +1,6 @@
-import React, {useState} from 'react';
-import {render, unmountComponentAtNode} from 'react-dom';
+import React from 'react';
+import {unmountComponentAtNode} from 'react-dom';
 import Tab from './tab';
-import {act} from 'react-dom/test-utils';
 import renderer from 'react-test-renderer';
 import DefaultTabInner from './defaulTabInner.js';
 import Api from '../utils/api/api.js';
@@ -12,7 +11,13 @@ const getDefaultApi = () => ({
     {id: '1', title: 'tab1', tooltip: 'tab1 tootltip', closable: false},
     {id: '2', title: 'tab2', iconClass: 'ui-icon ui-icon-disk'},
     {id: '3', title: 'tab3', disable: true},
-    {id: '4', title: 'tab4', panelComponent: (props) => <div>tab4</div>},
+    {
+      id: '4',
+      title: 'tab4',
+      panelComponent: function panelComponent4() {
+        return <div>tab4</div>;
+      },
+    },
   ],
 });
 const setMockUseContext = (op = {}) => {

src/tabList/tabList.js

@@ -3,7 +3,7 @@ import {ApiContext, StateContext} from '../utils/context.js';
 import Tab from '../tab/tab.js';
 import tablistPropsManager from './tablistPropsManager.js';
 const TabList = memo(
-  function TabList(props) {
+  function TabList() {
     const {openTabIDs, selectedTabID} = React.useContext(StateContext),
       api = React.useContext(ApiContext),
       tablistProps = tablistPropsManager({api});