C#: Update the Telemetry queries such that properties with both get and set are no longer interesting.

Author: michaelnebel

csharp/ql/src/Telemetry/ExternalApi.qll

@@ -13,8 +13,15 @@ private import TestLibrary
 
 /** Holds if the given callable is not worth supporting. */
 private predicate isUninteresting(Callable c) {
-  c.getDeclaringType() instanceof TestLibrary or
+  c.getDeclaringType() instanceof TestLibrary
+  or
   c.(Constructor).isParameterless()
+  or
+  // The data flow library uses read/store steps for properties, so we don't need to model them,
+  // if both a getter and a setter exist.
+  exists(Property p | p = c.(Accessor).getDeclaration() |
+    exists(p.getSetter()) and exists(p.getGetter())
+  )
 }
 
 /**