Skip to content

Commit 7a9911e

Browse files
HeniganHenigan
authored andcommitted
Align to latest iOS PR
1 parent 1f4cfcc commit 7a9911e

13 files changed

Lines changed: 439 additions & 422 deletions

File tree

android/sdl_android/src/androidTest/java/com/smartdevicelink/test/Validator.java

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,17 +3,16 @@
33
import com.smartdevicelink.managers.file.filetypes.SdlFile;
44
import com.smartdevicelink.protocol.enums.FrameDataControlFrameType;
55
import com.smartdevicelink.protocol.enums.FrameType;
6-
import com.smartdevicelink.protocol.enums.QueryErrorCode;
7-
import com.smartdevicelink.protocol.enums.QueryID;
8-
import com.smartdevicelink.protocol.enums.QueryType;
6+
import com.smartdevicelink.protocol.enums.SecurityQueryErrorCode;
7+
import com.smartdevicelink.protocol.enums.SecurityQueryID;
8+
import com.smartdevicelink.protocol.enums.SecurityQueryType;
99
import com.smartdevicelink.protocol.enums.SessionType;
1010
import com.smartdevicelink.proxy.rpc.*;
1111
import com.smartdevicelink.proxy.rpc.enums.AppServiceType;
1212
import com.smartdevicelink.proxy.rpc.enums.DefrostZone;
1313
import com.smartdevicelink.proxy.rpc.enums.FileType;
1414
import com.smartdevicelink.proxy.rpc.enums.HMILevel;
1515
import com.smartdevicelink.proxy.rpc.enums.HmiZoneCapabilities;
16-
import com.smartdevicelink.proxy.rpc.enums.KeyboardLayout;
1716
import com.smartdevicelink.proxy.rpc.enums.PRNDL;
1817
import com.smartdevicelink.proxy.rpc.enums.PrerecordedSpeech;
1918
import com.smartdevicelink.proxy.rpc.enums.SpeechCapabilities;
@@ -129,7 +128,7 @@ public static boolean validateSessionTypeArray(SessionType[] array1, SessionType
129128
return true;
130129
}
131130

