added additional condition to check for busybox existance, added tests for inode and hardlink test

Author: Stefan Friese

test/test_modules.py

@@ -350,6 +350,16 @@ def test_file(host):
     assert f == host.file("/d/f")
     assert not d == f
 
+    host.check_output("ln /d/f /d/h")
+    hardlink = host.file("/d/h")
+    assert hardlink.is_file
+    assert not hardlink.is_symlink
+    assert isinstance(hardlink.inode, int)
+    assert isinstance(f.inode, int)
+    assert hardlink.inode == f.inode
+    assert f == host.file("/d/f")
+    assert not d == f
+
     host.check_output("rm -f /d/p && mkfifo /d/p")
     assert host.file("/d/p").is_pipe
 

testinfra/modules/process.py

@@ -109,8 +109,9 @@ def _get_process_attribute_by_pid(self, pid, name):
     def get_module_class(cls, host):
         if host.file("/bin/ps").linked_to == "/bin/busybox":
             return BusyboxProcess
-        if host.file("/bin/ps").inode == host.file("/bin/busybox").inode:
-            return BusyboxProcess
+        if host.file("/bin/busybox").exists:
+            if host.file("/bin/ps").inode == host.file("/bin/busybox").inode:
+                return BusyboxProcess
         if host.system_info.type == "linux" or host.system_info.type.endswith("bsd"):
             return PosixProcess
         raise NotImplementedError