Skip to content

Commit

Permalink
Fixed Patchfinder test
Browse files Browse the repository at this point in the history
- This test needs to be rewritten since it performs querying of the actual webpage which changed and caused the assert to fail
  • Loading branch information
ctevse committed Oct 20, 2023
1 parent bf3a5b7 commit b87b527
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion patchfinder/src/test/java/patches/PatchFinderThreadTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@
*/
public class PatchFinderThreadTest {

//TODO: This needs to be re-written to utilize mocks. This test was failing because the apache airflow github added more patch commits
@Test
public void testRun() {
HashMap<String, ArrayList<String>> cvePatchEntry = new HashMap<>();
Expand All @@ -57,7 +58,7 @@ public void testRun() {
PatchFinder patchFinder = Mockito.mock(PatchFinder.class);
//check the patch commits
List<PatchCommit> patchCommits = PatchFinder.getPatchCommits();
assertEquals(24, patchCommits.size());
assertEquals(48, patchCommits.size());

}

Expand Down

0 comments on commit b87b527

Please sign in to comment.