fix: removing unnecessary for await for non-async iterables

Author: CDeltakai

tests/vaults/create.test.ts

@@ -67,7 +67,7 @@ describe('commandCreateVaults', () => {
     expect(result.exitCode).toBe(0);
     const list = (await polykeyAgent.vaultManager.listVaults()).keys();
     const namesList: string[] = [];
-    for await (const name of list) {
+    for (const name of list) {
       namesList.push(name);
     }
     expect(namesList).toContain('MyTestVault');

tests/vaults/remove.test.ts

@@ -85,7 +85,7 @@ describe('commandRemoveVault', () => {
 
     const list = (await polykeyAgent.vaultManager.listVaults()).keys();
     const namesList: string[] = [];
-    for await (const name of list) {
+    for (const name of list) {
       namesList.push(name);
     }
     expect(namesList).not.toContain(vaultName);

tests/vaults/rename.test.ts

@@ -83,7 +83,7 @@ describe('commandRenameVault', () => {
 
     const list = (await polykeyAgent.vaultManager.listVaults()).keys();
     const namesList: string[] = [];
-    for await (const name of list) {
+    for (const name of list) {
       namesList.push(name);
     }
     expect(namesList).toContain('RenamedVault');
@@ -110,7 +110,7 @@ describe('commandRenameVault', () => {
 
     const list = (await polykeyAgent.vaultManager.listVaults()).keys();
     const namesList: string[] = [];
-    for await (const name of list) {
+    for (const name of list) {
       namesList.push(name);
     }
     expect(namesList).toContain(vaultName);