| | 46 | Resolutions:Bugs are described and fixed in tickets |
| | 47 | == Analysis of bugs == |
| | 48 | Reporting an issue is the analysis of the bug. Bugs can be reported by all registered users and by anonymous users, but with limitations. If the following things are not described, the bug may be considered invalid. The text in ''italic'' is not mandatory. |
| | 49 | * Summary: |
| | 50 | * ''May start with "Crash:", "Tweak:", "Unexpected behavior:". Crash means that the report form is evoked. Error report is needed as an attachment'' |
| | 51 | * ''"TLID:" testlink id, if the bug can be reproduced by executing a testcase'' |
| | 52 | * Short summary, custom text, what exactly happens in few words |
| | 53 | * Type: bug |
| | 54 | * Priority: determine the priority of the bug. |
| | 55 | * ''Keywords - fill in related keywords, for example "text"'' |
| | 56 | * ''Components - fill in component if you can determine the component'' |
| | 57 | * reporter: DevID, automatically added by trac |
| | 58 | * Milestone - current milestone |
| | 59 | * ''cc: a reminder to someone'' |
| | 60 | * Analysis_owners: your DevID |
| | 61 | * Description: contains more detailed description and steps to recreate in order to reproduce the bug. |
| | 62 | * If a bug is a TLID type it must be clarified in the comment of the ticket on which step the application crashes. |
| | 63 | * ''Attachments: you may add attachments to your ticket (screenshots, crash logs, books) reffering to an attachment named "filename.ext" is done by ![attachment:filename.ext]'' |
| | 64 | * ''It is recommended that the attachments' names do not have spaces or any special symbols because you might not be able to link them properly in the ticket.'' |
| | 65 | |
| | 66 | When you create the ticket, move it's status to analysis finished. |
| | 67 | |
| | 68 | * Design |
| | 69 | * Design and implementation are done in a separate branch. |
| | 70 | * Design can be described in a comment of the ticket. If the space isn't enough (the fix is not trivial), the designer may create a wiki page named BUG_<ticket_id> to put his design section there (where <ticket_id> is the number of the ticket in trac. This page should be linked in the description field of the ticket. Bug pages are created using bug page template. When design is finished, the status should be updated and implementation may start without a design review. Same goes for the implementstion. |
| | 71 | * Design should have a system test that reproduces them. It should fail at design phase and pass at implementation. |
| | 72 | * In design and implementation the attachments must be linked properly if mentioned. |
| | 73 | * Implementation: Implementation changeset should be linked. The ticket status is moved to implementation finished. |
| | 74 | * Test: Test is performed by the reporter or in rare cases by an integrator. If the bug is not present, the ticket is closed. |
| | 75 | |
| | 76 | Reviews and resolutions: |
| | 77 | When the developer understands the bug, he moves it to analysis ok phase and starts the design. Review of the design and implementation is done by the integrators. |
| | 78 | * Analysis |
| | 79 | * The description describes the problem and the bug can be reproduced by following the instructions there - 3p |
| | 80 | * The ticket contains crash report +0.5p |
| | 81 | * The ticket contains attached books, screenshots, etc +0.5p (+1p) |
| | 82 | * The ticket contains linked test case +1p |
| | 83 | * The ticket contains links to related bugs, tasks, external links +0.5p |
| | 84 | Design and implementation are usually reviewed at once |
| | 85 | * Design |
| | 86 | * Explanation how it will be fixed |
| | 87 | * Methods |
| | 88 | * New classes if needed |
| | 89 | * Tests - system test that proves bug is present before the implementation and fixed after it. |
| | 90 | * What caused that issue? |
| | 91 | * Implementation |
| | 92 | * Lame implementation (bad code) and hacks are not allowed |
| | 93 | * Should follow the design |
| | 94 | * Test: Testing is performed by the reporter or in some needed cases by an integrator. If the problem is not present the ticket is closed. The test is not reviewed. If any testcases are related they should be executed. |
| | 95 | |
