Commit Graph

97 Commits

Author SHA1 Message Date
d13a66a96c Rework test scene by only relying on OnlineID 2022-03-08 20:11:56 +09:00
631c23ea3a Merge branch 'master' into beatmap-decoder-ruleset-store 2022-02-18 16:52:33 +09:00
0138f22c8d Update existing usages to point to RealmRulesetStore 2022-02-16 17:13:54 +09:00
bdc3b76df0 Remove beatmap bindable from PlaylistItem 2022-02-15 23:50:08 +09:00
efeba30b9f Remove ruleset and mod bindables from PlaylistItem 2022-02-15 16:28:15 +09:00
778d2a71b4 Remove Task from the inner-most Import method in RealmArchiveModelImporter
One of my pending work items for post-realm merge.

The lowest-level import task is no longer asynchronous, as we don't want
it to span multiple threads to allow easier interaction with realm.
Removing the `Task` spec simplifies a heap of usages.

Individual usages should decide whether they want to run the import
asynchronously, by either using an alternative override or spooling up a
thread themselves.
2022-01-25 15:30:29 +09:00
3e5c9e8436 Fix cases of Access instead of Realm 2022-01-25 12:58:15 +09:00
6eb2c28e41 Rename RealmContextFactory to RealmAccess 2022-01-24 20:38:07 +09:00
7df18c7767 Merge branch 'master' into playlist-test-coverage 2022-01-18 20:17:48 +01:00
73a1f27ce5 Merge branch 'master' into realm-integration/score-and-beatmaps 2022-01-18 17:31:37 +09:00
3e9e7a8fb6 Fix PLaylistsResultsScreen tests falling over due to missing beatmap 2022-01-18 16:48:27 +09:00
d26f4d50bd Add test coverage of aggregate room scores displaying correctly 2022-01-18 14:00:12 +09:00
c3d3c03f0f Fix TestScenePlaylistsScreen crashing on entering room 2022-01-18 13:42:41 +09:00
a59dcccab7 Add local ContextFactory caching to all remaining test scenes that create local managers 2022-01-14 19:59:21 +09:00
b2d09b7b10 Fix further warnings 2022-01-12 23:42:12 +09:00
ac3b7aa893 Fix more incorrect test access to ILive 2022-01-12 17:49:10 +09:00
31a3161189 Make tests compile again 2022-01-12 17:00:16 +09:00
00177a3ae1 Update usages to new naming 2022-01-06 22:54:43 +09:00
73b40e6833 Replace usage of .Result with .WaitSafelyForResult 2022-01-04 11:51:41 +09:00
031a40af6a Replace usages of Wait with WaitSafely 2022-01-04 11:51:41 +09:00
98524d60a4 Fix clear identifier typos 2021-12-27 20:26:28 -08:00
d79602a912 Add wait step to TestScenePlaylistResultScreen explicitly for screen load 2021-12-20 16:58:36 +09:00
da00c020be Remove whitespace 2021-12-15 07:33:49 +09:00
0950d8d327 Add back PlaylistRoomCreation test
Was spiritually removed in https://github.com/ppy/osu/pull/16045.

This implementation is mostly taken from that PR's comment thread
verbatim, and now works due to the associated changes to
`OnlinePlayBeatmapAvailabilityTracker`.
2021-12-14 19:16:52 +09:00
cebfeb5220 Merge pull request #16045 from smoogipoo/multiplayer-local-beatmap-query
Rework multiplayer's current item tracking
2021-12-14 18:33:30 +09:00
6e3558b222 Remove weird test 2021-12-14 06:38:57 +09:00
99ac71c1fe Simplify usages where the ruleset being used is osu! ruleset 2021-12-13 16:37:20 +09:00
b0d14526ea Move test ScoreInfo creation to TestResources
The main goal here is to remove the inheritance, since realm doesn't
like that. Unfortunate that we can't use object initialisers in a few of
these places, but no real way around that.
2021-12-13 16:34:50 +09:00
2cd2b10ce1 Fix results sometimes showing incorrect score position 2021-12-13 06:54:57 +09:00
dbb08f7d46 Use OnlineID for set operations 2021-12-10 16:11:48 +09:00
bf1418bafc Use OnlineID instead of legacy IDs for equality and lookups 2021-12-10 16:11:48 +09:00
2d125893fc Add tests and disable selection on invalid items 2021-11-16 17:23:52 +09:00
692e846acd Rename BeatmapSetInfo.OnlineBeatmapSetID to OnlineID to match interface 2021-11-12 17:52:44 +09:00
6a098a8634 Rename BeatmapInfo.OnlineBeatmapID to OnlineID to match interface 2021-11-12 17:46:24 +09:00
0ecf5f201c Rename User to APIUser and move to correct namespace 2021-11-07 11:26:01 +09:00
c96b07c625 Remove unnecessary/wrong code 2021-10-28 18:24:04 +09:00
c0848486d6 Add missing test coverage for attempt count display 2021-10-28 17:27:16 +09:00
8076e22e70 Replace weird import handling 2021-10-28 17:27:16 +09:00
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
e2c08fb2e5 Create room by clicking button 2021-10-28 15:41:59 +09:00
b0b27a47d5 Rename test scene 2021-10-28 15:28:16 +09:00
cf15282077 Merge branch 'master' into fix-playlists-room-subscreen-tests 2021-10-28 15:26:48 +09:00
f3dba49aae Rename room managers 2021-10-27 19:36:32 +09:00
58237f6fee Fix weird import flows 2021-10-27 14:26:37 +09:00
84bb3a962b Centralise room "creation" 2021-10-27 14:11:57 +09:00
c4a253cc75 Replace local instantiation of TestBeatmap with helper method 2021-10-27 13:55:14 +09:00
2bc2420012 Rename test scene
It's not testing the sub-screen. It's testing creating playlists.
2021-10-27 13:54:21 +09:00
d6ac6a5cd6 Fix intermittent results screen test failures 2021-10-08 15:18:01 +09:00
b339c149d8 Copy BaseDifficulty to Beatmap<T> and move all write operations across 2021-10-06 15:10:45 +09:00
a2e61883e3 Initial push to use ILive in import process 2021-09-30 22:55:25 +09:00