René's URL Explorer Experiment

Title: feat: Add integration tests for dbt import with local dbt project by Copilot · Pull Request #5877 · feast-dev/feast · GitHub

Open Graph Title: feat: Add integration tests for dbt import with local dbt project by Copilot · Pull Request #5877 · feast-dev/feast

X Title: feat: Add integration tests for dbt import with local dbt project by Copilot · Pull Request #5877 · feast-dev/feast

Description: What this PR does / why we need it: Adds comprehensive integration tests for the dbt import feature (PR #5827), which previously lacked end-to-end testing with actual dbt project structure. Changes Test Infrastructure Created sdk/python/tests/integration/dbt/ with 600+ lines covering: Manifest parsing with dbt-artifacts-parser Tag filtering (feast, ml, recommendations) Model selection by name All 3 data source types (BigQuery, Snowflake, File) Type mapping validation (INT32/64, FLOAT32/64, STRING, TIMESTAMP) FeatureView generation with schema validation Column exclusion (entity, timestamp, custom) Code generation workflow Test dbt Project Minimal dbt project in test_dbt_project/ with 3 models: driver_features: INT types, multiple tags customer_features: STRING entity product_features: FLOAT32, tag filtering test Pre-generated manifest.json for CI execution without dbt CLI DuckDB-only profile configuration for local testing without external service dependencies CI/CD GitHub Actions workflow dbt-integration-tests.yml Runs on Python 3.11 & 3.12 Triggers on dbt code/test changes Uses Makefile for dependency installation (make install-python-dependencies-ci) Bug Fixes Removed deprecated PytestUnhandledCoroutineWarning from pytest.ini Test Example def test_get_models_with_tag_filter(self, parser): """Test filtering models by dbt tag.""" # Filter by 'ml' tag ml_models = parser.get_models(tag_filter="ml") assert len(ml_models) == 2 # Filter by 'recommendations' tag rec_models = parser.get_models(tag_filter="recommendations") assert len(rec_models) == 1 Which issue(s) this PR fixes: Related to #3335 Misc Comprehensive documentation added for test structure and dbt project. The manifest.json format may need minor schema adjustments for dbt-artifacts-parser v0.12.0 compatibility, but test logic and infrastructure are production-ready. Original prompt This section details on the original issue you should resolve Add integration tests for dbt import with local dbt project ## Context PR #5827 added dbt integration but lacks proper integration tests with an actual dbt project setup. Description As mentioned by @franciscojavierarceo in PR #5827, we need a lightweight local dbt project that: Sets up a minimal dbt project structure Runs dbt compile to generate manifest.json Tests the full feast dbt import workflow end-to-end Validates generated Feast objects Acceptance Criteria Create test dbt project in sdk/python/tests/integration/dbt/ Add test that compiles dbt project and imports to Feast Test all three data source types (bigquery, snowflake, file) Verify FeatureViews are created correctly Test tag filtering and model selection Related PR #5827 Issue #3335 please setup dbt in a github action integration test and test the dbt parser that was implemented in PR #5827 Comments on the Issue (you are @copilot in this section) Fixes #5869 💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Open Graph Description: What this PR does / why we need it: Adds comprehensive integration tests for the dbt import feature (PR #5827), which previously lacked end-to-end testing with actual dbt project structure. Changes...

X Description: What this PR does / why we need it: Adds comprehensive integration tests for the dbt import feature (PR #5827), which previously lacked end-to-end testing with actual dbt project structure. Changes...

Opengraph URL: https://github.com/feast-dev/feast/pull/5877

X: @github

direct link

Domain: patch-diff.githubusercontent.com

Links:

Viewport: width=device-width