Skip to content

Commit 10ad1f1

Browse files
haritamarclaude
andcommitted
ci: stop hardcoding 'snapshots' target_schema in failed_snapshot fixture (CORE-687)
failed_snapshot.sql is a parse-time fixture (intentionally invalid SQL) that's never actually run by CI. The hardcoded target_schema='snapshots' was forcing a fixed Glue database name and breaking the per-run schema isolation pattern. Use target.schema instead so it lands in the same ephemeral schema as the rest of the run. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 863b43a commit 10ad1f1

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

tests/e2e_dbt_project/snapshots/failed_snapshot.sql

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,9 @@
11
{% snapshot failed_snapshot() %}
22

3+
{# target_schema is required by dbt; reuse target.schema so this lands in the per-run CI schema. #}
34
{{
45
config(
5-
target_schema='snapshots',
6+
target_schema=target.schema,
67
unique_key='unique_id',
78
strategy='timestamp',
89
updated_at='generated_at',

0 commit comments

Comments
 (0)