test: require named source databases for materialized views #17

Open

nsaspy wants to merge 4 commits from rage/15-view-source-database into master

pull from: rage/15-view-source-database

merge into: nsaspy:master

nsaspy:master

nsaspy:rage/13-index-range-duplicates-red

nsaspy:db/lmdb-auto-grow-red-20260831

nsaspy:merge/issue-7-nix-package

nsaspy:rage/7-nix-package-red

nsaspy:agent/composable-transactions-graph-lifecycle

nsaspy:agent/public-view-query-api

nsaspy:agent/lmdb-storage-engine-v0.2

Conversation 0 Commits 4 Files changed 3 +74 -7

nsaspy commented 2026-09-04 22:50:00 +00:00

Owner

Copy link

RED-first slice for #15, required by downstream lost-rob0t/hackmode#15.

This commit adds the deterministic acceptance fixture only. It defines a materialized view whose source is the named outbox database, proves rebuild and incremental application must read that source, and leaves the primary database untouched.

Expected RED on exact head 194df03706afa7ecbadad91c2461a0ea02c202ca: current new-view has no :source-database-name contract and both view application paths are hard-wired to the primary database.

No Hackmode-specific semantics are added to Tek9. The production fix will remain generic and preserve primary-database behavior by default.

RED-first slice for #15, required by downstream lost-rob0t/hackmode#15. This commit adds the deterministic acceptance fixture only. It defines a materialized view whose source is the named `outbox` database, proves rebuild and incremental application must read that source, and leaves the primary database untouched. Expected RED on exact head `194df03706afa7ecbadad91c2461a0ea02c202ca`: current `new-view` has no `:source-database-name` contract and both view application paths are hard-wired to the primary database. No Hackmode-specific semantics are added to Tek9. The production fix will remain generic and preserve primary-database behavior by default.

nsaspy added 4 commits 2026-09-04 22:50:00 +00:00

test: require named source databases for materialized views 194df03706

views: support declared named source databases f37160d38c

views: export source database identity 85042016be

test: require missing view sources to fail closed 18f0e81575

Some checks failed

Hide all checks

CI / benchmark-regression (pull_request) Failing after 16m49s

Details

CI / tests (ubuntu-22.04) (pull_request) Successful in 3m43s

Details

CI / tests (ubuntu-24.04) (pull_request) Successful in 2m19s

Details

CI / package-smoke (pull_request) Failing after 5s

Details

CI / merge-gate (pull_request) Failing after 2s

Details

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin rage/15-view-source-database:rage/15-view-source-database

git switch rage/15-view-source-database

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master

git merge --no-ff rage/15-view-source-database

git switch rage/15-view-source-database

git rebase master

git switch master

git merge --ff-only rage/15-view-source-database

git switch rage/15-view-source-database

git rebase master

git switch master

git merge --no-ff rage/15-view-source-database

git switch master

git merge --squash rage/15-view-source-database

git switch master

git merge --ff-only rage/15-view-source-database

git switch master

git merge rage/15-view-source-database

git push origin master

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

bug

Something isn't working

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

question

Further information is requested

  

wontfix

This will not be worked on

No labels

bug

documentation

duplicate

enhancement

good first issue

help wanted

invalid

question

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

nsaspy/tek9!17

No description provided.