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

Add very basic unit tests for S3 and HTTP #27

Merged
merged 5 commits into from
Apr 16, 2024

Conversation

jhiemstrawisc
Copy link
Member

This sets up a single unit test for both the S3 and HTTP plugins. Now that the
framework is in place, any contributions to the plugins should include unit
tests for the new work (and if the developer is generous, a unit test or two for
something that already exists -- we've got quite the backlog!)

This sets up a single unit test for both the S3 and HTTP plugins. Now that the
framework is in place, any contributions to the plugins should include unit
tests for the new work (and if the developer is generous, a unit test or two for
something that already exists -- we've got quite the backlog!)
src/S3Commands.cc Show resolved Hide resolved
test/http_tests.cc Outdated Show resolved Hide resolved
test/http_tests.cc Show resolved Hide resolved
test/http_tests.cc Outdated Show resolved Hide resolved
test/http_tests.cc Outdated Show resolved Hide resolved
test/s3_tests.cc Outdated Show resolved Hide resolved
test/s3_tests.cc Outdated Show resolved Hide resolved
test/s3_tests.cc Outdated Show resolved Hide resolved
test/s3_tests.cc Outdated Show resolved Hide resolved
test/s3_tests.cc Outdated Show resolved Hide resolved
@jhiemstrawisc jhiemstrawisc requested a review from turetske April 16, 2024 19:05
test/http_tests.cc Outdated Show resolved Hide resolved
test/http_tests.cc Show resolved Hide resolved
@jhiemstrawisc jhiemstrawisc removed the request for review from turetske April 16, 2024 19:10
@jhiemstrawisc jhiemstrawisc merged commit 010b9fb into PelicanPlatform:main Apr 16, 2024
2 checks passed
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.

1 participant