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

Pathogen tests associated with environment samples that were deleted and restored are not synchronized in the mobile app with regular synchronization #12853

Closed
adinaflorea9 opened this issue Dec 28, 2023 · 3 comments · Fixed by #12872
Assignees
Labels
android-app Affects the Android app bug An error or misbehavior of an existing feature (ticket type) environment laboratory Samples, pathogen tests and lab messages major A functional requirement is incorrect or incomplete, ... (Severity for bugs/defects) qa-verified Issue has been tested and verified by QA vaadin-app Affects the Vaadin application

Comments

@adinaflorea9
Copy link

Bug Description

If an environment sample that has pathogen test is deleted, cleaned up from the mobile app, and then restored, the pathogen test is not synchronized again in the mobile app with the regular synchronization that brings back the other restored entities, but only by a 'Re-synchronize' data type of synchronization.

Reproduction Steps

  1. Have the following configuration for the clean obsolete data job in Android Studio:
    image
  2. In the web app, create an environment that has a sample and pathogen test associated;
  3. Synchronize them in the mobile app;
  4. From the web app, delete the environment (sample and pathogen test get deleted with the environment);
  5. Do a synchronization in the mobile app (so that the obsolete data is removed);
  6. In the web app, restore the environment and the sample (the pathogen test gets restored with the sample);
  7. Do another synchronization in the mobile app;
  8. Observe the entities that are in the web app and those from the mobile app (that were used in the steps from above).
    Actual behavior: The pathogen test is active and associated with the environment sample in the web app, but in the mobile app it is not synchronized again with the regular synchronization that the other entities were synced with, but only by a 'Re-synchronize data' type of synchronization.

Expected Behavior

The pathogen test should be synchronized after restore in the mobile app with the regular synchronization as well, to be consistent with the way that the other restored entities are synchronized back into the mobile app after delete and restore.

Affected Area(s)

Android app

SORMAS Version

1.94.0; 1.93.0

Android version/Browser

Android Studio

User Role(s)

Admin+NationalUser, Default Surveillance Officer

Server URL

local machine

Additional Information

No response

@adinaflorea9 adinaflorea9 added bug An error or misbehavior of an existing feature (ticket type) android-app Affects the Android app laboratory Samples, pathogen tests and lab messages vaadin-app Affects the Vaadin application major A functional requirement is incorrect or incomplete, ... (Severity for bugs/defects) environment labels Dec 28, 2023
@MateStrysewske
Copy link
Contributor

@adinaflorea9 Does this work correctly with human samples and their pathogen tests?

@MateStrysewske MateStrysewske self-assigned this Jan 9, 2024
@MateStrysewske
Copy link
Contributor

@adinaflorea9 Tested it myself, samples work fine - on it!

@adinaflorea9
Copy link
Author

Verified ticket on the local machine using the latest version of SORMAS from the development branch - 1.94.0-SNAPSHOT(1c5762c).

@adinaflorea9 adinaflorea9 added the qa-verified Issue has been tested and verified by QA label Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android-app Affects the Android app bug An error or misbehavior of an existing feature (ticket type) environment laboratory Samples, pathogen tests and lab messages major A functional requirement is incorrect or incomplete, ... (Severity for bugs/defects) qa-verified Issue has been tested and verified by QA vaadin-app Affects the Vaadin application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants