Skip to content

Commit 4f39e01

Browse files
feat(#1663): Add one more puzzle and remove the old puzzle related to #1663 issue
1 parent 6adb0e9 commit 4f39e01

File tree

2 files changed

+6
-11
lines changed

2 files changed

+6
-11
lines changed

src/main/java/com/jcabi/github/RepositoryStatistics.java

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -45,17 +45,6 @@
4545
* @author Volodya Lombrozo ([email protected])
4646
* @version $Id $
4747
* @since 1.8.0
48-
* @todo #1660:90min Add RepositoryStatistics.Smart.
49-
* Implement RepositoryStatistics.Smart and use it to retrieve repository
50-
* statistics with the following methods:
51-
* - RepositoryStatistics.Smart#language()
52-
* - RepositoryStatistics.Smart#forksCount()
53-
* - RepositoryStatistics.Smart#stargazers()
54-
* - RepositoryStatistics.Smart#watchers()
55-
* - RepositoryStatistics.Smart#size()
56-
* - RepositoryStatistics.Smart#openIssues()
57-
* In other words, it would be convenient to have particular methods with
58-
* understandable names instead of using toMap() method.
5948
*/
6049
public final class RepositoryStatistics {
6150

src/test/java/com/jcabi/github/RepositoryStatisticsTest.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,12 @@
5050
* MkRepo should return different types like integer, double, long,
5151
* etc. When it is implemented, we can replace strings with concrete
5252
* types and remove that puzzle.
53+
* @todo #1663:90min Refactor RepositoryStatisticsTest.
54+
* RepositoryStatisticsTest has too many boilerplate code. Also it
55+
* has repeated variables and constants. Refactor it to make it more
56+
* readable and maintainable. Moreover all this variables don't follow
57+
* PMD and Checkstyle rules. When it is done, remove this puzzle and
58+
* all the checkstyle and PMD suppressions.
5359
* @checkstyle StaticVariableNameCheck (1000 lines)
5460
* @checkstyle MagicNumberCheck (1000 lines)
5561
* @checkstyle LineLengthCheck (2 lines)

0 commit comments

Comments
 (0)