Skip to content

Commit 425fa89

Browse files
committed
fix: cache test
1 parent ff2ba5f commit 425fa89

1 file changed

Lines changed: 5 additions & 7 deletions

File tree

tests/src/test/java/io/snyk/languageserver/protocolextension/SnykExtendedLanguageClientTest.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -155,11 +155,11 @@ void testSendsPluginInstalledEventAndRefreshFeatureFlagOnFirstStart() {
155155

156156
cut = new SnykExtendedLanguageClient();
157157

158-
ArgumentCaptor<Consumer<SnykExtendedLanguageClient>> captor =
158+
ArgumentCaptor<Consumer<SnykExtendedLanguageClient>> captor =
159159
ArgumentCaptor.forClass((Class<Consumer<SnykExtendedLanguageClient>>) (Class<?>) Consumer.class);
160160
verify(asMock, times(2)).registerTask(captor.capture(), any());
161161
verifyNoMoreInteractions(asMock);
162-
}
162+
}
163163
}
164164

165165

@@ -172,7 +172,7 @@ void testDoesNotSendPluginInstalledEventOnSecondStart() {
172172

173173
cut = new SnykExtendedLanguageClient();
174174

175-
ArgumentCaptor<Consumer<SnykExtendedLanguageClient>> captor =
175+
ArgumentCaptor<Consumer<SnykExtendedLanguageClient>> captor =
176176
ArgumentCaptor.forClass((Class<Consumer<SnykExtendedLanguageClient>>) (Class<?>) Consumer.class);
177177
verify(asMock, times(1)).registerTask(captor.capture(), any());
178178
verifyNoMoreInteractions(asMock);
@@ -183,10 +183,10 @@ void testDoesNotSendPluginInstalledEventOnSecondStart() {
183183
void testPublishDiagnosticsShouldChangeCache() {
184184
// Test Add to cache
185185
String folderPath = "/a/b";
186-
var issueCache = IssueCacheHolder.getInstance().getCacheInstance(folderPath);
187186

188187
var uri = "file://" + folderPath + "/c/";
189188
var filePath = LSPEclipseUtils.fromUri(URI.create(uri)).getAbsolutePath();
189+
var issueCache = IssueCacheHolder.getInstance().getCacheInstance(filePath);
190190

191191
var param = new PublishDiagnostics316Param();
192192
param.setUri(uri);
@@ -221,7 +221,6 @@ void testPublishDiagnosticsShouldChangeCache() {
221221
future = cut.publishDiagnostics316(param);
222222
try {
223223
future.thenRun(() -> {
224-
// FIXME This appears to be flaky at times
225224
assertEquals(true, issueCache.getCodeSecurityIssuesForPath(filePath).isEmpty());
226225
});
227226
} catch (Exception ex) {
@@ -345,7 +344,7 @@ void testSnykScanSuccessAddsInfoNodes_IssuesFound_onlyIgnoredDisplayed() {
345344
param.setStatus(SCAN_STATE_SUCCESS);
346345
param.setProduct(SCAN_PARAMS_CODE);
347346
param.setFolderPath("a/b/c");
348-
347+
349348
pref.store(Preferences.IS_GLOBAL_IGNORES_FEATURE_ENABLED, "true");
350349
pref.store(Preferences.FILTER_IGNORES_SHOW_OPEN_ISSUES, "true");
351350
pref.store(Preferences.FILTER_IGNORES_SHOW_IGNORED_ISSUES, "false");
@@ -363,7 +362,6 @@ void testSnykScanSuccessAddsInfoNodes_IssuesFound_onlyOpenDisplayed() {
363362
param.setStatus(SCAN_STATE_SUCCESS);
364363
param.setProduct(SCAN_PARAMS_CODE);
365364
param.setFolderPath("a/b/c");
366-
367365
pref.store(Preferences.IS_GLOBAL_IGNORES_FEATURE_ENABLED, "true");
368366
pref.store(Preferences.FILTER_IGNORES_SHOW_OPEN_ISSUES, "false");
369367
pref.store(Preferences.FILTER_IGNORES_SHOW_IGNORED_ISSUES, "true");

0 commit comments

Comments
 (0)