Trying to make CI better

.github/workflows/ci.yaml

@@ -9,6 +9,7 @@ concurrency:
 jobs:
   test-ipython:
     runs-on: ubuntu-latest
+    timeout-minutes: 5
 
     steps:
       - uses: actions/checkout@v3
@@ -33,6 +34,7 @@ jobs:
 
   test-evcxr:
     runs-on: ubuntu-latest
+    timeout-minutes: 5
 
     steps:
       - uses: actions/checkout@v3
@@ -59,6 +61,7 @@ jobs:
 
   test-irkernel:
     runs-on: ubuntu-latest
+    timeout-minutes: 5
 
     steps:
       - uses: actions/checkout@v3
@@ -67,12 +70,10 @@ jobs:
         uses: r-lib/actions/setup-r@v2
 
       - name: Cache R packages
-        uses: actions/cache@v2
+        uses: actions/cache@v3
         with:
-          path: ${{ runner.workspace }}/R/library
+          path: ${{ env.R_LIBS_USER }}
           key: R-packages-${{ runner.os }}
-          restore-keys: |
-            R-packages-${{ runner.os }}
 
       - name: Install IRkernel
         run: Rscript -e 'install.packages("IRkernel")'
@@ -87,6 +88,7 @@ jobs:
 
   test-deno:
     runs-on: ubuntu-latest
+    timeout-minutes: 5
 
     steps:
       - uses: actions/checkout@v3

src/main.rs

@@ -1,14 +1,18 @@
 use kernel_sidecar_rs::client::Client;
 use kernel_sidecar_rs::handlers::{DebugHandler, Handler};
 use kernel_sidecar_rs::kernels::JupyterKernel;
+use tokio::time::sleep;
 
 use std::sync::Arc;
+use std::time::Duration;
 
 #[tokio::main]
 async fn main() {
-    let kernel = JupyterKernel::ipython(false);
+    let kernel = JupyterKernel::deno(false);
     let client = Client::new(kernel.connection_info.clone()).await;
     client.heartbeat().await;
+    // small sleep to make sure iopub is connected,
+    sleep(Duration::from_millis(50)).await;
 
     let handler = DebugHandler::new();
     let handlers = vec![Arc::new(handler) as Arc<dyn Handler>];

tests/test_kernels.rs

@@ -24,7 +24,7 @@ async fn start_kernel() -> (JupyterKernel, Client) {
     // Could be totally wrong.
     // Separately, there may be an edge case where multiple JupyterKernel::ipython calls end up
     // with the same ports and it all blows up. TODO: fix that.
-    tokio::time::sleep(tokio::time::Duration::from_millis(100)).await;
+    tokio::time::sleep(tokio::time::Duration::from_millis(200)).await;
     (kernel, client)
 }