Dean Herbert
b8ee7e05f4
Expose more pieces of APIBeatmap for testing purposes
2021-10-29 18:35:27 +09:00
Dean Herbert
993a28b37e
Merge branch 'beatmap-refactor/difficulty-icons' into beatmap-refactor/beatmap-overlays
2021-10-29 18:35:26 +09:00
Dean Herbert
04acc7601c
Fix one more missed case
2021-10-29 18:35:15 +09:00
Dean Herbert
07e3ced315
Fix test scene and remove "impossible" nullable coalesce
2021-10-29 18:22:23 +09:00
Dean Herbert
7db8bdfb7c
Fix fallback logic not considering case where ruleset is not available
...
Occurs only in tests.
2021-10-29 18:16:54 +09:00
Dan Balasescu
159a7af556
Merge branch 'master' into beatmap-refactor/difficulty-icons
2021-10-29 18:03:17 +09:00
Dean Herbert
8ad33d43d0
Merge branch 'beatmap-refactor/get-and-present' into beatmap-refactor/beatmap-overlays
2021-10-29 18:01:17 +09:00
Dean Herbert
3be9c487f8
Merge branch 'beatmap-refactor/download-tracker' into beatmap-refactor/beatmap-overlays
2021-10-29 18:01:12 +09:00
Dean Herbert
90503f72ca
Update PresentBeatmap to support interface types
2021-10-29 17:43:39 +09:00
Dean Herbert
b5834044e0
Update GetScoreRequest to support requests with interface types
2021-10-29 17:43:39 +09:00
Dan Balasescu
32f89791c9
Merge pull request #15357 from peppy/beatmap-refactor/beatmap-difficulty-cache
...
Refactor `BeatmapDifficultyCache` to work with `IBeatmapInfo`
2021-10-29 17:31:14 +09:00
Dean Herbert
be0564f732
Update DifficultyIcon classes to use IBeatmapInfo
2021-10-29 17:05:06 +09:00
Dean Herbert
3598adb344
Fix test implementing old version of class
2021-10-29 17:04:56 +09:00
Dean Herbert
7583435901
Refactor BeatmapDifficultyCache to work with IBeatmapInfo
2021-10-29 16:45:10 +09:00
Dean Herbert
06b6bcfd29
Add xmldoc for CreateScoreInfo function for now
...
I don't actually know how temporary this one is going to be. The usages
are quite deep - ie. converting to a `ScoreInfo` to get a calculated
total score for ordering purposes.
2021-10-29 14:51:07 +09:00
Dean Herbert
f68d6dbc8f
Update score submission serialisation tests to use SubmittableScore
2021-10-29 14:32:31 +09:00
Dean Herbert
cd7bd935f6
Remove Statistics from interface until we figure how to properly deserialise
2021-10-29 14:18:10 +09:00
Dean Herbert
3f030cebf4
Remove local score copying in GetScoresRequest to allow APIScoreInfo.Beatmap to be APIBeatmap
2021-10-29 14:14:25 +09:00
Dean Herbert
54073d8a1e
Isolate score submissions model and remove serialisation from ScoreInfo
2021-10-29 14:13:43 +09:00
Dean Herbert
1944c255a7
Implement score interfaces
2021-10-29 13:49:30 +09:00
Dean Herbert
49b5de64be
Extract interface
2021-10-29 13:41:21 +09:00
Dean Herbert
817369903a
Rename API score classes
2021-10-29 13:40:56 +09:00
Dean Herbert
414d920ca2
Revert to previous exposure of RoomSubScreen.BeatmapAvailability
2021-10-29 12:02:10 +09:00
Dean Herbert
b252b176d4
Seal implementation and add comment about LoadComplete execution order
2021-10-29 11:59:07 +09:00
Dean Herbert
f3811edb0b
Add inline comment explaining usage of RequiresChildrenUpdate override
2021-10-29 11:57:54 +09:00
Dean Herbert
dc44734db2
Simplify null check in ScoreDownloadTracker and remove unnecessary ID check
2021-10-29 11:56:17 +09:00
Dean Herbert
75a088d4f4
Fix variable name mismatch
2021-10-29 11:54:42 +09:00
Dean Herbert
748003b016
Simplify Manager null check to a single pre-check
2021-10-29 11:54:19 +09:00
Dean Herbert
3b095e1626
Merge branch 'master' into beatmap-refactor/download-tracker
2021-10-29 11:50:54 +09:00
Dean Herbert
34d4715220
Allow for long online IDs and implement in ScoreInfo
2021-10-29 11:48:36 +09:00
Dan Balasescu
fe86ae51b0
Merge pull request #15349 from peppy/remap-mania-scroll-speed-adjust
...
Change scroll speed adjust defaults to F3/F4
2021-10-29 11:42:44 +09:00
Dean Herbert
3838fe5c6a
Fix typo in variable name
2021-10-29 11:39:15 +09:00
Dean Herbert
8d21f0b04b
Add migration to reset conflicting scroll speed key bindings
2021-10-29 11:14:22 +09:00
Dean Herbert
ce29d6fd81
Change scroll speed adjust defaults to F3/F4
...
Not the most obvious keys, but does match stable expectations so let's
go with it for now.
Rationale for not using +/- is that local audio offset was bound to
those, and people will expect it to be when we get to implementing
eventually.
Also, adjust the scroll speed is a pretty rare thing to do since it's
been constant across beatmaps for years now.
2021-10-29 11:13:07 +09:00
Dean Herbert
7ab68525b4
Merge pull request #15031 from Joehuu/ui-scaling-keybind
...
Add ability to change ui scaling via keyboard
2021-10-29 01:48:08 +09:00
Bartłomiej Dach
9199f2b337
Merge branch 'master' into ui-scaling-keybind
2021-10-28 18:20:34 +02:00
Dean Herbert
d14421723a
Merge pull request #15341 from peppy/always-unique-test-online-ids
...
Ensure all test beatmaps have unique OnlineIDs to avoid import conflicts
2021-10-29 00:21:19 +09:00
smoogipoo
c96b07c625
Remove unnecessary/wrong code
2021-10-28 18:24:04 +09:00
smoogipoo
2b3e63470d
Merge branch 'master' into always-unique-test-online-ids
2021-10-28 18:23:45 +09:00
Dan Balasescu
6e79a0947a
Merge pull request #15340 from peppy/add-number-setting-source-test
...
Add visual test for `SettingsNumberBox` usage in settings source context
2021-10-28 18:19:45 +09:00
Dean Herbert
d8d4839e43
Merge pull request #15339 from smoogipoo/fix-test-crash
...
Fix playlists room creation test crash
2021-10-28 17:28:46 +09:00
Dean Herbert
4f4f2503aa
Merge pull request #15335 from smoogipoo/cleanup-multiplayer-screen-tests
...
Clean up multiplayer-screen tests by removing intermediate screen
2021-10-28 17:28:26 +09:00
Dean Herbert
c0848486d6
Add missing test coverage for attempt count display
2021-10-28 17:27:16 +09:00
Dean Herbert
8076e22e70
Replace weird import handling
2021-10-28 17:27:16 +09:00
Dan Balasescu
b21f42ec76
Merge branch 'master' into add-number-setting-source-test
2021-10-28 17:25:52 +09:00
Dan Balasescu
e5a7589859
Merge pull request #15317 from peppy/beatmap-refactor/uncontested
...
Update `UpdateableBeatmapBackgroundSprite` to accept `IBeatmapInfo`
2021-10-28 17:24:49 +09:00
Dan Balasescu
e5107ae0f7
Merge pull request #15338 from peppy/add-progress-notification-icons
...
Add icons to various progress notification states
2021-10-28 17:24:39 +09:00
Dean Herbert
b096bf6cd1
Ensure all test beatmaps have unique OnlineIDs to avoid import conflicts
...
The order of operation of imports means that the already-imported
instances are considered most correct (they have usually been populated
using an online source, although that is skipped in tests), so on two
consecutive test imports in a test scene, the second will lose its
online IDs completely.
This aims to ensure that all test beatmaps will have online IDs, which
is the general expected behaviour.
2021-10-28 16:56:31 +09:00
smoogipoo
bacaa9d201
Fix playlists test crash
2021-10-28 16:48:17 +09:00
Dean Herbert
6f5040722a
Merge branch 'master' into beatmap-refactor/uncontested
2021-10-28 16:37:26 +09:00
