Skip to content

fix: resolve failing tests across utility modules - #357

Open
stooit wants to merge 1 commit into
mainfrom
quantcode/e2e-tier1-2199-1790578564
Open

stooit wants to merge 1 commit into
mainfrom
quantcode/e2e-tier1-2199-1790578564

Conversation

@stooit

@stooit stooit commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes all 16 failing tests (now 60/60 pass, tsc --noEmit clean). No test files or dependencies changed.

  • calculator: divide throws on zero divisor.
  • date-utils: day diff uses Math.round (consistent with minute/hour branches; 36h → "2 days ago").
  • string-utils: implemented truncate (word-boundary aware, ellipsis, never exceeds maxLength); wordCount splits on /\s+/.
  • task-manager: implemented remove, update (partial, returns false for unknown id), sortBy (priority/status/createdAt).
  • validator: isEmail allows long TLDs (plus 254-char length bound); isUrl accepts explicit ports.

Assumptions

  • sortBy("status") order is pending → in_progress → completed (untested; not specified).
  • isEmail 254-char cap added defensively (RFC 5321), not required by tests.

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.

1 participant