Backport 0.2

.github/workflows/test.yaml

@@ -8,6 +8,15 @@ on:
     branches: [ develop, develop-0.1, develop-0.2 ]
 
 jobs:
+  cargotest:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: dtolnay/rust-toolchain@stable
+
+      - run: cargo test --no-fail-fast
+
   testbuild:
     runs-on: ubuntu-latest
     steps:

src/cli.rs

@@ -559,7 +559,7 @@ Add new entry definitions to your zome with:
                 let file_tree = load_directory_into_memory(&current_dir)?;
                 let template_file_tree = choose_or_get_template_file_tree(&file_tree, &template)?;
 
-                let name: String = match name {
+                let name = match name {
                     Some(n) => {
                         check_case(&n, "entry type name", Case::Snake)?;
                         n

src/scaffold/app.rs

@@ -25,6 +25,7 @@ impl AppFileTree {
     pub fn file_tree(self) -> FileTree {
         self.file_tree
     }
+
     pub fn file_tree_ref<'a>(&'a self) -> &'a FileTree {
         &self.file_tree
     }

src/scaffold/entry_type/fields.rs

@@ -5,7 +5,7 @@ use dialoguer::{theme::ColorfulTheme, Confirm, Select};
 use regex::Regex;
 
 use crate::{
-    error::ScaffoldResult,
+    error::{ScaffoldError, ScaffoldResult},
     file_tree::{dir_content, FileTree},
     scaffold::zome::ZomeFileTree,
     utils::{check_case, input_with_case, input_with_case_and_initial_text},
@@ -261,6 +261,13 @@ pub fn choose_field(
                         ));
                     }
 
+                    if all_options.is_empty() {
+                        return Err(ScaffoldError::NoEntryTypesDefFoundForIntegrityZome(
+                            zome_file_tree.dna_file_tree.dna_manifest.name(),
+                            zome_file_tree.zome_manifest.name.to_string(),
+                        ))
+                    }
+
                     let selection = Select::with_theme(&ColorfulTheme::default())
                         .with_prompt(String::from("Which entry type is this field referring to?"))
                         .default(0)

src/templates.rs

@@ -295,89 +295,3 @@ pub fn choose_or_get_template(
 
     Ok(chosen_template_name)
 }
-
-#[cfg(test)]
-mod tests {
-    // Note this useful idiom: importing names from outer (for mod tests) scope.
-    use super::*;
-    use handlebars::Handlebars;
-    use serde_json::{Map, Value};
-
-    #[test]
-    fn test_get_scope_open_and_close_char_indexes() {
-        let text = String::from("const s = {};");
-        let scope_opener = String::from("const s = {");
-
-        let (scope_opener_index, scope_close_index) =
-            get_scope_open_and_close_char_indexes(&text, &scope_opener).unwrap();
-
-        assert_eq!(scope_opener_index, 10);
-        assert_eq!(scope_close_index, 11);
-    }
-
-    #[test]
-    fn test_merge_match_scope() {
-        let h = Handlebars::new();
-
-        let mut h = register_helpers(h);
-
-        let code = String::from(
-            r#"
-export class A {
-    nestedFn1() {
-
-    }
-}
-export class B {
-    nestedFn() {
-        // First line
-    }
-}
-            "#,
-        );
-        let mut map = Map::new();
-        map.insert(
-            String::from("previous_file_content"),
-            Value::String(String::from(code)),
-        );
-        let context = Context::from(Value::Object(map));
-        let template = r#"
-{{#merge previous_file_content}}
-    {{#match_scope "export class A {"}}
-    nestedFn2() {
-
-    }
-    {{previous_scope_content}}
-    {{/match_scope}}
-    {{#match_scope "export class B {"}}
-        {{#merge previous_scope_content}}
-            {{#match_scope "nestedFn() {"}}
-        {{previous_scope_content}}
-        // New line
-            {{/match_scope}}
-        {{/merge}}
-    {{/match_scope}}
-{{/merge}}
-            "#;
-
-        assert_eq!(
-            h.render_template_with_context(template, &context).unwrap(),
-            r#"
-export class A {
-    nestedFn2() {
-
-    }
-    nestedFn1() {
-
-    }
-}
-export class B {
-    nestedFn() {
-        // First line
-        // New line
-    }
-}
-            "#,
-        );
-    }
-}

src/templates/helpers/merge.rs

@@ -5,7 +5,7 @@ use handlebars::{
 use serde::{Deserialize, Serialize};
 use serde_json::{Map, Number, Value};
 
-fn get_scope_open_and_close_char_indexes(
+pub fn get_scope_open_and_close_char_indexes(
     text: &String,
     scope_opener: &String,
 ) -> Result<(usize, usize), RenderError> {
@@ -237,7 +237,7 @@ mod tests {
     // Note this useful idiom: importing names from outer (for mod tests) scope.
     use super::*;
     use handlebars::Handlebars;
-    use serde_json::{Map, Value};
+    use serde_json::json;
 
     #[test]
     fn test_get_scope_open_and_close_char_indexes() {
@@ -257,19 +257,14 @@ mod tests {
 
         let h = register_merge(h);
 
-        let code = String::from(
-            r#"class A {
+        let code = r#"
+        class A {
     // Multiline
     // Comment
 }
-"#,
-        );
-        let mut map = Map::new();
-        map.insert(
-            String::from("previous_file_content"),
-            Value::String(String::from(code.clone())),
-        );
-        let context = Context::from(Value::Object(map));
+"#;
+        let value = json!({"previous_file_content": code});
+        let context = Context::from(value);
         let template = r#"{{#merge previous_file_content}}
 {{/merge}}
 "#;
@@ -286,19 +281,13 @@ mod tests {
 
         let h = register_merge(h);
 
-        let code = String::from(
-            r#"class A {
+        let code = r#"class A {
     // Multiline
     // Comment
 }
-"#,
-        );
-        let mut map = Map::new();
-        map.insert(
-            String::from("previous_file_content"),
-            Value::String(String::from(code)),
-        );
-        let context = Context::from(Value::Object(map));
+"#;
+        let value = json!({"previous_file_content": code});
+        let context = Context::from(value);
         let template = r#"
 {{#merge previous_file_content}}
     {{#match_scope "class A {"}}
@@ -330,8 +319,7 @@ class A {
 
         let h = register_merge(h);
 
-        let code = String::from(
-            r#"export class A {
+        let code = r#"export class A {
     nestedFn1() {
         // First line
     }
@@ -341,21 +329,13 @@ export class B {
         // First line
     }
 }
-"#,
-        );
-        let mut map = Map::new();
-        map.insert(
-            String::from("previous_file_content"),
-            Value::String(String::from(code)),
-        );
-        map.insert(
-            String::from("class_functions"),
-            Value::Array(vec![
-                Value::String(String::from("nestedFn2")),
-                Value::String(String::from("nestedFn3")),
-            ]),
-        );
-        let context = Context::from(Value::Object(map));
+"#;
+        let value = json!({
+            "previous_file_content": code,
+            "class_functions": ["nestedFn2", "nestedFn3"]
+        });
+
+        let context = Context::from(value);
         let template = r#"{{#merge previous_file_content}}
     {{#match_scope "export class B {"}}
         {{#merge untrimmed_previous_scope_content}}

templates/lit/web-app/ui/.gitignore.hbs

@@ -6,7 +6,7 @@
 .DS_Store
 
 ## npm
-/node_modules/
+/**/node_modules/
 /npm-debug.log
 
 ## testing
@@ -19,6 +19,8 @@
 /_site/
 /dist/
 /out-tsc/
+/target/
+/.cargo
 
 storybook-static
 .rollup.cache

templates/svelte/web-app/ui/.gitignore.hbs

@@ -6,7 +6,7 @@
 .DS_Store
 
 ## npm
-/node_modules/
+/**/node_modules/
 /npm-debug.log
 
 ## testing
@@ -19,6 +19,7 @@
 /_site/
 /dist/
 /out-tsc/
+/target/
 
 storybook-static
 .rollup.cache

templates/vanilla/example/dnas/hello_world/zomes/integrity/hello_world/src/lib.rs.hbs

@@ -16,7 +16,7 @@ pub enum EntryTypes {
     Hello(Hello), 
 }
 
-/// Definition of a link type to be used for linking from an anchor to all created entreis
+/// Definition of a link type to be used for linking from an anchor to all created entries
 #[derive(Serialize, Deserialize)]
 #[hdk_link_types]
 pub enum LinkTypes {

templates/vanilla/web-app/.gitignore.hbs

@@ -6,7 +6,7 @@
 .DS_Store
 
 ## npm
-/node_modules/
+/**/node_modules/
 /npm-debug.log
 
 ## testing
@@ -19,6 +19,7 @@
 /_site/
 /dist/
 /out-tsc/
+/target/
 
 storybook-static
 .rollup.cache

templates/vue/web-app/ui/.gitignore.hbs

@@ -6,7 +6,7 @@
 .DS_Store
 
 ## npm
-/node_modules/
+/**/node_modules/
 /npm-debug.log
 
 ## testing
@@ -19,6 +19,7 @@
 /_site/
 /dist/
 /out-tsc/
+/target/
 
 storybook-static
 .rollup.cache