Skip to content

Commit 4916cf6

Browse files
committed
change comment
1 parent 90b334c commit 4916cf6

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

muzzle/src/test/java/io/opentelemetry/javaagent/tooling/muzzle/CacheProviderTest.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ void keySameRefEquivalence() {
4141
new AgentCachingPoolStrategy.TypeCacheKey(loaderHash, loaderRef, "foo");
4242

4343
assertThat(key1).hasSameHashCodeAs(key2).isEqualTo(key2);
44-
// ensures that loader isn't collected
44+
// use loader to ensures that it isn't collected before the test completes
4545
assertThat(loader).isNotNull();
4646
}
4747

@@ -60,7 +60,7 @@ void keyDifferentRefEquivalence() {
6060
assertThat(loaderRef1).isNotSameAs(loaderRef2);
6161

6262
assertThat(key1).hasSameHashCodeAs(key2).isEqualTo(key2);
63-
// ensures that loader isn't collected
63+
// use loader to ensures that it isn't collected before the test completes
6464
assertThat(loader).isNotNull();
6565
}
6666

@@ -77,7 +77,7 @@ void keyMismatchSameLoaderDifferentName() {
7777
// not strictly guaranteed -- but important for performance
7878
assertThat(fooKey.hashCode()).isNotEqualTo(barKey.hashCode());
7979
assertThat(fooKey).isNotEqualTo(barKey);
80-
// ensures that loader isn't collected
80+
// use loader to ensures that it isn't collected before the test completes
8181
assertThat(loader).isNotNull();
8282
}
8383

@@ -99,7 +99,7 @@ void keyMismatchSameNameDifferentLoader() {
9999
// not strictly guaranteed -- but important for performance
100100
assertThat(fooKey1.hashCode()).isNotEqualTo(fooKey2.hashCode());
101101
assertThat(fooKey1).isNotEqualTo(fooKey2);
102-
// ensures that loader isn't collected
102+
// use loader to ensures that it isn't collected before the test completes
103103
assertThat(loader1).isNotNull();
104104
assertThat(loader2).isNotNull();
105105
}
@@ -117,7 +117,7 @@ void testBasicCaching() {
117117

118118
// not strictly guaranteed, but fine for this test
119119
assertThat(cacheProvider.find("foo")).isNotNull();
120-
// ensures that loader isn't collected
120+
// use loader to ensures that it isn't collected before the test completes
121121
assertThat(loader).isNotNull();
122122
}
123123

@@ -138,7 +138,7 @@ void testLoaderEquivalence() {
138138

139139
assertThat(cacheProvider1A.find("foo")).isSameAs(cacheProvider1B.find("foo"));
140140

141-
// ensures that loader isn't collected
141+
// use loader to ensures that it isn't collected before the test completes
142142
assertThat(loader1).isNotNull();
143143
}
144144

@@ -161,7 +161,7 @@ void testLoaderSeparation() {
161161

162162
assertThat(cacheProvider1.find("foo")).isNotSameAs(cacheProvider2.find("foo"));
163163

164-
// ensures that loader isn't collected
164+
// use loader to ensures that it isn't collected before the test completes
165165
assertThat(loader1).isNotNull();
166166
assertThat(loader2).isNotNull();
167167
}

0 commit comments

Comments
 (0)