132-
public static boolean validateQueryTypeArray(QueryType[] array1, QueryType[] array2) {
131+
public static boolean validateQueryTypeArray(SecurityQueryType[] array1, SecurityQueryType[] array2) {
133132

134133
if (array1 == null) {
135134
return (array2 == null);
@@ -152,7 +151,7 @@ public static boolean validateQueryTypeArray(QueryType[] array1, QueryType[] arr
152151
return true;
153152
}
154153

155-
public static boolean validateQueryIDArray(QueryID[] array1, QueryID[] array2) {
154+
public static boolean validateQueryIDArray(SecurityQueryID[] array1, SecurityQueryID[] array2) {
156155

157156
if (array1 == null) {
158157
return (array2 == null);
@@ -175,7 +174,7 @@ public static boolean validateQueryIDArray(QueryID[] array1, QueryID[] array2) {
175174
return true;
176175
}
177176

178-
public static boolean validateQueryErrorCodeArray(QueryErrorCode[] array1, QueryErrorCode[] array2) {
177+
public static boolean validateQueryErrorCodeArray(SecurityQueryErrorCode[] array1, SecurityQueryErrorCode[] array2) {
179178

180179
if (array1 == null) {
181180
return (array2 == null);

android/sdl_android/src/androidTest/java/com/smartdevicelink/test/protocol/SecurityQueryPayloadTests.java

Lines changed: 11 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
import androidx.test.ext.junit.runners.AndroidJUnit4;
44

55
import com.smartdevicelink.protocol.SecurityQueryPayload;
6-
import com.smartdevicelink.protocol.enums.QueryID;
7-
import com.smartdevicelink.protocol.enums.QueryType;
6+
import com.smartdevicelink.protocol.enums.SecurityQueryID;
7+
import com.smartdevicelink.protocol.enums.SecurityQueryType;
88
import com.smartdevicelink.util.BitConverter;
99

1010
import org.junit.Test;
@@ -22,8 +22,8 @@ public class SecurityQueryPayloadTests {
2222
public static SecurityQueryPayload createDummyBqh() {
2323
SecurityQueryPayload bqh = new SecurityQueryPayload();
2424
bqh.setCorrelationID(123);
25-
bqh.setQueryID(QueryID.SEND_HANDSHAKE_DATA);
26-
bqh.setQueryType(QueryType.REQUEST);
25+
bqh.setQueryID(SecurityQueryID.SEND_HANDSHAKE_DATA);
26+
bqh.setQueryType(SecurityQueryType.REQUEST);
2727
bqh.setBulkData(null);
2828
bqh.setJsonSize(0);
2929
return bqh;
@@ -54,25 +54,25 @@ public void testCorrectParsing() {
5454
array[11] = 0;
5555

5656
SecurityQueryPayload parsedBqh = SecurityQueryPayload.parseBinaryQueryHeader(array);
57-
assertEquals(parsedBqh.getQueryType(), QueryType.REQUEST);
58-
assertEquals(parsedBqh.getQueryID(), QueryID.SEND_INTERNAL_ERROR);
57+
assertEquals(parsedBqh.getQueryType(), SecurityQueryType.REQUEST);
58+
assertEquals(parsedBqh.getQueryID(), SecurityQueryID.SEND_INTERNAL_ERROR);
5959
assertEquals(parsedBqh.getCorrelationID(), 3);
6060
assertEquals(parsedBqh.getJsonSize(), 0);
6161
}
6262

6363
@Test
6464
public void testCorrectHeaderAssembly() {
6565
SecurityQueryPayload dummyBqh = new SecurityQueryPayload();
66-
dummyBqh.setQueryType(QueryType.REQUEST);
67-
dummyBqh.setQueryID(QueryID.SEND_HANDSHAKE_DATA);
66+
dummyBqh.setQueryType(SecurityQueryType.REQUEST);
67+
dummyBqh.setQueryID(SecurityQueryID.SEND_HANDSHAKE_DATA);
6868
dummyBqh.setCorrelationID(3);
6969
dummyBqh.setJsonSize(0);
7070

7171
byte[] assembledHeader = dummyBqh.assembleHeaderBytes();
72-
assertEquals(dummyBqh.getQueryType(), QueryType.valueOf(assembledHeader[0]));
72+
assertEquals(dummyBqh.getQueryType(), SecurityQueryType.valueOf(assembledHeader[0]));
7373
byte[] queryIDFromHeader = new byte[3];
7474
System.arraycopy(assembledHeader, 1, queryIDFromHeader, 0, 3);
75-
assertEquals(dummyBqh.getQueryID(), QueryID.valueOf(queryIDFromHeader));
75+
assertEquals(dummyBqh.getQueryID(), SecurityQueryID.valueOf(queryIDFromHeader));
7676
assertEquals(dummyBqh.getCorrelationID(), BitConverter.intFromByteArray(assembledHeader, 4));
7777
assertEquals(dummyBqh.getJsonSize(), BitConverter.intFromByteArray(assembledHeader, 8));
7878
}
@@ -98,15 +98,10 @@ public void testAssemblyAndParse() {
9898
@Test
9999
public void testCorruptHeader() {
100100
SecurityQueryPayload bqh = createDummyBqh();
101-
bqh.setJsonSize(5);
102-
bqh.setJsonData(new byte[5]);
103-
bqh.setJsonSize(Integer.MAX_VALUE);
104-
105-
assertNotEquals(bqh.getJsonData().length, bqh.getJsonSize());
106101

107102
byte[] bqhBytes = bqh.assembleHeaderBytes();
108103

109-
assertNull(safeParse(bqhBytes));
104+
assertNotNull(safeParse(bqhBytes));
110105

111106
int size = bqhBytes.length;
112107
for (int i = 0; i < size; i++) {

android/sdl_android/src/androidTest/java/com/smartdevicelink/test/protocol/enums/QueryErrorCodeTests.java

Lines changed: 0 additions & 189 deletions
This file was deleted.

0 commit comments

Comments
 (0)