h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2024-09-21 17:27:24 +08:00
Code Issues Releases Wiki Activity

Revert "Add failing test case"

Browse Source 
This reverts commit ec89a149dd.
This commit is contained in:
Salman Ahmed 2021-05-19 21:35:06 +03:00
parent ec89a149dd
commit 434377aa52
1 changed files with 1 additions and 39 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
osu.Game.Rulesets.Catch.Tests/TestSceneCatchPlayerLegacySkin.cs
View File

@ -1,13 +1,7 @@
// 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 System.Linq;
using NUnit.Framework;
using osu.Framework.Graphics;
using osu.Framework.Testing;
using osu.Game.Beatmaps;
using osu.Game.Rulesets.Catch.Skinning.Legacy;
using osu.Game.Skinning;
using osu.Game.Tests.Visual;
namespace osu.Game.Rulesets.Catch.Tests
@ -15,38 +9,6 @@ namespace osu.Game.Rulesets.Catch.Tests
[TestFixture]
public class TestSceneCatchPlayerLegacySkin : LegacySkinPlayerTestScene
{
[Test]
public void TestUsingLegacySkin()
{
// check for the existence of a random legacy component to ensure using legacy skin.
// this should exist in LegacySkinPlayerTestScene but the weird transformer logic below needs to be "fixed" or otherwise first.
AddAssert("using legacy skin", () => this.ChildrenOfType<LegacyScoreCounter>().Any());
}
protected override Ruleset CreatePlayerRuleset() => new TestCatchRuleset();
private class TestCatchRuleset : CatchRuleset
{
public override ISkin CreateLegacySkinProvider(ISkinSource source, IBeatmap beatmap) => new TestCatchLegacySkinTransformer(source);
}
private class TestCatchLegacySkinTransformer : CatchLegacySkinTransformer
{
public TestCatchLegacySkinTransformer(ISkinSource source)
: base(source)
{
}
public override Drawable GetDrawableComponent(ISkinComponent component)
{
var drawable = base.GetDrawableComponent(component);
if (drawable != null)
return drawable;
// it shouldn't really matter whether to return null or return this,
// but returning null skips over the beatmap skin, so this needs to exist to test things properly.
return Source.GetDrawableComponent(component);
}
}
protected override Ruleset CreatePlayerRuleset() => new CatchRuleset();
}
}