chore: fixed tests

Author: icaldana

examples/bricks/CardPayment/3-CustomContainerId.tsx renamed to examples/bricks/CardPayment/3-Custom-ContainerId.tsx

@@ -0,0 +1,15 @@
+import React from 'react';
+import Wallet from '../../../src/bricks/wallet';
+import { PUBLIC_KEY } from '../../constants';
+
+import initMercadoPago from '../../../src/mercadoPago/initMercadoPago';
+
+initMercadoPago(PUBLIC_KEY);
+
+const ExampleSimpleWalletBrick = () => {
+  return (
+    <Wallet initialization={{ preferenceId: 'YOUR_PREFERENCE_ID' }} id="custom-container-id" />
+  );
+};
+
+export default ExampleSimpleWalletBrick;

examples/bricks/WalletBrick/6-Custom-ContainerId.tsx

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-empty-function */
 import { IBrickError } from '../types/common';
 
 const onSubmitDefault = async () => {};

src/bricks/util/initial/index.ts

@@ -3,7 +3,7 @@ import { onErrorDefault, onSubmitDefault, onReadyDefault } from './index';
 describe('Test default functions', () => {
   test('Console error should have been called when onErrorDefault is call', () => {
     // Mock the console.error to not generate a console.error when we run test.
-    const logSpy = jest.spyOn(global.console, 'error').mockImplementation(() => {});
+    const logSpy = jest.spyOn(global.console, 'error').mockImplementation(jest.fn());
     onErrorDefault({
       type: 'critical',
       cause: 'settings_empty',
@@ -15,10 +15,13 @@ describe('Test default functions', () => {
 
     logSpy.mockRestore();
   });
-  test('Should onSubmitDefault be called', () => {
-    expect(onSubmitDefault()).resolves.toBe({});
+
+  test('Should onSubmitDefault be called', async () => {
+    const response = await onSubmitDefault();
+    expect(response).toEqual(undefined);
   });
+
   test('Should onReadyDefault be called', () => {
-    expect(onReadyDefault()).toBe(undefined);
+    expect(onReadyDefault()).toEqual(undefined);
   });
 });

src/bricks/util/initial/initial.test.ts

@@ -9,12 +9,12 @@ declare global {
 
 describe('Test renderBrick', () => {
   test('should return a render Brick', async () => {
-    const mock = jest.fn();
+    const createMock = jest.fn();
     MercadoPagoInstance.publicKey = 'PUBLIC_KEY';
-    MercadoPagoInstance.instanceMercadoPago = {
+    MercadoPagoInstance.getInstance = jest.fn().mockResolvedValue({
       bricks: function () {
         return {
-          create: mock,
+          create: createMock,
         };
       },
       getIdentificationTypes: jest.fn(),
@@ -28,16 +28,17 @@ describe('Test renderBrick', () => {
         updateCardToken: jest.fn(),
         create: jest.fn(),
       },
-    };
+    });
 
     const WalletBrickConfig = {
       settings: {},
       name: 'brickTest',
-      containerIdcontainerId: 'brickTest_container',
       controller: 'brickTestController',
+      containerId: 'brickTest_container',
     };
 
     await initBrick(WalletBrickConfig);
-    expect(mock).toBeCalledTimes(1);
+    expect(createMock).toHaveBeenCalledTimes(1);
+    expect(createMock).toHaveBeenCalledWith('brickTest', 'brickTest_container', {});
   });
 });

src/bricks/util/renderBrick/renderBrick.test.ts

@@ -12,6 +12,7 @@ describe('Test Wallet Brick Component', () => {
 
     expect(element.container.querySelector('#walletBrick_container')).toBeTruthy();
   });
+
   test('should found the id of Wallet Brick div if specified', async () => {
     MercadoPagoInstance.publicKey = 'PUBLIC_KEY';
     const element = await render(

src/bricks/wallet/walletBrick.test.tsx

@@ -5,7 +5,7 @@ export class MercadoPagoInstance {
   static publicKey: string | null = null;
   static options: TOptions = {};
   static instanceMercadoPago?: TInstanceMercadoPago = undefined;
-  static loadedInstanceMercadoPago: boolean = false;
+  static loadedInstanceMercadoPago = false;
 
   static async getInstance() {
     if (this.publicKey) {