In /graph_json, check internal_only on Test entity and skip test for all Row entities.

dashboard/dashboard/datastore_hooks_test.py

Index: dashboard/dashboard/datastore_hooks_test.py
diff --git a/dashboard/dashboard/datastore_hooks_test.py b/dashboard/dashboard/datastore_hooks_test.py
index 503938bda3db93f6c5cc17532727076722be3435..5ca264d1d59dd214f2d15f49db33717ad163a46c 100644
--- a/dashboard/dashboard/datastore_hooks_test.py
+++ b/dashboard/dashboard/datastore_hooks_test.py
@@ -177,6 +177,21 @@ class DatastoreHooksTest(testing_common.TestCase):
     datastore_hooks.SetPrivilegedRequest()
     self._CheckQueryResults(True)
 
+  def testQuery_SinglePrivilegedRequest_InternalOnlyFetched(self):
+    self.UnsetCurrentUser()
+    datastore_hooks.SetSinglePrivilegedRequest()
+    # Not using _CheckQueryResults because this only affects a single query.
+    # First query has internal results.
+    rows = graph_data.Row.query(ndb.OR(
+        graph_data.Row.revision < 20, graph_data.Row.revision > 70)).filter(
+            graph_data.Row.value == 20).fetch()

[qyearsley, 2015/12/28 22:50:56]

Why `ndb.OR(graph_data.Row.revision < 20, graph_data.Row.revision > 70)`

and not just `graph_data.Row.revision < 20`?

It looks like the ndb.OR was used above in another test method, but I'm not sure
why. Does the test also pass if you change this line (and the one below) to the
following?

  `rows = graph_data.Row.query(graph_data.Row.revision == 10).fetch()`

[sullivan, 2015/12/29 03:36:06]

I modified this test to the simpler query you suggested.

I think the original test is trying to check that the hooks work properly even
if there are multiple conditions in the query. Not needed here.

+    self.assertEqual(2, len(rows))
+    # Second query does not.
+    rows = graph_data.Row.query(ndb.OR(
+        graph_data.Row.revision < 20, graph_data.Row.revision > 70)).filter(
+            graph_data.Row.value == 20).fetch()
+    self.assertEqual(1, len(rows))
+
   def _CheckGet(self, include_internal):
     m = ndb.Key('Master', 'ChromiumPerf').get()
     self.assertEqual(m.key.string_id(), 'ChromiumPerf')