Skip to content

test: Expand upload tests with fake_forge#250

Merged
jerry-skydio merged 1 commit into
mainfrom
jerry/revup/main/fakeforge
Jun 19, 2026
Merged

test: Expand upload tests with fake_forge#250
jerry-skydio merged 1 commit into
mainfrom
jerry/revup/main/fakeforge

Conversation

@jerry-skydio

Copy link
Copy Markdown
Collaborator

Now that we have a forge abstraction, we can improve
test coverage in upload. Add a fake forge class that
tracks query / update data in memory and a bunch of
tests that exercise the various flags.

@jerry-skydio

jerry-skydio commented May 20, 2026

Copy link
Copy Markdown
Collaborator Author

Reviews in this chain:
#250 test: Expand upload tests with fake_forge
 └#253 github: Add graphql utility class to contain query fields
  └#254 github: Add request retry splitting

@jerry-skydio

jerry-skydio commented May 20, 2026

Copy link
Copy Markdown
Collaborator Author
# head base diff date summary
0 48d65d5d bd7abad6 diff May 19 18:34 PM 2 files changed, 744 insertions(+)
1 0c04e95d bd7abad6 diff May 20 13:52 PM 1 file changed, 306 insertions(+), 2 deletions(-)
2 d68e6ac2 98287168 rebase May 20 14:15 PM 0 files changed
3 4599c0ba 98287168 diff May 20 14:42 PM 2 files changed, 67 insertions(+), 84 deletions(-)
4 b7cbff59 bbda170b diff Jun 19 10:47 AM 0 files changed

Base automatically changed from jerry/revup/main/forge to main June 19, 2026 17:35
Now that we have a forge abstraction, we can improve
test coverage in upload. Add a fake forge class that
tracks query / update data in memory and a bunch of
tests that exercise the various flags.
@jerry-skydio
jerry-skydio force-pushed the jerry/revup/main/fakeforge branch from 4599c0b to b7cbff5 Compare June 19, 2026 17:47
@jerry-skydio
jerry-skydio merged commit aab9089 into main Jun 19, 2026
5 checks passed
@jerry-skydio
jerry-skydio deleted the jerry/revup/main/fakeforge branch June 19, 2026 19:47
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.

2 participants