Skip to content

Commit

Permalink
Stop attempting to override Accept header in [GitHub] API provider (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
PyvesB authored Sep 12, 2021
1 parent 042ae1c commit ca63f21
Show file tree
Hide file tree
Showing 2 changed files with 56 additions and 1 deletion.
1 change: 0 additions & 1 deletion services/github/github-api-provider.js
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,6 @@ class GithubApiProvider {
baseUrl,
headers: {
'User-Agent': userAgent,
Accept: 'application/vnd.github.v3+json',
Authorization: `token ${tokenString}`,
...options.headers,
},
Expand Down
56 changes: 56 additions & 0 deletions services/github/github-auth-service.spec.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,56 @@
import Joi from 'joi'
import { expect } from 'chai'
import sinon from 'sinon'
import { GithubAuthV3Service } from './github-auth-service.js'
import GithubApiProvider from './github-api-provider.js'

describe('GithubAuthV3Service', function () {
class DummyGithubAuthV3Service extends GithubAuthV3Service {
static category = 'build'
static route = { base: 'runs' }

async handle() {
const { requiredString } = await this._requestJson({
schema: Joi.object({
requiredString: Joi.string().required(),
}).required(),
url: 'https://github-api.example.com/repos/badges/shields/check-runs',
options: {
headers: {
Accept: 'application/vnd.github.antiope-preview+json',
},
},
})
return { message: requiredString }
}
}

it('forwards custom Accept header', async function () {
const sendAndCacheRequestWithCallbacks = sinon.stub().returns(
Promise.resolve({
buffer: '{"requiredString": "some-string"}',
res: { statusCode: 200 },
})
)
const githubApiProvider = new GithubApiProvider({
baseUrl: 'https://github-api.example.com',
})
const mockToken = { update: sinon.mock(), invalidate: sinon.mock() }
sinon.stub(githubApiProvider.standardTokens, 'next').returns(mockToken)

DummyGithubAuthV3Service.invoke({
sendAndCacheRequestWithCallbacks,
githubApiProvider,
})

expect(sendAndCacheRequestWithCallbacks).to.have.been.calledOnceWith({
headers: {
'User-Agent': 'Shields.io/2003a',
Accept: 'application/vnd.github.antiope-preview+json',
Authorization: 'token undefined',
},
url: 'https://github-api.example.com/repos/badges/shields/check-runs',
baseUrl: 'https://github-api.example.com',
})
})
})

0 comments on commit ca63f21

Please # to comment.