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

Adaptación del componente de lista de problemas #4843

Merged
merged 12 commits into from Oct 24, 2020

Conversation

anscharivs
Copy link
Contributor

@anscharivs anscharivs commented Oct 21, 2020

Descripción

Adaptación del componente List.vue para mostrar problemas de calidad en CollectionList.vue.

image

Fixes: #4808

Checklist:

  • El código sigue la guía de
    estilo
    de
    omegaUp.
  • Se corrieron todas las pruebas y pasaron.
  • Si se está agregando funcionalidad nueva, se agregaron pruebas.
  • Si el cambio es grande (> 200 líneas), hay que intentar partirlo en
    varios pull requests. De preferencia uno para los controladores + phpunit
    y luego otro para la interfaz.

@codecov
Copy link

codecov bot commented Oct 21, 2020

Codecov Report

Merging #4843 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #4843   +/-   ##
=========================================
  Coverage     63.16%   63.16%           
  Complexity     5962     5962           
=========================================
  Files           361      361           
  Lines         27601    27601           
  Branches        740      740           
=========================================
  Hits          17435    17435           
  Misses        10166    10166           
Flag Coverage Δ Complexity Δ
#javascript 30.82% <100.00%> (ø) 0.00 <0.00> (ø)
#php 67.69% <100.00%> (ø) 5962.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
frontend/www/js/omegaup/api_types.ts 8.70% <ø> (ø) 0.00 <0.00> (ø)
frontend/server/src/Controllers/Problem.php 70.59% <100.00%> (ø) 580.00 <0.00> (ø)
...end/www/js/omegaup/components/problem/BaseList.vue 100.00% <100.00%> (ø) 0.00 <0.00> (?)
...w/js/omegaup/components/problem/CollectionList.vue 100.00% <100.00%> (ø) 0.00 <0.00> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b2ebd4...48ab18b. Read the comment docs.

@anscharivs anscharivs marked this pull request as ready for review October 21, 2020 18:22
@anscharivs
Copy link
Contributor Author

@tvanessa ¿El buscador y filtro por idioma de los problemas debe aparecer en la nueva página también?

@tvanessa
Copy link
Member

@tvanessa ¿El buscador y filtro por idioma de los problemas debe aparecer en la nueva página también?

No, los checkboxes y radiobuttons son suficientes por ahora.

frontend/www/js/omegaup/components/problem/List.vue Outdated Show resolved Hide resolved
frontend/www/js/omegaup/problem/collection_list.ts Outdated Show resolved Hide resolved
frontend/www/js/omegaup/problem/list.ts Outdated Show resolved Hide resolved
webpack.config-frontend.js Outdated Show resolved Hide resolved
frontend/www/js/omegaup/components/problem/List.vue Outdated Show resolved Hide resolved
frontend/www/js/omegaup/components/problem/ListSearch.vue Outdated Show resolved Hide resolved
frontend/www/js/omegaup/problem/collection_list.ts Outdated Show resolved Hide resolved
frontend/www/js/omegaup/problem/collection_list.ts Outdated Show resolved Hide resolved
@anscharivs
Copy link
Contributor Author

@lhchavez ¿a qué se debe el error de selenium que me está marcando? nunca lo había visto creo

@pabo99
Copy link
Collaborator

pabo99 commented Oct 23, 2020

@lhchavez ¿a qué se debe el error de selenium que me está marcando? nunca lo había visto creo

La razón por la cual están fallando las pruebas es porque en ellas se espera seleccionar un problema del componente problem-search-bar, pero ya no se logra encontrar los elementos debido a que en el nuevo diseño fue eliminada esa parte:


    search_box_element = driver.wait.until(
        EC.visibility_of_element_located(
            (By.CSS_SELECTOR,
             'input.tt-input[name="query"]')))

Aquí habrá que definir cuales son los pasos para seleccionar un problema. Una vez definidos, plasmarlos en la prueba test_create_problem del archivo test_smoke.py.

Yo te puedo echar la mano a modificarla :)

@anscharivs
Copy link
Contributor Author

@lhchavez ¿a qué se debe el error de selenium que me está marcando? nunca lo había visto creo

La razón por la cual están fallando las pruebas es porque en ellas se espera seleccionar un problema del componente problem-search-bar, pero ya no se logra encontrar los elementos debido a que en el nuevo diseño fue eliminada esa parte:


    search_box_element = driver.wait.until(
        EC.visibility_of_element_located(
            (By.CSS_SELECTOR,
             'input.tt-input[name="query"]')))

Aquí habrá que definir cuales son los pasos para seleccionar un problema. Una vez definidos, plasmarlos en la prueba test_create_problem del archivo test_smoke.py.

Yo te puedo echar la mano a modificarla :)

Por favor :( estas pruebas nunca las había tocado jajaja

@pabo99
Copy link
Collaborator

pabo99 commented Oct 23, 2020

Ok, Por lo pronto, para que te funcione la prueba hay que quitar el siguiente bloque:

search_box_element = driver.wait.until(
EC.visibility_of_element_located(
(By.CSS_SELECTOR,
'input.tt-input[name="query"]')))
search_box_element.send_keys(problem_alias)
with driver.page_transition():
search_box_element.submit()

Lo único que hace ese bloque es buscar el problema al escribir el problem alias en la caja de texto para que siempre aparezca en el listado de problemas.

Esto te garantiza que se va a encontrar el problema en una versión local y teniendo un listado relativo corto de problemas.

Lo que faltará es ver si con lo que se implementará hay alguna nueva forma de filtrar problemas, para garantizar que siempre funcione la prueba.

Copy link
Member

@lhchavez lhchavez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

creo que esto ya debería ser lo último!

@lhchavez lhchavez merged commit 5931834 into omegaup:master Oct 24, 2020
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

Successfully merging this pull request may close these issues.

[FEATURE] Listado de problemas en la nueva página de colecciones de problemas
4 participants