Merge ubuntu-manual-tests:reusehome into ubuntu-manual-tests:main

Proposed by Tim Andersson
Status: Merged
Merged at revision: 7f285b627464c61bee14b2858786a8a350de03ef
Proposed branch: ubuntu-manual-tests:reusehome
Merge into: ubuntu-manual-tests:main
Diff against target: 152 lines (+55/-7)
5 files modified
definitions/canary_desktop_cases.xml (+25/-2)
testcases/image/1470_Install (manual partitioning re-using home partition) (+9/-1)
testcases/image/1761_Canary Install (manual partitioning) (+1/-1)
testcases/image/1768_Canary Install (manual partitioning re-using home partition) (+18/-1)
testcases/image/1772_Canary Install (erase and re-install) (+2/-2)
Reviewer Review Type Date Requested Status
Brian Murray Approve
Review via email: mp+449640@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

I'm not exactly clear on where the test file should be located. Is it something like '/home/my-file-which-should-exist-later'? or is it '~/my-file-which-should-exist-later'?

I tested with the former, '/home/my-file-which-should-exist-later', and received a Permission denied error so I don't think you meant that. But if you did we should clarify the instructions so that we state that you need to create the file as root.

If it is the latter case I think we can use '~' as a shortcut without confusing too many people provided we explicitly call out the exact commands to use e.g. 'touch ~/my-file-which-I-want-preserved'.

review: Needs Information
Revision history for this message
Brian Murray (brian-murray) wrote :

Looking at this again it seems like the problem was sorted although its not clear from the git history. I think this line resolves the issue:

<dt>Create a file in the home directory to check for later.</dt>

Given that there is more ambiguity here I'm good with this MP, so feel free to merge it and update the server accordingly.

Although this might also need adding to test case id 1768.

review: Approve
Revision history for this message
Tim Andersson (andersson123) wrote :

I really messed up rebasing this after making some changes, I'll amend now

Revision history for this message
Tim Andersson (andersson123) wrote :

Okay this is amended, please re-review

Revision history for this message
Brian Murray (brian-murray) wrote :

