Tests: replace toHaveBeenCalledOnce and toBeCalledWith by toHaveBeenC…

…alledExactlyOnceWith
jhipster · Jan 26, 2025 · 10fd1ee · 10fd1ee
1 parent a33f47c
commit 10fd1ee
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 24 deletions.
diff --git a/src/test/webapp/unit/module/primary/landscape/LandscapeComponent.spec.ts b/src/test/webapp/unit/module/primary/landscape/LandscapeComponent.spec.ts
@@ -845,8 +845,7 @@ describe('Landscape', () => {
       const landscape = wrapper.find(wrappedElement('landscape-container'));
       await landscape.trigger('mousedown', mouseEvent);
 
-      expect(cursorUpdater.set).toHaveBeenCalledOnce();
-      expect(cursorUpdater.set).toBeCalledWith('grabbing');
+      expect(cursorUpdater.set).toHaveBeenCalledExactlyOnceWith('grabbing');
     });
 
     it('should prevent default clicking event when defined', async () => {
@@ -909,8 +908,7 @@ describe('Landscape', () => {
       await landscape.trigger('mousedown', mouseEventStart);
       await landscape.trigger('mousemove', mouseEventGrabbed);
 
-      expect(landscapeScroller.scroll).toHaveBeenCalledOnce();
-      expect(landscapeScroller.scroll).toBeCalledWith(expect.anything(), 20, 20);
+      expect(landscapeScroller.scroll).toHaveBeenCalledExactlyOnceWith(expect.anything(), 20, 20);
     });
 
     it('should not scroll without starting grabbing', async () => {
@@ -1277,8 +1275,7 @@ describe('Landscape', () => {
 
       expect(mockModuleRect).toHaveBeenCalled();
       expect(mockContainerRect).toHaveBeenCalled();
-      expect(landscapeScroller.scrollIntoView).toHaveBeenCalledOnce();
-      expect(landscapeScroller.scrollIntoView).toHaveBeenCalledWith(prettierModule);
+      expect(landscapeScroller.scrollIntoView).toHaveBeenCalledExactlyOnceWith(prettierModule);
     });
 
     it('should not scroll if the highlighted module is already visible', async () => {
@@ -1321,8 +1318,7 @@ describe('Landscape', () => {
 
       await performSearch(searchInput, '');
 
-      expect(landscapeScroller.scrollSmooth).toHaveBeenCalledOnce();
-      expect(landscapeScroller.scrollSmooth).toHaveBeenCalledWith(landscapeContainer, 0, 0);
+      expect(landscapeScroller.scrollSmooth).toHaveBeenCalledExactlyOnceWith(landscapeContainer, 0, 0);
     });
 
     const setupSearchTest = async () => {

diff --git a/src/test/webapp/unit/module/primary/landscape/LandscapeScroller.spec.ts b/src/test/webapp/unit/module/primary/landscape/LandscapeScroller.spec.ts
@@ -18,8 +18,7 @@ describe('LandscapeScroller', () => {
 
     landscapeScroller.scroll(element, 10, 10);
 
-    expect(element.scroll).toHaveBeenCalledOnce();
-    expect(element.scroll).toBeCalledWith(10, 10);
+    expect(element.scroll).toHaveBeenCalledExactlyOnceWith(10, 10);
   });
 
   it('should scroll smooth', () => {
@@ -28,8 +27,7 @@ describe('LandscapeScroller', () => {
 
     landscapeScroller.scrollSmooth(element, 10, 10);
 
-    expect(element.scroll).toHaveBeenCalledOnce();
-    expect(element.scroll).toBeCalledWith({ left: 10, top: 10, behavior: 'smooth' });
+    expect(element.scroll).toHaveBeenCalledExactlyOnceWith({ left: 10, top: 10, behavior: 'smooth' });
   });
 
   it('should scroll into view', () => {
@@ -38,7 +36,6 @@ describe('LandscapeScroller', () => {
 
     landscapeScroller.scrollIntoView(element);
 
-    expect(element.scrollIntoView).toHaveBeenCalledOnce();
-    expect(element.scrollIntoView).toBeCalledWith({ behavior: 'smooth', block: 'center', inline: 'center' });
+    expect(element.scrollIntoView).toHaveBeenCalledExactlyOnceWith({ behavior: 'smooth', block: 'center', inline: 'center' });
   });
 });
diff --git a/src/test/webapp/unit/shared/alert/infrastructure/secondary/MittAlertBus.spec.ts b/src/test/webapp/unit/shared/alert/infrastructure/secondary/MittAlertBus.spec.ts
@@ -10,8 +10,7 @@ describe('MittAlertBus', () => {
 
     mittAlertBus.success('A message');
 
-    expect(emitterStub.emit).toHaveBeenCalledOnce();
-    expect(emitterStub.emit).toBeCalledWith(AlertType.SUCCESS, 'A message');
+    expect(emitterStub.emit).toHaveBeenCalledExactlyOnceWith(AlertType.SUCCESS, 'A message');
   });
 
   it('should emit error', () => {
@@ -20,7 +19,6 @@ describe('MittAlertBus', () => {
 
     mittAlertBus.error('A message');
 
-    expect(emitterStub.emit).toHaveBeenCalledOnce();
-    expect(emitterStub.emit).toBeCalledWith(AlertType.ERROR, 'A message');
+    expect(emitterStub.emit).toHaveBeenCalledExactlyOnceWith(AlertType.ERROR, 'A message');
   });
 });
diff --git a/src/test/webapp/unit/shared/alert/infrastructure/secondary/MittAlertListener.spec.ts b/src/test/webapp/unit/shared/alert/infrastructure/secondary/MittAlertListener.spec.ts
@@ -12,8 +12,7 @@ describe('MittAlertListener', () => {
 
     emitter.emit(AlertType.SUCCESS, 'A message');
 
-    expect(spyAlerted).toHaveBeenCalledOnce();
-    expect(spyAlerted).toBeCalledWith('A message');
+    expect(spyAlerted).toHaveBeenCalledExactlyOnceWith('A message');
   });
 
   it('should unsubscribe success', () => {
@@ -39,8 +38,7 @@ describe('MittAlertListener', () => {
 
     emitter.emit(AlertType.ERROR, 'A message');
 
-    expect(spyAlerted).toHaveBeenCalledOnce();
-    expect(spyAlerted).toBeCalledWith('A message');
+    expect(spyAlerted).toHaveBeenCalledExactlyOnceWith('A message');
   });
 
   it('should unsubscribe error', () => {

diff --git a/src/test/webapp/unit/shared/logger/infrastructure/secondary/ConsoleLogger.spec.ts b/src/test/webapp/unit/shared/logger/infrastructure/secondary/ConsoleLogger.spec.ts
@@ -11,7 +11,6 @@ describe('ConsoleLogger', () => {
 
     consoleLogger.error('An error occurs', error);
 
-    expect(logger.error).toHaveBeenCalledOnce();
-    expect(logger.error).toBeCalledWith('An error occurs\n', error);
+    expect(logger.error).toHaveBeenCalledExactlyOnceWith('An error occurs\n', error);
   });
 });