External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2025-12-20 10:40:06 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

cleanup

Browse source
This commit is contained in:
Susucre 2023-09-19 10:39:49 +02:00
parent cf7045452d
commit dd7f755c9b
1 changed files with 6 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

14
Mage.Common/src/test/java/mage/view/ValidateViews.java
View file

@ -1,10 +1,8 @@
package mage.view; package mage.view;
import org.checkerframework.checker.units.qual.C;
import org.junit.Assert; import org.junit.Assert;
import org.junit.Test; import org.junit.Test;
import java.io.Serializable;
import java.lang.reflect.Field; import java.lang.reflect.Field;
import java.lang.reflect.Modifier; import java.lang.reflect.Modifier;
import java.lang.reflect.ParameterizedType; import java.lang.reflect.ParameterizedType;
@ -16,7 +14,7 @@ public class ValidateViews {
// All those class are sent to client as Views or inside Views. // All those class are sent to client as Views or inside Views.
// The test in this file is attempting to find any class that should // The test in this file is attempting to find any class that should
// not be sent. // not be sent.
private static List<Class<?>> toValidate = Arrays.asList( private static final List<Class<?>> toValidate = Arrays.asList(
// Views // Views
AbilityPickerView.class, AbilityView.class, CardsView.class, AbilityPickerView.class, AbilityView.class, CardsView.class,
CardView.class, ChatMessage.class, CombatGroupView.class, CardView.class, ChatMessage.class, CombatGroupView.class,
@ -89,18 +87,18 @@ public class ValidateViews {
); );
// Those are safe java class we allow in Views. // Those are safe java class we allow in Views.
private static List<Class<?>> safeClass = Arrays.asList( private static final List<Class<?>> safeClass = Arrays.asList(
boolean.class, int.class, long.class, float.class, boolean.class, int.class, long.class, float.class,
String.class, Date.class, UUID.class String.class, Date.class, UUID.class
); );
// Those are non-public class // Those are non-public class
private static List<String> nonPublicNames = Arrays.asList( private static final List<String> nonPublicNames = Arrays.asList(
"mage.players.PlayableObjectRecord" "mage.players.PlayableObjectRecord"
); );
// Those are excluded as Reflection on type of Map/Set/Collection is really hard. // Those are excluded as Reflection on type of Map/Set/Collection is really hard.
private static List<String> tooHardToRecurseInto = Arrays.asList( private static final List<String> tooHardToRecurseInto = Arrays.asList(
"mage.view.CardsView<java.util.LinkedHashMap>", // <UUID, CardView> "mage.view.CardsView<java.util.LinkedHashMap>", // <UUID, CardView>
"mage.view.CardsView<java.util.HashMap>", // <UUID, CardView> "mage.view.CardsView<java.util.HashMap>", // <UUID, CardView>
"mage.view.CardsView<java.util.AbstractMap>", // <UUID, CardView> "mage.view.CardsView<java.util.AbstractMap>", // <UUID, CardView>
@ -118,7 +116,7 @@ public class ValidateViews {
); );
// Same, but for fields. // Same, but for fields.
private static List<String> tooHardToRecurseIntoField = Arrays.asList( private static final List<String> tooHardToRecurseIntoField = Arrays.asList(
"mage.view.GameClientMessage: <mage.view.GameClientMessage>::targets<java.util.Set>" // <UUID> "mage.view.GameClientMessage: <mage.view.GameClientMessage>::targets<java.util.Set>" // <UUID>
); );
@ -173,7 +171,7 @@ public class ValidateViews {
} }
msg = msg + "<" + clazz.getName() + ">"; msg = msg + "<" + clazz.getName() + ">";
if(tooHardToRecurseIntoField.contains(msg)) { if (tooHardToRecurseIntoField.contains(msg)) {
return; return;
} }