Upstream/bentobox
1
0
Fork 0
mirror of https://github.com/BentoBoxWorld/BentoBox.git synced 2024-11-01 00:10:40 +01:00
Code Issues Projects Releases Wiki Activity

Fixed addon api-version compatibility check

Browse Source 
Fixes https://github.com/BentoBoxWorld/BentoBox/issues/1346
Added a unit test to prevent regressions

Newer BentoBox releases are now properly considered "compatible" to addons expecting an older patch release
This commit is contained in:
Florian CUNY 2020-05-09 11:26:00 +02:00
parent 11510b15fc
commit 937052d378
2 changed files with 28 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/main/java/world/bentobox/bentobox/managers/AddonsManager.java
View File

@ -288,12 +288,17 @@ public class AddonsManager {
bentoboxNumber = Integer.parseInt(bentoboxVersion[i]);
}
int apiNumber = Util.isInteger(apiVersion[i], false) ? Integer.parseInt(apiVersion[i]) : -1;
if (bentoboxNumber < apiNumber) {
return false;
if (bentoboxNumber > apiNumber) {
return true; // BentoBox version is greater than the required version -> compatible
}
if (bentoboxNumber < apiNumber) {
return false; // BentoBox is definitely outdated
}
// If it is equal, go to the next number
}
return true;
return true; // Everything is equal, so return true
}
/**

29
src/test/java/world/bentobox/bentobox/managers/AddonsManagerTest.java
View File

@ -252,7 +252,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxSnapshotNoAPIVersion() {
@ -263,7 +263,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxReleaseAPIVersion() {
@ -274,7 +274,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxSnapshotAPIVersion() {
@ -285,7 +285,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxReleaseNoAPIVersion() {
@ -296,7 +296,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxSnapshotAPIVersionVariableDigits() {
@ -307,7 +307,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxOldSnapshot() {
@ -318,7 +318,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxOldRelease() {
@ -329,7 +329,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxOldReleaseLong() {
@ -340,7 +340,7 @@ public class AddonsManagerTest {
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon)}.
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxOldReleaseLongAPI() {
@ -350,6 +350,17 @@ public class AddonsManagerTest {
assertFalse(am.isAddonCompatibleWithBentoBox(addon, "1.11.1"));
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#isAddonCompatibleWithBentoBox(Addon, String)}.
* Prevents regression on https://github.com/BentoBoxWorld/BentoBox/issues/1346.
*/
@Test
public void testIsAddonCompatibleWithBentoBoxNewRelease() {
Addon addon = mock(Addon.class);
AddonDescription addonDesc = new AddonDescription.Builder("main.class", "Addon-name", "1.0.0").apiVersion("1.13.1").build();
when(addon.getDescription()).thenReturn(addonDesc);
assertTrue(am.isAddonCompatibleWithBentoBox(addon, "1.14.0-SNAPSHOT-b1777"));
}
/**
* Test method for {@link world.bentobox.bentobox.managers.AddonsManager#setPerms(Addon)}