Skip to content

test: Expand upload tests with fake_forge#250

Open
jerry-skydio wants to merge 1 commit into
jerry/revup/main/forgefrom
jerry/revup/main/fakeforge
Open

test: Expand upload tests with fake_forge#250
jerry-skydio wants to merge 1 commit into
jerry/revup/main/forgefrom
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.

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:
#249 forge: Genericize github handling into a forge abstraction
 └#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(-)

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