From a692425f5fbad2c7242d672dd1aa113b6f1ccd5d Mon Sep 17 00:00:00 2001
From: svalouch <svalouch@valouch.com>
Date: Mon, 23 Dec 2019 09:35:29 +0100
Subject: [PATCH] zfs: add __repr__

---
 src/simplezfs/zfs.py        | 3 +++
 src/simplezfs/zfs_cli.py    | 3 +++
 src/simplezfs/zfs_native.py | 3 +++
 3 files changed, 9 insertions(+)

diff --git a/src/simplezfs/zfs.py b/src/simplezfs/zfs.py
index 98bb874..20671ab 100644
--- a/src/simplezfs/zfs.py
+++ b/src/simplezfs/zfs.py
@@ -62,6 +62,9 @@ class ZFS:
         self.pe_helper = pe_helper
         self.use_pe_helper = use_pe_helper
 
+    def __repr__(self) -> str:
+        return f'<ZFS(pe_helper="{self._pe_helper}", use_pe_helper="{self._use_pe_helper}")>'
+
     @property
     def metadata_namespace(self) -> Optional[str]:
         '''
diff --git a/src/simplezfs/zfs_cli.py b/src/simplezfs/zfs_cli.py
index b3d9f81..7245263 100644
--- a/src/simplezfs/zfs_cli.py
+++ b/src/simplezfs/zfs_cli.py
@@ -33,6 +33,9 @@ class ZFSCli(ZFS):
         super().__init__(metadata_namespace=metadata_namespace)
         self.find_executable(path=zfs_exe)
 
+    def __repr__(self) -> str:
+        return f'<ZFSCli(exe="{self.__exe}", pe_helper="{self._pe_helper}", use_pe_helper="{self._use_pe_helper}")>'
+
     def find_executable(self, path: str = None):
         '''
         Tries to find the executable ``zfs(8)``. If ``path`` points to an executable, it is used instead of relying on
diff --git a/src/simplezfs/zfs_native.py b/src/simplezfs/zfs_native.py
index dda8d6c..7c9f546 100644
--- a/src/simplezfs/zfs_native.py
+++ b/src/simplezfs/zfs_native.py
@@ -22,6 +22,9 @@ class ZFSNative(ZFS):
                  use_pe_helper: bool = False, **kwargs) -> None:
         super().__init__(metadata_namespace=metadata_namespace)
 
+    def __repr__(self) -> str:
+        return f'<ZFSNative(pe_helper="{self._pe_helper}", use_pe_helper="{self._use_pe_helper}")>'
+
     def set_property(self, dataset: str, key: str, value: str, *, metadata: bool = False, overwrite_metadata_namespace: Optional[str] = None) -> None:
         raise NotImplementedError