feat(meta): do not jsonized version meta

s-pace · Sylvain Pace · commit 2b452ff08e8d · 2020-05-27T14:05:50.000+02:00
diff --git a/scraper/src/strategies/default_strategy.py b/scraper/src/strategies/default_strategy.py
@@ -177,7 +177,7 @@ def get_records_from_dom(self, current_page_url=None):
                         record[name] = content
 
                     if name == "version":
-                        record[name] = str(record[name])
+                        record[name] = str(content)
 
             if current_page_url is not None:
                 # Add variables to the record
diff --git a/scraper/src/tests/default_strategy/meta_test.py b/scraper/src/tests/default_strategy/meta_test.py
@@ -187,6 +187,37 @@ def test_meta_numbered_version(self):
         assert actual[0]['version'] == "1.0"
         assert actual[1]['version'] != 1
 
+    def test_meta_decimal_version(self):
+        # Given
+        strategy = get_strategy({
+            'selectors': {
+                'lvl0': "h1",
+                'content': 'p'
+            }
+        })
+        strategy.dom = lxml.html.fromstring("""
+        <html>
+            <header>
+                <meta name="docsearch:version" content='5.20'>
+            </header>
+            <body>
+                <h1>Foo</h1>
+                <p>text</p>
+                <h2>Bar</h2>
+                <h3>Baz</h3>
+            </body>
+        </html>
+        """)
+
+        # When
+        actual = strategy.get_records_from_dom()
+
+        # Then
+
+        assert len(actual) == 2
+        assert actual[0]['version'] == "5.20"
+        assert actual[1]['version'] != 5.2
+
     def test_meta_escaped_string(self):
         # Given
         strategy = get_strategy({
