review comment updates

src/core/protocols-grant-authorization.ts

@@ -32,6 +32,10 @@ export class ProtocolsGrantAuthorization {
     ProtocolsGrantAuthorization.verifyScope(protocolsConfigureMessage, permissionGrant.scope as ProtocolPermissionScope);
   }
 
+  /**
+   * Authorizes the scope of a permission grant for a ProtocolsQuery message.
+   * @param messageStore Used to check if the grant has been revoked.
+   */
   public static async authorizeQuery(input: {
     expectedGrantor: string,
     expectedGrantee: string,

tests/features/author-delegated-grant.spec.ts

@@ -73,7 +73,7 @@ export function testAuthorDelegatedGrant(): void {
         const alice = await TestDataGenerator.generateDidKeyPersona();
         const bob = await TestDataGenerator.generateDidKeyPersona();
 
-        // Alice grants Bob to configure the email protocol
+        // Alice grants Bob ability to configure any protocol, bob will use it to configure the email protocol
         const scope: PermissionScope = {
           interface : DwnInterfaceName.Protocols,
           method    : DwnMethodName.Configure,

tests/handlers/protocols-query.spec.ts

@@ -376,6 +376,17 @@ export function testProtocolsQueryHandler(): void {
           expect(protocolQueryPublishedReply.status.code).to.equal(200);
           expect(protocolQueryPublishedReply.entries?.length).to.equal(1);
           expect(protocolQueryPublishedReply.entries![0].descriptor.definition.protocol).to.deep.equal(publishedProtocol);
+
+          // Bob tries to ProtocolsQuery to Alice's DWN with no filters, using the same grant
+          const protocolQueryNoFilters = await ProtocolsQuery.create({
+            signer            : Jws.createSigner(bob),
+            permissionGrantId : permissionGrant.recordsWrite.message.recordId,
+          });
+
+          const protocolQueryNoFiltersReply = await dwn.processMessage(alice.did, protocolQueryNoFilters.message);
+          expect(protocolQueryNoFiltersReply.status.code).to.equal(200);
+          expect(protocolQueryNoFiltersReply.entries?.length).to.equal(1);
+          expect(protocolQueryNoFiltersReply.entries![0].descriptor.definition.protocol).to.deep.equal(publishedProtocol);
         });
 
         it('rejects with 401 when an external party attempts to ProtocolsQuery if they present an expired grant', async () => {