Skip to content

chore: ignore .DS_Store in test fixtures#4422

Merged
spiffcs merged 1 commit intomainfrom
chore-ignore-ds-store-in-test-fixtures
Dec 1, 2025
Merged

chore: ignore .DS_Store in test fixtures#4422
spiffcs merged 1 commit intomainfrom
chore-ignore-ds-store-in-test-fixtures

Conversation

@willmurphyscode
Copy link
Contributor

Otherwise, we get test failures on macOS if macOS has decided to put .DS_Store entries in the test fixtures.

Description

Otherwise, I sometimes get errors like unable to get logical key for snippet "classifiers/snippets/arangodb/.DS_Store": invalid managed binary path: "arangodb/.DS_Store" when trying to run make unit locally on a macbook.w

Type of change

  • Chore (improve the developer experience, fix a test flake, etc, without changing the visible behavior of Syft)

Checklist:

Checklist is not really applicable as only test fixture handling code is changed, not production code.

  • I have added unit tests that cover changed behavior
  • I have tested my code in common scenarios and confirmed there are no regressions
  • I have added comments to my code, particularly in hard-to-understand sections

Otherwise, we get test failures on macOS if macOS has decided to put
.DS_Store entries in the test fixtures.

Signed-off-by: Will Murphy <willmurphyscode@users.noreply.github.com>
@spiffcs spiffcs merged commit a0f7148 into main Dec 1, 2025
12 checks passed
@spiffcs spiffcs deleted the chore-ignore-ds-store-in-test-fixtures branch December 1, 2025 15:15
VictorHuu pushed a commit to VictorHuu/syft that referenced this pull request Dec 5, 2025
Otherwise, we get test failures on macOS if macOS has decided to put
.DS_Store entries in the test fixtures.

Signed-off-by: Will Murphy <willmurphyscode@users.noreply.github.com>
Signed-off-by: Yuntao Hu <victorhu493@gmail.com>
@willmurphyscode willmurphyscode added the changelog-ignore Don't include this issue in the release changelog label Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-ignore Don't include this issue in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants