fix: ensure duplicated are excluded from child bundles

Author: shakyShane

rjs-parse/src/modules.rs

@@ -52,6 +52,7 @@ pub fn generate_modules(
         &mut modules,
         req_log,
         &config.bundles,
+        &mut vec![],
         vec!["requirejs/require".into()],
     );
     modules.to_vec()
@@ -64,13 +65,17 @@ pub fn collect<'a>(
     modules: &'a mut Vec<BuildModule>,
     req_log: &Vec<ModuleData>,
     children: &Vec<ConfigItem>,
+    prev: &mut Vec<String>,
     exclude: Vec<String>,
 ) {
     children.iter().for_each(|conf_item| {
         let mut include: Vec<String> = vec![];
         req_log.iter().for_each(|item| {
             if let Some(..) = conf_item.urls.iter().find(|x| **x == item.referrer) {
-                include.push(create_entry_point(&item));
+                let next_id = create_entry_point(&item);
+                if let None = prev.iter().find(|x| **x == next_id) {
+                    include.push(next_id);
+                }
             }
         });
         include.sort();
@@ -84,7 +89,8 @@ pub fn collect<'a>(
         modules.push(this_item);
         let mut exclude = exclude.clone();
         exclude.push(conf_item.name.to_string());
-        collect(modules, req_log, &conf_item.children, exclude);
+        prev.extend(include);
+        collect(modules, req_log, &conf_item.children, prev, exclude);
     });
 }
 

rjs-parse/tests/e2e.rs

@@ -2,7 +2,7 @@ extern crate rjs;
 extern crate from_file;
 extern crate serde_json;
 
-use rjs::{RequireJsBuildConfig, RequireJsClientConfig, bundle_config};
+use rjs::{RequireJsBuildConfig, bundle_config};
 
 #[test]
 fn test_all_strings() {
@@ -16,14 +16,48 @@ fn test_all_strings() {
     })();
     "#;
     let bundle_config = r#"
-        {"bundles": [{"name": "main", "children": [], "urls": ["/"]}]}
+        {
+            "bundles": [
+                {
+                    "name": "main",
+                    "urls": ["/"],
+                    "children": [
+                        {
+                            "name": "other",
+                            "urls": ["/about"],
+                            "children": []
+                        },
+                        {
+                            "name": "product",
+                            "urls": ["/product"],
+                            "children": []
+                        }
+                    ]
+                }
+            ]
+        }
     "#;
     let req_log = r#"
         [
             {
                 "url": "https://example.com/jquery",
                 "id": "jquery",
                 "referrer": "/"
+            },
+            {
+                "url": "https://example.com/jquery",
+                "id": "jquery",
+                "referrer": "/about"
+            },
+            {
+                "url": "https://example.com/jquery-ui",
+                "id": "jquery-ui",
+                "referrer": "/about"
+            },
+            {
+                "url": "https://example.com/gallery.js",
+                "id": "gallery",
+                "referrer": "/product"
             }
         ]
     "#;
@@ -57,6 +91,24 @@ fn test_all_strings() {
             "requirejs/require"
           ],
           "create": true
+        },
+        {
+          "name": "other",
+          "include": ["jquery-ui"],
+          "exclude": [
+            "requirejs/require",
+            "main"
+          ],
+          "create": true
+        },
+        {
+          "name": "product",
+          "include": ["gallery"],
+          "exclude": [
+            "requirejs/require",
+            "main"
+          ],
+          "create": true
         }
       ]
     }
@@ -72,5 +124,7 @@ fn test_all_strings() {
     let actual_as_value: serde_json::Value = serde_json::from_str(&as_string).expect("serde actual");
     let expected_as_value: serde_json::Value = serde_json::from_str(&expected).expect("serde expected");
 
+//    println!("{}", as_string);
+
     assert_eq!(actual_as_value, expected_as_value);
 }