diff --git a/pyanaconda/modules/storage/devicetree/viewer.py b/pyanaconda/modules/storage/devicetree/viewer.py index 89d269c6c1a9..541dafed8b27 100644 --- a/pyanaconda/modules/storage/devicetree/viewer.py +++ b/pyanaconda/modules/storage/devicetree/viewer.py @@ -541,19 +541,3 @@ def get_required_mount_points(self): [p for p in platform.partitions if p.mountpoint and p.mountpoint != "/boot"] + [root_partition])) return ret - - def get_recommended_mount_points(self): - """Get list of recommended mount points for the current platform - - Currently it contains only /boot partition if it is default the for - platform. - - :return: a list of mount points with its constraints - """ - # FIXME in general /boot is not required, just recommended. Depending - # on the filesystem on the root partition it may be required (ie - # crypted root). - recommended = ["/boot"] - ret = list(map(self._get_platform_mount_point_data, - [p for p in platform.partitions if p.mountpoint in recommended])) - return ret diff --git a/pyanaconda/modules/storage/devicetree/viewer_interface.py b/pyanaconda/modules/storage/devicetree/viewer_interface.py index 416d994507a7..667e6e9f1c63 100644 --- a/pyanaconda/modules/storage/devicetree/viewer_interface.py +++ b/pyanaconda/modules/storage/devicetree/viewer_interface.py @@ -221,14 +221,3 @@ def GetRequiredMountPoints(self) -> List[Structure]: """ return MountPointConstraintsData.to_structure_list( self.implementation.get_required_mount_points()) - - def GetRecommendedMountPoints(self) -> List[Structure]: - """Get list of recommended mount points for the current platform - - Currently it contains only /boot partition if it is default the for - platform. - - :return: a list of mount points with its constraints - """ - return MountPointConstraintsData.to_structure_list( - self.implementation.get_recommended_mount_points()) diff --git a/tests/unit_tests/pyanaconda_tests/modules/storage/test_module_device_tree.py b/tests/unit_tests/pyanaconda_tests/modules/storage/test_module_device_tree.py index e8b5da6fab40..5a18cbab1d02 100644 --- a/tests/unit_tests/pyanaconda_tests/modules/storage/test_module_device_tree.py +++ b/tests/unit_tests/pyanaconda_tests/modules/storage/test_module_device_tree.py @@ -874,20 +874,6 @@ def test_get_required_mount_points(self): assert root.mount_point == "/" assert root.required_filesystem_type == "" - def test_get_recommended_mount_points(self): - """Test GetRecommendedMountPoints.""" - result = self.interface.GetRecommendedMountPoints() - assert isinstance(result, list) - assert len(result) == 1 - - result = MountPointConstraintsData.from_structure_list(self.interface.GetRecommendedMountPoints()) - boot = result[0] - assert boot is not None - assert boot.encryption_allowed is False - assert boot.logical_volume_allowed is False - assert boot.mount_point == "/boot" - assert boot.required_filesystem_type == "" - class DeviceTreeTasksTestCase(unittest.TestCase): """Test the storage tasks."""