Error item links (#89)

* Add failing test case

* Handle raw keys

CHANGES.md

@@ -13,6 +13,8 @@ Note that the top-most release is changes in the unreleased master branch on Git
 ### Added
 ### Changed
 ### Fixed
+- `Arche.glance()`, #88
+- Item links in Schema validation errors, #89
 - Empty NAN bars on category graphs, #93
 ### Removed
 

src/arche/report.py

@@ -1,5 +1,6 @@
 from typing import Dict
 
+from arche import SH_URL
 from arche.rules.result import Level, Outcome, Result
 from colorama import Fore, Style
 from IPython.display import display, HTML
@@ -100,8 +101,15 @@ def sample_keys(keys: pd.Series, limit: int) -> str:
             sample = keys.sample(limit)
         else:
             sample = keys
+
+        def url(x: str) -> str:
+            if SH_URL in x:
+                return f"<a href='{x}'>{x.split('/')[-1]}</a>"
+            key, number = x.rsplit("/", 1)
+            return f"<a href='{SH_URL}/{key}/item/{number}'>{number}</a>"
+
         # make links only for Cloud data
         if keys.dtype == np.dtype("object") and "/" in keys.iloc[0]:
-            sample = [f"<a href='{k}'>{k.split('/')[-1]}</a>" for k in sample]
-        sample = ", ".join(sample)
-        return sample
+            sample = sample.apply(url)
+
+        return ", ".join(sample)

src/arche/rules/json_schema.py

@@ -22,7 +22,6 @@ def validate(schema: Schema, raw_items: RawItems, fast: bool = False) -> Result:
     schema_result_message = (
         f"{len(raw_items)} items were checked, {len(errors)} error(s)"
     )
-
     if errors:
         result.add_error(schema_result_message, errors=errors)
     else:

tests/test_arche.py

@@ -1,4 +1,4 @@
-from arche import arche
+from arche import arche, SH_URL
 from arche.arche import Arche
 from arche.rules.result import Level
 from conftest import create_result
@@ -245,6 +245,34 @@ def test_validate_with_json_schema(mocker, get_job_items, get_schema):
     assert a.report.results.get("JSON Schema Validation") == res
 
 
+def test_validate_with_json_schema_fails(mocker, get_job_items, get_schema):
+    mocked_html = mocker.patch("arche.report.HTML", autospec=True)
+    key = f"112358/13/21"
+    url_base = f"{SH_URL}/{key}/item"
+    res = create_result(
+        "JSON Schema Validation",
+        {
+            Level.ERROR: [
+                (
+                    "4 items were checked, 1 error(s)",
+                    None,
+                    {"'price' is a required property": {f"{key}/1"}},
+                )
+            ]
+        },
+    )
+    schema = {"type": "object", "required": ["price"]}
+    a = Arche("source", schema=schema)
+    a._source_items = get_job_items
+    a.validate_with_json_schema()
+
+    assert len(a.report.results) == 1
+    assert a.report.results.get("JSON Schema Validation") == res
+    mocked_html.assert_any_call(
+        f"1 items affected - 'price' is a required property: <a href='{url_base}/1'>1</a>"
+    )
+
+
 @pytest.mark.parametrize(
     "source, expected_message",
     [

tests/test_report.py

@@ -131,16 +131,22 @@ def test_write_detailed_errors(mocker, errors, short, keys_limit, expected_messa
         (
             pd.Series(f"{SH_URL}/112358/13/21/item/0"),
             5,
-            [f"{SH_URL}/112358/13/21/item/5"],
+            pd.Series(f"{SH_URL}/112358/13/21/item/5"),
             f"<a href='{SH_URL}/112358/13/21/item/0'>0</a>",
         ),
         (
             pd.Series([f"{SH_URL}/112358/13/21/item/{i}" for i in range(20)]),
             10,
-            [f"{SH_URL}/112358/13/21/item/5"],
+            pd.Series(f"{SH_URL}/112358/13/21/item/5"),
             f"<a href='{SH_URL}/112358/13/21/item/5'>5</a>",
         ),
         (pd.Series([str(i) for i in range(5)]), 1, ["0"], "0"),
+        (
+            pd.Series("112358/13/21/0"),
+            1,
+            pd.Series("112358/13/21/0"),
+            f"<a href='{SH_URL}/112358/13/21/item/0'>0</a>",
+        ),
     ],
 )
 def test_sample_keys(mocker, keys, limit, sample_mock, expected_sample):