Skip to content
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

Arreglar los tests del consumer #1205

Closed
2 tasks done
ezequielPereyra opened this issue Dec 16, 2022 · 4 comments
Closed
2 tasks done

Arreglar los tests del consumer #1205

ezequielPereyra opened this issue Dec 16, 2022 · 4 comments
Assignees

Comments

@ezequielPereyra
Copy link
Contributor

ezequielPereyra commented Dec 16, 2022

  • Pareciera que a veces puede tomar la configuración correctamente del .env.test y otras veces no. Hay que investigar la razón para poder solucionarlo y evitar que rompan muchos tests.
  • Luego hay un problemita con el map reduce. Varios tests fallan porque dice que el context es undefined y otros problemas de tipos.
FAIL  test/consumer/persistence/analyzer.spec.ts
  ● Analyzer › should group by challenge and user

    MongoError: MapReduce internal error :: caused by :: TypeError: context is undefined :
    @:3:9

      at MessageStream.messageHandler (node_modules/mongodb/lib/cmap/connection.js:299:20)
      at processIncomingData (node_modules/mongodb/lib/cmap/message_stream.js:144:12)
      at MessageStream._write (node_modules/mongodb/lib/cmap/message_stream.js:42:5)

  ● Analyzer › should group by challenge and user

    TypeError: Cannot read property 'length' of undefined

      37 |     test('collection', async () => {
      38 |       const collection = 'TEST'
    > 39 |       const response = await analyzer.experiences({ collection })
         |                                                                  ^
      40 |       expect(response).toHaveProperty('collection')
      41 |       const count = await analyzer.experiencesCount(collection)
      42 |       expect(count).toBeGreaterThan(0)

      at Object.<anonymous>.__spreadArray (test/consumer/persistence/analyzer.spec.ts:39:66)
      at sorted (test/consumer/persistence/analyzer.spec.ts:21:28)
      at Object.<anonymous> (test/consumer/persistence/analyzer.spec.ts:22:12)

@asanzo asanzo transferred this issue from Program-AR/pilas-bloques-backend Dec 16, 2022
@asanzo
Copy link
Contributor

asanzo commented Dec 16, 2022

Lo pasé a pilas-bloques, carguemos todo ahí por simplicidad.

@dlopezalvas dlopezalvas self-assigned this Dec 26, 2022
@dlopezalvas
Copy link
Contributor

El .env.test lo está tomando bien, el tema es que por alguna razón está corriendo los tests de api (usando el .env.test del consumer porque eso le dijimos que haga en el script de los tests del consumer), como en el consumer no existe API_DB_CONNECTION_URI por ejemplo, explotan varios. No llegué encontrar la razón por la que corre también los tests de api (y a veces los de analytics).

@asanzo
Copy link
Contributor

asanzo commented Jan 5, 2023

Bueno, sobre que corra a veces tests de analytics y de api es porque en el comando de los tests faltaba la carpeta de parámetro para jest. Sigo viendo.

@asanzo
Copy link
Contributor

asanzo commented Jan 6, 2023

El problema era que se estaba seedeando mal la colección de solutions. En lugar de crear 5 o 6 soluciones, creaba un objeto único que tenía un atributo "solutions". Entonces luego los find que tiraba no encontraban objetos, porque el único que había no cumplía el filtro.

@asanzo asanzo closed this as completed Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants