Bugzilla – Full Text Bug Listing
| Summary: | [Usability] Switch to Test Perspective after creating a test asset. | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Paul Slauenwhite <paulslau> | ||||||
| Component: | TPTP | Assignee: | Tony Wang <newboya1988> | ||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||
| Severity: | normal | ||||||||
| Priority: | P1 | CC: | jkubasta, sluiman | ||||||
| Version: | unspecified | Keywords: | plan | ||||||
| Target Milestone: | --- | Flags: | paulslau:
review-
|
||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| Whiteboard: | |||||||||
| Bug Depends on: | |||||||||
| Bug Blocks: | 214671, 230280, 231376 | ||||||||
| Attachments: |
|
||||||||
|
Description
Paul Slauenwhite
Bulk update of the target field to reflect the change in iteration numbers. See https://bugs.eclipse.org/bugs/show_bug.cgi?id=186148#c0 for the code to open the Test Perspective. Deferred from I5 to I6 with PMC approval. Created attachment 90672 [details]
Changes for defect 206356
Paul, please review this patch. Patch reviewed. A couple of comments:
-When creating patches for review, ensure only modified code is included to
make the review process easier. For example, does not generate a patch for a
class that has whitespace changes, unless they are meaningful to the code
changes.
-There is no need for the following in src/org/eclipse/hyades/test/ui/internal/wizard/ArtifactNewWizard.java:
+ public void run()
+ {
+ }
-I have created a utility method in org.eclipse.hyades.test.ui.util.TestUIUtil.java for opening the Test Perspective that can be called when the editors are opened.
-Please make sure the copyright end date is updated on each modified file to
2008.
-Please make sure author and version tags are added to the class JavaDoc
comments for each new/modified file. For example:
@author Tony Wang
@version February 28, 2008
Created attachment 90997 [details]
Patch Version 2
Tony, please see the changes I made to your original patch.
Patch Version 2 checked into CVS (HEAD). I have also added the dialog to prompt the user if they want to switch to the Test Perspective and the associated preference. In addition, the same dialog and preference was added for opening the editor after creating a test asset. Since removed under https://bugs.eclipse.org/bugs/show_bug.cgi?id=230280. Closing. |