Rename test classes to have suffix Test in singular form #407
Labels
Comments
|
Thanks!!! |
dionisiydk
added a commit
to dionisiydk/Calypso
that referenced
this issue
Jan 4, 2019
- all queries are renamed to have suffix Query - all queriy tests are rename accordingly pharo-ide#407: - all tests with singular suffix Test instead of Tests
|
Script to perform rename: allTests := TestCase allSubclasses select: [ :each | each package name beginsWith: 'Calypso' ].
toRename := allTests select: [ :each | each name endsWith: 'Tests' ].
"toRename collect: [ :each | each name allButLast ]."
toRename do: [ :each | | c |
c := SycRenameClassCommand new targetClass: each; newName: (each name allButLast) asSymbol.
c execute ]. |
dionisiydk
added a commit
to dionisiydk/Calypso
that referenced
this issue
Jan 10, 2019
- all queries are renamed to have suffix Query - all queriy tests are rename accordingly Changes are done with following script: ```Smalltalk toRename := ClyQuery allSubclasses reject: [ :each | each name includesSubstring: 'Query' ]. toRename do: [ :each | | c | c := SycRenameClassCommand new targetClass: each; newName: (each name, 'Query') asSymbol. c execute ]. testsToRename := toRename collect: [ :each | (each name withoutSuffix: 'Query'), 'Tests' ] thenSelect: [ :each | (self environment at: each asSymbol ifAbsent: [ nil ]) notNil ]. testsToRename do: [ :each | | c | c := SycRenameClassCommand new targetClass: each asClass; newName: ((each withoutSuffix: 'Tests'), 'QueryTest') asSymbol. c execute ]. ``` pharo-ide#407: - all tests are renamed to have singular suffix Test instead of Tests Changes are done with following script: ```Smalltalk allTests := TestCase allSubclasses select: [ :each | each package name beginsWith: 'Calypso' ]. toRename := allTests select: [ :each | each name endsWith: 'Tests' ]. "toRename collect: [ :each | each name allButLast ]." toRename do: [ :each | | c | c := SycRenameClassCommand new targetClass: each; newName: (each name allButLast) asSymbol. c execute ]. ```
dionisiydk
added a commit
to dionisiydk/Calypso
that referenced
this issue
Jan 10, 2019
- all queries are renamed to have suffix Query - all queriy tests are rename accordingly Changes are done with following script: ```Smalltalk toRename := ClyQuery allSubclasses reject: [ :each | each name includesSubstring: 'Query' ]. toRename do: [ :each | | c | c := SycRenameClassCommand new targetClass: each; newName: (each name, 'Query') asSymbol. c execute ]. testsToRename := toRename collect: [ :each | (each name withoutSuffix: 'Query'), 'Tests' ] thenSelect: [ :each | (self environment at: each asSymbol ifAbsent: [ nil ]) notNil ]. testsToRename do: [ :each | | c | c := SycRenameClassCommand new targetClass: each asClass; newName: ((each withoutSuffix: 'Tests'), 'QueryTest') asSymbol. c execute ]. ``` pharo-ide#407: - all tests are renamed to have singular suffix Test instead of Tests Changes are done with following script: ```Smalltalk allTests := TestCase allSubclasses select: [ :each | each package name beginsWith: 'Calypso' ]. toRename := allTests select: [ :each | each name endsWith: 'Tests' ]. "toRename collect: [ :each | each name allButLast ]." toRename do: [ :each | | c | c := SycRenameClassCommand new targetClass: each; newName: (each name allButLast) asSymbol. c execute ]. ```
|
It's done now |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Now there are explicit rules in Pharo which kindly force naming test cases with Test suffix instead of Tests.
The text was updated successfully, but these errors were encountered: