Skip to content

Commit e2385ad

Browse files
committed
rename vars, extend ApiTracer
1 parent e7bc1c3 commit e2385ad

File tree

1 file changed

+14
-18
lines changed

1 file changed

+14
-18
lines changed

gax-java/gax-httpjson/src/test/java/com/google/api/gax/httpjson/testing/TestApiTracer.java

Lines changed: 14 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
*/
3030
package com.google.api.gax.httpjson.testing;
3131

32-
import com.google.api.gax.tracing.BaseApiTracer;
32+
import com.google.api.gax.tracing.ApiTracer;
3333
import java.util.concurrent.atomic.AtomicBoolean;
3434
import java.util.concurrent.atomic.AtomicInteger;
3535
import org.threeten.bp.Duration;
@@ -38,45 +38,41 @@
3838
* Test tracer that keeps count of different events. See {@link TestApiTracerFactory} for more
3939
* details.
4040
*/
41-
public class TestApiTracer extends BaseApiTracer {
41+
public class TestApiTracer implements ApiTracer {
4242

43-
private final AtomicInteger tracerAttempts;
44-
private final AtomicInteger tracerAttemptsFailed;
45-
private final AtomicBoolean tracerOperationFailed;
46-
private final AtomicBoolean tracerFailedRetriesExhausted;
43+
private final AtomicInteger attemptsStarted;
44+
private final AtomicInteger attemptsFailed;
45+
private final AtomicBoolean operationFailed;
46+
private final AtomicBoolean retriesExhausted;
4747

4848
public TestApiTracer(
4949
AtomicInteger tracerAttempts,
5050
AtomicInteger tracerAttemptsFailed,
5151
AtomicBoolean tracerOperationFailed,
5252
AtomicBoolean tracerFailedRetriesExhausted) {
53-
this.tracerAttempts = tracerAttempts;
54-
this.tracerAttemptsFailed = tracerAttemptsFailed;
55-
this.tracerOperationFailed = tracerOperationFailed;
56-
this.tracerFailedRetriesExhausted = tracerFailedRetriesExhausted;
53+
this.attemptsStarted = tracerAttempts;
54+
this.attemptsFailed = tracerAttemptsFailed;
55+
this.operationFailed = tracerOperationFailed;
56+
this.retriesExhausted = tracerFailedRetriesExhausted;
5757
}
5858

5959
@Override
6060
public void attemptFailed(Throwable error, Duration delay) {
61-
tracerAttemptsFailed.incrementAndGet();
62-
super.attemptFailed(error, delay);
61+
attemptsFailed.incrementAndGet();
6362
}
6463

6564
@Override
6665
public void attemptStarted(int attemptNumber) {
67-
tracerAttempts.incrementAndGet();
68-
super.attemptStarted(attemptNumber);
66+
attemptsStarted.incrementAndGet();
6967
}
7068

7169
@Override
7270
public void operationFailed(Throwable error) {
73-
tracerOperationFailed.set(true);
74-
super.operationFailed(error);
71+
operationFailed.set(true);
7572
}
7673

7774
@Override
7875
public void attemptFailedRetriesExhausted(Throwable error) {
79-
tracerFailedRetriesExhausted.set(true);
80-
super.attemptFailedRetriesExhausted(error);
76+
retriesExhausted.set(true);
8177
}
8278
};

0 commit comments

Comments
 (0)