codechecker/SmalllintReportGenerator.st
author Claus Gittinger <cg@exept.de>
Sun, 07 Aug 2011 11:30:17 +0200
changeset 36 2c0987f7230d
child 37 0a356190f3f3
permissions -rw-r--r--
initial checkin
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
36
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     1
"{ Package: 'stx:goodies/builder/codechecker' }"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     2
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     3
Object subclass:#SmalllintReportGenerator
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     4
	instanceVariableNames:'rules environment'
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     5
	classVariableNames:''
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     6
	poolDictionaries:''
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     7
	category:'Refactory-Lint'
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     8
!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
     9
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    10
!SmalllintReportGenerator class methodsFor:'documentation'!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    11
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    12
documentation
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    13
"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    14
    a standalone smallint runner.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    15
    Generates an xml report compatible to pmd, to be processed by hudson.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    16
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    17
    [author:]
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    18
        Claus Gittinger
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    19
"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    20
!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    21
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    22
examples
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    23
"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    24
    |checker|
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    25
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    26
    checker := self new.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    27
    checker addPackage:'exept:workflow'.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    28
    checker performChecks.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    29
    checker generateReportAs:'checkstyle.xml'
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    30
"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    31
! !
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    32
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    33
!SmalllintReportGenerator class methodsFor:'instance creation'!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    34
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    35
new
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    36
    "return an initialized instance"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    37
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    38
    ^ self basicNew initialize.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    39
! !
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    40
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    41
!SmalllintReportGenerator methodsFor:'checking'!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    42
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    43
performChecks
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    44
    rules withIndexDo:[:rule :index|
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    45
       Stdout showCR:('Checking: ', rule name).
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    46
       (SmalllintChecker runRule: rule onEnvironment: environment)
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    47
    ].
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    48
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    49
    "Created: / 07-08-2011 / 01:10:00 / cg"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    50
! !
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    51
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    52
!SmalllintReportGenerator methodsFor:'initialization'!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    53
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    54
initialize
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    55
    self setupRules.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    56
    self setupEnvironment
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    57
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    58
    "Modified: / 07-08-2011 / 01:14:51 / cg"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    59
! !
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    60
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    61
!SmalllintReportGenerator methodsFor:'reporting'!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    62
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    63
generateReportAs:aFilename
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    64
    aFilename asFilename writingFileDo:[:s | self generateReportOn:s].
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    65
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    66
    "Created: / 07-08-2011 / 01:17:46 / cg"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    67
!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    68
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    69
generateReportOn:aStream
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    70
    aStream nextPutLine: '<?xml version="1.0"?>'.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    71
    aStream nextPutLine: '<pmd>'.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    72
    rules do:[:eachRule | 
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    73
        eachRule problemCount > 0 ifTrue:[
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    74
            eachRule failedMethods do:[:method |
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    75
                |class classFileName s selector fullSource 
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    76
                 charPos lineNumber ruleName rationale|
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    77
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    78
                class := method mclass.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    79
                s := class localSourceStreamFor:class theNonMetaclass classFilename.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    80
                s notNil ifTrue:[
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    81
                    classFileName := s pathName.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    82
                    s close.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    83
                ] ifFalse:[
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    84
                    classFileName := class sourceFilename.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    85
                ].
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    86
                fullSource := class source.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    87
                charPos := method sourcePosition ? 1.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    88
                "/ q&d hack - editor knows
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    89
                lineNumber := (TextView new contents:fullSource)
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    90
                                lineOfCharacterPosition:charPos.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    91
                ruleName := eachRule name.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    92
                rationale := eachRule rationale.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    93
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    94
    aStream nextPutLine:('  <file name="%1">' bindWith:classFileName).
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    95
    aStream nextPutLine:('    <violation line="%1" rule="%2">' bindWith:lineNumber with:ruleName).
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    96
    aStream nextPutLine:('%1' bindWith:rationale).
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    97
    aStream nextPutLine: '    </violation>'.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    98
    aStream nextPutLine: '  </file>'.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
    99
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   100
            ].
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   101
        ]
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   102
    ].
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   103
    aStream nextPutLine: '</pmd>'.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   104
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   105
    "Created: / 07-08-2011 / 01:17:00 / cg"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   106
! !
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   107
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   108
!SmalllintReportGenerator methodsFor:'setup'!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   109
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   110
addClass:aClass
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   111
    environment addClass: aClass.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   112
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   113
    "Created: / 07-08-2011 / 01:11:33 / cg"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   114
!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   115
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   116
addPackage:aPackage
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   117
    Smalltalk loadPackage:aPackage.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   118
    Smalltalk allClassesInPackage:aPackage do:[:cls | self addClass:cls]
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   119
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   120
    "Created: / 07-08-2011 / 01:12:31 / cg"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   121
!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   122
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   123
setupEnvironment
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   124
    environment := ClassEnvironment new.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   125
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   126
    "Created: / 07-08-2011 / 01:10:56 / cg"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   127
!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   128
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   129
setupRules
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   130
    | allRule checks|
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   131
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   132
    allRule := RBCompositeLintRule allRules.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   133
    checks := allRule rules detect:[ :each | each name = 'Lint checks' ].
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   134
    checks rules: (checks rules reject: [ :each | each name = 'Squeak bugs' ]).
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   135
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   136
    rules := allRule flattened.
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   137
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   138
    "Created: / 07-08-2011 / 01:08:56 / cg"
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   139
! !
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   140
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   141
!SmalllintReportGenerator class methodsFor:'documentation'!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   142
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   143
version
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   144
    ^ '$Header$'
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   145
!
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   146
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   147
version_CVS
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   148
    ^ '$Header$'
2c0987f7230d initial checkin
Claus Gittinger <cg@exept.de>
parents:
diff changeset
   149
! !