Harmony
  1. Harmony
  2. HARMONY-1997

[classlib]adopt CheckStyle to follow junit best practice

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Classlib
    • Labels:
      None

      Description

      There are many violations found by Mark's tools(Harmony-1960). To fix and avoid them, I suggest to use CheckStyle as a Eclipse plugin, which will highlights all possiable violate lines in your editor windows.

      I post my configuration here as an initial. It must be not very precise, you can customize it by yourself. Any bug report are welcome.

      HOW TO
      1,Install CheckStyle, you can find it at http://eclipse-cs.sourceforge.net/ .
      2,After that you can find an item named checkstyle in your eclipse menu-windows-preferences, click the new button and then import my configuration.
      3,In the properties of a project, there is also an item named checkstyle, you can enable checkstyle for your project here. remember to select the right configuration you just import.
      4,If you get an out of memory error, you can do something in its exclude options. for example, exclude files not opened in the editor.

      1. junitBP.xml
        11 kB
        Tony Wu
      2. junitBP.xml
        10 kB
        Tony Wu
      3. junitBP.xml
        10 kB
        Tony Wu

        Activity

        Hide
        Tim Ellison added a comment -

        Tony: I'm closing this as it is left to the discretion of the contributors to adhere to the guidelines.

        Show
        Tim Ellison added a comment - Tony: I'm closing this as it is left to the discretion of the contributors to adhere to the guidelines.
        Hide
        Tony Wu added a comment -

        update checking for assertSame

        Show
        Tony Wu added a comment - update checking for assertSame
        Hide
        Tony Wu added a comment -

        updated

        Show
        Tony Wu added a comment - updated

          People

          • Assignee:
            Tim Ellison
            Reporter:
            Tony Wu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development