Yicong-Huang opened a new issue, #5210:
URL: https://github.com/apache/texera/issues/5210

   ### Task Summary
   
   Follow-up to #5208. The pure-logic cluster under `org.apache.texera.web` is 
now covered (#5209), but two DB-touching helpers remain uncovered:
   
   - `web/service/ExecutionsMetadataPersistService.scala` — 
`insertNewExecution`, `tryGetExistingExecution`, `tryUpdateExistingExecution` — 
all three swallow `Throwable` silently and only log, so a regression that 
breaks the update path would surface only via missing-data downstream.
   - `web/storage/ExecutionStateStore.updateWorkflowState` — wraps 
`tryUpdateExistingExecution` to also flip the in-memory `metadataStore`.
   
   Use the existing `common/dao` `MockTexeraDB` trait (EmbeddedPostgres + 
texera_ddl.sql) — same pattern already used by 
`WorkflowExecutionsResourceSpec`, `WorkflowAccessResourceSpec`, etc. Seed a 
`WorkflowExecutions` row in `beforeEach`, exercise the update, assert the row 
is mutated; pin the silent-failure behavior of `tryUpdateExistingExecution` on 
a non-existent eid with a comment so a future fix breaks the spec deliberately.
   
   ### Task Type
   
   - [x] Testing / QA


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to