Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tracking issue for P1 blocked integration/unit tests #24

Open
36 tasks
NotStirred opened this issue Jan 24, 2024 · 0 comments
Open
36 tasks

Tracking issue for P1 blocked integration/unit tests #24

NotStirred opened this issue Jan 24, 2024 · 0 comments
Assignees
Labels

Comments

@NotStirred
Copy link
Member

NotStirred commented Jan 24, 2024

ChunkMap

  • Only cc_updateChunkScheduling and calls from ChunkHolder#cc_getOrScheduleFuture are tested

ChunkHolder

  • Only cc_getOrScheduleFuture is tested

MixinMinecraftServer

  • setInitialSpawn, only unit test, requires integration test
    • cc_replaceChunkPosInSetInitialSpawn
    • cc_replaceGetWorldPositionInSetInitialSpawn
  • prepareLevels only unit test, requires integration test
    • cc_replaceAddRegionTicketInPrepareLevels
    • cc_waitUntilCubicGenerationComplete

MixinLevel

  • Mixins

    • setBlock only unit test, requires integration test
      • cc_replaceLevelChunkInGetChunkAt
      • cc_replaceLevelChunkInSetBlockState
      • cc_replaceLevelChunkInMarkAndNotifyBlock
    • getBlockState only unit test, requires integration test
      • cc_replaceLevelChunkInGetBlockState
      • cc_replaceLevelChunkInGetBlockState (overload)
    • getBlockEntity only unit test, requires integration test
      • cc_replaceGetChunkAtInSetBlockEntity
    • getFluidState only unit test, requires integration test
      • cc_replaceGetChunkAtInGetFluidState
    • setBlockEntity only unit test, requires integration test | might break if vanilla code is added after the cancel.
      • cc_replaceLevelChunkInSetBlockEntity
    • removeBlockEntity only unit test, requires integration test
      • cc_replaceGetChunkAtInRemoveBlockEntity
      • cc_replaceLevelChunkInRemoveBlockEntity
    • isLoaded only unit test, requires integration test
      • cc_replaceHasChunkInIsLoaded
    • loadedAndEntityCanStandOnFace only unit test, requires integration test
      • cc_replaceGetChunkAtInLoadedAndEntityCanStandOnFace
    • blockEntityChanged only unit test, requires integration test
      • cc_replaceBlockEntityChanged
    • getCurrentDifficultyAt only unit test, requires integration test
      • cc_replaceGetCurrentDifficultyAt
  • CC methods

    • LevelCube cc_getCube only unit test, requires integration test
    • CubeAccess cc_getCube only unit test, requires integration test
    • cc_getCubeAt only unit test, requires integration test
    • cc_hasCubeAt no test
    • cc_hasCubesAt no test

CubicChunkSource

  • None tested, missing ServerChunkCache and ClientChunkCache to test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

No branches or pull requests

3 participants