AlphaX/qortal
1
0
Fork 0
forked from Qortal/qortal
Code Issues Pull Requests Projects Releases Wiki Activity

Removed localAuthBypassEnabled override in unit tests.

Browse Source 
Hopefully this will allow us to proactively catch any missing API keys in the future.
This commit is contained in:
CalDescent 2022-01-23 12:48:37 +00:00
parent ea10eec926
commit ff6ec83b1c
2 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/test/java/org/qortal/test/api/AdminApiTests.java
View File

@ -2,10 +2,12 @@ package org.qortal.test.api;
import static org.junit.Assert.*; import static org.junit.Assert.*;
import org.apache.commons.lang3.reflect.FieldUtils;
import org.junit.Before; import org.junit.Before;
import org.junit.Test; import org.junit.Test;
import org.qortal.api.resource.AdminResource; import org.qortal.api.resource.AdminResource;
import org.qortal.repository.DataException; import org.qortal.repository.DataException;
import org.qortal.settings.Settings;
import org.qortal.test.common.ApiCommon; import org.qortal.test.common.ApiCommon;
import org.qortal.test.common.Common; import org.qortal.test.common.Common;
@ -29,7 +31,10 @@ public class AdminApiTests extends ApiCommon {
} }
@Test @Test
public void testSummary() { public void testSummary() throws IllegalAccessException {
// Set localAuthBypassEnabled to true, since we don't need to test authentication here
FieldUtils.writeField(Settings.getInstance(), "localAuthBypassEnabled", true, true);
assertNotNull(this.adminResource.summary("testApiKey")); assertNotNull(this.adminResource.summary("testApiKey"));
} }

3
src/test/resources/test-settings-v2.json
View File

@ -15,6 +15,5 @@
"tempDataPath": "data-test/_temp", "tempDataPath": "data-test/_temp",
"listsPath": "lists-test", "listsPath": "lists-test",
"storagePolicy": "FOLLOWED_OR_VIEWED", "storagePolicy": "FOLLOWED_OR_VIEWED",
"maxStorageCapacity": 104857600, "maxStorageCapacity": 104857600
"localAuthBypassEnabled": true
} }