I think there is one issue I've identified in an in-line comment but I'll just fix it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/definitions/canary_desktop_cases.xml b/definitions/canary_desktop_cases.xml
2index a616525..db4847c 100644
3--- a/definitions/canary_desktop_cases.xml
4+++ b/definitions/canary_desktop_cases.xml
5@@ -90,7 +90,7 @@
6 </ut:test>
7
8 <ut:test id="custom-partition">
9- <dt>Select 'Something Else' and click 'Continue'</dt>
10+ <dt>Select 'Manual Partitioning' and click 'Next'</dt>
11 <dd>A screen showing the current hard disks and partition layouts is displayed</dd>
12 <dt>Select the drive you wish to partition and use the Add '+', Change 'Change', and Delete '-' buttons to create your desired scheme</dt>
13 <dd>The screen updates showing your desired partitions and mount points</dd>
14@@ -274,7 +274,8 @@
15 </ut:case>
16
17 <ut:test id="reuse-postinst">
18- <dt>Login, ensure that the /home partition is still in use and the data from the previous installation have been preserved</dt>
19+ <dt>Login, and check the following command runs without any errors:</dt>
20+ <dd>echo /home/$USER/dummy_file</dd>
21 </ut:test>
22
23 <ut:test id="entire-disk-zfs">
24@@ -347,6 +348,25 @@
25 <dt>Click on the Next button</dt>
26 </ut:test>
27
28+ <ut:test id="reuse-home-initial-check">
29+ <dt>Open a terminal using your FAMILY commands or GUI to do so, in terminal enter <code>df -h</code> and press enter</dt>
30+ <dt>Terminal echoes the output of the df command</dt>
31+ <dd>Check that / and /home use different partitions</dd>
32+ </ut:test>
33+
34+ <ut:test id="reuse-home-setup-test-file">
35+ <dt>Create a file in the /home/$USER/ directory to check for later.</dt>
36+ <dt>Run the following command to create a file we'll check for later:</dt>
37+ <dd>echo "dummy file" > /home/$USER/dummy_file</dd>
38+ </ut:test>
39+
40+ <ut:test id="reinstall-reuse-home">
41+ <dt>Shutdown the installed system.</dt>
42+ <dt>Put the installation medium back in, and boot into the installation of a new system.</dt>
43+ <dd>Do everything as before, but when you get to the manual partitioning screen, make sure to remove the old '/' partition, create a new one, and make sure the partition used for '/home' has also been selected to have '/home' as the mount point (highlight the existing home partition, and click the 'Change' button), but make sure the reformat check box is left unchecked.</dd>
44+ <dt>Continue installation as normal, and reboot once finished.</dt>
45+ </ut:test>
46+
47 <ut:case id="1761_Canary Install (manual partitioning)">
48 <ut:include ref="grub" />
49 <ut:include ref="try-or-install" />
50@@ -429,6 +449,9 @@
51 <ut:include ref="reuse-partition" />
52 <ut:include ref="setup" />
53 <ut:include ref="reboot-ubuntu" />
54+ <ut:include ref="reuse-home-initial-check"/>
55+ <ut:include ref="reuse-home-setup-test-file"/>
56+ <ut:include ref="reinstall-reuse-home"/>
57 <ut:include ref="reuse-postinst" />
58 </ut:case>
59
60diff --git a/testcases/image/1470_Install (manual partitioning re-using home partition) b/testcases/image/1470_Install (manual partitioning re-using home partition)
61index ab63cfd..fa621f1 100755
62--- a/testcases/image/1470_Install (manual partitioning re-using home partition)
63+++ b/testcases/image/1470_Install (manual partitioning re-using home partition)
64@@ -19,7 +19,7 @@
65 </dd>
66 <dt>Click on the continue button</dt>
67 <dd>The 'Installation type' screen is displayed</dd>
68- <dt>Select Something Else</dt>
69+ <dt>Select Manual Partitioning</dt>
70 <dd>A screen showing the current hard disks and partition layouts is displayed</dd>
71 <dt>Select the drive you wish to partition and use the 'Add', 'Change', and 'Delete' buttons to create your desired scheme</dt>
72 <dd>The screen updates showing your desired partitions and mount points</dd>
73@@ -47,6 +47,14 @@
74 <dt>Open a terminal using your FAMILY commands or GUI to do so, in terminal enter <code>df -h</code> and press enter</dt>
75 <dd>Terminal echoes the output of the df command</dd>
76 <dt>Check that / and /home use different partitions</dt>
77+ <dt>Create a file in the /home/$USER/ directory to check for later.</dt>
78+ <dt>Run the following command to create a file we'll check for later:</dt>
79+ <dd>echo "dummy file" > /home/$USER/dummy_file</dd>
80+ <dt>Shutdown the installed system.</dt>
81+ <dt>Put the installation medium back in, and boot into the installation of a new system.</dt>
82+ <dd>Do everything as before, but when you get to the manual partitioning screen, make sure to remove the old '/' partition, create a new one, and make sure the partition used for '/home' has also been selected to have '/home' as the mount point (highlight the existing home partition, and click the 'Change' button), but make sure the reformat check box is left unchecked.</dd>
83+ <dt>Continue installation as normal, and reboot once finished.</dt>
84+ <dt>Boot into the installed system and check the file you created in the previous install is present in the '/home/' directory</dt>
85 </dl>
86
87 <strong>If all actions produce the expected results listed, please <a href="results#add_result">submit</a> a 'passed' result.<br />
88diff --git a/testcases/image/1761_Canary Install (manual partitioning) b/testcases/image/1761_Canary Install (manual partitioning)
89index 8679d60..16694ea 100644
90--- a/testcases/image/1761_Canary Install (manual partitioning)
91+++ b/testcases/image/1761_Canary Install (manual partitioning)
92@@ -45,7 +45,7 @@
93
94
95
96- <dt>Select 'Something Else' and click 'Continue'</dt>
97+ <dt>Select 'Manual Partitioning' and click 'Next'</dt>
98 <dd>A screen showing the current hard disks and partition layouts is displayed</dd>
99 <dt>Select the drive you wish to partition and use the Add '+', Change 'Change', and Delete '-' buttons to create your desired scheme</dt>
100 <dd>The screen updates showing your desired partitions and mount points</dd>
101diff --git a/testcases/image/1768_Canary Install (manual partitioning re-using home partition) b/testcases/image/1768_Canary Install (manual partitioning re-using home partition)
102index 127d487..4a20870 100644
103--- a/testcases/image/1768_Canary Install (manual partitioning re-using home partition)
104+++ b/testcases/image/1768_Canary Install (manual partitioning re-using home partition)
105@@ -80,7 +80,24 @@
106 <dd>The system boots properly and loads into FAMILY showing username selected</dd>
107
108
109- <dt>Login, ensure that the /home partition is still in use and the data from the previous installation have been preserved</dt>
110+ <dt>Open a terminal using your FAMILY commands or GUI to do so, in terminal enter <code>df -h</code> and press enter</dt>
111+ <dt>Terminal echoes the output of the df command</dt>
112+ <dd>Check that / and /home use different partitions</dd>
113+
114+
115+ <dt>Create a file in the /home/$USER/ directory to check for later.</dt>
116+ <dt>Run the following command to create a file we'll check for later:</dt>
117+ <dd>echo "dummy file" &gt; /home/$USER/dummy_file</dd>
118+
119+
120+ <dt>Shutdown the installed system.</dt>
121+ <dt>Put the installation medium back in, and boot into the installation of a new system.</dt>
122+ <dd>Do everything as before, but when you get to the manual partitioning screen, make sure to remove the old '/' partition, create a new one, and make sure the partition used for '/home' has also been selected to have '/home' as the mount point (highlight the existing home partition, and click the 'Change' button), but make sure the reformat check box is left unchecked.</dd>
123+ <dt>Continue installation as normal, and reboot once finished.</dt>
124+
125+
126+ <dt>Login, and check the following command runs without any errors:</dt>
127+ <dd>echo /home/$USER/dummy_file</dd>
128
129
130 </dl>
131diff --git a/testcases/image/1772_Canary Install (erase and re-install) b/testcases/image/1772_Canary Install (erase and re-install)
132index 6e4196b..055c415 100644
133--- a/testcases/image/1772_Canary Install (erase and re-install)
134+++ b/testcases/image/1772_Canary Install (erase and re-install)
135@@ -44,7 +44,7 @@
136 <dd>The 'Installation type' screen is displayed</dd>
137
138
139- <dt>Select 'Something Else' and click 'Continue'</dt>
140+ <dt>Select 'Manual Partitioning' and click 'Next'</dt>
141 <dd>A screen showing the current hard disks and partition layouts is displayed</dd>
142 <dt>Select the drive you wish to partition and use the Add '+', Change 'Change', and Delete '-' buttons to create your desired scheme</dt>
143 <dd>The screen updates showing your desired partitions and mount points</dd>
144@@ -116,7 +116,7 @@
145
146
147
148- <dt>Select 'Something Else' and click 'Continue'</dt>
149+ <dt>Select 'Manual Partitioning' and click 'Next'</dt>
150 <dd>A screen showing the current hard disks and partition layouts is displayed</dd>
151 <dt>Select the drive you wish to partition and use the Add '+', Change 'Change', and Delete '-' buttons to create your desired scheme</dt>
152 <dd>The screen updates showing your desired partitions and mount points</dd>

Subscribers

People subscribed via source and target branches

to all changes: