Skip to content

Increase code coverage for angular app#5

Merged
bdgould merged 2 commits intomainfrom
claude/increase-test-coverage-01RFdA3ehzs59BAtRaxVcey5
Dec 2, 2025
Merged

Increase code coverage for angular app#5
bdgould merged 2 commits intomainfrom
claude/increase-test-coverage-01RFdA3ehzs59BAtRaxVcey5

Conversation

@bdgould
Copy link
Member

@bdgould bdgould commented Dec 2, 2025

No description provided.

claude and others added 2 commits December 2, 2025 03:07
- Created comprehensive tests for services without spec files:
  - drawer.service.spec.ts (new)
  - cache.service.spec.ts (new)
  - sparqi.service.spec.ts (new)
  - ontology.service.spec.ts (new)

- Created tests for components without spec files:
  - sparqi-metrics-modal.component.spec.ts (new)
  - ontology-visualization-dialog.component.spec.ts (new)
  - sparqi-chat.component.spec.ts (new)

- Enhanced existing minimal service tests:
  - datasource.service.spec.ts
  - settings.service.spec.ts
  - metrics.service.spec.ts
  - graphmarts.service.spec.ts
  - layers.service.spec.ts

All tests include comprehensive coverage of:
- Success scenarios
- Error handling
- Edge cases
- HTTP request/response validation
- Observable subscriptions
- Component lifecycle hooks
- User interactions

This significantly increases the overall code coverage for the query-service-web Angular application.
@bdgould bdgould self-assigned this Dec 2, 2025
@bdgould bdgould merged commit 4f4e92f into main Dec 2, 2025
1 check passed
@bdgould bdgould deleted the claude/increase-test-coverage-01RFdA3ehzs59BAtRaxVcey5 branch December 19, 2025 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants