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

        1. docker_container_log.txt
          3 kB
          Aaliya Haque
        2. Intellij_log.txt
          8 kB
          Aaliya Haque

        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