Merge lp://staging/~wxl/ubuntu-manual-tests/alternate-manual into lp://staging/ubuntu-manual-tests

Proposed by ԜаӀtеr Ⅼарсһуnѕkі
Status: Merged
Merged at revision: 319
Proposed branch: lp://staging/~wxl/ubuntu-manual-tests/alternate-manual
Merge into: lp://staging/ubuntu-manual-tests
Diff against target: 54 lines (+50/-0)
1 file modified
testcases/image/Alternate Install (Manual Partitioning) (+50/-0)
To merge this branch: bzr merge lp://staging/~wxl/ubuntu-manual-tests/alternate-manual
Reviewer Review Type Date Requested Status
Nicholas Skaggs (community) Approve
Elfy Pending
Ubuntu Testcase Admins Pending
Review via email: mp+251026@code.staging.launchpad.net

Description of the change

I just got rid of Lubuntu's Alternate image encrypted/LVM testcase from our testsuite. We don't have a similar testcase in Desktop and frankly, we really don't need one. There isn't, however, a manual partitioning testcase for Alternate images, so I made one up. Now we have the same requirements for both sets of images.

To post a comment you must log in.
309. By ԜаӀtеr Ⅼарсһуnѕkі

adding alternate manual partition testcase

Revision history for this message
David Pires (slickymaster) wrote :

Walter, I trust you won't mind my input, but I'd like to point just three minor details.

There's an extra space after the opening parenthesis in:
dd>You should be prompted to confirm ( If you entered a weak password, confirm that you want to keep it or select no and enter a stronger one).</dd>

There's also an extra space between the hyphen and 'The' in:
<dd>Wait for a while - The install may seem like it has locked at a percentage but is actually working in the background. You can check and see if it is working by switching to tty4 (alt + f4).</dd>

The utc nomenclature should perhaps be capitalized in:
<dt>Select Yes to setting your clock to utc and press Enter</dt>

Revision history for this message
ԜаӀtеr Ⅼарсһуnѕkі (wxl) wrote :

Actually, these are problems with the "Alternate" testcases in general, it seems. I'll do that up as a separate merge proposal. Thanks!

Revision history for this message
Istimsak (saqman2060) wrote :

Yeah I see that same errors mentioned by David. Overall, the testcase seems pretty clear to follow. Strange, I never tested the alternative images, but there was no testcase for manual partitioning?

Revision history for this message
ԜаӀtеr Ⅼарсһуnѕkі (wxl) wrote :

Yeah, it's funny— I hadn't paid attention to it, either! ☺

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

Merged with the aforementioned changes. :-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches