Skip to content

Commit f613c73

Browse files
authored
Cleaned up Javadoc (#1433)
Signed-off-by: dhoard <doug.hoard@gmail.com>
1 parent e00b279 commit f613c73

File tree

47 files changed

+2270
-495
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+2270
-495
lines changed

integration_test_suite/integration_tests/src/main/java/io/prometheus/jmx/test/support/environment/IsolatorExporterTestEnvironment.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,9 @@ public String getBaseUrl(int index) {
128128
return baseUrl + ":" + port;
129129
}
130130

131-
/** Method to destroy the test environment */
131+
/**
132+
* Method to destroy the test environment.
133+
*/
132134
public void destroy() {
133135
if (javaAgentApplicationContainer != null) {
134136
javaAgentApplicationContainer.stop();

integration_test_suite/integration_tests/src/main/java/io/prometheus/jmx/test/support/environment/JmxExporterTestEnvironment.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,9 @@ public String getBaseUrl() {
166166
return baseUrl + ":" + port;
167167
}
168168

169-
/** Method to destroy the test environment */
169+
/**
170+
* Method to destroy the test environment.
171+
*/
170172
public void destroy() {
171173
if (javaAgentApplicationContainer != null) {
172174
javaAgentApplicationContainer.stop();

integration_test_suite/integration_tests/src/main/java/io/prometheus/jmx/test/support/environment/PrometheusTestEnvironment.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,9 @@ private String getPrometheusBaseUrl() {
134134
return baseUrl + ":" + prometheusContainer.getMappedPort(9090);
135135
}
136136

137-
/** Method to wait for the Prometheus container to be ready */
137+
/**
138+
* Method to wait for the Prometheus container to be ready.
139+
*/
138140
public void waitForReady() {
139141
waitForReady(null, null);
140142
}
@@ -175,7 +177,9 @@ public void waitForReady(String username, String password) {
175177
prometheusContainer.getDockerImageName(), System.currentTimeMillis() - startMilliseconds));
176178
}
177179

178-
/** Method to destroy the test environment */
180+
/**
181+
* Method to destroy the test environment.
182+
*/
179183
public void destroy() {
180184
if (prometheusContainer != null) {
181185
prometheusContainer.stop();

integration_test_suite/integration_tests/src/main/java/io/prometheus/jmx/test/support/http/HttpRequest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,9 @@ public static Builder builder() {
112112
return new Builder();
113113
}
114114

115-
/** Class to implement Builder */
115+
/**
116+
* Class to implement Builder.
117+
*/
116118
public static class Builder {
117119

118120
private String url;

integration_test_suite/integration_tests/src/main/java/io/prometheus/jmx/test/support/metrics/MetricsParser.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -409,7 +409,9 @@ private static TreeMap<String, String> toLabels(List<Metrics.LabelPair> labelPai
409409
return labels;
410410
}
411411

412-
/** Class to read a Reader line by line with the ability to push a line back to the Reader */
412+
/**
413+
* Class to read a Reader line by line with the ability to push a line back to the Reader.
414+
*/
413415
private static class LineReader implements AutoCloseable {
414416

415417
private final LinkedList<String> lineBuffer;

integration_test_suite/integration_tests/src/main/java/io/prometheus/jmx/test/support/util/Repeater.java

Lines changed: 21 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,9 @@ public void run() throws Throwable {
174174
}
175175
}
176176

177-
/** Method to abort a test */
177+
/**
178+
* Method to abort a test.
179+
*/
178180
public static void abort() {
179181
throw new RuntimeException("4dbf35e7-415c-4e66-a61e-f6a7057e382e");
180182
}
@@ -191,25 +193,33 @@ public static void rethrow(Throwable throwable) throws Throwable {
191193
}
192194
}
193195

194-
/** Interface to define code */
196+
/**
197+
* Interface to define code.
198+
*/
195199
public interface ThrowableRunnable {
196200

197201
void run() throws Throwable;
198202
}
199203

200-
/** Interface to consume the result */
204+
/**
205+
* Interface to consume the result.
206+
*/
201207
public interface ThrowableConsumer {
202208

203209
void accept(int counter, Throwable throwable) throws Throwable;
204210
}
205211

206-
/** Interface to implement a Throttle */
212+
/**
213+
* Interface to implement a Throttle.
214+
*/
207215
public interface Throttle {
208216

209217
void throttle();
210218
}
211219

212-
/** Class to implement a fixed throttle */
220+
/**
221+
* Class to implement a fixed throttle.
222+
*/
213223
public static class FixedThrottle implements Throttle {
214224

215225
private final long milliseconds;
@@ -233,7 +243,9 @@ public void throttle() {
233243
}
234244
}
235245

236-
/** Class to implement a random throttle */
246+
/**
247+
* Class to implement a random throttle.
248+
*/
237249
public static class RandomThrottle implements Throttle {
238250

239251
private final long minMilliseconds;
@@ -273,7 +285,9 @@ public void throttle() {
273285
}
274286
}
275287

276-
/** Class to implement an exponential backoff throttle */
288+
/**
289+
* Class to implement an exponential backoff throttle.
290+
*/
277291
public static class ExponentialBackoffThrottle implements Throttle {
278292

279293
private final long maxMilliseconds;

integration_test_suite/integration_tests/src/test/java/io/prometheus/jmx/test/opentelemetry/BasicAuthenticationTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,9 @@ public void beforeAll(ArgumentContext argumentContext) {
7171
jmxExporterTestEnvironment.initialize(testClass, network);
7272
}
7373

74-
/** Method to test that metrics exist in Prometheus */
74+
/**
75+
* Method to test that metrics exist in Prometheus.
76+
*/
7577
@Verifyica.Test
7678
public void testPrometheusHasMetrics(ArgumentContext argumentContext) throws IOException {
7779
OpenTelemetryTestEnvironment openTelemetryTestEnvironment =

integration_test_suite/integration_tests/src/test/java/io/prometheus/jmx/test/opentelemetry/BasicTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,9 @@ public void beforeAll(ArgumentContext argumentContext) {
6868
jmxExporterTestEnvironment.initialize(testClass, network);
6969
}
7070

71-
/** Method to test that metrics exist in Prometheus */
71+
/**
72+
* Method to test that metrics exist in Prometheus.
73+
*/
7274
@Verifyica.Test
7375
public void testPrometheusHasMetrics(OpenTelemetryTestEnvironment openTelemetryTestEnvironment) throws IOException {
7476
PrometheusTestEnvironment prometheusTestEnvironment = openTelemetryTestEnvironment.prometheusTestEnvironment();

integration_test_suite/integration_tests/src/test/java/io/prometheus/jmx/test/opentelemetry/CombinedModeTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,9 @@ public void testPrometheusProtobufMetrics(OpenTelemetryTestEnvironment openTelem
149149
assertMetricsResponse(jmxExporterTestEnvironment, httpResponse, MetricsContentType.PROMETHEUS_PROTOBUF_METRICS);
150150
}
151151

152-
/** Method to test that metrics exist in Prometheus */
152+
/**
153+
* Method to test that metrics exist in Prometheus.
154+
*/
153155
@Verifyica.Test
154156
@Verifyica.Order(6)
155157
public void testPrometheusHasMetrics(OpenTelemetryTestEnvironment openTelemetryTestEnvironment) throws IOException {

integration_test_suite/integration_tests/src/test/java/io/prometheus/jmx/test/opentelemetry/ConfigurationTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,9 @@ public void beforeAll(ArgumentContext argumentContext) {
8080
jmxExporterTestEnvironment.initialize(testClass, network);
8181
}
8282

83-
/** Method to test that metrics exist in Prometheus */
83+
/**
84+
* Method to test that metrics exist in Prometheus.
85+
*/
8486
@Verifyica.Test
8587
public void testPrometheusHasMetrics(OpenTelemetryTestEnvironment openTelemetryTestEnvironment) throws IOException {
8688
JmxExporterTestEnvironment jmxExporterTestEnvironment = openTelemetryTestEnvironment.exporterTestEnvironment();

0 commit comments

Comments
 (0)