fix: address copilot review comments

Author: DevipriyaS17

src/app/devices/device-toolbar/device-toolbar.component.spec.ts

@@ -138,6 +138,24 @@ describe('DeviceToolbarComponent', () => {
     expect(component.isLoading()()).toBeFalse()
   })
 
+  it('should fetch power state using non-cached API', () => {
+    devicesService.getPowerState.calls.reset()
+    devicesService.getPowerStateCached.calls.reset()
+
+    component.getPowerState()
+
+    expect(devicesService.getPowerState).toHaveBeenCalledWith('guid')
+    expect(devicesService.getPowerStateCached).not.toHaveBeenCalled()
+  })
+
+  it('should reset loading state when power state request fails', () => {
+    devicesService.getPowerState.and.returnValue(throwError(() => new Error('Network error')))
+
+    component.getPowerState()
+
+    expect(component.isLoading()()).toBeFalse()
+  })
+
   it('should navigate to device', async () => {
     fixture.componentRef.setInput('deviceId', '12345-pokli-456772')
     const routerSpy = spyOn(component.router, 'navigate')

src/app/devices/device-toolbar/device-toolbar.component.ts

@@ -219,20 +219,29 @@ export class DeviceToolbarComponent implements OnInit {
 
   getPowerState(): void {
     this.isLoading().set(true)
-    // Bypass the cache so a manual refresh always fetches the latest state from
-    // the device rather than returning a previously cached value.
+    // Bypass the cache so each request fetches the latest state from the
+    // device rather than returning a previously cached value.
     this.devicesService
       .getPowerState(this.deviceId())
-      .pipe(takeUntilDestroyed(this.destroyRef))
-      .subscribe((powerState) => {
-        this.powerState.set(
-          powerState.powerstate.toString() === '2'
-            ? 'deviceToolbar.power.on.value'
-            : powerState.powerstate.toString() === '3' || powerState.powerstate.toString() === '4'
-              ? 'deviceToolbar.power.sleep.value'
-              : 'deviceToolbar.power.off.value'
-        )
-        this.isLoading().set(false)
+      .pipe(
+        finalize(() => {
+          this.isLoading().set(false)
+        }),
+        takeUntilDestroyed(this.destroyRef)
+      )
+      .subscribe({
+        next: (powerState) => {
+          this.powerState.set(
+            powerState.powerstate.toString() === '2'
+              ? 'deviceToolbar.power.on.value'
+              : powerState.powerstate.toString() === '3' || powerState.powerstate.toString() === '4'
+                ? 'deviceToolbar.power.sleep.value'
+                : 'deviceToolbar.power.off.value'
+          )
+        },
+        error: () => {
+          // Keep existing error propagation behavior from DevicesService; finalize handles UI state.
+        }
       })
   }