Skip to content

Commit 9373d7a

Browse files
SnipxiText-CI
authored andcommitted
Avoid using Assert.assertSame for enums and primitive types
There is a difference in how Java and .NET implement this assertion and it does not make a lot of sense for primitives or enums anyway DEVSIX-3585
1 parent c262a8c commit 9373d7a

File tree

1 file changed

+24
-24
lines changed

1 file changed

+24
-24
lines changed

kernel/src/test/java/com/itextpdf/kernel/pdf/PdfTokenizerTest.java

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -231,104 +231,104 @@ public void primitivesTest() throws Exception {
231231
PdfTokenizer tok = new PdfTokenizer(new RandomAccessFileOrArray(factory.createSource(data.getBytes(StandardCharsets.ISO_8859_1))));
232232

233233
tok.nextValidToken();
234-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.StartDic);
234+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.StartDic);
235235

236236
tok.nextValidToken();
237-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Name);
237+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Name);
238238
PdfName name = new PdfName(tok.getByteContent());
239239
Assert.assertEquals("Size", name.getValue());
240240

241241
tok.nextValidToken();
242-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Number);
242+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Number);
243243
PdfNumber num = new PdfNumber(tok.getByteContent());
244244
Assert.assertEquals("70.", num.toString());
245245

246246
tok.nextValidToken();
247-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Name);
247+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Name);
248248
name = new PdfName(tok.getByteContent());
249249
Assert.assertEquals("Value ", name.getValue());
250250

251251
tok.nextValidToken();
252-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Number);
252+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Number);
253253
num = new PdfNumber(tok.getByteContent());
254254
Assert.assertNotSame("0.1", num.toString());
255255

256256
tok.nextValidToken();
257-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Name);
257+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Name);
258258
name = new PdfName(tok.getByteContent());
259259
Assert.assertEquals("Root", name.getValue());
260260

261261
tok.nextValidToken();
262-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Ref);
262+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Ref);
263263
PdfIndirectReference ref = new PdfIndirectReference(null, tok.getObjNr(), tok.getGenNr());
264264
Assert.assertEquals("46 0 R", ref.toString());
265265

266266
tok.nextValidToken();
267-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Name);
267+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Name);
268268
name = new PdfName(tok.getByteContent());
269269
Assert.assertEquals("Info", name.getValue());
270270

271271
tok.nextValidToken();
272-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Ref);
272+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Ref);
273273
ref = new PdfIndirectReference(null, tok.getObjNr(), tok.getGenNr());
274274
Assert.assertEquals("44 0 R", ref.toString());
275275

276276
tok.nextValidToken();
277-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Name);
277+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Name);
278278
name = new PdfName(tok.getByteContent());
279279
Assert.assertEquals("ID", name.getValue());
280280

281281
tok.nextValidToken();
282-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.StartArray);
282+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.StartArray);
283283

284284
tok.nextValidToken();
285-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.String);
286-
Assert.assertSame(tok.isHexString(), true);
285+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.String);
286+
Assert.assertTrue(tok.isHexString());
287287
PdfString str = new PdfString(tok.getByteContent(), tok.isHexString());
288288
Assert.assertEquals("some hex string ", str.getValue());
289289

290290
tok.nextValidToken();
291-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.String);
292-
Assert.assertSame(tok.isHexString(), false);
291+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.String);
292+
Assert.assertFalse(tok.isHexString());
293293
str = new PdfString(tok.getByteContent(), tok.isHexString());
294294
Assert.assertEquals("some simple string ", str.getValue());
295295

296296
tok.nextValidToken();
297-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.String);
298-
Assert.assertSame(tok.isHexString(), true);
297+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.String);
298+
Assert.assertTrue(tok.isHexString());
299299
str = new PdfString(tok.getByteContent(), tok.isHexString());
300300
Assert.assertEquals("\u008C%G\u00D5\u008DK\u00D2\u00C6\u00F3\u00D3+\u0083\u000B\u00E3%\u009D ", str.getValue());
301301

302302
tok.nextValidToken();
303-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Number);
303+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Number);
304304
num = new PdfNumber(tok.getByteContent());
305305
Assert.assertEquals("-70.1", num.toString());
306306

307307
tok.nextValidToken();
308-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Number);
308+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Number);
309309
num = new PdfNumber(tok.getByteContent());
310310
Assert.assertEquals("-0.2", num.toString());
311311

312312
tok.nextValidToken();
313-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.EndArray);
313+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.EndArray);
314314

315315
tok.nextValidToken();
316-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Name);
316+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Name);
317317
name = new PdfName(tok.getByteContent());
318318
Assert.assertEquals("Name1", name.getValue());
319319

320320
tok.nextValidToken();
321-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Number);
321+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Number);
322322
num = new PdfNumber(tok.getByteContent());
323323
Assert.assertEquals("0", num.toString());
324324

325325
tok.nextValidToken();
326-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Name);
326+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Name);
327327
name = new PdfName(tok.getByteContent());
328328
Assert.assertEquals("Prev", name.getValue());
329329

330330
tok.nextValidToken();
331-
Assert.assertSame(tok.getTokenType(), PdfTokenizer.TokenType.Number);
331+
Assert.assertEquals(tok.getTokenType(), PdfTokenizer.TokenType.Number);
332332
num = new PdfNumber(tok.getByteContent());
333333
Assert.assertEquals("-116.23", num.toString());
334334
}

0 commit comments

Comments
 (0)