#7190 - Renew tokenMultimedia #2272

jsegarra · 2024-04-09T11:19:24Z

jsegarra commented

2024-04-09 11:19:24 +00:00

Aprovechamos la PR para subir el uso del tokenMultimedia en paralelo con los otros sistemas dependientes

jsegarra changed title from ~~#7190 - Renew tokenMultimedia~~ to WIP: #7190 - Renew tokenMultimedia

2024-04-09 11:22:50 +00:00

jsegarra added 2 commits 2024-04-15 06:48:41 +00:00

253465cf13 refs #7190 feat: renewToken for multimedia

gitea/salix/pipeline/pr-dev This commit looks good Details

25b5c35dc6 refs #6835 fix: issue

jsegarra changed title from ~~WIP: #7190 - Renew tokenMultimedia~~ to #7190 - Renew tokenMultimedia

2024-04-15 06:50:44 +00:00

jsegarra added 1 commit 2024-04-15 06:51:06 +00:00

gitea/salix/pipeline/pr-dev This commit looks good Details

6db0eba72c Merge branch 'dev' into 7190_renewTokenMultimedia

jgallego approved these changes 2024-04-15 07:06:01 +00:00

jsegarra requested review from juan 2024-04-15 07:28:44 +00:00

jsegarra requested review from alexm 2024-04-15 07:28:44 +00:00

alexm requested changes 2024-04-16 05:48:47 +00:00

Dismissed

back/methods/vn-user/specs/share-token.spec.js

						
				@ -26,0 +36,4 @@

				        expect(multimediaToken.multimediaToken.scopes[0]).toEqual(TOKEN_MULTIMEDIA);

				    });

				    it('NOT should renew', async() => {

alexm commented

2024-04-16 05:47:34 +00:00

Tal cual está este test en renew-token , hace falta repetirlo?

Tal cual está este test en [renew-token](https://gitea.verdnatura.es/verdnatura/salix/src/commit/ba94bcdb1998d6b3f9e5d837d4f7fbbd6f8921b3/back/methods/vn-user/specs/renew-token.spec.js#L38) , hace falta repetirlo?

jsegarra commented

2024-04-16 05:58:10 +00:00

Lo sé, pero

Al ctx hay que pasarle el tokenMultimedia y no el token normal, entonces habria que hacer mas código
El test es para shareToken, entonces si delego la funcionalidad de shareToken en renewToken, puede quedar confuso y da la sensacion que para sharetoken no se hace la prueba de cuando no debe renovar el token

Entonces, preferí dejarlo así, se duplica pero queda aislada la funcionalidad de testear

Lo sé, pero 1. Al ctx hay que pasarle el tokenMultimedia y no el token normal, entonces habria que hacer mas código 2. El test es para shareToken, entonces si delego la funcionalidad de shareToken en renewToken, puede quedar confuso y da la sensacion que para sharetoken no se hace la prueba de cuando no debe renovar el token Entonces, preferí dejarlo así, se duplica pero queda aislada la funcionalidad de testear

👍 1