SlimefunGuguProject / Slimefun4

Slimefun4 (粘液科技) 汉化版 | Slimefun modified version in Simplified Chinese
GNU General Public License v3.0
255 stars 60 forks source link

Refactor:移除源代码中有关单元测试的代码 #903

Closed mcchampions closed 3 months ago

mcchampions commented 3 months ago

简介

删除了源代码中有关单元测试的代码。

在源代码中,不少见到Slimefun#isUnitTest的判断。可将minecraftVersion赋值为MinecraftVersion.UNIT_TEST却仅出现在Slimefun(JavaPluginLoader,PluginDescriptionFile,File,File)这个构造方法中可是在源代码中,我未发现有关的初始化代码片段,在e2e-tester也未出现相关代码片段。 当我翻阅与这个 Slimefun/Slimefun4#1892 时,我发现当初跟它一起提交的单元测试也并未出现有关代码(鬼知道最开始为什么写这个构造方法) 翻阅 Slimefun/Slimefun4 现在的代码,发现有关内容已变为

public Slimefun() {
        super();

        // Check that we got loaded by MockBukkit rather than Bukkit's loader
        // TODO: This is very much a hack and we can hopefully move to a more native way in the future
        if (getClassLoader().getClass().getPackageName().startsWith("be.seeseemelk.mockbukkit")) {
            minecraftVersion = MinecraftVersion.UNIT_TEST;
        }
    }

(好像真没用上过那个构造方法) 如今,该分支不再使用相关单元测试类,取而代之的是Slimefun-e2e-tester,那么源代码中有关单元测试的代码是否还有存在的必要呢,我认为应该是没有必要的的

StarWishsama commented 3 months ago

预期行为,减少与上游差异。