Fix web/ payload data_type json serialization, (gluesql#938)

Fix payload data_type json serialization. Fix gluesql-js web/ integration tests.
arbfay · Oct 14, 2022 · 3468ccc · 3468ccc
1 parent bcadf6c
commit 3468ccc
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/pkg/javascript/web/src/payload.rs b/pkg/javascript/web/src/payload.rs
@@ -50,7 +50,7 @@ fn convert_payload(payload: Payload) -> Json {
                 .map(|(name, data_type)| {
                     json!({
                         "name": name,
-                        "type": data_type.to_owned(),
+                        "type": data_type.to_string(),
                     })
                 })
                 .collect();

diff --git a/pkg/javascript/web/tests/memory_storage.rs b/pkg/javascript/web/tests/memory_storage.rs
@@ -24,5 +24,4 @@ impl Tester<MemoryStorage> for MemoryTester {
 }
 
 generate_store_tests!(wasm_bindgen_test, MemoryTester);
-generate_dictionary_tests!(wasm_bindgen_test, MemoryTester);
 generate_alter_table_tests!(wasm_bindgen_test, MemoryTester);
diff --git a/pkg/javascript/web/tests/payload.rs b/pkg/javascript/web/tests/payload.rs
@@ -4,7 +4,6 @@ wasm_bindgen_test_configure!(run_in_browser);
 
 use {
     gluesql_js::Glue,
-    memory_storage::MemoryStorage,
     serde_json::{json, Value as Json},
     wasm_bindgen_futures::JsFuture,
     wasm_bindgen_test::*,
@@ -79,7 +78,7 @@ async fn queries() {
             "SHOW VERSION",
             json!([{
                 "type": "SHOW VERSION",
-                "version": MemoryStorage::default().version()
+                "version": env!("CARGO_PKG_VERSION"),
             }]),
         ),
         (