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

test: add tests for error when renaming a saved visualization (DHIS2-16154) #455

Merged
merged 8 commits into from
Dec 8, 2023

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented Nov 6, 2023

Fixes DHIS2-16154

Requires dhis2/analytics#1593


Key features

  1. bump analytics dep with the actual fix
  2. add some Cypress tests for File Menu rename

Description

The rename feature stopped working when support for deprecated PATCH request format was removed.
json+patch format is required for PATCH requests now.
The implementation for File Menu rename and update of custom calculations is in analytics (see linked PR).

This PR adds some tests for the File Menu rename feature.


TODO

  • Cypress tests
  • Update docs
  • KFMT
  • update dep once analytics PR 1593 is merged

@edoardo edoardo requested a review from a team as a code owner November 6, 2023 13:40
@dhis2-bot
Copy link
Contributor

dhis2-bot commented Nov 6, 2023

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

@dhis2-bot dhis2-bot temporarily deployed to netlify November 6, 2023 13:43 Inactive
Copy link

cypress bot commented Nov 6, 2023

Passing run #2142 ↗︎

0 520 0 0 Flakiness 0

Details:

Merge 086e52a into 41b04d5...
Project: line-listing-app Commit: a69176fef3 ℹ️
Status: Passed Duration: 13:11 💡
Started: Dec 7, 2023 1:42 PM Ended: Dec 7, 2023 1:55 PM

Review all test suite changes for PR #455 ↗︎

@dhis2-bot dhis2-bot temporarily deployed to netlify November 7, 2023 09:42 Inactive
@edoardo edoardo changed the title fix: add Cypress tests for rename patch fix fix: fix visualization rename after deprecation of PATCH request payload format Nov 13, 2023
@dhis2-bot dhis2-bot temporarily deployed to netlify November 16, 2023 13:05 Inactive
@edoardo edoardo force-pushed the fix/rename-patch-tests branch from 1f1c1b3 to 0ff8cd2 Compare November 29, 2023 14:55
@dhis2-bot dhis2-bot temporarily deployed to netlify November 29, 2023 14:58 Inactive
@martinkrulltott martinkrulltott changed the title fix: fix visualization rename after deprecation of PATCH request payload format fix: visualization rename after deprecation of PATCH request payload format Nov 30, 2023
@janhenrikoverland janhenrikoverland changed the title fix: visualization rename after deprecation of PATCH request payload format fix: renaming saved visualizations throws an error (DHIS2-16154) Nov 30, 2023
@dhis2-bot dhis2-bot temporarily deployed to netlify November 30, 2023 09:17 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify November 30, 2023 10:36 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify November 30, 2023 10:51 Inactive
@edoardo edoardo force-pushed the fix/rename-patch-tests branch from 8f6b636 to bb38eb7 Compare November 30, 2023 12:53
@dhis2-bot dhis2-bot temporarily deployed to netlify November 30, 2023 12:57 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify November 30, 2023 13:02 Inactive
@edoardo edoardo force-pushed the fix/rename-patch-tests branch from 8631fee to 32c903e Compare November 30, 2023 13:39
@dhis2-bot dhis2-bot temporarily deployed to netlify November 30, 2023 13:41 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify November 30, 2023 15:06 Inactive
@edoardo edoardo changed the title fix: renaming saved visualizations throws an error (DHIS2-16154) test: add tests for error when renaming a saved visualization (DHIS2-16154) Dec 1, 2023
@edoardo edoardo force-pushed the fix/rename-patch-tests branch from 9c5f3dc to 086e52a Compare December 7, 2023 13:28
@dhis2-bot dhis2-bot temporarily deployed to netlify December 7, 2023 13:30 Inactive
@edoardo edoardo merged commit 1f77798 into master Dec 8, 2023
28 checks passed
@edoardo edoardo deleted the fix/rename-patch-tests branch December 8, 2023 09:41
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.11.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants