fix log list e2e test

rockcarver · Sep 28, 2023 · e5b0c8c · e5b0c8c
1 parent d35dea3
commit e5b0c8c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 11 deletions.
diff --git a/src/cli/log/log-list.ts b/src/cli/log/log-list.ts
@@ -29,9 +29,9 @@ program
         );
       } else {
         printMessage(`Log sources from ${state.getHost()}`);
-        sources.forEach((source) => {
+        for (const source of sources) {
           printMessage(`${source}`, 'data');
-        });
+        }
         printMessage(
           'Use any combination of comma separated sources, example:',
           'info'

diff --git a/test/e2e/log-list.e2e.test.js b/test/e2e/log-list.e2e.test.js
@@ -65,13 +65,9 @@ const env = {
 };
 
 describe('frodo log list', () => {
-  // flaky test. skipping in github pipeline until we can figure out to make it succeed
-  testif(!process.env['GITHUB_TOKEN'])(
-    '"frodo log list": should list the names of the logs sources',
-    async () => {
-      const CMD = `frodo log list`;
-      const { stdout } = await exec(CMD, env);
-      expect(removeAnsiEscapeCodes(stdout)).toMatchSnapshot();
-    }
-  );
+  test('"frodo log list": should list the names of the logs sources', async () => {
+    const CMD = `frodo log list`;
+    const { stdout } = await exec(CMD, env);
+    expect(removeAnsiEscapeCodes(stdout)).toMatchSnapshot();
+  });
 });