Material UI: Study programme overview - Update statistics tab #453
rikurauhalapull_request.yaml
on: pull_request
Lint backend and frontend
1m 30s
Matrix: test_end_to_end
Annotations
2 errors and 10 warnings
|
Test population statistics
Cypress tests: 1 failed
|
|
Test study programme overview
Cypress tests: 1 failed
|
|
Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/FacultiesTab/index.jsx#L39
React Hook useMemo has a missing dependency: 'selectedSemesters'. Either include it or remove the dependency array
|
|
Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/FacultiesTab/index.jsx#L65
React Hook useMemo has a missing dependency: 'totalRow'. Either include it or remove the dependency array
|
|
Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/index.jsx#L51
React Hook useEffect has a missing dependency: 'semesterFilter'. Either include it or remove the dependency array
|
|
Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/selectorComponents.jsx#L76
React Hook useMemo has a missing dependency: 'getTextIn'. Either include it or remove the dependency array
|
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L37
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L44
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L51
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L84
React Hook useEffect has missing dependencies: 'parseQueryFromUrl' and 'setValues'. Either include them or remove the dependency array. If 'setValues' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/OpenUniPopulationResults/index.jsx#L232
React Hook useEffect has missing dependencies: 'getTextIn', 'isError', and 'isFetchingOrLoading'. Either include them or remove the dependency array
|
|
Lint backend and frontend:
services/frontend/src/components/CustomPopulation/index.jsx#L44
The 'allSemesters' logical expression could make the dependencies of useMemo Hook (at line 84) change on every render. Move it inside the useMemo callback. Alternatively, wrap the initialization of 'allSemesters' in its own useMemo() Hook
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
Cypress videos - Test population statistics
|
27.7 MB |
|
|
Cypress videos - Test study programme overview
|
29.9 MB |
|