From 382a978859a7a7a4351542be818bb2e59523429c Mon Sep 17 00:00:00 2001 From: Leo Kettmeir Date: Thu, 27 Jan 2022 15:09:47 +0100 Subject: refactor: factor out CDP message types (#13501) --- cli/tools/coverage/mod.rs | 29 ++++++++++++++++++++--------- 1 file changed, 20 insertions(+), 9 deletions(-) (limited to 'cli/tools/coverage/mod.rs') diff --git a/cli/tools/coverage/mod.rs b/cli/tools/coverage/mod.rs index 1c14859ea..6177295a6 100644 --- a/cli/tools/coverage/mod.rs +++ b/cli/tools/coverage/mod.rs @@ -43,22 +43,34 @@ impl CoverageCollector { } async fn enable_debugger(&mut self) -> Result<(), AnyError> { - self.session.post_message("Debugger.enable", None).await?; + self + .session + .post_message::<()>("Debugger.enable", None) + .await?; Ok(()) } async fn enable_profiler(&mut self) -> Result<(), AnyError> { - self.session.post_message("Profiler.enable", None).await?; + self + .session + .post_message::<()>("Profiler.enable", None) + .await?; Ok(()) } async fn disable_debugger(&mut self) -> Result<(), AnyError> { - self.session.post_message("Debugger.disable", None).await?; + self + .session + .post_message::<()>("Debugger.disable", None) + .await?; Ok(()) } async fn disable_profiler(&mut self) -> Result<(), AnyError> { - self.session.post_message("Profiler.disable", None).await?; + self + .session + .post_message::<()>("Profiler.disable", None) + .await?; Ok(()) } @@ -66,10 +78,9 @@ impl CoverageCollector { &mut self, parameters: StartPreciseCoverageParameters, ) -> Result { - let parameters_value = serde_json::to_value(parameters)?; let return_value = self .session - .post_message("Profiler.startPreciseCoverage", Some(parameters_value)) + .post_message("Profiler.startPreciseCoverage", Some(parameters)) .await?; let return_object = serde_json::from_value(return_value)?; @@ -82,7 +93,7 @@ impl CoverageCollector { ) -> Result { let return_value = self .session - .post_message("Profiler.takePreciseCoverage", None) + .post_message::<()>("Profiler.takePreciseCoverage", None) .await?; let return_object = serde_json::from_value(return_value)?; @@ -592,8 +603,8 @@ pub async fn cover_files( })? }; let file = maybe_file.ok_or_else(|| { - anyhow!("Failed to fetch \"{}\" from cache. - Before generating coverage report, run `deno test --coverage` to ensure consistent state.", + anyhow!("Failed to fetch \"{}\" from cache. + Before generating coverage report, run `deno test --coverage` to ensure consistent state.", module_specifier ) })?; -- cgit v1.2.3