1
0
mirror of https://github.com/ppy/osu.git synced 2025-01-12 17:43:05 +08:00

Add test ensuring correct break lightening behaviour

This commit is contained in:
Salman Ahmed 2019-12-12 00:32:39 +03:00
parent bc02cfc2e2
commit 4f6b85e5ea
2 changed files with 86 additions and 1 deletions

View File

@ -0,0 +1,82 @@
// Copyright (c) ppy Pty Ltd <contact@ppy.sh>. Licensed under the MIT Licence.
// See the LICENCE file in the repository root for full licence text.
using NUnit.Framework;
using osu.Framework.Allocation;
using osu.Framework.Bindables;
using osu.Framework.Graphics;
using osu.Framework.Graphics.Shapes;
using osu.Game.Configuration;
using osu.Game.Graphics;
using osu.Game.Graphics.Containers;
using osuTK.Graphics;
namespace osu.Game.Tests.Visual.Background
{
public class TestSceneUserDimContainer : OsuTestScene
{
private readonly TestUserDimContainer container;
private readonly BindableBool isBreakTime = new BindableBool();
private readonly Bindable<bool> lightenDuringBreaks = new Bindable<bool>();
public TestSceneUserDimContainer()
{
Add(container = new TestUserDimContainer
{
RelativeSizeAxes = Axes.Both,
Child = new Box
{
Colour = Color4.White,
RelativeSizeAxes = Axes.Both,
},
});
container.IsBreakTime.BindTo(isBreakTime);
}
[BackgroundDependencyLoader]
private void load(OsuConfigManager config)
{
config.BindWith(OsuSetting.LightenDuringBreaks, lightenDuringBreaks);
}
[SetUp]
public void SetUp()
{
isBreakTime.Value = false;
lightenDuringBreaks.Value = false;
}
[TestCase(0.6f, 0.3f)]
[TestCase(0.2f, 0.0f)]
[TestCase(0.0f, 0.0f)]
public void TestBreakLightening(float userDim, float expectedBreakDim)
{
AddStep($"set dim level {userDim}", () => container.UserDimLevel.Value = userDim);
AddStep("set break", () => isBreakTime.Value = true);
AddWaitStep("wait for dim", 3);
AddAssert($"is current dim {userDim}", () => container.DimEqual(userDim));
AddStep("set lighten during break", () => lightenDuringBreaks.Value = true);
AddWaitStep("wait for dim", 3);
AddAssert($"is current dim {expectedBreakDim}", () => container.DimEqual(expectedBreakDim));
AddStep("clear lighten during break", () => lightenDuringBreaks.Value = false);
AddWaitStep("wait for dim", 3);
AddAssert($"is current dim {userDim}", () => container.DimEqual(userDim));
AddStep("clear break", () => isBreakTime.Value = false);
AddStep("set lighten during break", () => lightenDuringBreaks.Value = true);
AddWaitStep("wait for dim", 3);
AddAssert($"is current dim {userDim}", () => container.DimEqual(userDim));
}
private class TestUserDimContainer : UserDimContainer
{
public bool DimEqual(float expectedDimLevel) => Content.Colour == OsuColour.Gray(1f - expectedDimLevel);
public new Bindable<double> UserDimLevel => base.UserDimLevel;
}
}
}

View File

@ -16,6 +16,9 @@ namespace osu.Game.Graphics.Containers
/// </summary>
public abstract class UserDimContainer : Container
{
/// <summary>
/// Amount of lightening to apply to current dim level during break times.
/// </summary>
private const float break_lighten_amount = 0.3f;
protected const double BACKGROUND_FADE_DURATION = 800;
@ -34,7 +37,7 @@ namespace osu.Game.Graphics.Containers
/// Whether player is in break time.
/// Must be bound to <see cref="BreakOverlay.IsBreakTime"/> to allow for dim adjustments in gameplay.
/// </summary>
internal readonly IBindable<bool> IsBreakTime = new Bindable<bool>();
public readonly IBindable<bool> IsBreakTime = new Bindable<bool>();
/// <summary>
/// Whether the content of this container is currently being displayed.