Enable backtrace / remove other leftover dbg lines

.github/workflows/per-pr.yml

@@ -49,6 +49,8 @@ jobs:
   test:
     name: Unit Tests
     runs-on: ubuntu-latest
+    env:
+      RUST_BACKTRACE: 1
     steps:
       - uses: actions/checkout@v2
       - uses: actions-rs/toolchain@v1

core/src/worker/activities/activity_heartbeat_manager.rs

@@ -450,8 +450,7 @@ mod test {
         hm.shutdown().await;
     }
 
-    /// Ensure that heartbeat can be called from a tight loop without any throttle_interval, resulting in two
-    /// interactions with the server - one immediately and one after 500ms after the throttle_interval.
+    /// Ensure that heartbeat can be called from a tight loop and correctly throttle
     #[tokio::test]
     async fn process_tight_loop_and_shutdown() {
         let mut mock_client = mock_workflow_client();

core/src/worker/activities/local_activities.rs

@@ -1246,7 +1246,7 @@ mod tests {
         // Wait more than the timeout before grabbing the task
         sleep(timeout + Duration::from_millis(10)).await;
 
-        assert!(dbg!(lam.next_pending().await.unwrap()).is_timeout(false));
+        assert!(lam.next_pending().await.unwrap().is_timeout(false));
         assert_eq!(lam.num_in_backoff(), 0);
         assert_eq!(lam.num_outstanding(), 0);
     }

core/src/worker/workflow/machines/local_activity_state_machine.rs

@@ -1250,7 +1250,6 @@ mod tests {
         });
 
         let mut worker = build_fake_sdk(mock_cfg);
-        dbg!("Past thing");
         worker.register_wf(DEFAULT_WORKFLOW_TYPE, la_wf);
         worker.register_activity(
             DEFAULT_ACTIVITY_TYPE,