-
Notifications
You must be signed in to change notification settings - Fork 3
Start CKHub sharing service as a part of the GHA integration tests #73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
agriyakhetarpal
wants to merge
18
commits into
JupyterEverywhere:main
Choose a base branch
from
agriyakhetarpal:start-ckhub-api-gha
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 4 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
ac6b5ad
Check out sharing service
agriyakhetarpal 3c7e2d7
Don't persist credentials for this repository too
agriyakhetarpal c10945b
Set up Docker Compose
agriyakhetarpal 4ec0f15
Start CKHub sharing service
agriyakhetarpal 9b17b09
Add explicit contents read permissions
agriyakhetarpal 73b836d
Remove token
agriyakhetarpal ed9678c
Merge branch 'main' into start-ckhub-api-gha
agriyakhetarpal fd159cc
Don't mock the sharing service token
agriyakhetarpal 113df1e
Temporarily expose the `SharingService`
agriyakhetarpal 77a90e2
Try to clear token before each test
agriyakhetarpal 2ffa275
Remove mocks, try turning off kernel selection
agriyakhetarpal 4d8edb9
Dismiss kernel dialog for File uploads test
agriyakhetarpal a4b67da
Merge branch 'main' into start-ckhub-api-gha
agriyakhetarpal bce631a
Drop all modifications to tests and restore mocks
agriyakhetarpal ef753f8
Add functional test for the sharing service
agriyakhetarpal 54fbc2f
Start sharing service after installation is done
agriyakhetarpal ffd71ce
Start service + Playwright tests in same terminal
agriyakhetarpal 80687cd
Fix path to sharing service
agriyakhetarpal File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.