-
Notifications
You must be signed in to change notification settings - Fork 56
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
[FEATURE] Change l'URL du endpoint Parcoursup vers /api/parcoursup/certification/search?ine={ine}
(PIX-15913).
#10931
[FEATURE] Change l'URL du endpoint Parcoursup vers /api/parcoursup/certification/search?ine={ine}
(PIX-15913).
#10931
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
handler: certificationController.getCertificationResult, | ||
tags: ['api', 'parcoursup'], | ||
notes: [ | ||
'- **Cette route est accessible uniquement à Parcours Sup**\n' + |
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.
'- **Cette route est accessible uniquement à Parcours Sup**\n' + | |
'- **Cette route est accessible uniquement à Parcoursup**\n' + |
context('return 400 error when the param is missing', function () { | ||
let httpTestServer, headers; | ||
beforeEach(async function () { | ||
httpTestServer = new HttpTestServer(); | ||
httpTestServer.setupAuthentication(); | ||
await httpTestServer.register(moduleUnderTest); | ||
|
||
const PARCOURSUP_CLIENT_ID = 'test-parcoursupClientId'; | ||
const PARCOURSUP_SCOPE = 'parcoursup'; | ||
const PARCOURSUP_SOURCE = 'parcoursup'; | ||
|
||
headers = { | ||
authorization: generateValidRequestAuthorizationHeaderForApplication( | ||
PARCOURSUP_CLIENT_ID, | ||
PARCOURSUP_SOURCE, | ||
PARCOURSUP_SCOPE, | ||
), | ||
}; | ||
}); | ||
|
||
it('case of organizationUai', async function () { |
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.
context('return 400 error when the param is missing', function () { | |
let httpTestServer, headers; | |
beforeEach(async function () { | |
httpTestServer = new HttpTestServer(); | |
httpTestServer.setupAuthentication(); | |
await httpTestServer.register(moduleUnderTest); | |
const PARCOURSUP_CLIENT_ID = 'test-parcoursupClientId'; | |
const PARCOURSUP_SCOPE = 'parcoursup'; | |
const PARCOURSUP_SOURCE = 'parcoursup'; | |
headers = { | |
authorization: generateValidRequestAuthorizationHeaderForApplication( | |
PARCOURSUP_CLIENT_ID, | |
PARCOURSUP_SOURCE, | |
PARCOURSUP_SCOPE, | |
), | |
}; | |
}); | |
it('case of organizationUai', async function () { | |
context('when the param is missing', function () { | |
let httpTestServer, headers; | |
beforeEach(async function () { | |
httpTestServer = new HttpTestServer(); | |
httpTestServer.setupAuthentication(); | |
await httpTestServer.register(moduleUnderTest); | |
const PARCOURSUP_CLIENT_ID = 'test-parcoursupClientId'; | |
const PARCOURSUP_SCOPE = 'parcoursup'; | |
const PARCOURSUP_SOURCE = 'parcoursup'; | |
headers = { | |
authorization: generateValidRequestAuthorizationHeaderForApplication( | |
PARCOURSUP_CLIENT_ID, | |
PARCOURSUP_SOURCE, | |
PARCOURSUP_SCOPE, | |
), | |
}; | |
}); | |
it('return 400 error', async function () { |
Co-authored-by: Alexandre Coin <[email protected]> Co-Authored-By: GUL <[email protected]>
Co-authored-by: Alexandre Coin <[email protected]>
cd4a7aa
to
4e02346
Compare
🎄 Problème
Les 2 endpoints Parcoursup ne sont pas cohérents.
🎁 Proposition
Utiliser un endpoint unique de recherche avec des paramètres de query spécifiques aux usages.
🧦 Remarques
🎅 Pour tester
Sur
https://api-pr10931.review.pix.fr/api/documentation
, générer un token valide avec les bonsclient_id
,client_secret
etscope