From 8237052ef5f3e334eb5915b1a84502c33a4f794a Mon Sep 17 00:00:00 2001 From: AudricV <74829229+AudricV@users.noreply.github.com> Date: Mon, 7 Aug 2023 19:12:17 +0200 Subject: [PATCH] Fix wrong assertion in assertNotEmpty The non-null assertion was made on the exception message instead of the string to check, causing a NullPointerException if the string to check was null. --- .../java/org/schabi/newpipe/extractor/ExtractorAsserts.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/extractor/src/test/java/org/schabi/newpipe/extractor/ExtractorAsserts.java b/extractor/src/test/java/org/schabi/newpipe/extractor/ExtractorAsserts.java index 07e58f4aa..b47f6d5ee 100644 --- a/extractor/src/test/java/org/schabi/newpipe/extractor/ExtractorAsserts.java +++ b/extractor/src/test/java/org/schabi/newpipe/extractor/ExtractorAsserts.java @@ -56,7 +56,7 @@ public class ExtractorAsserts { } public static void assertNotEmpty(@Nullable String message, String stringToCheck) { - assertNotNull(message, stringToCheck); + assertNotNull(stringToCheck, message); assertFalse(stringToCheck.isEmpty(), message); }