Skip to content

fix: resolve failing tests and implement missing utilities - #462

Open
stooit wants to merge 1 commit into
mainfrom
quantcode/e2e-tier1-895-1794327244
Open

stooit wants to merge 1 commit into
mainfrom
quantcode/e2e-tier1-895-1794327244

Conversation

@stooit

@stooit stooit commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • calculator: divide throws Division by zero when divisor is 0
  • date-utils: formatRelative rounds days (Math.round(|hours|/24)) — fixes off-by-one
  • string-utils: wordCount splits on whitespace runs; implemented truncate (word boundary, ... counts toward maxLength)
  • task-manager: implemented remove, update, sortBy (priority/status/createdAt)
  • validator: isEmail allows TLDs of 2+ letters; isUrl accepts ports

bun test: 60 pass, 0 fail. No test files modified, no new deps.

Assumptions

  • Behaviour choices (sort orders, truncation fallback to hard cut when no space) follow test expectations.
  • Stale // BUG:/// TODO: comments left in place (cosmetic).

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