Skip to content

feat(poly test diff): extract related bricks from changed tests - #460

Merged
DavidVujic merged 7 commits into
mainfrom
test-diff-related-bricks
Aug 7, 2026
Merged

feat(poly test diff): extract related bricks from changed tests#460
DavidVujic merged 7 commits into
mainfrom
test-diff-related-bricks

Conversation

@DavidVujic

@DavidVujic DavidVujic commented Aug 5, 2026

Copy link
Copy Markdown
Owner

Description

Adding the possibility to find related bricks for changed tests. The existing behaviour of the poly test diff command is to inspect the tests, and find bricks by the imports made in the tests.

This new feature adds a --strategy path option to find the corresponding brick according to the folder structure.

Example

A changed test in test/components/my_namespace/my_brick/the_test_for_the_brick.py,
the poly tool will look for a corresponding/related brick in components/my_namespace/my_brick.

fixes issue #459

Motivation and Context

The idea and implementation suggestions is from issue #459

How Has This Been Tested?

✅ CI
✅ added unit tests
✅ local run of the poly test diff command.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

@DavidVujic DavidVujic closed this Aug 5, 2026
@DavidVujic DavidVujic reopened this Aug 5, 2026
@DavidVujic
DavidVujic marked this pull request as draft August 5, 2026 08:38
@DavidVujic DavidVujic changed the title wip(poly test diff): extract related/owner bricks from changed tests feat(poly test diff): extract related bricks from changed tests Aug 6, 2026
@DavidVujic
DavidVujic marked this pull request as ready for review August 6, 2026 08:49
@DavidVujic
DavidVujic force-pushed the test-diff-related-bricks branch from d4ea600 to faed46b Compare August 7, 2026 06:33
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@DavidVujic
DavidVujic merged commit cb0ac4b into main Aug 7, 2026
3 of 4 checks passed
@DavidVujic
DavidVujic deleted the test-diff-related-bricks branch August 7, 2026 06:45
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.

1 participant