summaryrefslogtreecommitdiff
path: root/tests/integration
diff options
context:
space:
mode:
authorKyle Kelley <rgbkrk@gmail.com>2024-05-21 13:35:21 -0700
committerGitHub <noreply@github.com>2024-05-21 22:35:21 +0200
commit8698e80304815353ec52be1b16f96483ebe559a0 (patch)
tree9abd53d5b656cd8cc0c1aa3940684f3ce1d9c8ef /tests/integration
parentcc8c0609ebec9f101a1739a0c42c91718ca2abba (diff)
refactor(jupyter): use runtimelib for Jupyter structures and directory paths (#23826)
This brings in [`runtimelib`](https://github.com/runtimed/runtimed) to use: ## Fully typed structs for Jupyter Messages ```rust let msg = connection.read().await?; self .send_iopub( runtimelib::Status::busy().as_child_of(msg), ) .await?; ``` ## Jupyter paths Jupyter paths are implemented in Rust, allowing the Deno kernel to be installed completely via Deno without a requirement on Python or Jupyter. Deno users will be able to install and use the kernel with just VS Code or other editors that support Jupyter. ```rust pub fn status() -> Result<(), AnyError> { let user_data_dir = user_data_dir()?; let kernel_spec_dir_path = user_data_dir.join("kernels").join("deno"); let kernel_spec_path = kernel_spec_dir_path.join("kernel.json"); if kernel_spec_path.exists() { log::info!("✅ Deno kernel already installed"); Ok(()) } else { log::warn!("ℹ️ Deno kernel is not yet installed, run `deno jupyter --install` to set it up"); Ok(()) } } ``` Closes https://github.com/denoland/deno/issues/21619
Diffstat (limited to 'tests/integration')
-rw-r--r--tests/integration/jupyter_tests.rs10
1 files changed, 6 insertions, 4 deletions
diff --git a/tests/integration/jupyter_tests.rs b/tests/integration/jupyter_tests.rs
index 3c4efbdac..c7b2712e8 100644
--- a/tests/integration/jupyter_tests.rs
+++ b/tests/integration/jupyter_tests.rs
@@ -10,6 +10,8 @@ use test_util::DenoChild;
use test_util::TestContext;
use test_util::TestContextBuilder;
+use chrono::DateTime;
+use chrono::Utc;
use deno_core::anyhow::Result;
use deno_core::serde_json;
use deno_core::serde_json::json;
@@ -119,7 +121,7 @@ impl Default for JupyterMsg {
struct MsgHeader {
msg_id: Uuid,
session: Uuid,
- date: String,
+ date: DateTime<Utc>,
username: String,
msg_type: String,
version: String,
@@ -136,7 +138,7 @@ impl Default for MsgHeader {
Self {
msg_id: Uuid::new_v4(),
session: Uuid::new_v4(),
- date: utc_now().to_rfc3339(),
+ date: utc_now(),
username: "test".into(),
msg_type: "kernel_info_request".into(),
version: "5.3".into(),
@@ -517,7 +519,7 @@ async fn jupyter_kernel_info() -> Result<()> {
"mimetype": "text/x.typescript",
"file_extension": ".ts",
"pygments_lexer": "typescript",
- "nb_converter": "script"
+ "nbconvert_exporter": "script"
},
}),
);
@@ -612,7 +614,7 @@ async fn jupyter_store_history_false() -> Result<()> {
json!({
"silent": false,
"store_history": false,
- "code": "console.log(\"asdf\")"
+ "code": "console.log(\"asdf\")",
}),
)
.await?;