From 836f1ee7c49bcde993e082010213202b73fd1409 Mon Sep 17 00:00:00 2001
From: Mike McLean <mikem@redhat.com>
Date: Oct 11 2017 04:40:48 +0000
Subject: PR#629: fix wrong mock.patch target


Merges #629
https://pagure.io/koji/pull-request/629

Fixes #627
https://pagure.io/koji/issue/627

---

diff --git a/tests/test_lib_py2only/test_tasks.py b/tests/test_lib_py2only/test_tasks.py
index 5cee010..6cbc298 100644
--- a/tests/test_lib_py2only/test_tasks.py
+++ b/tests/test_lib_py2only/test_tasks.py
@@ -36,6 +36,8 @@ def get_fake_mounts_file():
         'cgroup /sys/fs/cgroup/memory cgroup rw,nosuid,nodev,noexec,relatime,memory 0 0\n'
         'cgroup /sys/fs/cgroup/hugetlb cgroup rw,nosuid,nodev,noexec,relatime,hugetlb 0 0\n'
         'configfs /sys/kernel/config configfs rw,relatime 0 0\n'
+        'hugetlbfs /dev/hugepages hugetlbfs rw,seclabel,relatime 0 0\n'
+        'mqueue /dev/mqueue mqueue rw,seclabel,relatime 0 0\n'
     )))
 
 
@@ -80,9 +82,8 @@ class TasksTestCase(TestCase):
         """
         fake_mounts_file_contents = get_fake_mounts_file()
 
-        with patch('{0}.open'.format(__name__), return_value=fake_mounts_file_contents, create=True):
-            self.assertIn(scan_mounts('/dev'), [['/dev/shm', '/dev/pts', '/dev/mqueue', '/dev/hugepages', '/dev'],
-                                                ['/dev/shm', '/dev/pts', '/dev/mqueue', '/dev/console', '/dev']])
+        with patch('koji.tasks.open', return_value=fake_mounts_file_contents, create=True):
+            self.assertIn(scan_mounts('/dev'), [['/dev/shm', '/dev/pts', '/dev/mqueue', '/dev/hugepages', '/dev']])
 
     def test_scan_mounts_no_results(self):
         """ Tests the scan_mounts function with a mocked /proc/mounts file. An argument of /nonexistent/path
@@ -90,7 +91,7 @@ class TasksTestCase(TestCase):
         """
         fake_mounts_file_contents = get_fake_mounts_file()
 
-        with patch('{0}.open'.format(__name__), return_value=fake_mounts_file_contents, create=True):
+        with patch('koji.tasks.open', return_value=fake_mounts_file_contents, create=True):
             self.assertEquals(scan_mounts('/nonexistent/path'), [])
 
     # Patching the scan_mounts function instead of the built-in open function because this is only testing umount_all