Skip to content

Commit

Permalink
fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
blampe authored and pulumi-renovate[bot] committed Dec 7, 2024
1 parent 1789a6c commit e511a49
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
6 changes: 3 additions & 3 deletions aws-ts-pulumi-webhooks/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,15 +20,15 @@ const stackConfig = {
};

// Just logs information from an incoming webhook request.
function logRequest(req: awsx.apigateway.Request) {
function logRequest(req: awsx.classic.apigateway.Request) {
const webhookID = req.headers !== undefined ? req.headers["pulumi-webhook-id"] : "";
const webhookKind = req.headers !== undefined ? req.headers["pulumi-webhook-kind"] : "";
console.log(`Received webhook from Pulumi ${webhookID} [${webhookKind}]`);
}

// Webhooks can optionally be configured with a shared secret, so that webhook handlers like this app can authenticate
// message integrity. Rejects any incoming requests that don't have a valid "pulumi-webhook-signature" header.
function authenticateRequest(req: awsx.apigateway.Request): awsx.apigateway.Response | undefined {
function authenticateRequest(req: awsx.classic.apigateway.Request): awsx.classic.apigateway.Response | undefined {
const webhookSig = req.headers !== undefined ? req.headers["pulumi-webhook-signature"] : "";
if (!stackConfig.sharedSecret || !webhookSig) {
return undefined;
Expand All @@ -50,7 +50,7 @@ function authenticateRequest(req: awsx.apigateway.Request): awsx.apigateway.Resp
// unsecret the webhook so we can add it to the handler
(<any>stackConfig.slackWebhook).isSecret = false;

const webhookHandler = new awsx.apigateway.API("pulumi-webhook-handler", {
const webhookHandler = new awsx.classic.apigateway.API("pulumi-webhook-handler", {
restApiArgs: {
binaryMediaTypes: ["application/json"],
},
Expand Down
2 changes: 1 addition & 1 deletion aws-ts-thumbnailer/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ const repo = new awsx.ecr.Repository("repo", {
// A task which runs a containerized FFMPEG job to extract a thumbnail image.
const ffmpegThumbnailTask = new awsx.classic.ecs.FargateTaskDefinition("ffmpegThumbTask", {
container: {
image: new awsx.ecr.Image("ffmpegThumbTask", { repositoryUrl: repo.url, path: "./docker-ffmpeg-thumb" }).imageUri,
image: new awsx.ecr.Image("ffmpegThumbTask", { repositoryUrl: repo.url, dockerfile: "./docker-ffmpeg-thumb/Dockerfile" }).imageUri,
memoryReservation: 512,
},
});
Expand Down
1 change: 1 addition & 0 deletions misc/test/aws_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -566,6 +566,7 @@ func TestAccAwsTsTwitterAthena(t *testing.T) {
}

func TestAccAwsTsLambdaEfs(t *testing.T) {
t.Skip("times out https://github.com/pulumi/examples/issues/1728")
test := getAWSBase(t).
With(integration.ProgramTestOptions{
Dir: path.Join(getCwd(t), "..", "..", "aws-ts-lambda-efs"),
Expand Down

0 comments on commit e511a49

Please sign in to comment.