Skip to content

Unused test data in brace-expansion.test.ts #17576

Closed
@ediamin

Description

@ediamin

I found an issue in the brace-expansion.test.ts file where there's an unused test case at line 20.

File: packages/tailwindcss/src/utils/brace-expansion.test.ts

Issue: The test case for 'a/{0..10..5}/b' is incorrectly structured. The third array element ['a/{10..0..5}/b', ['a/10/b', 'a/5/b', 'a/0/b']] has no effect on the test execution.

Current code:

...
[
  'a/{0..10..5}/b',
  ['a/0/b', 'a/5/b', 'a/10/b'],
  ['a/{10..0..5}/b', ['a/10/b', 'a/5/b', 'a/0/b']],
],
...

Expected structure:

...
['a/{0..10..5}/b', ['a/0/b', 'a/5/b', 'a/10/b']],
['a/{10..0..5}/b', ['a/10/b', 'a/5/b', 'a/0/b']],
...

I'm not sure what the original intention of this array was. If I move this array to the parent level, the test fails, meaning expand('a/{10..0..5}/b') does not return ['a/10/b', 'a/5/b', 'a/0/b']. The actual output is ['a/0/b', 'a/10/b', 'a/5/b'].

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions