Skip to content

Commit 13b030a

Browse files
authored
test: align shared OIDC mock helper callback naming
1 parent 56010d3 commit 13b030a

4 files changed

Lines changed: 6 additions & 6 deletions

File tree

containers/api-proxy/aws-oidc-token-provider.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const { AwsOidcTokenProvider } = require('./aws-oidc-token-provider');
55
const { createBaseMockServer } = require('./test-helpers/mock-oidc-server');
66

77
function createMockServer(handlers = {}) {
8-
return createBaseMockServer((url, req, res, body, routeHandlers) => {
8+
return createBaseMockServer((url, req, res, routeHandlers) => {
99
if (url.pathname === '/' && url.searchParams.get('Action') === 'AssumeRoleWithWebIdentity') {
1010
const handler = routeHandlers.stsAssume || (() => ({
1111
statusCode: 200,

containers/api-proxy/gcp-oidc-token-provider.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const { GcpOidcTokenProvider } = require('./gcp-oidc-token-provider');
55
const { createBaseMockServer } = require('./test-helpers/mock-oidc-server');
66

77
function createMockServer(handlers = {}) {
8-
return createBaseMockServer((url, req, res, body, routeHandlers) => {
8+
return createBaseMockServer((url, req, res, routeHandlers, body) => {
99
if (url.pathname === '/v1/token' && req.method === 'POST') {
1010
const handler = routeHandlers.stsToken || (() => ({
1111
statusCode: 200,

containers/api-proxy/oidc-token-provider.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const { createBaseMockServer } = require('./test-helpers/mock-oidc-server');
77

88
// Helper to create a mock OIDC server with Azure token exchange support
99
function createMockOidcServer(handlers = {}) {
10-
return createBaseMockServer((url, req, res, body, routeHandlers) => {
10+
return createBaseMockServer((url, req, res, routeHandlers, body) => {
1111
if (url.pathname.includes('/oauth2/v2.0/token') && req.method === 'POST') {
1212
const handler = routeHandlers.azureToken || (() => ({
1313
statusCode: 200,

containers/api-proxy/test-helpers/mock-oidc-server.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22

33
const http = require('http');
44

5-
function createBaseMockServer(providerRouteHandler, handlers = {}) {
6-
const routeHandler = providerRouteHandler || (() => false);
5+
function createBaseMockServer(handleProviderRoute, handlers = {}) {
6+
const routeHandler = handleProviderRoute || (() => false);
77

88
return http.createServer((req, res) => {
99
let body = '';
@@ -22,7 +22,7 @@ function createBaseMockServer(providerRouteHandler, handlers = {}) {
2222
return;
2323
}
2424

25-
const handled = routeHandler(url, req, res, body, handlers);
25+
const handled = routeHandler(url, req, res, handlers, body);
2626
if (handled) {
2727
return;
2828
}

0 commit comments

Comments
 (0)