Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-3457

Lack of MutationResult from KV remove causing FIT failures

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • None
    • None
    • None
    • None
    • 1
    • SDK34

    Description

      assertFalse(result.hasException()) of .NET FIT Performer is throwing error but the same is passing in Java.

      Below are the failures from jenkins: 

      https://sdk.jenkins.couchbase.com/job/dotnet/job/sdk/job/TransactionsFITPerformer_SinglePerformer/1297/testReport/junit/com.couchbase.client.kv/RemoveTest/Setup_test_Infra_and_Execute_Tests___6_6_release_Vs_DOTNET_MASTER___basic/ https://sdk.jenkins.couchbase.com/job/dotnet/job/sdk/job/TransactionsFITPerformer_SinglePerformer/1297/testReport/junit/com.couchbase.client.kv/RemoveTest/Setup_test_Infra_and_Execute_Tests___7_0_release_Vs_DOTNET_MASTER___basic/ https://sdk.jenkins.couchbase.com/job/dotnet/job/sdk/job/TransactionsFITPerformer_SinglePerformer/1297/testReport/junit/com.couchbase.client.kv/RemoveTest/Setup_test_Infra_and_Execute_Tests___7_1_stable_Vs_DOTNET_MASTER___basic/ https://sdk.jenkins.couchbase.com/job/dotnet/job/sdk/job/TransactionsFITPerformer_SinglePerformer/1297/testReport/junit/com.couchbase.client.kv/RemoveTest/Setup_test_Infra_and_Execute_Tests___7_2_stable_Vs_DOTNET_MASTER___basic/

       

      Below is the error message:

       

      com.couchbase.client.kv.RemoveTest.basic expected: <false> but was: <true

       

      Below is the test case that is failing:

      @TestFactory
      Stream<DynamicTest> basic() {
          return shared.firstPerformer().supportedApis().stream().map(api -> DynamicTest.dynamicTest(api.toString(), () -> {
              var id = UUID.randomUUID().toString();
              collection.insert(id, initial);
       
              var result = execute(shared, command(remove(collection, id), api));
       
              assertFalse(result.hasException());
       
              // Dotnet currently does not return a mutation result for remove. When this is added this will be modified to check dotnet version
              if (!shared.firstPerformer().sdk().equals("dotnet")) {
                  assertTrue(result.hasMutationResult());
                  long cas = result.getMutationResult().getCas();
                  assertTrue(cas != 0);
                  assertTrue(result.getMutationResult().hasMutationToken());
                  assertThrows(DocumentNotFoundException.class, () ->
                          collection.get(id)
                  );
              }
          }));
      } 

      Attaching docker and IntelliJ logs for your reference. Please feel free to reach out for any further assistance.

      Thanks!

       

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              richard.ponton Richard Ponton
              aaliya.haque Aaliya Haque
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty