Skip to content

Add edge case test for whitespace-only input - #174

Closed
BlocksecPHD wants to merge 1 commit into
un33k:masterfrom
BlocksecPHD:whitespace-only-input-test
Closed

BlocksecPHD wants to merge 1 commit into
un33k:masterfrom
BlocksecPHD:whitespace-only-input-test

Conversation

@BlocksecPHD

Copy link
Copy Markdown

This PR adds a test case to verify that correctly returns an empty string when the input contains only whitespace (spaces, tabs, newlines) or is empty.

Test cases covered:

  • Spaces only ()
  • Tabs and newlines ()
  • Mixed whitespace characters ()
  • Empty string ()

All 83 existing tests pass. No breaking changes introduced.

This addresses issue #169.

This test verifies that slugify correctly returns an empty string
when the input contains only whitespace (spaces, tabs, newlines).

Test cases include:
- Spaces only
- Tabs and newlines
- Mixed whitespace characters
- Empty string

No breaking changes introduced.
@un33k

un33k commented Sep 8, 2026

Copy link
Copy Markdown
Owner

This is Dojo, posting a maintainer-authorized follow-up linking this PR to #191.

The broader whitespace examples are consolidated into the new regression suite; this standalone PR is superseded.

Closing this item under the maintainer’s consolidation decision. The default algorithm remains legacy; improved output rules are opt-in. This note does not announce a published release, and no individual PR is being merged by this follow-up. Thank you for the contribution and discussion.

🚀 Generated with Dojo ⛩️

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