Skip to content

test: add opt-in devnet scenarios - #188

Draft
galargh wants to merge 6 commits into
mainfrom
galargh/optional-test-scenarios
Draft

galargh wants to merge 6 commits into
mainfrom
galargh/optional-test-scenarios

Conversation

@galargh

@galargh galargh commented Aug 29, 2026 •

Copy link
Copy Markdown
Contributor

Related to #127

It adds tests for access control, batch uploads, and service termination (parts of what was mentioned in the issue). Also, as suggested there, it keeps the longer running tests optional - they can be enabled via run-optional-scenarios label.

I added labeled/unlabeled events to the CI workflow so that adding/removing the optional scenarios label would trigger them immediately. These tests are quite expensive in general so if we start using labels more here we might want to consider removing that auto-trigger. Then, the rerun would have to be forced via close/reopen PR or pushing an empty commit for example.

Added scenarios:

  • Negative permissions: requests from another user and/or with invalid data are rejected without changing the data set
  • Bulk add (optional): uploads 40 distinct pieces in one batch, then checks the count and looks for each piece
  • Termination controls (optional): checks provider-relayed and direct termination, fees, etc.

@FilOzzy FilOzzy added this to FOC Aug 29, 2026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC Aug 29, 2026
@FilOzzy FilOzzy moved this from 📌 Triage to ⌨️ In Progress in FOC Aug 29, 2026
@galargh galargh closed this Aug 30, 2026
@github-project-automation github-project-automation Bot moved this from ⌨️ In Progress to 🎉 Done in FOC Aug 30, 2026
@galargh galargh reopened this Aug 30, 2026
@github-project-automation github-project-automation Bot moved this from 🎉 Done to ⌨️ In Progress in FOC Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ⌨️ In Progress

Development

Successfully merging this pull request may close these issues.

2 participants