Parse content types when MetaData nodes are in the html

ccouzens · Jul 16, 2020 · d6e538f · d6e538f
1 parent b63b5ce
commit d6e538f
Show file tree

Hide file tree

Showing 9 changed files with 1,860 additions and 243 deletions.
diff --git a/transformer/src/parsers/doc/detail_page.rs b/transformer/src/parsers/doc/detail_page.rs
@@ -59,7 +59,7 @@ impl<'a> TryFrom<&scraper::ElementRef<'a>> for DefinitionListValue {
     type Error = DefinitionListValueFromElError;
 
     fn try_from(el: &scraper::ElementRef) -> Result<Self, Self::Error> {
-        let top_selector = scraper::Selector::parse(":scope > dl, :scope > a")
+        let top_selector = scraper::Selector::parse(":scope > dl, :scope > MetadataType > dl")
             .map_err(|e| Self::Error::SelectorParseError(format!("{:?}", e)))?;
         let child = el.select(&top_selector).next();
         match (child, child.map(|c| c.value().name())) {

diff --git a/transformer/src/parsers/doc/etc/object_type.rs b/transformer/src/parsers/doc/etc/object_type.rs
@@ -468,16 +468,16 @@ fn removed_field_test() {
     assert_eq!(
         serde_json::to_value(value).unwrap(),
         json!({
-            "title": "test_BaseType",
-            "description": "A base abstract type for all the types.",
-            "type": "object",
-            "properties": {
-              "fieldB": {
-                "description": "A field that has not been removed",
-                "type": "string"
-              }
-            },
-            "additionalProperties": false
-          })
+          "title": "test_BaseType",
+          "description": "A base abstract type for all the types.",
+          "type": "object",
+          "properties": {
+            "fieldB": {
+              "description": "A field that has not been removed",
+              "type": "string"
+            }
+          },
+          "additionalProperties": false
+        })
     );
 }