Handle getting summary in Testkit Backend when a Record Iterator exists (#1258)

Author: MaxAake

packages/testkit-backend/src/request-handlers.js

@@ -241,7 +241,10 @@ export function ResultConsume (_, context, data, wire) {
   const { resultId } = data
   const result = context.getResult(resultId)
 
-  return result.summary().then(summary => {
+  let summaryPromise = 'recordIt' in result
+    ? (async () => {return (await result.recordIt.return()).value})()
+    : result.summary()
+  return summaryPromise.then(summary => {
     wire.writeResponse(responses.Summary({ summary }, { binder: context.binder }))
   }).catch(e => wire.writeError(e))
 }

packages/testkit-backend/src/skipped-tests/common.js

@@ -56,6 +56,7 @@ const skippedTests = [
   skip(
     'ResultSummary.notifications defaults to empty array instead of return null/undefined',
     ifEquals('stub.summary.test_summary.TestSummaryNotifications4x4.test_no_notifications'),
+    ifEquals('stub.summary.test_summary.TestSummaryNotifications4x4Discard.test_no_notifications'),
     ifEquals('neo4j.test_summary.TestSummary.test_no_notification_info')
   ),
   skip(