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

Objects are being persisted with a modified timestamp > persisted timestamp #5919

Closed
2 of 7 tasks
akhenry opened this issue Oct 27, 2022 · 2 comments
Closed
2 of 7 tasks

Comments

@akhenry
Copy link
Contributor

akhenry commented Oct 27, 2022

Summary

Reported in this comment - #5915 (comment)

Expected vs Current Behavior

Expected: The persisted timestamp on persisted objects should always be later than or equal to the modified timestamp.
Current: Objects are being persisted with a modified timestamp that is later than the persisted timestamp.

Steps to Reproduce

In an Open MCT instance with Couch DB configured (either locally, or in the VIPER test environment):

  1. Open Chrome dev tools and open the network tab
  2. Clear the current content of the network tab (if not already empty)
  3. From the create menu, create a new object of any kind
  4. In the network tab in Chrome dev tools find the POST request corresponding to the new object
  5. Inspect the payload of the request and verify that the persisted timestamp is >= the modified timestamp.

Environment

  • Open MCT Version: 8e3146e
  • Deployment Type: Open w/ Couch DB

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

@akhenry
Copy link
Contributor Author

akhenry commented Oct 27, 2022

Testing instructions

To verify Object API fix

Follow reproduction steps above to verify that persisted timestamp is always >= modified timestamp.

To test for regressions in Notebooks

This PR also includes changes to the notebook, so please check for regressions in notebooks.

  1. Verify that notebooks can be created
  2. Verify that sections and pages can be added to, and deleted from notebooks.
  3. Verify that entries can be added to, and deleted from notebooks

@ozyx
Copy link
Member

ozyx commented Oct 27, 2022

Verified fixed -- Testathon 10/27/22

  • Verified that created objects have persisted >= modified

Regression tested notebooks. Was able to add/remove/rename sections and pages. Spammed a bunch of entries in multiple notebook pages. No network or console errors were observed.

Clicked through a bunch of payloads from these actions and persisted was always >= modified

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

No branches or pull requests

4 participants