Skip to content

Add skylib tests for compile_commands.bzl - #284

Open
furtib wants to merge 3 commits into
Ericsson:mainfrom
furtib:test-bazel-mock-002
Open

Add skylib tests for compile_commands.bzl#284
furtib wants to merge 3 commits into
Ericsson:mainfrom
furtib:test-bazel-mock-002

Conversation

@furtib

@furtib furtib commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Why:
We want to properly test compile_commands generation.

What:

  • Added skylib tests to check whether compile flags have been correctly obtained.
  • Have 4 bug, tests:
    • implementation_deps does not collect defines
    • implementation_deps does not collect local_defines
    • implementaion_deps does not collect qoute_includes
    • Compile flags are duplicated

Addresses:
none

@furtib
furtib requested a review from Szelethus August 13, 2026 08:51
@furtib furtib self-assigned this Aug 13, 2026
@furtib furtib added the test ☑️ Adding or refactoring tests label Aug 13, 2026
furtib added 3 commits August 13, 2026 10:56
Add qoute include test

Remove most tests

Undo refactors for compile_commands.bzl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test ☑️ Adding or refactoring tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant