Difference between revisions of "User:Torley Linden/Bug reporting"

From Second Life Wiki
Jump to navigation Jump to search
Line 10: Line 10:


== The recipe (parts of a repro) ==
== The recipe (parts of a repro) ==
* '''-''' - used to indicate a general step.
* '''-''' - A dash is used to at the beginning of a general step.
* '''- Verify:''' - I don't use these very often, but they're useful to include in longer, detailed test plans to guide someone along to ensure they're either following the steps correctly, or found a problem with the repro.
* '''- Verify:''' - I don't use these very often, but they're useful to include in longer, detailed test plans to guide someone along to ensure they're either following the steps correctly, or found a problem with the repro.
* '''* Observe:''' - A special step that tells the person following the repro what goes wrong.
* '''* Observe:''' - A special step that tells the person following the repro what goes wrong.
Line 19: Line 19:


=== Example bug repro ===
=== Example bug repro ===
'''Summary:''' WindLight: Changing Light color and right-clicking Color box crashes viewer
This bug is fictional and solely provided for educational purposes.
- Right-click an attachment on your avatar and select Edit.
- In the Edit tools, check "Edit linked parts".
- Click the Features tab and check "Light".
- Click the Color box.
- Verify: The Color Picker appears.
- Select a new color (any works) and click Select.
- Verify: The Color box changes color.
- Right-click the Color box.
* Observe: The viewer crashes.
= Expected: The viewer shouldn't crash.
> Workaround: Disabling Water Reflections in Preferences > Graphics tab (with Custom checked) stops this from happening.


[NEEDS LINK See how this issue looks on the actual Issue Tracker!]
: <font size="3"><tt>'''''Summary:'' WindLight: Changing Light color and right-clicking Color box crashes viewer'''<br>
: - Right-click an attachment on your avatar and select Edit.<br>
: - In the Edit tools, check "Edit linked parts".<br>
: - Click the Features tab and check "Light".<br>
: - Click the Color box.<br>
: - Verify: The Color Picker appears.<br>
: - Choose a new color (any works) and click Select.<br>
: - Verify: The Color box changes color.<br>
: - Right-click the Color box.<br>
: <nowiki>*</nowiki> Observe: The viewer crashes.<br>
: = Expected: The viewer shouldn't crash.<br>
: > Workaround: Disabling Water Reflections in Preferences > Graphics tab (with Custom checked) stops the crash from happening.</tt></font>
 
<font color=#ff0099>[NEEDS LINK See how this issue looks on the actual Issue Tracker!]</font>


=== Additional notes ===
=== Additional notes ===

Revision as of 14:51, 4 December 2007

OMG WIP (WORK IN PROGRESS)!!!!

Torley's bug reporting best practices

The following owes a lot to Linden QA Engineers and helpful Residents, in particular Brent Linden the repro format, and I'm writing it down because I know it'll be useful to others. These aren't the rules for bug-reporting, but merely how I, Torley Linden, do it. Feel free to better 'em!

What makes a good bug repro?

In case you aren't yet familiar, "repro" is short for "reproduction", which means: a way to reliably make a bug happen. Why? So the bug can be investigated and fixed — which is a very good thing (TM). Think of modern medicine, where symptoms may point to a cause, or consider detectives & CSI, who look for clues to solve crimes. Finding a good repro is analogous and essential.

Second Life is unique in that, more than other technology-based experiences, we have a strong emphasis in getting our community's help in fixing bugs, especially through our public Issue Tracker. I know, JIRA (software made by Atlassian that powers the Issue Tracker) may look confusing at first, but if you've ever filled out a web survey (even one of those fun questionnaires on MySpace or FaceBook), the process is basically the same. You'll feel a lot more comfortable actually using the Issue Tracker a few times, so don't let anxiety cloud your mind, and take it easy at your own pace. If you're smart enough to use Second Life, you're smart enough to report bugs — it need not be a technical, complicated process, as you'll see.

The recipe (parts of a repro)

  • - - A dash is used to at the beginning of a general step.
  • - Verify: - I don't use these very often, but they're useful to include in longer, detailed test plans to guide someone along to ensure they're either following the steps correctly, or found a problem with the repro.
  • * Observe: - A special step that tells the person following the repro what goes wrong.
  • = Expected: - A special step describing the correct behavior. This is what needs to be fixed.
  • > Workaround: - If there's a way around the bug until a better fix is in place, share it here. This can help mitigate the pain a bug causes, so I recommend workarounds to be included whenever applicable.

Cook 'em together and you get a dish like this...

Example bug repro

This bug is fictional and solely provided for educational purposes.

Summary: WindLight: Changing Light color and right-clicking Color box crashes viewer
- Right-click an attachment on your avatar and select Edit.
- In the Edit tools, check "Edit linked parts".
- Click the Features tab and check "Light".
- Click the Color box.
- Verify: The Color Picker appears.
- Choose a new color (any works) and click Select.
- Verify: The Color box changes color.
- Right-click the Color box.
* Observe: The viewer crashes.
= Expected: The viewer shouldn't crash.
> Workaround: Disabling Water Reflections in Preferences > Graphics tab (with Custom checked) stops the crash from happening.

[NEEDS LINK See how this issue looks on the actual Issue Tracker!]

Additional notes

  • "Viewer" refers to the Second Life application you run on your computer, not to be confused with "Second Life" as an online world in its entirety. If a bug caused all of Second Life to crash, that would be very bad indeed! I generally prefer "viewer" instead of "Second Life" because it's more clear what I'm referring to. If a bug crashes a region (also known as a "sim"), then "crashes region" would be appropriate.
  • If the bug is specific to a certain viewer or feature-set, e.g., "WindLight", "Havok4", "Search", etc. I find it useful to include that at the beginning of the summary. Here's another example: "Search: Looking for avatars with last names ending in 'x' fails". This is largely up to personal preference and isn't required, but does make it easier to sort through issues at-a-glance.
  • You can include other notes before or after the repro. This isn't a strict format, but a simple outline to base your own repros on! Improvise and have fun. :)

The meal (essentials)

  • A clear summary - A summary — the title, or subject line — should tell us what the bug is about in generally 80 characters or less. "Second Life crashes" is a sucky summary, "Viewer crashes after right-clicking on avatar" is much better. "OMG INVENTORY IZ TOO BIG" is crappy, "Inventory window can't be resized smaller like in previous versions" is a clear improvement. Get the idea?
    • An even clearer summary - As an additional guide,
  • Easy-to-follow steps - A repro is worthless without these, just like poor driving directions will get you lost and frustrated. Keep your steps as terse as possible (e.g., not big paragraphs of eye-hurting text), and if a step looks like it's getting too long to read in a single breath, split it into two steps, each on a new line. Universal rule-of-thumb: if two trusted friends can't follow your repro, it's not helpful.
  • Exact wording of user interface elements - If a window is labeled "Communicate", don't refer to it as the "Talk window". If a checkbox says "Edit linked parts", don't call it "Edit linked things". Simply refer to things as they appear. And if you don't know the difference between a button and a menu, or the different types of windows, learn more about elements of graphical user interfaces. If you use a computer, which you do, these are good-to-know.
  • Be specific - This reinforces the above two: if someone told you to "click on the thing", would you have a clue which "thing" they're talking about? Of course not. Details can make or break a good repro. If you're really not sure how to address something, ask for help, or post a screenshot (keep reading for more on that).

Sauces & spices (make your repro even better)

  • Include as much useful info as possible -
  • Communicate visually - A wonderful thing about our Issue Tracker is that you can attach screenshots and even videos using the "Attach file" and "Attach screenshot" links on the left. In both cases, you'll see an comment field, which is useful for saying something like, "See the above image I just attached for more info".
    • Attach file - Clicking on this brings you to a new screen where you can click the "Browse..." button and select a file from your hard drive to upload. Don't upload BMP images, they're too big and don't automatically get a thumbnail preview. I recommend using JPGs in most instances, and PNGs when you need lossless quality (for example, you're trying to show the subtle graphical differences between two screenshots and don't want JPG compression to degrade how things look). And if you're uploading videos, make sure they're viewable with QuickTime Player because Second Life uses QT and many Residents already have it installed. Don't use an obscure codec that others can't easily view.
    • Attach screenshot - Clicking on this pops up a new window (make sure your popup blocker is off or allows this), where you can easily select an image in another application (even in Photoshop), copy it, then paste it in here. You'll see a preview before it goes through, and you can give it a descriptive name if needed.

Desserts

The finished dish

Here's what one of my finished bug reports looks like:

Doggie bag (further help)

Understand that not all bugs have repros. But when they do, that definitely should be included.