NCBC-3974 FIT test issues

Motivation
==========
Green tests are best.  This greens them a bit, more coming...

Modification
============
* Fixed small issue in handling CasMismatch during staging inserts,
  where we were not adding it to the staged mutations when we are
  only resolving ambiguity.  This involved allowing an empty
  mutation token in this case.
* Raising proper error (TransactionFailedPostCommit) when a document
  was changed during unstaging.
* Properly encoding the durabilityLevel for transactional queries.
  Also needed to correct the multiple places it was being set
  unnecessarily.

Results
=======
Test fails down to 26 locally for me (from 32-ish) out of 3242.
Change-Id: Id52c89e171ef45b527b485bd655e5e2b29612047

Change-Id: I682ee665fd940a3f00746c33d074094ac351cea9
Reviewed-on: https://review.couchbase.org/c/couchbase-net-client/+/226161
Reviewed-by: Emilien Bevierre <emilien.bevierre@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

Author: davidkelly

src/Couchbase/Client/Transactions/AttemptContext.cs

@@ -792,7 +792,10 @@ await ForwardCompatibility.Check(this,
                                                     "update cas as we are only resolving ambiguity");
                                                 docAlreadyExistsResult =
                                                     docWithMeta.GetPostTransactionResult();
-                                                docAlreadyExistsResult.Cas = docWithMeta!.Cas;
+                                                var stagedMutation =
+                                                    new StagedMutation(docAlreadyExistsResult,
+                                                        content, StagedMutationType.Insert);
+                                                _stagedMutations.Add(stagedMutation);
                                                 return (RepeatAction.NoRepeat,
                                                     RepeatAction.NoRepeat);
                                             }
@@ -1445,6 +1448,7 @@ await RepeatUntilSuccessOrThrow(async () =>
                     .DoNotRollbackAttempt()
                     .Cause(new AttemptExpiredException(this,
                         "Commit expired in HandleDocChangedDuringCommit"))
+                    .RaiseException(TransactionOperationFailedException.FinalError.TransactionFailedPostCommit)
                     .Build();
             }
             try
@@ -2392,12 +2396,28 @@ private async Task<IQueryResult<T>> QueryWrapperLocked<T>(
             }
         }
 
+        private static string DurabilityLevelToString(DurabilityLevel durabilityLevel)
+        {
+            return durabilityLevel switch
+            {
+                DurabilityLevel.None => "NONE",
+                DurabilityLevel.Majority => "MAJORITY",
+                DurabilityLevel.MajorityAndPersistToActive => "MAJORITY_AND_PERSIST_TO_ACTIVE",
+                DurabilityLevel.PersistToMajority => "PERSIST_TO_MAJORITY",
+                _ => durabilityLevel.ToString()
+            };
+        }
+
         private QueryTxData CreateBeginWorkTxData()
         {
-            var state = new TxDataState((long)_overallContext.RemainingUntilExpiration.TotalMilliseconds);
-            var txConfig = new TxDataReportedConfig((long?)_config?.KeyValueTimeout?.TotalMilliseconds ?? 10_000, AtrIds.NumAtrs, _effectiveDurabilityLevel.ToString().ToUpperInvariant());
+            var state =
+                new TxDataState((long)_overallContext.RemainingUntilExpiration.TotalMilliseconds);
+            var txConfig = new TxDataReportedConfig(
+                (long?)_config?.KeyValueTimeout?.TotalMilliseconds ?? 10_000, AtrIds.NumAtrs,
+                DurabilityLevelToString(_effectiveDurabilityLevel));
 
-            var mutations = _stagedMutations?.ToList().Select(sm => sm.AsTxData()) ?? Array.Empty<TxDataMutation>();
+            var mutations = _stagedMutations?.ToList().Select(sm => sm.AsTxData()) ??
+                            Array.Empty<TxDataMutation>();
             var txid = new CompositeId()
             {
                 Transactionid = _overallContext.TransactionId,
@@ -2628,14 +2648,6 @@ private QueryOptions InitializeBeginWorkQueryOptions(QueryOptions queryOptions)
         {
             queryOptions
                 .ScanConsistency(_config.ScanConsistency ?? QueryScanConsistency.RequestPlus)
-                .Raw("durability_level", _effectiveDurabilityLevel switch
-                {
-                    DurabilityLevel.None => "none",
-                    DurabilityLevel.Majority => "majority",
-                    DurabilityLevel.MajorityAndPersistToActive => "majorityAndPersistActive",
-                    DurabilityLevel.PersistToMajority => "persistToMajority",
-                    _ => _effectiveDurabilityLevel.ToString()
-                })
                 .Raw("txtimeout", $"{_overallContext.RemainingUntilExpiration.TotalMilliseconds}ms");
 
             if (_config.MetadataCollection != null)

src/Couchbase/Client/Transactions/Components/StagedMutation.cs

@@ -14,12 +14,12 @@ internal class StagedMutation
         public StagedMutationType Type { get; }
         public MutationToken MutationToken { get; }
 
-        public StagedMutation(TransactionGetResult doc, object? content, StagedMutationType type, MutationToken mutationToken)
+        public StagedMutation(TransactionGetResult doc, object? content, StagedMutationType type, MutationToken? mutationToken = null)
         {
             Doc = doc;
             Content = content;
             Type = type;
-            MutationToken = mutationToken;
+            MutationToken = mutationToken ?? MutationToken.Empty;
         }
 
         public JObject ForAtr() => new JObject(

src/Couchbase/Client/Transactions/Error/Attempts/ErrorTriage.cs

@@ -191,8 +191,7 @@ internal void ThrowIfCommitWithPreviousErrors(IEnumerable<TransactionOperationFa
             var previousErrors = previousErrorOperations.ToList();
             var retryTransaction = previousErrors.All(ex => ex.RetryTransaction);
             var rollback = previousErrors.All(ex => ex.AutoRollbackAttempt);
-            var cause = new PreviousOperationFailedException(previousErrors);
-            var builder = CreateError(_ctx, FailOther, cause);
+            var builder = CreateError(_ctx, FailOther);
             if (retryTransaction)
             {
                 builder.RetryTransaction();