Skip to content

Commit 01730bc

Browse files
committed
add logs to attempt to find source of null pointer
1 parent a0ce453 commit 01730bc

File tree

2 files changed

+17
-0
lines changed

2 files changed

+17
-0
lines changed

src/test/java/unittest/mock/systems/behaviors/MockBehavior.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package unittest.mock.systems.behaviors;
22

33
import tech.fastj.graphics.game.GameObject;
4+
import tech.fastj.logging.Log;
45
import tech.fastj.math.Pointf;
56
import tech.fastj.systems.behaviors.Behavior;
67

@@ -15,6 +16,14 @@ public void init(GameObject gameObject) {
1516

1617
@Override
1718
public void fixedUpdate(GameObject gameObject) {
19+
if (pointf == null) {
20+
Log.warn(
21+
MockBehavior.class,
22+
"Game Object {} containing mock behavior tried calling fixed update while pointf was null",
23+
gameObject
24+
);
25+
}
26+
1827
pointf.add(1f);
1928
}
2029

src/test/java/unittest/testcases/graphics/game/GameObjectTests.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package unittest.testcases.graphics.game;
22

33
import tech.fastj.graphics.game.GameObject;
4+
import tech.fastj.logging.Log;
45
import tech.fastj.math.Maths;
56
import tech.fastj.systems.behaviors.Behavior;
67
import tech.fastj.systems.control.Scene;
@@ -98,6 +99,7 @@ void checkAddAndRemoveBehaviorsFromGameObject_usingMethodChaining() {
9899
@Test
99100
void checkInitBehaviors_shouldInitializePointf() {
100101
GameObject gameObject = new MockGameObject();
102+
Log.info("created game object {}", gameObject);
101103
MockBehavior mockBehavior = new MockBehavior();
102104
Scene mockScene = new MockEmptyScene();
103105

@@ -110,6 +112,7 @@ void checkInitBehaviors_shouldInitializePointf() {
110112
@Test
111113
void checkUpdateBehaviors_shouldIncrementPointf() {
112114
GameObject gameObject = new MockGameObject();
115+
Log.info("created game object {}", gameObject);
113116
MockBehavior mockBehavior = new MockBehavior();
114117
Scene mockScene = new MockEmptyScene();
115118

@@ -129,6 +132,7 @@ void checkUpdateBehaviors_shouldIncrementPointf() {
129132
@Test
130133
void checkDestroyBehaviors_shouldMakePointfNull() {
131134
GameObject gameObject = new MockGameObject();
135+
Log.info("created game object {}", gameObject);
132136
MockBehavior mockBehavior = new MockBehavior();
133137
Scene mockScene = new MockEmptyScene();
134138

@@ -143,6 +147,7 @@ void checkDestroyBehaviors_shouldMakePointfNull() {
143147
@Test
144148
void checkInitBehavior_withAddLateBehavior_shouldInitializePointf() {
145149
GameObject gameObject = new MockGameObject();
150+
Log.info("created game object {}", gameObject);
146151
MockBehavior mockBehavior = new MockBehavior();
147152
Scene mockScene = new MockEmptyScene();
148153

@@ -154,6 +159,7 @@ void checkInitBehavior_withAddLateBehavior_shouldInitializePointf() {
154159
@Test
155160
void checkUpdateBehavior_withAddLateBehavior_shouldIncrementPointf() {
156161
GameObject gameObject = new MockGameObject();
162+
Log.info("created game object {}", gameObject);
157163
MockBehavior mockBehavior = new MockBehavior();
158164
Scene mockScene = new MockEmptyScene();
159165

@@ -172,6 +178,7 @@ void checkUpdateBehavior_withAddLateBehavior_shouldIncrementPointf() {
172178
@Test
173179
void checkDestroyBehavior_withAddLateBehavior_shouldMakePointfNull() {
174180
GameObject gameObject = new MockGameObject();
181+
Log.info("created game object {}", gameObject);
175182
MockBehavior mockBehavior = new MockBehavior();
176183
Scene mockScene = new MockEmptyScene();
177184

@@ -185,6 +192,7 @@ void checkDestroyBehavior_withAddLateBehavior_shouldMakePointfNull() {
185192
@Test
186193
void tryUpdateBehaviorWithoutInitializing_shouldThrowNullPointerException() {
187194
GameObject gameObject = new MockGameObject();
195+
Log.info("created game object {}", gameObject);
188196
MockBehavior mockBehavior = new MockBehavior();
189197
Scene mockScene = new MockEmptyScene();
190198

0 commit comments

Comments
 (0)