Remove todo for cursor iteration

Author: aherlihy

packages/shell-api/src/aggregation-cursor.ts

@@ -30,7 +30,7 @@ export default class AggregationCursor extends ShellApiClass {
       return results;
     }
 
-    for (let i = 0; i < 20; i++) { // TODO: ensure that assigning cursor doesn't iterate
+    for (let i = 0; i < 20; i++) {
       if (!await this.hasNext()) {
         break;
       }

packages/shell-api/src/cursor.ts

@@ -33,7 +33,7 @@ export default class Cursor extends ShellApiClass {
       return results;
     }
 
-    for (let i = 0; i < 20; i++) { // TODO: ensure that assigning cursor doesn't iterate
+    for (let i = 0; i < 20; i++) {
       if (!await this.hasNext()) {
         break;
       }

packages/shell-api/src/shell-api.ts

@@ -81,7 +81,6 @@ export default class ShellApi extends ShellApiClass {
   @returnsPromise
   async it(): Promise<any> {
     if (!this.internalState.currentCursor) {
-      // TODO: warn here
       return new CursorIterationResult();
     }
     return await this.internalState.currentCursor._it();