Improve JSON serialization and deserialization (#28)

 - Make all service types JSON serializable, behind a json feature.
 - Fixes and improvements to serialization of some core types, including a special JSON variant for extension object bodies.
 - Custom serializer for enums as numbers.
 - More tests.

.github/workflows/ci_code_coverage.yml

@@ -13,12 +13,6 @@ jobs:
       - name: Test with profiling data
         run: cargo llvm-cov --workspace --exclude opcua-crypto --codecov --output-path codecov.json
 
-      - name: Run crypto tests
-        run: cargo test -p opcua-crypto
-
-      - name: Run core tests
-        run: cargo test -p opcua-core
-
       - name: Upload lcov to codecov.io
         uses: codecov/codecov-action@v4
         with:

Cargo.toml

@@ -26,6 +26,7 @@ futures = "0.3"
 hashbrown = "0.14.5"
 serde = { version = "1.0", features = ["derive"] }
 serde_json = { version = "1.0", features = ["arbitrary_precision"] }
+serde_with = "3.9.0"
 serde_yaml = "0.9"
 log = "0.4"
 regex = "1.7"

lib/Cargo.toml

@@ -28,6 +28,8 @@ server = ["opcua-server", "opcua-nodes", "generated-address-space", "discovery-s
 client = ["opcua-client"]
 # Console logging just installs a logger that writes out to the screen, useful for general logging
 console-logging = ["env_logger"]
+# The json feature adds serialize/deserialize to all OPC-UA types.
+json = ["opcua-types/json"]
 # Includes all the code to populate the address space with the default node set. This is something that embedded
 # systems may or may not require.
 generated-address-space = ["opcua-server/generated-address-space"]
@@ -58,5 +60,5 @@ tempdir = "0.3"
 tokio = { workspace = true }
 tokio-util = { workspace = true }
 
-# Include console-logging when building tests
-opcua = { path = ".", features = ["console-logging"] }
+# Include console-logging and json when building tests
+opcua = { path = ".", features = ["console-logging", "json"] }

opcua-codegen/src/lib.rs

@@ -15,9 +15,9 @@ use opcua_xml::load_nodeset2_file;
 use serde::{Deserialize, Serialize};
 use syn::File;
 pub use types::{
-    base_ignored_types, base_json_serialized_types, base_native_type_mappings,
-    basic_types_import_map, BsdTypeLoader, CodeGenItemConfig, GeneratedItem, ItemDefinition,
-    LoadedType, LoadedTypes, StructureField, StructureFieldType, StructuredType,
+    base_ignored_types, base_native_type_mappings, basic_types_import_map, BsdTypeLoader,
+    CodeGenItemConfig, GeneratedItem, ItemDefinition, LoadedType, LoadedTypes, StructureField,
+    StructureFieldType, StructuredType,
 };
 use types::{generate_types, ExternalType};
 pub use utils::{create_module_file, GeneratedOutput};
@@ -165,8 +165,6 @@ pub struct TypeCodeGenTarget {
     #[serde(default)]
     pub ignore: Vec<String>,
     #[serde(default)]
-    pub json_serialized_types: Vec<String>,
-    #[serde(default)]
     pub types_import_map: HashMap<String, ExternalType>,
     #[serde(default)]
     pub default_excluded: HashSet<String>,

opcua-codegen/src/types/base_constants.rs

@@ -95,40 +95,6 @@ pub fn basic_types_import_map() -> HashMap<String, ExternalType> {
     .collect()
 }
 
-pub fn base_json_serialized_types() -> HashSet<String> {
-    [
-        "ReadValueId",
-        "DataChangeFilter",
-        "EventFilter",
-        "SimpleAttributeOperand",
-        "ContentFilter",
-        "ContentFilterElement",
-        "MonitoredItemNotification",
-        "ServerDiagnosticsSummaryDataType",
-        "EventFieldList",
-        "DataChangeTrigger",
-        "FilterOperator",
-        "TimestampsToReturn",
-        "MonitoringMode",
-        "ConfigurationVersionDataType",
-        "DataSetMetaDataType",
-        "StructureDescription",
-        "EnumDescription",
-        "SimpleTypeDescription",
-        "StructureDefinition",
-        "EnumDefinition",
-        "FieldMetaData",
-        "KeyValuePair",
-        "DataSetFieldFlags",
-        "StructureType",
-        "StructureField",
-        "EnumField",
-    ]
-    .into_iter()
-    .map(|v| v.to_owned())
-    .collect()
-}
-
 pub fn base_native_type_mappings() -> HashMap<String, String> {
     [
         ("String", "UAString"),