Remove continue-on-error from unit test steps and fix TypeScript error in test

Co-authored-by: renemadsen <[email protected]>

Author: Copilot

.github/workflows/dotnet-core-master.yml

@@ -91,14 +91,13 @@ jobs:
         # Try different test commands based on what's available in package.json
         if grep -q '"test:ci"' package.json 2>/dev/null; then
           echo "Running with test:ci script..."
-          npm run test:ci -- --include='**/time-planning-pn/**/*.spec.ts' || true
+          npm run test:ci -- --include='**/time-planning-pn/**/*.spec.ts'
         elif grep -q '"test"' package.json 2>/dev/null; then
           echo "Running with test script..."
-          npm run test -- --watch=false --browsers=ChromeHeadless --include='**/time-planning-pn/**/*.spec.ts' || true
+          npm run test -- --watch=false --browsers=ChromeHeadless --include='**/time-planning-pn/**/*.spec.ts'
         else
           echo "No test script found in package.json, skipping unit tests"
         fi
-      continue-on-error: true
   pn-test:
     needs: build
     runs-on: ubuntu-latest

.github/workflows/dotnet-core-pr.yml

@@ -85,14 +85,13 @@ jobs:
         # Try different test commands based on what's available in package.json
         if grep -q '"test:ci"' package.json 2>/dev/null; then
           echo "Running with test:ci script..."
-          npm run test:ci -- --include='**/time-planning-pn/**/*.spec.ts' || true
+          npm run test:ci -- --include='**/time-planning-pn/**/*.spec.ts'
         elif grep -q '"test"' package.json 2>/dev/null; then
           echo "Running with test script..."
-          npm run test -- --watch=false --browsers=ChromeHeadless --include='**/time-planning-pn/**/*.spec.ts' || true
+          npm run test -- --watch=false --browsers=ChromeHeadless --include='**/time-planning-pn/**/*.spec.ts'
         else
           echo "No test script found in package.json, skipping unit tests"
         fi
-      continue-on-error: true
   pn-test:
     needs: build
     runs-on: ubuntu-22.04

eform-client/src/app/plugins/modules/time-planning-pn/components/plannings/time-planning-actions/assigned-site/assigned-site-dialog.component.spec.ts

@@ -59,7 +59,7 @@ describe('AssignedSiteDialogComponent', () => {
 
     mockStore.select.and.returnValue(of(true));
     mockSettingsService.getGlobalAutoBreakCalculationSettings.and.returnValue(
-      of({ success: true, model: {} as any })
+      of({ success: true, model: {} }) as any
     );
 
     await TestBed.configureTestingModule({