Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit 7f05e63

Browse files
committed
Merge master into shana/fix-tests
2 parents d4d6175 + 09fca34 commit 7f05e63

File tree

1 file changed

+8
-15
lines changed

1 file changed

+8
-15
lines changed

src/UnitTests/GitHub.App/Caches/CredentialCacheTests.cs

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -172,13 +172,13 @@ public class TheInvalidateMethod : TestBaseClass
172172
[Fact]
173173
public async Task InvalidatesTheCredential()
174174
{
175+
const string key = nameof(InvalidatesTheCredential);
175176
using (var credentialCache = new CredentialCache())
176177
{
177178
var credential = Tuple.Create("somebody", "somebody's secret");
178-
await credentialCache.InsertObject(nameof(InvalidatesTheCredential), credential);
179-
await credentialCache.Invalidate(nameof(InvalidatesTheCredential));
180-
181-
await Assert.ThrowsAsync<KeyNotFoundException>(async () => await credentialCache.Get(nameof(InvalidatesTheCredential)));
179+
await credentialCache.InsertObject(key, credential);
180+
await credentialCache.Invalidate(key);
181+
await Assert.ThrowsAsync<KeyNotFoundException>(async () => await credentialCache.Get(key));
182182
}
183183
}
184184

@@ -214,17 +214,10 @@ public async Task InvalidatesTheCredential()
214214
const string key = nameof(InvalidatesTheCredential);
215215
using (var credentialCache = new CredentialCache())
216216
{
217-
try
218-
{
219-
var credential = Tuple.Create("somebody", "somebody's secret");
220-
await credentialCache.InsertObject(key, credential);
221-
}
222-
finally
223-
{
224-
await credentialCache.InvalidateObject<Tuple<string, string>>(key);
225-
226-
await Assert.ThrowsAsync<KeyNotFoundException>(async () => await credentialCache.Get("unknownkey"));
227-
}
217+
var credential = Tuple.Create("somebody", "somebody's secret");
218+
await credentialCache.InsertObject(key, credential);
219+
await credentialCache.InvalidateObject<Tuple<string, string>>(key);
220+
await Assert.ThrowsAsync<KeyNotFoundException>(async () => await credentialCache.Get(key));
228221
}
229222
}
230223

0 commit comments

Comments
 (0)