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

feat: prefix time dimension ids with program id when tracked entity output type #480

Conversation

jenniferarnesen
Copy link
Collaborator

@jenniferarnesen jenniferarnesen commented Jan 12, 2024

Implements https://dhis2.atlassian.net/browse/DHIS2-16023

Description

Time dimensions need to be prefixed with the programId when using TE output type. The following works:

  • adding time dimensions from different programs to layout - they get suffixed with the program name
  • analytics query works - data shows
  • saving LL with time dimensions works
  • opening a saved LL with time dimensions - layout is empty but table shows correctly (metadata issue to be solved with backend changes)

The failing cypress tests are the same as the target branch

image image

@jenniferarnesen jenniferarnesen requested a review from a team as a code owner January 12, 2024 15:47
@dhis2-bot
Copy link
Contributor

dhis2-bot commented Jan 12, 2024

🚀 Deployed on https://pr-480--dhis2-line-listing.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify January 12, 2024 15:50 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 12, 2024 15:53 Inactive
@jenniferarnesen jenniferarnesen marked this pull request as draft January 12, 2024 15:58
Copy link

cypress bot commented Jan 12, 2024

120 failed tests on run #2406 ↗︎

120 342 0 33 Flakiness 0

Details:

Merge 9d42ee1 into 5f41e28...
Project: line-listing-app Commit: a624af7e4c ℹ️
Status: Failed Duration: 36:41 💡
Started: Jan 23, 2024 9:34 AM Ended: Jan 23, 2024 10:11 AM
Failed  conditions/numericConditions.cy.js • 3 failed tests • e2e-chrome-parallel-2.38

View Output Video

Test Artifacts
integer > integer with negative value Test Replay Screenshots Video
integer > integer with positive value Test Replay Screenshots Video
integer > integer with 0 Test Replay Screenshots Video
Failed  conditions/numericConditions.cy.js • 17 failed tests • e2e-chrome-parallel-2.39

View Output Video

Test Artifacts
number conditions (event) > equal to Test Replay Screenshots Video
integer > integer with negative value Test Replay Screenshots Video
preset options > set only Test Replay Screenshots Video
numeric types > E2E - Number has all operators Test Replay Screenshots Video
numeric types > E2E - Number can be used in a visualization Test Replay Screenshots Video
numeric types > E2E - Unit interval has all operators Test Replay Screenshots Video
numeric types > E2E - Unit interval can be used in a visualization Test Replay Screenshots Video
numeric types > E2E - Percentage has all operators Test Replay Screenshots Video
numeric types > E2E - Percentage can be used in a visualization Test Replay Screenshots Video
numeric types > E2E - Integer has all operators Test Replay Screenshots Video
The first 10 failed tests are shown, see all 17 tests in Cypress Cloud.
Failed  conditions/alphanumericConditions.cy.js • 15 failed tests • e2e-chrome-parallel-2.39

View Output Video

Test Artifacts
text conditions (event) > exactly Test Replay Screenshots Video
alphanumeric types > E2E - Text has all operators Test Replay Screenshots Video
alphanumeric types > E2E - Text can be used in a visualization Test Replay Screenshots Video
alphanumeric types > E2E - Letter has all operators Test Replay Screenshots Video
alphanumeric types > E2E - Letter can be used in a visualization Test Replay Screenshots Video
alphanumeric types > E2E - Long text has all operators Test Replay Screenshots Video
alphanumeric types > E2E - Long text can be used in a visualization Test Replay Screenshots Video
alphanumeric types > E2E - Email has all operators Test Replay Screenshots Video
alphanumeric types > E2E - Email can be used in a visualization Test Replay Screenshots Video
alphanumeric types > E2E - Username has all operators Test Replay Screenshots Video
The first 10 failed tests are shown, see all 15 tests in Cypress Cloud.
Failed  conditions/dateConditions.cy.js • 7 failed tests • e2e-chrome-parallel-2.39

View Output Video

Test Artifacts
date conditions (Date) > exactly Test Replay Screenshots Video
date types > E2E - Date has all operators Test Replay Screenshots Video
date types > E2E - Date can be used in a visualization Test Replay Screenshots Video
date types > E2E - Time has all operators Test Replay Screenshots Video
date types > E2E - Time can be used in a visualization Test Replay Screenshots Video
date types > E2E - Date & Time has all operators Test Replay Screenshots Video
date types > E2E - Date & Time can be used in a visualization Test Replay Screenshots Video
Failed  save.cy.js • 5 failed tests • e2e-chrome-parallel-2.39

View Output Video

Test Artifacts
rename > replace existing name works correctly Test Replay Screenshots Video
rename > add non existing description works correctly Test Replay Screenshots Video
save > new AO with name saves correctly (event) Test Replay Screenshots Video
save > new AO without name saves correctly Test Replay Screenshots Video
save > "save" a copied AO created by others works after editing Test Replay Screenshots Video

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

Review all test suite changes for PR #480 ↗︎

Copy link

cypress bot commented Jan 12, 2024

7 failed tests on run #2410 ↗︎

7 170 13 0 Flakiness 0

Details:

Merge branch 'feat/tracked-entity-input' into feat/timedimensions-with-programid...
Project: line-listing-app Commit: eed6cddafc
Status: Failed Duration: 14:32 💡
Started: Jan 23, 2024 2:25 PM Ended: Jan 23, 2024 2:40 PM
Failed  conditions/numericConditions.cy.js • 3 failed tests • e2e-chrome-parallel-dev

View Output Video

Test Artifacts
integer > integer with negative value Test Replay Screenshots Video
integer > integer with positive value Test Replay Screenshots Video
integer > integer with 0 Test Replay Screenshots Video
Failed  repeatedEvents.cy.js • 2 failed tests • e2e-chrome-parallel-dev

View Output Video

Test Artifacts
repeated events > can use repetition for enrollments Test Replay Screenshots Video
repeated events > repetition out of bounds returns as empty value Test Replay Screenshots Video
Failed  programStatus.cy.js • 1 failed test • e2e-chrome-parallel-dev

View Output Video

Test Artifacts
program status (TE) > can be filtered by status COMPLETED Test Replay Screenshots Video
Failed  createTrackedEntity.cy.js • 1 failed test • e2e-chrome-parallel-dev

View Output Video

Test Artifacts
tracked entity > creates a line list with dimensions Test Replay Screenshots Video

Review all test suite changes for PR #480 ↗︎

@dhis2-bot dhis2-bot temporarily deployed to netlify January 15, 2024 19:03 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 17, 2024 14:06 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 19, 2024 07:54 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 19, 2024 11:25 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 19, 2024 11:53 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 19, 2024 13:15 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 22, 2024 10:21 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 22, 2024 10:28 Inactive
@jenniferarnesen jenniferarnesen marked this pull request as ready for review January 22, 2024 11:19
@dhis2-bot dhis2-bot temporarily deployed to netlify January 22, 2024 11:21 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify January 23, 2024 14:25 Inactive
@jenniferarnesen jenniferarnesen merged commit e0398d0 into feat/tracked-entity-input Jan 24, 2024
12 of 18 checks passed
@jenniferarnesen jenniferarnesen deleted the feat/timedimensions-with-programid-prefix branch January 24, 2024 08:33
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.

3 participants