unit tests: Add cases for btrfs subvol

Add test cases to verify new and modified logic for handling btrfs
subvolumes as kdump targets.

Assisted-by: Claude (Anthropic)
Signed-off-by: Lichen Liu <[email protected]>

Author: licliu

spec/kdump-lib-initramfs_spec.sh

@@ -67,6 +67,12 @@ Describe 'kdump-lib-initramfs'
 				else
 					printf '/var %s[/ostree/deploy/default/var]\n/sysroot %s\n' "$7" "$7"
 				fi
+			elif [[ "$7" == '/dev/nvme0n1p3' ]]; then
+				if [[ "$8" == "-f" ]]; then
+					printf '/ /dev/nvme0n1p3[/root]\n'
+				else
+					printf '/ /dev/nvme0n1p3[/root]\n/var /dev/nvme0n1p3[/var]\n'
+				fi
 			fi
 		}
 
@@ -76,19 +82,51 @@ Describe 'kdump-lib-initramfs'
 			#  - IPv6 NFS target also contain '[' in the export
 			#  - local dumping target that has '[' in the name
 			#  - has bind mint
+			#  - has subvol
 			Parameters
-				'eng.redhat.com:/srv/[nfs]' '/mnt'
-				'[2620:52:0:a1:217:38ff:fe01:131]:/srv/[nfs]' '/mnt'
-				'/dev/mapper/rhel[disk]' '/'
-				'/dev/vda4' '/sysroot'
+				'eng.redhat.com:/srv/[nfs]' '' '/mnt'
+				'[2620:52:0:a1:217:38ff:fe01:131]:/srv/[nfs]' '' '/mnt'
+				'/dev/mapper/rhel[disk]' '' '/'
+				'/dev/vda4' '' '/sysroot'
+				'/dev/nvme0n1p3' '/var' '/var'
 			End
 
 			It 'should handle all cases correctly'
-				When call get_mntpoint_from_target "$1"
-				The output should equal "$2"
+				When call get_mntpoint_from_target "$1" "$2"
+				The output should equal "$3"
 			End
 		End
 
 	End
 
+	Describe 'Test get_btrfs_subvol_from_mntopt'
+		Context 'Given different mount option formats'
+			# Test the following cases:
+			#  - Standard subvol option at the beginning
+			#  - Standard subvol option in the middle
+			#  - Standard subvol option at the end
+			#  - Subvol with @ prefix (common btrfs pattern)
+			#  - Subvol with complex path
+			#  - No subvol option present
+			#  - Empty mount options
+			#  - Subvol option with no value
+			#  - Similar but different option names
+			Parameters
+				"subvol=/home,rw,relatime" "/home"
+				"rw,subvol=/root,defaults" "/root"
+				"rw,relatime,subvol=/root" "/root"
+				"subvol=@home" "@home"
+				"subvol=/path/to/subvolume,rw" "/path/to/subvolume"
+				"rw,relatime,defaults" ""
+				true ''
+				"subvol=,rw" ""
+				"rw,subvolume=@home" ""
+			End
+
+			It 'should extract subvolume correctly'
+				When call get_btrfs_subvol_from_mntopt "$1"
+				The output should equal "$2"
+			End
+		End
+	End
 End

spec/kdump-lib_spec.sh

@@ -133,4 +133,52 @@ Describe 'kdump-lib'
 		End
 	End
 
+	Describe "get_kdump_mntpoint_from_target with btrfs subvolume support"
+		# Mock get_mntpoint_from_target to simulate different mount scenarios
+		get_mntpoint_from_target() {
+			local device="$1" subvol="$2"
+			case "$device" in
+				'/dev/sda1')
+					if [[ "$subvol" == "@home" ]]; then
+						echo "/mnt/btrfs"
+					elif [[ "$subvol" == "/root" ]]; then
+						echo "/"
+					else
+						echo "/"
+					fi
+					;;
+				'/dev/sdb1')
+					if [[ "$subvol" == "@backup" ]]; then
+						echo "/backup"
+					else
+						echo "/backup"
+					fi
+					;;
+				*)
+					echo "/"
+					;;
+			esac
+		}
+
+		Context 'Given various mount point scenarios with subvolumes'
+			# Test the following cases:
+			#  - Root filesystem with @ subvolume -> /sysroot
+			#  - Home subvolume on non-root -> /kdumproot/mnt/btrfs
+			#  - Backup subvolume -> /kdumproot/backup
+			#  - Regular filesystem without subvolume -> /sysroot
+			Parameters
+				'/dev/sda1 /root /sysroot'
+				'/dev/sda1 @home /kdumproot/mnt/btrfs'
+				'/dev/sdb1 @backup /kdumproot/backup'
+				'/dev/vda2 "" /sysroot'
+			End
+
+			It 'should handle btrfs subvolumes correctly'
+				# Parse parameters: device subvol expected_result
+				read -r device subvol expected_result <<< "$1"
+				When call get_kdump_mntpoint_from_target "$device" "$subvol"
+				The output should equal "$expected_result"
+			End
+		End
+	End
 End