Skip to content

include a workflow ID for each deposition form open in the browser, c… - #43

Merged
jkalayan merged 1 commit into
mainfrom
40-bug-sessions-shared-across-tabs2
Aug 25, 2026
Merged

include a workflow ID for each deposition form open in the browser, c…#43
jkalayan merged 1 commit into
mainfrom
40-bug-sessions-shared-across-tabs2

Conversation

@jkalayan

Copy link
Copy Markdown
Collaborator

…reate separate tmpdir for each workflow and delete once submitted. Update tests.

Summary

  • A workflow ID for each extraction in each open browser window
  • Delete the tmpdir and workflow instance once submission is completed/failed/clear fields

…reate separate tmpdir for each workflow and delete once submitted. Update tests.
@jkalayan jkalayan linked an issue Aug 25, 2026 that may be closed by this pull request
@jkalayan
jkalayan merged commit d6554d1 into main Aug 25, 2026
12 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 32872461981

Coverage decreased (-1.5%) to 78.825%

Details

  • Coverage decreased (-1.5%) from the base build.
  • Patch coverage: 34 uncovered changes across 4 files (103 of 137 lines covered, 75.18%).
  • 28 coverage regressions across 3 files.

Uncovered Changes

File Changed Covered %
biosimdb_interface/form/webform.py 42 25 59.52%
biosimdb_interface/form/workflows.py 62 48 77.42%
biosimdb_interface/form/extract.py 16 14 87.5%
biosimdb_interface/form/upload.py 15 14 93.33%
Total (5 files) 137 103 75.18%

Coverage Regressions

28 previously-covered lines in 3 files lost coverage.

File Lines Losing Coverage Coverage
biosimdb_interface/form/upload.py 21 58.45%
biosimdb_interface/form/webform.py 6 67.32%
biosimdb_interface/form/extract.py 1 89.47%

Coverage Stats

Coverage Status
Relevant Lines: 732
Covered Lines: 577
Line Coverage: 78.83%
Coverage Strength: 7.09 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: sessions shared across tabs

2 participants