Skip to content

Commit 62bc812

Browse files
committed
Fix tests
1 parent 58c847a commit 62bc812

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

Tests/Runtime/MonkeyTest.cs

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -616,7 +616,7 @@ public void LotteryOperator_NotReachableObjectOnly_LogNotLottery()
616616
[TestFixture]
617617
public class Visualizer
618618
{
619-
private const float IndicatorLifetime = 0.2f;
619+
private const float IndicatorLifetime = 0.5f;
620620
private DefaultDebugVisualizer _visualizer;
621621

622622
[OneTimeSetUp]
@@ -636,19 +636,18 @@ public void OneTimeTearDown()
636636
public async Task LotteryOperator_IgnoredObjectOnly_IgnoredIndicatorIsShown()
637637
{
638638
var cube = GameObject.Find("Cube");
639-
cube.AddComponent<SpyOnPointerClickHandler>();
640639
cube.transform.position = new Vector3(0, 0, 0);
641640
cube.AddComponent<IgnoreAnnotation>(); // ignored
642641

643-
await UniTask.NextFrame();
642+
await UniTask.DelayFrame(5); // warm up for physics raycaster (maybe)
644643

645644
var operators = new List<(GameObject, IOperator)> { (cube, new UguiClickOperator()), };
646645
var random = new RandomWrapper();
647646
var ignoreStrategy = new DefaultIgnoreStrategy();
648647
var reachableStrategy = new DefaultReachableStrategy();
649648
Monkey.LotteryOperator(operators, random, ignoreStrategy, reachableStrategy, visualizer: _visualizer);
650649

651-
await UniTask.DelayFrame(5);
650+
await UniTask.NextFrame();
652651

653652
var indicator = GameObject.Find("Indicator"); // exist multiple, so only one
654653
Assert.That(indicator, Is.Not.Null);
@@ -664,21 +663,21 @@ public async Task LotteryOperator_IgnoredObjectOnly_IgnoredIndicatorIsShown()
664663
public async Task LotteryOperator_NotReachableObjectOnly_NotReachableIndicatorIsShown()
665664
{
666665
var cube = GameObject.Find("Cube");
667-
cube.AddComponent<SpyOnPointerClickHandler>();
668666
cube.transform.position = new Vector3(0, 0, 0);
669667

670668
var blocker = GameObject.CreatePrimitive(PrimitiveType.Quad);
671-
blocker.transform.position = new Vector3(0, 1, -9);
669+
blocker.transform.position = new Vector3(0, 1, -7);
670+
blocker.GetComponent<MeshRenderer>().materials[0].color = Color.gray;
672671

673-
await UniTask.NextFrame();
672+
await UniTask.DelayFrame(5); // warm up for physics raycaster (maybe)
674673

675674
var operators = new List<(GameObject, IOperator)> { (cube, new UguiClickOperator()), };
676675
var random = new RandomWrapper();
677676
var ignoreStrategy = new DefaultIgnoreStrategy();
678677
var reachableStrategy = new DefaultReachableStrategy();
679678
Monkey.LotteryOperator(operators, random, ignoreStrategy, reachableStrategy, visualizer: _visualizer);
680679

681-
await UniTask.DelayFrame(5);
680+
await UniTask.NextFrame();
682681

683682
var indicator = GameObject.Find("Indicator"); // exist multiple, so only one
684683
Assert.That(indicator, Is.Not.Null);

Tests/Runtime/Visualizers/DefaultDebugVisualizerTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ public async Task ShowNotReachableIndicator_Horizontal([Values] GameViewResoluti
8282
public async Task ShowNotReachableIndicator_Vertical([Values] GameViewResolution resolution)
8383
{
8484
var (width, height, name) = resolution.GetParameter();
85-
GameViewControlHelper.SetResolution(height, width, name); // flip
85+
GameViewControlHelper.SetResolution(height, width, $"{name} Portrait");
8686
await UniTask.NextFrame();
8787

8888
foreach (var reference in _referenceObjects)

0 commit comments

Comments
 (0)