feat(#1660): Add test of RepositoryStatistics by using MkGithub

volodya-lombrozo · volodya-lombrozo · commit 42f248fb9b37 · 2023-08-04T12:38:40.000+03:00
diff --git a/src/main/java/com/jcabi/github/RepositoryStatistics.java b/src/main/java/com/jcabi/github/RepositoryStatistics.java
@@ -5,7 +5,6 @@
 import java.util.Map;
 import java.util.stream.Collectors;
 import javax.json.JsonObject;
-import javax.json.JsonValue;
 import org.cactoos.map.MapEntry;
 import org.cactoos.map.MapOf;
 
@@ -40,13 +39,7 @@ private enum KEY {
         }
 
         private MapEntry<String, Object> toEntry(final JsonObject object) {
-            final String k = this.getKey();
-            final JsonValue value = object.get(k);
-            return new MapEntry<>(k, value.toString());
-        }
-
-        public String getKey() {
-            return key;
+            return new MapEntry<>(this.key, object.get(this.key).toString());
         }
     }
 }
diff --git a/src/test/java/com/jcabi/github/RepositoryStatisticsTest.java b/src/test/java/com/jcabi/github/RepositoryStatisticsTest.java
@@ -1,7 +1,7 @@
 package com.jcabi.github;
 
+import com.jcabi.github.mock.MkGithub;
 import java.io.IOException;
-import java.util.Map;
 import org.hamcrest.MatcherAssert;
 import org.hamcrest.Matchers;
 import org.junit.Test;
@@ -16,7 +16,13 @@ public class RepositoryStatisticsTest {
     public void gathersBasicStatisticsFromRepo() throws IOException {
         MatcherAssert.assertThat(
             "We expect to have basic statistics from repo",
-            new RepositoryStatistics(null).toMap(),
+            new RepositoryStatistics(
+                new MkGithub()
+                    .repos()
+                    .create(
+                        new Repos.RepoCreate("volodya-lombrozo", false)
+                    )
+            ).toMap(),
             Matchers.allOf(
                 Matchers.hasKey("language"),
                 Matchers.hasKey("forks_count"),

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,6 @@`
`5`	`5`	`import java.util.Map;`
`6`	`6`	`import java.util.stream.Collectors;`
`7`	`7`	`import javax.json.JsonObject;`
`8`		`-import javax.json.JsonValue;`
`9`	`8`	`import org.cactoos.map.MapEntry;`
`10`	`9`	`import org.cactoos.map.MapOf;`
`11`	`10`
`@@ -40,13 +39,7 @@ private enum KEY {`
`40`	`39`	`}`
`41`	`40`
`42`	`41`	`private MapEntry<String, Object> toEntry(final JsonObject object) {`
`43`		`- final String k = this.getKey();`
`44`		`- final JsonValue value = object.get(k);`
`45`		`- return new MapEntry<>(k, value.toString());`
`46`		`- }`
`47`		`-`
`48`		`- public String getKey() {`
`49`		`- return key;`
	`42`	`+ return new MapEntry<>(this.key, object.get(this.key).toString());`
`50`	`43`	`}`
`51`	`44`	`}`
`52`	`45`	`}`