typechecl

cryostatio · Sep 4, 2024 · a787215 · a787215
1 parent 6bbf632
commit a787215
Show file tree

Hide file tree

Showing 7 changed files with 25 additions and 1 deletion.
diff --git a/src/test/CreateRecording/SnapshotRecordingForm.test.tsx b/src/test/CreateRecording/SnapshotRecordingForm.test.tsx
@@ -16,6 +16,7 @@
 
 import { SnapshotRecordingForm } from '@app/CreateRecording/SnapshotRecordingForm';
 import { authFailMessage } from '@app/ErrorView/types';
+import { ActiveRecording, RecordingState } from '@app/Shared/Services/api.types';
 import { ServiceContext, Services, defaultServices } from '@app/Shared/Services/Services';
 import { TargetService } from '@app/Shared/Services/Target.service';
 import { screen, cleanup, act as doAct } from '@testing-library/react';
@@ -31,6 +32,23 @@ const mockTarget = {
   labels: [],
   annotations: { cryostat: [], platform: [] },
 };
+const mockRecording: ActiveRecording = {
+  id: 100,
+  state: RecordingState.RUNNING,
+  duration: 1010,
+  startTime: 9999,
+  continuous: false,
+  toDisk: false,
+  maxSize: 55,
+  maxAge: 66,
+  remoteId: 77,
+  name: 'snapshot-10',
+  downloadUrl: 'http://localhost:8080/api/v4/targets/1/recordings/77',
+  reportUrl: 'http://localhost:8080/api/v4/targets/1/reports/77',
+  metadata: {
+    labels: [],
+  },
+};
 
 jest.spyOn(defaultServices.target, 'authFailure').mockReturnValue(of());
 jest.spyOn(defaultServices.target, 'target').mockReturnValue(of(mockTarget));
@@ -67,7 +85,7 @@ describe('<SnapshotRecordingForm />', () => {
   });
 
   it('should create Recording when create is clicked', async () => {
-    const onCreateSpy = jest.spyOn(defaultServices.api, 'createSnapshot').mockReturnValue(of(true));
+    const onCreateSpy = jest.spyOn(defaultServices.api, 'createSnapshot').mockReturnValue(of(mockRecording));
     const { user } = render({
       routerConfigs: {
         routes: [

diff --git a/src/test/RecordingMetadata/BulkEditLabels.test.tsx b/src/test/RecordingMetadata/BulkEditLabels.test.tsx
@@ -72,6 +72,7 @@ const mockActiveRecording: ActiveRecording = {
   toDisk: false,
   maxSize: 0,
   maxAge: 0,
+  remoteId: 9876,
 };
 
 const mockActiveLabelsNotification = {

diff --git a/src/test/Recordings/Filters/DurationFilter.test.tsx b/src/test/Recordings/Filters/DurationFilter.test.tsx
@@ -38,6 +38,7 @@ const mockRecording: ActiveRecording = {
   toDisk: false,
   maxSize: 0,
   maxAge: 0,
+  remoteId: 8765,
 };
 
 const onDurationInput = jest.fn((_durationInput) => undefined);

diff --git a/src/test/Recordings/Filters/LabelFilter.test.tsx b/src/test/Recordings/Filters/LabelFilter.test.tsx
@@ -46,6 +46,7 @@ const mockRecording: ActiveRecording = {
   toDisk: false,
   maxSize: 0,
   maxAge: 0,
+  remoteId: 6543,
 };
 const mockAnotherRecording = {
   ...mockRecording,

diff --git a/src/test/Recordings/Filters/NameFilter.test.tsx b/src/test/Recordings/Filters/NameFilter.test.tsx
@@ -38,6 +38,7 @@ const mockRecording: ActiveRecording = {
   toDisk: false,
   maxSize: 0,
   maxAge: 0,
+  remoteId: 5432,
 };
 const mockAnotherRecording = { ...mockRecording, name: 'anotherRecording' };
 const mockRecordingList = [mockRecording, mockAnotherRecording];

diff --git a/src/test/Recordings/Filters/RecordingStateFilter.test.tsx b/src/test/Recordings/Filters/RecordingStateFilter.test.tsx
@@ -38,6 +38,7 @@ const mockRecording: ActiveRecording = {
   toDisk: false,
   maxSize: 0,
   maxAge: 0,
+  remoteId: 4321,
 };
 const mockAnotherRecording = {
   ...mockRecording,

diff --git a/src/test/Recordings/RecordingFilters.test.tsx b/src/test/Recordings/RecordingFilters.test.tsx
@@ -65,6 +65,7 @@ const mockActiveRecording: ActiveRecording = {
   toDisk: false,
   maxSize: 0,
   maxAge: 0,
+  remoteId: 3210,
 };
 const mockActiveRecordingList = [
   mockActiveRecording,