Skip to content

Conversation

@forsyth2
Copy link
Collaborator

Summary

Objectives:

  • Streamline the blocking/non-blocking test.

Issue resolution:

Select one: This pull request is...

  • a bug fix: increment the patch version
  • a small improvement: increment the minor version
  • a new feature: increment the minor version
  • an incompatible (non-backwards compatible) API change: increment the major version

Small Change

  • To merge, I will use "Squash and merge". That is, this change should be a single commit.
  • Logic: I have visually inspected the entire pull request myself.
  • Pre-commit checks: All the pre-commits checks have passed.

@forsyth2 forsyth2 self-assigned this Oct 20, 2025
@forsyth2 forsyth2 added the Testing Files in `tests` modified label Oct 20, 2025
@forsyth2 forsyth2 changed the title Issue 385 blocking test Improvements to blocking test Oct 20, 2025
@forsyth2 forsyth2 force-pushed the issue-385-blocking-test branch from 9408a54 to 96b3f8f Compare October 20, 2025 21:59
@forsyth2 forsyth2 force-pushed the issue-385-blocking-test branch from 96b3f8f to fca1525 Compare October 20, 2025 22:16
@forsyth2
Copy link
Collaborator Author

Still in progress. Remaining action items:

  • Add assertions that certain lines appear/don't appear in the logs
  • Make sure https://app.globus.org/activity shows successful transfers
  • Make sure the BLOCKING and NON-BLOCKING cases work

@forsyth2 forsyth2 added the Globus Globus label Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Globus Globus Testing Files in `tests` modified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants