Merge lp:~ki7mt/ubuntu-manual-tests/lp1168084-fix into lp:ubuntu-manual-tests

Proposed by Greg Beam
Status: Merged
Merged at revision: 267
Proposed branch: lp:~ki7mt/ubuntu-manual-tests/lp1168084-fix
Merge into: lp:ubuntu-manual-tests
Diff against target: 83 lines (+74/-0)
2 files modified
testcases/image/Install_Minimal_System (+37/-0)
testcases/image/Install_Minimal_Virtual_System (+37/-0)
To merge this branch: bzr merge lp:~ki7mt/ubuntu-manual-tests/lp1168084-fix
Reviewer Review Type Date Requested Status
Elfy (community) Approve
Nicholas Skaggs (community) Needs Fixing
Review via email: mp+201878@code.launchpad.net

Description of the change

To post a comment you must log in.
262. By Greg Beam

Fixed a couple </dt> Tags to Orignal MP

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

I would make the same comments regarding grammar and tags on this merge as on the other merge; https://code.launchpad.net/~ki7mt/ubuntu-manual-tests/server-upgrade-cli/+merge/202185/comments/474664

A couple other thoughts;

1) Don't supply an example username and password; let the user choose
2) I'm confused by the keyboard layout step;

15 + <li>Select layout, "No"</li>
16 + <li>Select country</li>
17 + <li>Select layout</li>

What does this mean?

Overall, good work. Thanks!

review: Needs Fixing
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Are you able to finish this? Just a couple minor edits are needed. Thanks!

Revision history for this message
Greg Beam (ki7mt) wrote :

Hi Nicholas,

I had forgotten about this actually. If you know what needs doing, go
ahead and fix it up, otherwise I'll have to dig into it again, as it's
been a while, and I certainly don't recall all the in's & out's.

Greg,

On 05/13/2014 05:54 PM, Nicholas Skaggs wrote:
> Are you able to finish this? Just a couple minor edits are needed. Thanks!

263. By Greg Beam

update from review comments

Revision history for this message
Elfy (elfy) wrote :

There are syntax errors in the Install_Minimal_System testcase - have a look at https://wiki.ubuntu.com/QATeam/ContributingTestcases/ManualStyleGuide/test_case_format_script

Typo "ncruses"

Not too sure what nskaggs comments re keyboard layout refer to, but it appears to be the same.

Virtual system testcase - ncruses typo.

Asa with the other MP - name me as reviewer and I'll get to it as soon as I can

Revision history for this message
Elfy (elfy) wrote :

missed the review type ...

review: Needs Fixing
264. By Greg Beam

Fix syntax and edit grammer

265. By Greg Beam

remove hostname, username and password suggestions

266. By Greg Beam

replace double quotes with <em>

267. By Greg Beam

fix more syntax errors

Revision history for this message
Elfy (elfy) wrote :

Thanks ki7mt - they look good to me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== added file 'testcases/image/Install_Minimal_System'
--- testcases/image/Install_Minimal_System 1970-01-01 00:00:00 +0000
+++ testcases/image/Install_Minimal_System 2014-06-25 10:54:40 +0000
@@ -0,0 +1,37 @@
1<dl>
2 <dt>Boot up the server image</dt>
3 <dt>Select language</dt>
4 <dt>Press F4, arrow down, select <em>Install a minimal system</em></dt>
5 <dt>Press Enter, selecting <em>Install FAMILY Server</em></dt>
6 <dt>Choose the language</dt>
7 <dt>Select your location</dt>
8 <dt>Configure the keyboard</dt>
9 <ul>
10 <li>Select layout, <em>No</em></li>
11 <li>Select country</li>
12 </ul>
13 <dt>Choose a hostname</dt>
14 <dt>Choose a username</dt>
15 <dt>Choose a password</dt>
16 <dt>Re-enter password</dt>
17 <dt>Encrypt your home directory, <em>No</em></dt>
18 <dt>Is this time zone correct? <em>Yes</em> or <em>No</em> to setup</dt>
19 <dt>Partitioning method, <em>Guided - Use entire disk</em></dt>
20 <dt>Select disk to partition, use the listed device</dt>
21 <dt>Write changes to disks, verify targets, if correct, <em>Yes</em> to start</dt>
22 <dt>http proxy request, leave blank</dt>
23 <dt>How do you want to manage upgrades on this system? <em>No automatic updates</em></dt>
24 <dt>Choose software, Tab to <em>continue</em>, no additional selections</dt>
25 <dt>Install the GRUB boot loader to the master boot record, <em>Yes</em></dt>
26 <dt>Remove the installation media (CDROM or USB key), then <em>continue</em></dt>
27 <dt>Wait for the system reboot, then login</dt>
28 <dt>Verify Internet access: sudo apt-get update</dt>
29 <dd>Provide sudo password</dd>
30 <dd>apt-get will perform a normal update</dd>
31 <dt>Verify access to tasksel: sudo tasksel</dt>
32 <dd>ncurses GUI will render, listing upgrade options</dd>
33 <dt>Select Tab, then <em>Ok</em> to exit</dt>
34 <dt>Logout</dt>
35</dl>
36<strong>If all actions produce the expected results listed, please <a href="results#add_result">submit</a> a 'passed' result.
37 If an action fails, or produces an unexpected result, please <a href="results#add_result">submit</a> a 'failed' result and <a href="../../buginstructions">file a bug</a>. Please be sure to include the bug number when you <a href="results#add_result">submit</a> your result.</strong>
038
=== added file 'testcases/image/Install_Minimal_Virtual_System'
--- testcases/image/Install_Minimal_Virtual_System 1970-01-01 00:00:00 +0000
+++ testcases/image/Install_Minimal_Virtual_System 2014-06-25 10:54:40 +0000
@@ -0,0 +1,37 @@
1<dl>
2 <dt>Boot up the server image</dt>
3 <dt>Select language</dt>
4 <dt>Press F4, arrow down, select <em>Install a virtual minimal system</em>, press Enter</dt>
5 <dt>Press Enter a 2nd time to <em>Install FAMILY Server</em></dt>
6 <dt>Choose the language</dt>
7 <dt>Select your location</dt>
8 <dt>Configure the keyboard</dt>
9 <ul>
10 <li>Select layout, <em>No</em></li>
11 <li>Select country</li>
12 </ul>
13 <dt>Choose a hostname</dt>
14 <dt>Choose a username</dt>
15 <dt>Choose a password</dt>
16 <dt>Re-enter password</dt>
17 <dt>Encrypt your home directory, <em>No</em></dt>
18 <dt>Is this time zone correct? <em>Yes</em> if correct, <em>No</em> to setup</dt>
19 <dt>Partitioning method, <em>Guided - Use entire disk</em></dt>
20 <dt>Select disk to partition, use the listed device</dt>
21 <dt>Write changes to disks, verify targets, if correct, <em>Yes</em> to start</dt>
22 <dt>http proxy request, leave blank, <em>continue</em></dt>
23 <dt>How do you want to manage upgrades on this system? <em>No automatic updates</em></dt>
24 <dt>Choose software to install, Tab to <em>continue</em>, no additional selections</dt>
25 <dt>Install the GRUB boot loader to the master boot record, <em>Yes</em></dt>
26 <dt>Remove the installation media (CDROM or USB key), then <em>continue</em></dt>
27 <dt>Wait for the system reboot and login</dt>
28 <dt>Verify internet access with: sudo apt-get update</dt>
29 <dd>When asked, add sudo password</dd>
30 <dd>apt-get will perform a normal update</dd>
31 <dt>Verify access to tasksel: sudo tasksel</dt>
32 <dd>ncurses GUI will render, listing upgrade options</dd>
33 <dt>Hit Tab, then <em>Ok</em> to exit.</dt>
34 <dt>Logout by typing: <em>logout</em></dt>
35</dl>
36<strong>If all actions produce the expected results listed, please <a href="results#add_result">submit</a> a 'passed' result.
37 If an action fails, or produces an unexpected result, please <a href="results#add_result">submit</a> a 'failed' result and <a href="../../buginstructions">file a bug</a>. Please be sure to include the bug number when you <a href="results#add_result">submit</a> your result.</strong>

Subscribers

People subscribed via source and target branches