Skip to content
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

feat: test directories for acc pac with ci #1484

Closed
wants to merge 2 commits into from
Closed

feat: test directories for acc pac with ci #1484

wants to merge 2 commits into from

Conversation

JasonPaulGithub
Copy link
Contributor

@JasonPaulGithub JasonPaulGithub commented Mar 21, 2024

Description

Upload and download on files and folders:
Solve for issue #1468 which covers CI tests for Acc Pac.
More creative tests can come from this, such as detecting if an empty directory has been attempted for upload, double download, sync files etc.

@JasonPaulGithub JasonPaulGithub self-assigned this Mar 21, 2024
@JasonPaulGithub JasonPaulGithub marked this pull request as draft March 21, 2024 14:49
@JasonPaulGithub JasonPaulGithub changed the title feat: test file and upload for acc pac ci test feat: test file and upload test for acc pac ci test Mar 21, 2024
@JasonPaulGithub JasonPaulGithub changed the title feat: test file and upload test for acc pac ci test feat: test directories for acc pac with ci Mar 21, 2024
@JasonPaulGithub JasonPaulGithub changed the title feat: test directories for acc pac with ci feat: test directories for acc pac with ci (blocker) Mar 21, 2024
@JasonPaulGithub JasonPaulGithub changed the title feat: test directories for acc pac with ci (blocker) feat: test directories for acc pac with ci Mar 21, 2024
@JasonPaulGithub JasonPaulGithub added Question Further information is requested Help Needed Extra attention is needed labels Mar 21, 2024
@JasonPaulGithub
Copy link
Contributor Author

JasonPaulGithub commented Mar 25, 2024

@bochaco @joshuef

Facing an issue here on writing a CI level check for the downloads. The download command requires a network address, which is given after the upload has finished. It's a different address each time it's uploaded. I need to somehow feed this into the CI-CLI. Perhaps this why the tests at the moment are ran from 'sn_client/tests/folders_api.rs'.

Issue is blocked for now

@JasonPaulGithub JasonPaulGithub deleted the ci_accpac_file branch April 1, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help Needed Extra attention is needed Infrastructure Question Further information is requested Small Pull request is small Tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant