add eslint rule no-floating-promises in docs and demos dirs

software-mansion · Sep 12, 2024 · 1563059 · 1563059
1 parent 65c443c
commit 1563059
Show file tree

Hide file tree

Showing 8 changed files with 22 additions and 16 deletions.
diff --git a/demos/.eslintrc.json b/demos/.eslintrc.json
@@ -14,9 +14,15 @@
     "prettier"
   ],
   "parser": "@typescript-eslint/parser",
+  "parserOptions": {
+    "project": [
+      "tsconfig.json"
+    ]
+  },
   "rules": {
     "prettier/prettier": ["error"],
     "@typescript-eslint/no-explicit-any": [0, {}],
+    "@typescript-eslint/no-floating-promises": "error",
     "no-constant-condition": [0]
   }
 }
diff --git a/demos/1-videoconferencing/index.ts b/demos/1-videoconferencing/index.ts
@@ -65,11 +65,11 @@ async function exampleAsync() {
   });
 
   if (useWebCam) {
-    gstStreamWebcam(IP, INPUT_PORT, DISPLAY_LOGS);
+    void gstStreamWebcam(IP, INPUT_PORT, DISPLAY_LOGS);
   } else {
     const callPath = path.join(__dirname, '../assets/call.mp4');
     await downloadAsync(CALL_URL, callPath);
-    ffmpegSendVideoFromMp4(INPUT_PORT, callPath, DISPLAY_LOGS);
+    void ffmpegSendVideoFromMp4(INPUT_PORT, callPath, DISPLAY_LOGS);
   }
   await startAsync();
 
@@ -166,4 +166,4 @@ function sceneWithInputs(n: number): Component {
   };
 }
 
-runCompositorExample(exampleAsync, DISPLAY_LOGS);
+void runCompositorExample(exampleAsync, DISPLAY_LOGS);
diff --git a/demos/2-tv_broadcast/index.ts b/demos/2-tv_broadcast/index.ts
@@ -37,7 +37,7 @@ const LOGO_URL =
   'https://raw.githubusercontent.com/membraneframework-labs/video_compositor_snapshot_tests/main/demo_assets/logo.png';
 
 async function exampleAsync() {
-  ffplayStartPlayerAsync(IP, DISPLAY_LOGS, VIDEO_OUTPUT_PORT, AUDIO_OUTPUT_PORT);
+  void ffplayStartPlayerAsync(IP, DISPLAY_LOGS, VIDEO_OUTPUT_PORT, AUDIO_OUTPUT_PORT);
   await sleepAsync(2000);
 
   process.env.LIVE_COMPOSITOR_LOGGER_LEVEL = 'debug';
@@ -103,7 +103,7 @@ async function exampleAsync() {
     },
   });
 
-  ffmpegSendVideoFromMp4(INPUT_PORT, TV_PATH, DISPLAY_LOGS);
+  void ffmpegSendVideoFromMp4(INPUT_PORT, TV_PATH, DISPLAY_LOGS);
   await startAsync();
 
   // First update to set start position of the bunny for transition
@@ -319,4 +319,4 @@ function logo(): Component {
   };
 }
 
-runCompositorExample(exampleAsync, DISPLAY_LOGS);
+void runCompositorExample(exampleAsync, DISPLAY_LOGS);
diff --git a/demos/3-live_stream/index.ts b/demos/3-live_stream/index.ts
@@ -100,13 +100,13 @@ async function exampleAsync() {
   });
 
   if (useWebCam) {
-    gstStreamWebcam(IP, WEBCAM_INPUT_PORT, DISPLAY_LOGS);
+    void gstStreamWebcam(IP, WEBCAM_INPUT_PORT, DISPLAY_LOGS);
   } else {
     const callPath = path.join(__dirname, '../assets/call.mp4');
     await downloadAsync(CALL_URL, callPath);
-    ffmpegSendVideoFromMp4(WEBCAM_INPUT_PORT, callPath, DISPLAY_LOGS);
+    void ffmpegSendVideoFromMp4(WEBCAM_INPUT_PORT, callPath, DISPLAY_LOGS);
   }
-  ffmpegSendVideoFromMp4(GAMEPLAY_PORT, gameplayPath, DISPLAY_LOGS);
+  void ffmpegSendVideoFromMp4(GAMEPLAY_PORT, gameplayPath, DISPLAY_LOGS);
 
   await sleepAsync(2000);
   await startAsync();
@@ -232,4 +232,4 @@ function baseScene(): Component {
   };
 }
 
-runCompositorExample(exampleAsync, DISPLAY_LOGS);
+void runCompositorExample(exampleAsync, DISPLAY_LOGS);
diff --git a/demos/utils/ffmpeg.ts b/demos/utils/ffmpeg.ts
@@ -81,7 +81,7 @@ async function writeVideoAudioSdpFile(
   audio_port: number,
   destination: string
 ): Promise<void> {
-  fs.writeFile(
+  await fs.writeFile(
     destination,
     `
 v=0
@@ -100,7 +100,7 @@ a=rtcp-mux
 }
 
 async function writeVideoSdpFile(ip: string, port: number, destination: string): Promise<void> {
-  fs.writeFile(
+  await fs.writeFile(
     destination,
     `
 v=0

diff --git a/demos/utils/generate_types.ts b/demos/utils/generate_types.ts
@@ -12,4 +12,4 @@ async function generateTypes() {
   fs.writeFileSync(tsOutputPath, typesTs);
 }
 
-generateTypes();
+void generateTypes();
diff --git a/demos/utils/gst.ts b/demos/utils/gst.ts
@@ -31,7 +31,7 @@ export function gstStreamWebcam(ip: string, port: number, displayOutput: boolean
 
 function checkGstPlugins(plugins: string[]) {
   plugins.forEach(plugin => {
-    isGstPluginAvailable(plugin).then(isAvailable => {
+    void isGstPluginAvailable(plugin).then(isAvailable => {
       if (!isAvailable) {
         throw Error(`Gstreamer plugin: ${plugin} is not available.`);
       }

diff --git a/demos/utils/run.ts b/demos/utils/run.ts
@@ -11,7 +11,7 @@ export async function runCompositorExample(
   await ensureCompositorReadyAsync();
   const { command, args, cwd } = getCompositorRunCmd();
   try {
-    spawn(command, args, {
+    void spawn(command, args, {
       displayOutput: displayOutput,
       cwd: cwd ?? process.cwd(),
     });
@@ -20,7 +20,7 @@ export async function runCompositorExample(
 
     await fn();
   } catch (err) {
-    logError(err);
+    void logError(err);
     throw err;
   }
 }