-
Notifications
You must be signed in to change notification settings - Fork 252
Improvement/cldsrv 724 bucket related functional tests #5976
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
Improvement/cldsrv 724 bucket related functional tests #5976
Conversation
tests/functional/aws-node-sdk/test/bucket/aclUsingPredefinedGroups.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/aclUsingPredefinedGroups.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/aclUsingPredefinedGroups.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/bucketPolicyWithResourceStatements.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/bucketPolicyWithResourceStatements.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/putBucketReplication.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/putBucketReplication.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/putBucketReplication.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/putBucketObjectLock.js
Outdated
Show resolved
Hide resolved
tests/functional/aws-node-sdk/test/bucket/putBucketObjectLock.js
Outdated
Show resolved
Hide resolved
| after(async () => { | ||
| await s3ObjectLock.send(new DeleteBucketCommand({ Bucket: bucket })); | ||
| }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| after(async () => { | |
| await s3ObjectLock.send(new DeleteBucketCommand({ Bucket: bucket })); | |
| }); | |
| after(() => s3ObjectLock.send(new DeleteBucketCommand({ Bucket: bucket }))); |
tests/functional/aws-node-sdk/test/bucket/testBucketVersioning.js
Outdated
Show resolved
Hide resolved
| // Ensure host includes port for canonical request | ||
| const hostname = host.split(':')[0]; // Extract 127.0.0.1 | ||
| const port = parseInt(host.split(':')[1] || '8000', 10); // Default to 8000 | ||
| const [pathBase, queryString] = path.split('?'); | ||
| const query = queryString ? Object.fromEntries(new URLSearchParams(queryString)) : {}; | ||
|
|
||
| // Create HTTP request (mimics AWS.HttpRequest with v2-like endpoint structure) | ||
| const request = new HttpRequest({ | ||
| protocol: 'http:', // Match Scality CloudServer | ||
| hostname, // 127.0.0.1 | ||
| port, // 8000 | ||
| method: method.toUpperCase(), | ||
| path: pathBase, | ||
| query, | ||
| body, | ||
| headers: { | ||
| Host: host, // Explicitly set Host: 127.0.0.1:8000 | ||
| 'X-Amz-Date': signingDate.toISOString().replace(/[:\-]|\.\d{3}/g, ''), | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comments to be removed probably
❌ 119 Tests Failed:
View the top 3 failed test(s) by shortest run time
View the full list of 92 ❄️ flaky test(s)
To view more test analytics, go to the Test Analytics Dashboard |
69b2a4a to
c56ac68
Compare
f6cbcbd to
cc2320a
Compare
No description provided.