bug: add assertions to service spec tests (SonarQube S2699)

[devin-ai-integration]

Summary

Resolves the 9 open SonarQube S2699 blockers ("Add at least one assertion to this test case") in the Angular service specs. These tests subscribed to a service call and used httpMock.expectOne(...) to drive the request, but had no expect(...) call — expectOne is not recognized by SonarQube as an assertion, so the tests were flagged as assertion-less.

Each flagged test now asserts the captured request's HTTP method, matching the existing convention used in the surrounding tests (e.g. articles.service.spec.ts:114, comments.service.spec.ts:173).

  const req = httpMock.expectOne(`/profiles/${username}`);
+ expect(req.request.method).toBe('GET');
  req.flush({ profile: { ...mockProfile, username } });

Files / tests touched:

• articles.service.spec.ts — "should handle pagination parameters" (GET)
• comments.service.spec.ts — "slug with special characters" (GET), "numeric comment ID" / "UUID comment ID" (DELETE)
• profile.service.spec.ts — "username with special characters", "with numbers", "with hyphens", "with underscores", "very long username" (GET)

Notes

• The reported rule was java:S2699, but there are no java:S2699 issues in this org's SonarQube — the matching blockers are typescript:S2699 in this repo. Same rule (S2699), TypeScript analyzer.
• Verified the 3 edited spec files pass (78 tests green). Note: the repo's unit test suite is currently broken on main independently of this change — every spec imports zone.js which is not a declared dependency, so all spec files fail to load on a clean checkout. Tests here were validated by temporarily installing zone.js locally (not part of this PR). This pre-existing test-setup gap may want a separate fix.

Link to Devin session: https://app.devin.ai/sessions/8bf682121cf4422e9d34860899031849
Requested by: @Colhodm

---

Devin Review

Status	Commit
🟢 Reviewed	eda2fcf

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment, CI, and merge conflict monitoring

[staging-devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Debug

Playground