174 Commits

• Reduce the allowed length by 5 to account for (99) suffix. Move truncating logic to GetFilename. Update tests.
  Cootz · 2023-02-21 18:35:53 +03:00
  90aa4288d0
• I don't know what .ots is but let's not use random file extension that make no sense
  Dean Herbert · 2023-02-17 13:46:27 +09:00
  372b6b794c
• More constants and assert fixes
  Dean Herbert · 2023-02-17 13:46:27 +09:00
  8ef3fb26e0
• Simplify test storage by removing nested storage
  Dean Herbert · 2023-02-17 13:43:42 +09:00
  86d110e893
• Mark test as fixture
  Dean Herbert · 2023-02-17 13:39:31 +09:00
  4560ae6b02
• Move long filename to fixture level
  Dean Herbert · 2023-02-17 13:39:24 +09:00
  99236f0ae8
• Rename max length variable to make sense (it's a filename limit, not path)
  Dean Herbert · 2023-02-17 13:33:22 +09:00
  96b1498932
• Tidy up new test method code quality
  Dean Herbert · 2023-02-17 13:30:00 +09:00
  415220a447
• Fix inspectCode issues
  Cootz · 2023-02-17 01:50:24 +03:00
  f4038a49a1
• Improve naming
  Cootz · 2023-02-17 01:46:15 +03:00
  1d8b348e4c
• Add more test cases
  Cootz · 2023-02-17 01:44:45 +03:00
  6cb00cd42f
• Improve code slightly
  Cootz · 2023-02-16 16:42:07 +03:00
  6819a45a1b
• Add tests
  Cootz · 2023-02-16 16:26:57 +03:00
  f1da213bea
• Hook up ImportParameter flow with IModelImporter caller methods
  Dean Herbert · 2022-12-13 21:29:32 +09:00
  cb16d62700
• Convert batchImport parameter to parameters class to allow further import configuration
  Dean Herbert · 2022-12-13 19:55:18 +09:00
  cf2719d4c0
• Partial everything
  Dan Balasescu · 2022-11-27 00:00:27 +09:00
  7bc8908ca9
• Suppress nullable warning for now
  Salman Ahmed · 2022-11-18 09:46:50 +03:00
  e2aca8dc90
• Add test coverage of ruleset being marked unavailable if methods are throwing
  Dean Herbert · 2022-08-29 14:28:55 +09:00
  892f43da43
• Add test coverage for outdated ruleset
  Dean Herbert · 2022-08-29 14:26:12 +09:00
  5ff4e6a4fe
• Add more missing realm Refresh() calls to new beatmap import tests

  As noticed at https://github.com/ppy/osu/runs/7605101313?check_suite_focus=true

  Dean Herbert · 2022-08-01 17:06:46 +09:00
  ee68113913
• Merge branch 'master' into collections-track-beatmap-updates
  Dan Balasescu · 2022-07-28 22:05:34 +09:00
  a59d7f6710
• Adjust comment
  Dan Balasescu · 2022-07-28 16:56:11 +09:00
  6d4023b933
• Add realm Refresh calls to ensure tests are stable
  Dean Herbert · 2022-07-28 15:18:04 +09:00
  67c44552cb
• Add realm refresh calls to fix intermittent test failures on new update tests
  Dean Herbert · 2022-07-28 15:15:41 +09:00
  485d140a21
• Add test coverage of beatmap updates transferring collection hashes
  Dean Herbert · 2022-07-28 15:02:58 +09:00
  2ae5a34c0e
• Fix potential test failure if scores are added to the beatmap which is subsequently removed
  Dean Herbert · 2022-07-27 16:27:10 +09:00
  494486ad09
• Merge branch 'master' into beatmap-update-test
  Dean Herbert · 2022-07-26 17:27:36 +09:00
  7f75184a01
• Add test coverage of no online ID scenario
  Dean Herbert · 2022-07-26 17:27:23 +09:00
  ee694c1257
• Add comprehensive test coverage of update scenarios
  Dean Herbert · 2022-07-26 17:22:52 +09:00
  1221cb1a42
• Refactor new tests to not suck as much as the old importer tests
  Dean Herbert · 2022-07-26 16:33:00 +09:00
  846291d203
• Add ImportAsUpdate method to IModelImporter to avoid otehr changes
  Dean Herbert · 2022-07-26 15:49:04 +09:00
  8370ca9765
• Fix left over resource file which isn't actually imported
  Dean Herbert · 2022-07-26 15:08:18 +09:00
  ac553d22ea
• Avoid leaving left-over files after test run completes
  Dean Herbert · 2022-07-26 15:08:18 +09:00
  59d3cc52c4
• Move tests to own class
  Dean Herbert · 2022-07-26 14:53:20 +09:00
  7d8a78ef01
• Fix test nullability assertions
  Dean Herbert · 2022-07-25 20:06:37 +09:00
  9c411c2250
• Add test coverage of actual update flow
  Dean Herbert · 2022-07-25 19:51:19 +09:00
  92dd1bcddb
• Centralise calls to reset online info of a BeatmapInfo
  Dean Herbert · 2022-07-25 19:51:19 +09:00
  6a3e8e31de
• Add basic coverage of current behaviour of beatmap reimport
  Dean Herbert · 2022-07-25 19:51:19 +09:00
  ec477a3ebf
• Add test coverage
  Dean Herbert · 2022-07-21 18:20:46 +09:00
  1374738a0d
• Add test coverage of nested rollback for good measure
  Dean Herbert · 2022-07-07 18:19:01 +09:00
  bf10f2db2e
• Fix transaction not being disposed
  Dean Herbert · 2022-07-07 18:15:38 +09:00
  ac216d94a8
• Add automatic transaction handling to realm helper methods
  Dean Herbert · 2022-07-07 17:36:25 +09:00
  5197d0fa9e
• Add more verbose logging to realm blocking process
  Dean Herbert · 2022-07-02 12:36:56 +09:00
  4fd47b5fa0
• Merge pull request #18835 from peppy/beatmap-update-flow

  Split out beatmap update tasks to `BeatmapUpdater` and invoke from editor save flow

  Dan Balasescu · 2022-07-01 20:28:04 +09:00
  3b1842a2c2
• Merge branch 'master' into realm-fix-async-write-after-disposal
  Dean Herbert · 2022-06-30 19:07:15 +09:00
  447b496eff
• Merge branch 'master' into beatmap-update-flow
  Dean Herbert · 2022-06-30 16:44:17 +09:00
  98938821e5
• Add reproduction of transaction not being exited correctly
  Dean Herbert · 2022-06-30 15:15:09 +09:00
  0abb400f64
• Merge branch 'master' into realm-fix-async-write-after-disposal
  Dean Herbert · 2022-06-29 22:39:05 +09:00
  a3b4a515fc
• Add test coverage of async writes during a blocking operation
  Dean Herbert · 2022-06-28 17:07:49 +09:00
  d64959ad0c
• Add test coverage of realm async writes
  Dean Herbert · 2022-06-27 19:38:14 +09:00
  6203885040