Uploaded image for project: 'Nuxeo Platform'
  1. Nuxeo Platform
  2. NXP-27660

Fix TestSQLRepositoryAPI.testLocking on MongoDB

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 11.1, 2021.0
    • Component/s: Core

      Description

      Since NXP-24359, we can now lock twice a document without error.

      The test TestSQLRepositoryAPI.testLocking has been updated to reflect this change but it has been failing on MongoDB since, see:

      java.lang.AssertionError: expected:<org.nuxeo.ecm.core.api.Lock@6e009215> but was:<org.nuxeo.ecm.core.api.Lock@364a1b2f>
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failNotEquals(Assert.java:834)
	at org.junit.Assert.assertEquals(Assert.java:118)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at org.nuxeo.ecm.core.TestSQLRepositoryAPI.testLocking(TestSQLRepositoryAPI.java:4369)

      Issue is probably just the lack of equals implementation in Lock and it's not visible on VCS because VCSLockManager has a cache and so we get the same object.

        Attachments

          Issue Links

          is caused by

          Bug - A problem which impairs or prevents the functions of the product. NXP-24359 Document.Lock called twice should not fail

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 day
                  1d