[Stats] update action returns before updating stats for NONE operat…

…ions

We keep around a noop stats indicating how many update operations ended up not updating the document (typically because it didn't change). However, the TransportUpdateAction update that counter only after returning the result. This can throw off stats check which are done immediately after, potentially causing test failures.

Closes #7639

src/main/java/org/elasticsearch/action/update/TransportUpdateAction.java

@@ -270,14 +270,14 @@ protected void doRun() {
                 break;
             case NONE:
                 UpdateResponse update = result.action();
-                listener.onResponse(update);
                 IndexService indexServiceOrNull = indicesService.indexService(request.concreteIndex());
                 if (indexServiceOrNull !=  null) {
                     IndexShard shard = indexService.shard(request.request().shardId());
                     if (shard != null) {
                         shard.indexingService().noopUpdate(request.request().type());
                     }
                 }
+                listener.onResponse(update);
                 break;
             default:
                 throw new ElasticsearchIllegalStateException("Illegal operation " + result.operation());