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

Consulta sobre resultado del test #40

Open
diegomeretta opened this issue May 16, 2019 · 4 comments
Open

Consulta sobre resultado del test #40

diegomeretta opened this issue May 16, 2019 · 4 comments
Labels
question Further information is requested

Comments

@diegomeretta
Copy link

diegomeretta commented May 16, 2019

Chicos, gracias por la corrección del repo, ahora andan todos bien menos el mío.
Ya miré los videos que pasaron pero no logro entender donde está el error en el test.

test('El modal de agregar película debería abrirse al hacer click en Agregar', async () => {
    await page.reload();
    const agregarBtn = await page.$('.card-header-actions button:nth-child(1)');
    await page.$eval('.card-header-actions button:nth-child(1)', firstCheck => firstCheck.click());
    const visibility = await isVisible('#modal');
    expect(visibility).toBe(true);
})

usé otras funciones como click("button#addMovieBtn") pero el resultado es el mismo

Lo edito para poder ver bien el código.

@RodrigoJacznik
Copy link
Contributor

Cuando ejecutas el test que tira la consola?

@RodrigoJacznik
Copy link
Contributor

RodrigoJacznik commented May 16, 2019

Ya lo pude revisar mejor.
El test esta casi bien. Pasale el elemento a isVisible en lugar del selector y funciona.

el reload de la primera linea lo haces por algo en particular?

con respecto a estas lineas:

const agregarBtn = await page.$('.card-header-actions button:nth-child(1)');
await page.$eval('.card-header-actions button:nth-child(1)', firstCheck => firstCheck.click());

fijate que el selector '.card-header-actions button:nth-child(1)' es el mismo en los 2 casos. La primera linea no es necesaria ya que no haces nada con el agregarBtn en todo el test. Esto es un detalle para que el test quede 10 puntos.

Saludos.

@RodrigoJacznik RodrigoJacznik added the question Further information is requested label May 16, 2019
@diegomeretta
Copy link
Author

diegomeretta commented May 17, 2019

Simplemente la recargaba para que no quede basura de otro test anterior

test('El modal de agregar película debería abrirse al hacer click en Agregar', async () => {
    await page.$eval('.card-header-actions button:nth-child(1)', firstCheck => firstCheck.click());
    const visibility = await isVisible('.modal-card');
    expect(visibility).toBe(true);
})

Le estoy pasando el nombre del modal (.modal) ó un boton del modal y no funciona

@RodrigoJacznik
Copy link
Contributor

No debería hacer falta recargar la pagina en cada test. Estos son independientes.

Aclaro esto: "Pasale el elemento a isVisible en lugar del selector y funciona":

elemento: lo que retorna page.$()
selector: selector del elemento que usas en page.$();

Por ejemplo en:

const agregarBtn = await page.$('.card-header-actions button:nth-child(1)');

agregarBtn es el elemento y .card-header-actions button:nth-child(1) el selector.

Resumiendo:

isVisible(agregarBtn) funciona correctamente
isVisible('.card-header-actions button:nth-child(1)') no funciona

Saludos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants