fixes #682

Author: jph00

fastcore/_modidx.py

@@ -514,8 +514,10 @@
                               'fastcore.xdg.xdg_state_home': ('xdg.html#xdg_state_home', 'fastcore/xdg.py')},
             'fastcore.xml': { 'fastcore.xml.FT': ('xml.html#ft', 'fastcore/xml.py'),
                               'fastcore.xml.FT.__call__': ('xml.html#ft.__call__', 'fastcore/xml.py'),
+                              'fastcore.xml.FT.__eq__': ('xml.html#ft.__eq__', 'fastcore/xml.py'),
                               'fastcore.xml.FT.__getattr__': ('xml.html#ft.__getattr__', 'fastcore/xml.py'),
                               'fastcore.xml.FT.__getitem__': ('xml.html#ft.__getitem__', 'fastcore/xml.py'),
+                              'fastcore.xml.FT.__hash__': ('xml.html#ft.__hash__', 'fastcore/xml.py'),
                               'fastcore.xml.FT.__html__': ('xml.html#ft.__html__', 'fastcore/xml.py'),
                               'fastcore.xml.FT.__init__': ('xml.html#ft.__init__', 'fastcore/xml.py'),
                               'fastcore.xml.FT.__iter__': ('xml.html#ft.__iter__', 'fastcore/xml.py'),

fastcore/xml.py

@@ -220,6 +220,15 @@ def to_xml(elm, lvl=0, indent=True, do_escape=True):
 def __html__(self:FT): return to_xml(self, indent=False)
 FT.__str__ = FT.__html__
 
+# %% ../nbs/09_xml.ipynb
+@patch
+def __eq__(self:FT, other):
+    if not isinstance(other, FT): return False
+    return self.tag==other.tag and self.attrs==other.attrs and self.children==other.children
+
+@patch
+def __hash__(self:FT): return hash((self.tag, tuple(sorted(self.attrs.items())), self.children))
+
 # %% ../nbs/09_xml.ipynb
 def highlight(s, lang='html'):
     "Markdown to syntax-highlight `s` in language `lang`"

nbs/09_xml.ipynb

@@ -48,7 +48,7 @@
     "from IPython.display import Markdown\n",
     "from pprint import pprint\n",
     "\n",
-    "from fastcore.test import test_eq"
+    "from fastcore.test import test_eq, test_ne"
    ]
   },
   {
@@ -743,6 +743,47 @@
     "print(Div('ho'))"
    ]
   },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "id": "cd8a6aff",
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "#| export\n",
+    "@patch\n",
+    "def __eq__(self:FT, other):\n",
+    "    if not isinstance(other, FT): return False\n",
+    "    return self.tag==other.tag and self.attrs==other.attrs and self.children==other.children\n",
+    "\n",
+    "@patch\n",
+    "def __hash__(self:FT): return hash((self.tag, tuple(sorted(self.attrs.items())), self.children))"
+   ]
+  },
+  {
+   "cell_type": "markdown",
+   "id": "1abc3f67",
+   "metadata": {},
+   "source": [
+    "`FT` object equality and hashing is based on tag, attrs, and children."
+   ]
+  },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "id": "8f665b3d",
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "test_eq(Div('hello', id='x'), Div('hello', id='x'))\n",
+    "test_ne(Div('hello'), Div('goodbye'))\n",
+    "test_ne(Div('hello', id='a'), Div('hello', id='b'))\n",
+    "test_ne(P('hello'), Div('hello'))\n",
+    "\n",
+    "test_eq(hash(Div('hello', id='x')), hash(Div('hello', id='x')))\n",
+    "assert hash(Div('hello')), hash(Div('goodbye'))"
+   ]
+  },
   {
    "cell_type": "markdown",
    "id": "5ad30d7c",
@@ -869,13 +910,7 @@
    "source": []
   }
  ],
- "metadata": {
-  "kernelspec": {
-   "display_name": "python3",
-   "language": "python",
-   "name": "python3"
-  }
- },
+ "metadata": {},
  "nbformat": 4,
  "nbformat_minor": 5
 }