mirror of
https://github.com/ppy/osu.git
synced 2025-03-28 03:07:19 +08:00
Merge pull request #8750 from bdach/fix-classic-scoring
Fix scoring in classic mode not awarding exact numerical value for judgement
This commit is contained in:
commit
f9bc64ebca
57
osu.Game.Tests/Rulesets/Scoring/ScoreProcessorTest.cs
Normal file
57
osu.Game.Tests/Rulesets/Scoring/ScoreProcessorTest.cs
Normal file
@ -0,0 +1,57 @@
|
|||||||
|
// 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.Collections.Generic;
|
||||||
|
using System.Linq;
|
||||||
|
using NUnit.Framework;
|
||||||
|
using osu.Framework.Utils;
|
||||||
|
using osu.Game.Beatmaps;
|
||||||
|
using osu.Game.Rulesets;
|
||||||
|
using osu.Game.Rulesets.Judgements;
|
||||||
|
using osu.Game.Rulesets.Objects;
|
||||||
|
using osu.Game.Rulesets.Osu.Judgements;
|
||||||
|
using osu.Game.Rulesets.Osu.Objects;
|
||||||
|
using osu.Game.Rulesets.Scoring;
|
||||||
|
using osu.Game.Tests.Beatmaps;
|
||||||
|
|
||||||
|
namespace osu.Game.Tests.Rulesets.Scoring
|
||||||
|
{
|
||||||
|
public class ScoreProcessorTest
|
||||||
|
{
|
||||||
|
private ScoreProcessor scoreProcessor;
|
||||||
|
private IBeatmap beatmap;
|
||||||
|
|
||||||
|
[SetUp]
|
||||||
|
public void SetUp()
|
||||||
|
{
|
||||||
|
scoreProcessor = new ScoreProcessor();
|
||||||
|
beatmap = new TestBeatmap(new RulesetInfo())
|
||||||
|
{
|
||||||
|
HitObjects = new List<HitObject>
|
||||||
|
{
|
||||||
|
new HitCircle()
|
||||||
|
}
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
[TestCase(ScoringMode.Standardised, HitResult.Meh, 750_000)]
|
||||||
|
[TestCase(ScoringMode.Standardised, HitResult.Good, 800_000)]
|
||||||
|
[TestCase(ScoringMode.Standardised, HitResult.Great, 1_000_000)]
|
||||||
|
[TestCase(ScoringMode.Classic, HitResult.Meh, 50)]
|
||||||
|
[TestCase(ScoringMode.Classic, HitResult.Good, 100)]
|
||||||
|
[TestCase(ScoringMode.Classic, HitResult.Great, 300)]
|
||||||
|
public void TestSingleOsuHit(ScoringMode scoringMode, HitResult hitResult, int expectedScore)
|
||||||
|
{
|
||||||
|
scoreProcessor.Mode.Value = scoringMode;
|
||||||
|
scoreProcessor.ApplyBeatmap(beatmap);
|
||||||
|
|
||||||
|
var judgementResult = new JudgementResult(beatmap.HitObjects.Single(), new OsuJudgement())
|
||||||
|
{
|
||||||
|
Type = hitResult
|
||||||
|
};
|
||||||
|
scoreProcessor.ApplyResult(judgementResult);
|
||||||
|
|
||||||
|
Assert.IsTrue(Precision.AlmostEquals(expectedScore, scoreProcessor.TotalScore.Value));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
@ -174,7 +174,7 @@ namespace osu.Game.Rulesets.Scoring
|
|||||||
|
|
||||||
case ScoringMode.Classic:
|
case ScoringMode.Classic:
|
||||||
// should emulate osu-stable's scoring as closely as we can (https://osu.ppy.sh/help/wiki/Score/ScoreV1)
|
// should emulate osu-stable's scoring as closely as we can (https://osu.ppy.sh/help/wiki/Score/ScoreV1)
|
||||||
return bonusScore + baseScore * ((1 + Math.Max(0, HighestCombo.Value - 1) * scoreMultiplier) / 25);
|
return bonusScore + baseScore * (1 + Math.Max(0, HighestCombo.Value - 1) * scoreMultiplier / 25);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user