Skip to content

Commit

Permalink
improve test cases
Browse files Browse the repository at this point in the history
  • Loading branch information
michaeloffner committed Jul 13, 2024
1 parent f4bb3b2 commit 2d931f5
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
1 change: 0 additions & 1 deletion tests/functions/S3Download.cfc
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ Sorglos";
it(title="download as binary", body = function( currentSpec ) {
var cred=currentSpec.cred;
var bucketName=currentSpec.bucketName;
createBucket(cred,bucketName);
var data=s3Download(bucket:bucketName,object:objectName,accessKeyId:cred.ACCESS_KEY_ID,secretAccessKey:cred.SECRET_KEY);
assertTrue(isBinary(data));
assertEquals(len(data),12);
Expand Down
3 changes: 2 additions & 1 deletion tests/general/BigBucket.cfc
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ component extends="org.lucee.cfml.test.LuceeTestCase" labels="s3" {
accessKeyId:cred.ACCESS_KEY_ID, secretAccessKey:cred.SECRET_KEY, host:(isNull(cred.HOST)?nullvalue():cred.HOST));
}
}
arguments.spec.body({bucketName:bucketName,cred:cred});
arguments.spec.body({bucketName:bucketName,cred:cred,records:records});
}
finally {
Util::deleteBucketEL(cred,bucketName);
Expand All @@ -46,6 +46,7 @@ component extends="org.lucee.cfml.test.LuceeTestCase" labels="s3" {
it(title="checking with S3ListBucket", skip=Util::isAWSNotSupported(), body = function( currentSpec ) {
var cred=currentSpec.cred;
var bucketName=currentSpec.bucketName;
var records=currentSpec.records;
var cred=Util::getAWSCredentials()

if(records==0){
Expand Down
3 changes: 1 addition & 2 deletions tests/general/Region.cfc
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,7 @@ component extends="org.lucee.cfml.test.LuceeTestCase" labels="s3" {
,secretAccessKey:cred.SECRET_KEY
);

// if it has a dot it should not start with http://bundle.download.s3.amazonaws.com
assertEquals("http://s3.amazonaws.com/bundle.download", res);

});
});
}
Expand Down

0 comments on commit 2d931f5

Please sign in to comment.