Skip to content

chore: yarn upgrade, node lts, lint warnings, code smells#3461

Draft
janhenrikoverland wants to merge 11 commits intomasterfrom
yarn-upgrade-and-node-lts
Draft

chore: yarn upgrade, node lts, lint warnings, code smells#3461
janhenrikoverland wants to merge 11 commits intomasterfrom
yarn-upgrade-and-node-lts

Conversation

@janhenrikoverland
Copy link
Copy Markdown
Collaborator

@janhenrikoverland janhenrikoverland commented Mar 23, 2026

Spring cleaning:

  • yarn upgrade
  • node lts
  • fix lint warnings
  • fix code smells
Screenshot_2026-03-25_14-44-13 Screenshot_2026-03-25_15-17-09

@dhis2-bot
Copy link
Copy Markdown
Contributor

dhis2-bot commented Mar 23, 2026

🚀 Deployed on https://pr-3461.data-visualizer.netlify.dhis2.org

@dhis2-bot dhis2-bot temporarily deployed to netlify March 23, 2026 11:07 Inactive
@janhenrikoverland janhenrikoverland changed the title Yarn upgrade and node lts chore: yarn upgrade and node lts Mar 23, 2026
@dhis2-bot dhis2-bot temporarily deployed to netlify March 23, 2026 14:54 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify March 25, 2026 13:04 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify March 25, 2026 13:30 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify March 25, 2026 14:14 Inactive
@janhenrikoverland janhenrikoverland changed the title chore: yarn upgrade and node lts chore: yarn upgrade, node lts, lint warnings, code smells Mar 25, 2026
@dhis2-bot dhis2-bot temporarily deployed to netlify March 25, 2026 14:29 Inactive
@sonarqubecloud
Copy link
Copy Markdown

@janhenrikoverland janhenrikoverland added the e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud label Mar 25, 2026
@cypress
Copy link
Copy Markdown

cypress bot commented Mar 25, 2026

data-visualizer-app    Run #3141

Run Properties:  status check failed Failed #3141  •  git commit a9e86c3a7b ℹ️: Merge 32693e971dce1483113abb523a65d80dd9fd3c6e into 8f45e41bd5260b82ee4d5b156147...
Project data-visualizer-app
Branch Review yarn-upgrade-and-node-lts
Run status status check failed Failed #3141
Run duration 11m 42s
Commit git commit a9e86c3a7b ℹ️: Merge 32693e971dce1483113abb523a65d80dd9fd3c6e into 8f45e41bd5260b82ee4d5b156147...
Committer Jan Henrik Øverland
View all properties for this run ↗︎

Test results
Tests that failed  Failures 27
Tests that were flaky  Flaky 2
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 716
View all changes introduced in this branch ↗︎

Tests for review

Failed  options/legend.cy.js • 1 failed test • e2e-chrome-parallel-42-3

View Output Video

Test Artifacts
Options - Legend > applies different styles of legend to a Single Value chart Test Replay Screenshots Video
Failed  dimensions/orgUnit.cy.js • 6 failed tests • e2e-chrome-parallel-40-1

View Output Video

Test Artifacts
Org unit dimension > selects a level - District Test Replay Screenshots Video
Org unit dimension > deselects District Test Replay Screenshots Video
Org unit dimension > selects a group - Eastern Area Test Replay Screenshots Video
Org unit dimension > deselects Eastern Area Test Replay Screenshots Video
Org unit dimension > selects a user org unit - 'User sub-units' Test Replay Screenshots Video
Org unit dimension > deselects 'User sub-units' Test Replay Screenshots Video
Failed  visTypes/scatter.cy.js • 1 failed test • e2e-chrome-parallel-40-1

View Output Video

Test Artifacts
Scatter chart > works correctly (create, save, load, swap etc) Test Replay Screenshots Video
Failed  options/legend.cy.js • 1 failed test • e2e-chrome-parallel-41-3

View Output Video

Test Artifacts
Options - Legend > applies different styles of legend to a Single Value chart Test Replay Screenshots Video
Failed  options/axes.cy.js • 1 failed test • e2e-chrome-parallel-40-4

View Output Video

Test Artifacts
Options - Auto-generated axis title > Multi items - single axis > config has vertical axis title "2 items and default black color Test Replay Screenshots Video

The first 5 failed specs are shown, see all 12 specs in Cypress Cloud.

Flakiness  dimensions/data.cy.js • 1 flaky test • e2e-chrome-parallel-40-4

View Output Video

Test Artifacts
Data dimension > displays Data elements correctly Test Replay Screenshots Video
Flakiness  drillDown.cy.js • 1 flaky test • e2e-chrome-parallel-41-4

View Output Video

Test Artifacts
Drill-down: Pivot table > navigates to the start page, changes vis type to Pivot table and adds a data item Test Replay Screenshots Video

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e record Apply this label to a pull request to trigger recording of E2E tests on Cypress Cloud

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants