Skip to content

test(fixtures): add include-path multi-value handling tests#1456

Open
niklasmarderx wants to merge 1 commit intoorhun:mainfrom
niklasmarderx:test/include-path-fixtures-1455
Open

test(fixtures): add include-path multi-value handling tests#1456
niklasmarderx wants to merge 1 commit intoorhun:mainfrom
niklasmarderx:test/include-path-fixtures-1455

Conversation

@niklasmarderx
Copy link
Copy Markdown

@niklasmarderx niklasmarderx commented Mar 31, 2026

Add two fixture tests that verify --include-path handles multiple glob patterns correctly (#1455):

  • test-include-path-repeated-flag: --include-path 'website/**/*' --include-path 'docs/**/*'
  • test-include-path-space-separated: --include-path 'website/**/* docs/**/*'

Both use the same commit history (files in website/, src/, and docs/) and expect the same filtered output — only commits touching website/ or docs/ paths.

The workflow entries for test-fixtures.yml need to be added separately since fork pushes can't modify workflow files. Here's what to add to the matrix:

- fixtures-name: test-include-path-repeated-flag
  command: --include-path 'website/**/*' --include-path 'docs/**/*'
- fixtures-name: test-include-path-space-separated
  command: --include-path 'website/**/* docs/**/*'

Closes #1455
@niklasmarderx
test(fixtures): add include-path multi-value handling tests
f9a6c10 
Add two fixture tests that verify --include-path works correctly with
multiple glob patterns:

- test-include-path-repeated-flag: uses separate --include-path flags
- test-include-path-space-separated: uses a single flag with space-separated globs

Both should produce identical output, filtering commits to only those
that touch files matching the given patterns.

Closes orhun#1455
@niklasmarderx niklasmarderx requested a review from orhun as a code owner March 31, 2026 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@niklasmarderx