test: should not emit deprecation warning in test (#7356)

src/collapse-item/index.js

@@ -46,7 +46,7 @@ export default createComponent({
       const { value, accordion } = this.parent;
 
       if (
-        process.env.NODE_ENV !== 'production' &&
+        process.env.NODE_ENV === 'development' &&
         !accordion &&
         !Array.isArray(value)
       ) {

src/collapse/test/index.spec.js

@@ -123,28 +123,6 @@ test('lazy render collapse content', async () => {
   expect(wrapper).toMatchSnapshot();
 });
 
-test('warn when value type is incorrect', () => {
-  const originConsoleError = console.error;
-  const error = jest.fn();
-  console.error = error;
-
-  mount({
-    template: `
-    <van-collapse v-model="active">
-      <van-collapse-item title="a" name="first"></van-collapse-item>
-    </van-collapse>
-    `,
-    data() {
-      return {
-        active: 0,
-      };
-    },
-  });
-
-  expect(error).toHaveBeenCalledTimes(1);
-  console.error = originConsoleError;
-});
-
 test('toggle method', (done) => {
   mount({
     template: `

src/goods-action-icon/index.js

@@ -31,7 +31,7 @@ export default createComponent({
       const slot = this.slots('icon');
       const info = this.badge ?? this.info;
 
-      if (process.env.NODE_ENV !== 'production' && this.info) {
+      if (process.env.NODE_ENV === 'development' && this.info) {
         console.warn(
           '[Vant] GoodsActionIcon: "info" prop is deprecated, use "badge" prop instead.'
         );

src/goods-action/test/__snapshots__/index.spec.js.snap

@@ -22,6 +22,13 @@ exports[`Icon render icon slot 1`] = `
 </div>
 `;
 
+exports[`Icon render icon slot with badge 1`] = `
+<div role="button" tabindex="0" class="van-goods-action-icon">
+  <div class="van-goods-action-icon__icon">Custom Icon<div class="van-info">1</div>
+  </div>Text
+</div>
+`;
+
 exports[`Icon render icon slot with dot 1`] = `
 <div role="button" tabindex="0" class="van-goods-action-icon">
   <div class="van-goods-action-icon__icon">Custom Icon<div class="van-info van-info--dot"></div>
@@ -29,11 +36,4 @@ exports[`Icon render icon slot with dot 1`] = `
 </div>
 `;
 
-exports[`Icon render icon slot with info 1`] = `
-<div role="button" tabindex="0" class="van-goods-action-icon">
-  <div class="van-goods-action-icon__icon">Custom Icon<div class="van-info">1</div>
-  </div>Text
-</div>
-`;
-
 exports[`disable safe-area-inset-bottom prop 1`] = `<div class="van-goods-action van-goods-action--unfit"></div>`;

src/goods-action/test/index.spec.js

@@ -50,12 +50,12 @@ test('Icon render icon slot', () => {
   expect(wrapper).toMatchSnapshot();
 });
 
-test('Icon render icon slot with info', () => {
+test('Icon render icon slot with badge', () => {
   const wrapper = mount({
     render(h) {
       return h(Icon, {
         props: {
-          info: '1',
+          badge: '1',
         },
         scopedSlots: {
           default: () => 'Text',

src/grid-item/index.js

@@ -74,7 +74,7 @@ export default createComponent({
       const iconSlot = this.slots('icon');
       const info = this.badge ?? this.info;
 
-      if (process.env.NODE_ENV !== 'production' && this.info) {
+      if (process.env.NODE_ENV === 'development' && this.info) {
         console.warn(
           '[Vant] GridItem: "info" prop is deprecated, use "badge" prop instead.'
         );

src/grid/test/index.spec.js

@@ -49,7 +49,7 @@ test('render icon-slot', () => {
   const wrapper = mount({
     template: `
       <van-grid icon-size="10">
-        <van-grid-item info="1">
+        <van-grid-item badge="1">
           <template #icon>
             <div>Custom Icon</div>
           </template>

src/icon/index.tsx

@@ -50,7 +50,7 @@ function Icon(
   const name = correctName(props.name);
   const imageIcon = isImage(name);
 
-  if (process.env.NODE_ENV !== 'production' && props.info) {
+  if (process.env.NODE_ENV === 'development' && props.info) {
     console.warn(
       '[Vant] Icon: "info" prop is deprecated, use "badge" prop instead.'
     );

src/info/test/__snapshots__/index.spec.js.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`should not render when info is empty string 1`] = ``;
+exports[`should not render when badge is empty string 1`] = ``;
 
-exports[`should not render when info is empty undefined 1`] = ``;
+exports[`should not render when badge is empty undefined 1`] = ``;
 
-exports[`should render when info is zero 1`] = `<div class="van-info">0</div>`;
+exports[`should render when badge is zero 1`] = ``;

src/info/test/index.spec.js

@@ -1,30 +1,30 @@
 import Info from '..';
 import { mount } from '../../../test';
 
-test('should not render when info is empty string', () => {
+test('should not render when badge is empty string', () => {
   const wrapper = mount(Info, {
     propsData: {
-      info: '',
+      badge: '',
     },
   });
 
   expect(wrapper).toMatchSnapshot();
 });
 
-test('should not render when info is empty undefined', () => {
+test('should not render when badge is empty undefined', () => {
   const wrapper = mount(Info, {
     propsData: {
-      info: undefined,
+      badge: undefined,
     },
   });
 
   expect(wrapper).toMatchSnapshot();
 });
 
-test('should render when info is zero', () => {
+test('should render when badge is zero', () => {
   const wrapper = mount(Info, {
     propsData: {
-      info: 0,
+      badge: 0,
     },
   });
 

src/sidebar-item/index.js

@@ -38,7 +38,7 @@ export default createComponent({
   },
 
   render() {
-    if (process.env.NODE_ENV !== 'production' && this.info) {
+    if (process.env.NODE_ENV === 'development' && this.info) {
       console.warn(
         '[Vant] SidebarItem: "info" prop is deprecated, use "badge" prop instead.'
       );

src/switch-cell/index.tsx

@@ -30,7 +30,7 @@ function SwitchCell(
   slots: DefaultSlots,
   ctx: RenderContext<SwitchCellProps>
 ) {
-  if (process.env.NODE_ENV !== 'production') {
+  if (process.env.NODE_ENV === 'development') {
     console.warn(
       '[Vant] "SwitchCell" component is deprecated, see: https://youzan.github.io/vant/#/zh-CN/switch-cell.'
     );

src/tab/index.js

@@ -58,7 +58,7 @@ export default createComponent({
     const { slots, parent, isActive } = this;
     const slotContent = slots();
 
-    if (process.env.NODE_ENV !== 'production' && this.info) {
+    if (process.env.NODE_ENV === 'development' && this.info) {
       console.warn(
         '[Vant] Tab: "info" prop is deprecated, use "badge" prop instead.'
       );

src/tab/test/__snapshots__/index.spec.js.snap

@@ -1,5 +1,21 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`badge prop 1`] = `
+<div class="van-tabs van-tabs--line">
+  <div class="van-tabs__wrap">
+    <div role="tablist" class="van-tabs__nav van-tabs__nav--line">
+      <div role="tab" aria-selected="true" class="van-tab van-tab--active"><span class="van-tab__text-wrapper"><span class="van-tab__text van-tab__text--ellipsis"></span>
+        <div class="van-info">10</div>
+        </span></div>
+      <div class="van-tabs__line"></div>
+    </div>
+  </div>
+  <div class="van-tabs__content">
+    <div role="tabpanel" class="van-tab__pane">Text</div>
+  </div>
+</div>
+`;
+
 exports[`border props 1`] = `
 <div class="van-tabs van-tabs--line">
   <div class="van-tabs__wrap">
@@ -116,22 +132,6 @@ exports[`dot prop 1`] = `
 </div>
 `;
 
-exports[`info prop 1`] = `
-<div class="van-tabs van-tabs--line">
-  <div class="van-tabs__wrap">
-    <div role="tablist" class="van-tabs__nav van-tabs__nav--line">
-      <div role="tab" aria-selected="true" class="van-tab van-tab--active"><span class="van-tab__text-wrapper"><span class="van-tab__text van-tab__text--ellipsis"></span>
-        <div class="van-info">10</div>
-        </span></div>
-      <div class="van-tabs__line"></div>
-    </div>
-  </div>
-  <div class="van-tabs__content">
-    <div role="tabpanel" class="van-tab__pane">Text</div>
-  </div>
-</div>
-`;
-
 exports[`lazy render 1`] = `
 <div class="van-tabs van-tabs--line">
   <div>

src/tab/test/index.spec.js

@@ -246,11 +246,11 @@ test('dot prop', () => {
   expect(wrapper).toMatchSnapshot();
 });
 
-test('info prop', () => {
+test('badge prop', () => {
   const wrapper = mount({
     template: `
       <van-tabs>
-        <van-tab info="10">Text</van-tab>
+        <van-tab badge="10">Text</van-tab>
       </van-tabs>
     `,
   });

src/tabbar-item/index.js

@@ -68,7 +68,7 @@ export default createComponent({
     const active = this.parent.route ? this.routeActive : this.active;
     const color = this.parent[active ? 'activeColor' : 'inactiveColor'];
 
-    if (process.env.NODE_ENV !== 'production' && this.info) {
+    if (process.env.NODE_ENV === 'development' && this.info) {
       console.warn(
         '[Vant] TabbarItem: "info" prop is deprecated, use "badge" prop instead.'
       );

src/toast/index.js

@@ -97,7 +97,7 @@ function Toast(options = {}) {
     ...options,
   };
 
-  if (process.env.NODE_ENV !== 'production' && options.mask) {
+  if (process.env.NODE_ENV === 'development' && options.mask) {
     console.warn(
       '[Vant] Toast: "mask" option is deprecated, use "overlay" option instead.'
     );

src/tree-select/index.tsx

@@ -52,7 +52,7 @@ function TreeSelect(
 ) {
   const { items, height, activeId, selectedIcon, mainActiveIndex } = props;
 
-  if (process.env.NODE_ENV !== 'production') {
+  if (process.env.NODE_ENV === 'development') {
     if (ctx.listeners.navclick) {
       console.warn(
         '[Vant] TreeSelect: "navclick" event is deprecated, use "click-nav" instead.'

src/tree-select/test/__snapshots__/index.spec.js.snap

@@ -25,7 +25,7 @@ exports[`height prop 1`] = `
 </div>
 `;
 
-exports[`nav info 1`] = `
+exports[`nav render badge 1`] = `
 <div class="van-tree-select" style="height: 300px;">
   <div class="van-sidebar van-tree-select__nav"><a class="van-sidebar-item van-sidebar-item--select van-tree-select__nav-item">
       <div class="van-sidebar-item__text">group1<div class="van-info van-sidebar-item__info">3</div>

src/tree-select/test/index.spec.js

@@ -27,7 +27,6 @@ const mockItems = [
 ];
 
 test('click-nav event', () => {
-  const onNavClick = jest.fn();
   const onClickNav = jest.fn();
 
   const wrapper = mount(TreeSelect, {
@@ -36,7 +35,6 @@ test('click-nav event', () => {
     },
     context: {
       on: {
-        navclick: onNavClick,
         'click-nav': onClickNav,
       },
     },
@@ -45,12 +43,10 @@ test('click-nav event', () => {
   const navItems = wrapper.findAll('.van-tree-select__nav-item');
   navItems.at(1).trigger('click');
 
-  expect(onNavClick).toHaveBeenCalledWith(1);
   expect(onClickNav).toHaveBeenCalledWith(1);
 });
 
 test('click-item event', () => {
-  const onItemClick = jest.fn();
   const onClickItem = jest.fn();
 
   const wrapper = mount(TreeSelect, {
@@ -59,7 +55,6 @@ test('click-item event', () => {
     },
     context: {
       on: {
-        itemclick: onItemClick,
         'click-item': onClickItem,
       },
     },
@@ -67,7 +62,6 @@ test('click-item event', () => {
 
   const items = wrapper.findAll('.van-tree-select__item');
   items.at(0).trigger('click');
-  expect(onItemClick).toHaveBeenCalledWith(mockItem);
   expect(onClickItem).toHaveBeenCalledWith(mockItem);
 });
 
@@ -151,13 +145,13 @@ test('height prop', () => {
   expect(wrapper).toMatchSnapshot();
 });
 
-test('nav info', () => {
+test('nav render badge', () => {
   const wrapper = mount(TreeSelect, {
     propsData: {
       items: [
         {
           text: 'group1',
-          info: 3,
+          badge: 3,
         },
       ],
     },