move tests outside of constructor

cryostatio · Jan 28, 2025 · ae6c3ef · ae6c3ef
1 parent 01f39c7
commit ae6c3ef
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 9 deletions.
diff --git a/src/app/Shared/Services/Api.service.tsx b/src/app/Shared/Services/Api.service.tsx
@@ -91,16 +91,9 @@ export class ApiService {
     private readonly ctx: CryostatContext,
     private readonly target: TargetService,
     private readonly notifications: NotificationService,
-  ) {
-    this.doGet('recordings')
-      .pipe(
-        catchError(() => {
-          this.archiveEnabled.next(false);
-          return EMPTY;
-        }),
-      )
-      .subscribe();
+  ) {}
 
+  testHealth() {
     const getDatasourceURL: Observable<GrafanaDashboardUrlGetResponse> = this.ctx
       .url('/api/v4/grafana_datasource_url')
       .pipe(
@@ -381,6 +374,17 @@ export class ApiService {
     );
   }
 
+  testArchiveAvailability() {
+    this.doGet('recordings')
+      .pipe(
+        catchError(() => {
+          this.archiveEnabled.next(false);
+          return EMPTY;
+        }),
+      )
+      .subscribe();
+  }
+
   isArchiveEnabled(): Observable<boolean> {
     return this.archiveEnabled.asObservable();
   }

diff --git a/src/app/Shared/Services/Services.tsx b/src/app/Shared/Services/Services.tsx
@@ -53,6 +53,9 @@ const notificationChannel = new NotificationChannel(NotificationsInstance, login
 const reports = new ReportService(NotificationsInstance, notificationChannel);
 const targets = new TargetsService(api, NotificationsInstance, notificationChannel);
 
+api.testHealth();
+api.testArchiveAvailability();
+
 const defaultServices: Services = {
   target,
   targets,