◐ Shell
clean mode source ↗

feat: Add integration tests for dbt import by YassinNouh21 · Pull Request #5899 · feast-dev/feast

franciscojavierarceo

franciscojavierarceo

YassinNouh21

devin-ai-integration[bot]

devin-ai-integration[bot]

franciscojavierarceo

- Add 82 integration tests covering dbt manifest parsing, mapper, codegen, and CLI
- Add real dbt project with DuckDB for testing
- Add GitHub Actions workflow for dbt integration tests
- Fix entity column handling: codegen now includes entity columns in schema
  to match mapper behavior (FeatureView.__init__ expects them)
- Fix type mapping: Int32→ValueType.INT32, Float32→ValueType.FLOAT
  (instead of widening to INT64/DOUBLE)
- Add multi-entity column support tests
- Add comprehensive test coverage for edge cases, validation errors,
  special characters, and CLI commands

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>

ntkathole

- Add PR trigger types (opened, synchronize, labeled)
- Add label check condition requiring ok-to-test/approved/lgtm
- Prevents resource spin-up for every PR

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>

ntkathole

soooojinlee pushed a commit to soooojinlee/feast that referenced this pull request

Feb 18, 2026

jyejare pushed a commit to opendatahub-io/feast that referenced this pull request

Mar 9, 2026
* Add comprehensive integration tests for dbt import feature

- Add 82 integration tests covering dbt manifest parsing, mapper, codegen, and CLI
- Add real dbt project with DuckDB for testing
- Add GitHub Actions workflow for dbt integration tests
- Fix entity column handling: codegen now includes entity columns in schema
  to match mapper behavior (FeatureView.__init__ expects them)
- Fix type mapping: Int32→ValueType.INT32, Float32→ValueType.FLOAT
  (instead of widening to INT64/DOUBLE)
- Add multi-entity column support tests
- Add comprehensive test coverage for edge cases, validation errors,
  special characters, and CLI commands

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>

* Add ok-to-test label check to dbt integration workflow

- Add PR trigger types (opened, synchronize, labeled)
- Add label check condition requiring ok-to-test/approved/lgtm
- Prevents resource spin-up for every PR

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>

---------

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>

Shizoqua pushed a commit to Shizoqua/feast that referenced this pull request

Mar 18, 2026
* Add comprehensive integration tests for dbt import feature

- Add 82 integration tests covering dbt manifest parsing, mapper, codegen, and CLI
- Add real dbt project with DuckDB for testing
- Add GitHub Actions workflow for dbt integration tests
- Fix entity column handling: codegen now includes entity columns in schema
  to match mapper behavior (FeatureView.__init__ expects them)
- Fix type mapping: Int32→ValueType.INT32, Float32→ValueType.FLOAT
  (instead of widening to INT64/DOUBLE)
- Add multi-entity column support tests
- Add comprehensive test coverage for edge cases, validation errors,
  special characters, and CLI commands

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>

* Add ok-to-test label check to dbt integration workflow

- Add PR trigger types (opened, synchronize, labeled)
- Add label check condition requiring ok-to-test/approved/lgtm
- Prevents resource spin-up for every PR

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>

---------

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>
Signed-off-by: Shizoqua <hr.lanreshittu@gmail.com>

Shizoqua pushed a commit to Shizoqua/feast that referenced this pull request

Mar 18, 2026