From 3485ef33a726b1caee8a0245188691d79da38674 Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Wed, 25 Dec 2019 15:05:08 +0900
Subject: [PATCH] Rename tests and add missing coverage

---
 .../TestSceneLegacyBeatmapSkin.cs                     | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/osu.Game.Rulesets.Osu.Tests/TestSceneLegacyBeatmapSkin.cs b/osu.Game.Rulesets.Osu.Tests/TestSceneLegacyBeatmapSkin.cs
index e26b671d57..4676f14655 100644
--- a/osu.Game.Rulesets.Osu.Tests/TestSceneLegacyBeatmapSkin.cs
+++ b/osu.Game.Rulesets.Osu.Tests/TestSceneLegacyBeatmapSkin.cs
@@ -26,19 +26,20 @@ namespace osu.Game.Rulesets.Osu.Tests
         [Resolved]
         private AudioManager audio { get; set; }
 
-        [Test]
-        public void TestBeatmapComboColours()
+        [TestCase(true)]
+        [TestCase(false)]
+        public void TestBeatmapComboColours(bool customSkinColoursPresent)
         {
             ExposedPlayer player = null;
 
-            AddStep("load coloured beatmap", () => player = loadBeatmap(false, true));
+            AddStep("load coloured beatmap", () => player = loadBeatmap(customSkinColoursPresent, true));
             AddUntilStep("wait for player", () => player.IsLoaded);
 
             AddAssert("is beatmap skin colours", () => player.UsableComboColours.SequenceEqual(TestBeatmapSkin.Colours));
         }
 
         [Test]
-        public void TestEmptyBeatmapComboColours()
+        public void TestBeatmapNoComboColours()
         {
             ExposedPlayer player = null;
 
@@ -49,7 +50,7 @@ namespace osu.Game.Rulesets.Osu.Tests
         }
 
         [Test]
-        public void TestEmptyBeatmapCustomSkinColours()
+        public void TestBeatmapNoComboColoursSkinOverride()
         {
             ExposedPlayer player = null;