Skip to content

Commit c1bc083

Browse files
sparkhisparkhi
committed
nested calls simplified by extracting request the first time
1 parent 959123a commit c1bc083

2 files changed

Lines changed: 48 additions & 47 deletions

File tree

‎README.md‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,8 @@ List<FailMessage> messages = validateWithStringNames
8585
.usingProgress(progress)
8686
.usingSkipFileChecks(skipFileChecks)
8787
.usingMaxCharsPerCell(maxCharsPerCell)
88-
.runValidation().getErrors();
88+
.runValidation()
89+
.getErrors();
8990

9091
if(messages.isEmpty()) {
9192
System.out.println("All worked OK");

‎csv-validator-java-api/src/test/java/uk/gov/nationalarchives/csv/validator/api/java/CsvValidatorTest.java‎

Lines changed: 46 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -24,33 +24,33 @@ public class CsvValidatorTest {
2424
@Test
2525
public void defaultValuesAreUsedWhenUsingTextFileMethod() {
2626
Result result = new CsvValidator.ValidatorBuilder("csvFile", "csvSchema").runValidation();
27-
CsvValidatorJavaBridge.ValidationResult validationResult = (CsvValidatorJavaBridge.ValidationResult) result;
28-
Assert.assertEquals("csvFile", validationResult.validatorRequest().csvFile());
29-
Assert.assertEquals(StandardCharsets.UTF_8, validationResult.validatorRequest().csvEncoding());
30-
Assert.assertEquals("csvSchema", validationResult.validatorRequest().csvSchemaFile());
31-
Assert.assertEquals(StandardCharsets.UTF_8, validationResult.validatorRequest().csvSchemaEncoding());
32-
Assert.assertFalse(validationResult.validatorRequest().failFast());
33-
Assert.assertEquals(new ArrayList<Substitution>(), validationResult.validatorRequest().pathSubstitutionsList());
34-
Assert.assertFalse(validationResult.validatorRequest().enforceCaseSensitivePathChecks());
35-
Assert.assertFalse(validationResult.validatorRequest().trace());
36-
Assert.assertNull(validationResult.validatorRequest().progress());
37-
Assert.assertFalse(validationResult.validatorRequest().skipFileChecks());
38-
Assert.assertEquals(4096, validationResult.validatorRequest().maxCharsPerCellLimit());
27+
CsvValidatorJavaBridge.ValidationRequest validationRequest = ((CsvValidatorJavaBridge.ValidationResult) result).validatorRequest();
28+
Assert.assertEquals("csvFile", validationRequest.csvFile());
29+
Assert.assertEquals(StandardCharsets.UTF_8, validationRequest.csvEncoding());
30+
Assert.assertEquals("csvSchema", validationRequest.csvSchemaFile());
31+
Assert.assertEquals(StandardCharsets.UTF_8, validationRequest.csvSchemaEncoding());
32+
Assert.assertFalse(validationRequest.failFast());
33+
Assert.assertEquals(new ArrayList<Substitution>(), validationRequest.pathSubstitutionsList());
34+
Assert.assertFalse(validationRequest.enforceCaseSensitivePathChecks());
35+
Assert.assertFalse(validationRequest.trace());
36+
Assert.assertNull(validationRequest.progress());
37+
Assert.assertFalse(validationRequest.skipFileChecks());
38+
Assert.assertEquals(4096, validationRequest.maxCharsPerCellLimit());
3939
}
4040

4141
@Test
4242
public void defaultValuesAreUsedWhenUsingReaderMethod() {
4343
Result result = new CsvValidator.ValidatorBuilder(csvFileReader, csvSchemaReader).runValidation();
44-
CsvValidatorJavaBridge.ReaderValidationResult validationResult = (CsvValidatorJavaBridge.ReaderValidationResult) result;
45-
Assert.assertEquals(csvFileReader, validationResult.validatorRequest().csvReader());
46-
Assert.assertEquals(csvSchemaReader, validationResult.validatorRequest().csvSchemaReader());
47-
Assert.assertFalse(validationResult.validatorRequest().failFast());
48-
Assert.assertEquals(new ArrayList<Substitution>(), validationResult.validatorRequest().pathSubstitutionsList());
49-
Assert.assertFalse(validationResult.validatorRequest().enforceCaseSensitivePathChecks());
50-
Assert.assertFalse(validationResult.validatorRequest().trace());
51-
Assert.assertNull(validationResult.validatorRequest().progress());
52-
Assert.assertFalse(validationResult.validatorRequest().skipFileChecks());
53-
Assert.assertEquals(4096, validationResult.validatorRequest().maxCharsPerCellLimit());
44+
CsvValidatorJavaBridge.ReaderValidationRequest validationRequest = ((CsvValidatorJavaBridge.ReaderValidationResult) result).validatorRequest();
45+
Assert.assertEquals(csvFileReader, validationRequest.csvReader());
46+
Assert.assertEquals(csvSchemaReader, validationRequest.csvSchemaReader());
47+
Assert.assertFalse(validationRequest.failFast());
48+
Assert.assertEquals(new ArrayList<Substitution>(), validationRequest.pathSubstitutionsList());
49+
Assert.assertFalse(validationRequest.enforceCaseSensitivePathChecks());
50+
Assert.assertFalse(validationRequest.trace());
51+
Assert.assertNull(validationRequest.progress());
52+
Assert.assertFalse(validationRequest.skipFileChecks());
53+
Assert.assertEquals(4096, validationRequest.maxCharsPerCellLimit());
5454
}
5555

5656
@Test
@@ -79,19 +79,19 @@ public void correctValuesArePassedIntoValidateMethodUsingTextFileMethod() throws
7979
.usingSkipFileChecks(true)
8080
.usingMaxCharsPerCell(8096)
8181
.runValidation();
82-
CsvValidatorJavaBridge.ValidationResult validationResult = (CsvValidatorJavaBridge.ValidationResult) result;
83-
Assert.assertEquals("csvFile", validationResult.validatorRequest().csvFile());
84-
Assert.assertEquals(StandardCharsets.ISO_8859_1, validationResult.validatorRequest().csvEncoding());
85-
Assert.assertEquals("csvSchema", validationResult.validatorRequest().csvSchemaFile());
86-
Assert.assertEquals(StandardCharsets.ISO_8859_1, validationResult.validatorRequest().csvSchemaEncoding());
87-
Assert.assertTrue(validationResult.validatorRequest().failFast());
88-
Assert.assertEquals("fromPath", validationResult.validatorRequest().pathSubstitutionsList().get(0).getFrom());
89-
Assert.assertEquals("toPath", validationResult.validatorRequest().pathSubstitutionsList().get(0).getTo());
90-
Assert.assertTrue(validationResult.validatorRequest().enforceCaseSensitivePathChecks());
91-
Assert.assertTrue(validationResult.validatorRequest().trace());
92-
Assert.assertEquals(progressCallback, validationResult.validatorRequest().progress());
93-
Assert.assertTrue(validationResult.validatorRequest().skipFileChecks());
94-
Assert.assertEquals(8096, validationResult.validatorRequest().maxCharsPerCellLimit());
82+
CsvValidatorJavaBridge.ValidationRequest validationRequest = ((CsvValidatorJavaBridge.ValidationResult) result).validatorRequest();
83+
Assert.assertEquals("csvFile", validationRequest.csvFile());
84+
Assert.assertEquals(StandardCharsets.ISO_8859_1, validationRequest.csvEncoding());
85+
Assert.assertEquals("csvSchema", validationRequest.csvSchemaFile());
86+
Assert.assertEquals(StandardCharsets.ISO_8859_1, validationRequest.csvSchemaEncoding());
87+
Assert.assertTrue(validationRequest.failFast());
88+
Assert.assertEquals("fromPath", validationRequest.pathSubstitutionsList().get(0).getFrom());
89+
Assert.assertEquals("toPath", validationRequest.pathSubstitutionsList().get(0).getTo());
90+
Assert.assertTrue(validationRequest.enforceCaseSensitivePathChecks());
91+
Assert.assertTrue(validationRequest.trace());
92+
Assert.assertEquals(progressCallback, validationRequest.progress());
93+
Assert.assertTrue(validationRequest.skipFileChecks());
94+
Assert.assertEquals(8096, validationRequest.maxCharsPerCellLimit());
9595
}
9696

9797
@Test
@@ -109,17 +109,17 @@ public void correctValuesArePassedIntoValidateMethodUsingReaderMethod() {
109109
.usingMaxCharsPerCell(16096)
110110
.runValidation();
111111

112-
CsvValidatorJavaBridge.ReaderValidationResult validationResult = (CsvValidatorJavaBridge.ReaderValidationResult) result;
113-
Assert.assertEquals(csvFileReader, validationResult.validatorRequest().csvReader());
114-
Assert.assertEquals(csvSchemaReader, validationResult.validatorRequest().csvSchemaReader());
115-
Assert.assertTrue(validationResult.validatorRequest().failFast());
116-
Assert.assertEquals("fromPath", validationResult.validatorRequest().pathSubstitutionsList().get(0).getFrom());
117-
Assert.assertEquals("toPath", validationResult.validatorRequest().pathSubstitutionsList().get(0).getTo());
118-
Assert.assertTrue(validationResult.validatorRequest().enforceCaseSensitivePathChecks());
119-
Assert.assertTrue(validationResult.validatorRequest().trace());
120-
Assert.assertEquals(progressCallback, validationResult.validatorRequest().progress());
121-
Assert.assertTrue(validationResult.validatorRequest().skipFileChecks());
122-
Assert.assertEquals(16096, validationResult.validatorRequest().maxCharsPerCellLimit());
112+
CsvValidatorJavaBridge.ReaderValidationRequest validationRequest = ((CsvValidatorJavaBridge.ReaderValidationResult) result).validatorRequest();
113+
Assert.assertEquals(csvFileReader, validationRequest.csvReader());
114+
Assert.assertEquals(csvSchemaReader, validationRequest.csvSchemaReader());
115+
Assert.assertTrue(validationRequest.failFast());
116+
Assert.assertEquals("fromPath", validationRequest.pathSubstitutionsList().get(0).getFrom());
117+
Assert.assertEquals("toPath", validationRequest.pathSubstitutionsList().get(0).getTo());
118+
Assert.assertTrue(validationRequest.enforceCaseSensitivePathChecks());
119+
Assert.assertTrue(validationRequest.trace());
120+
Assert.assertEquals(progressCallback, validationRequest.progress());
121+
Assert.assertTrue(validationRequest.skipFileChecks());
122+
Assert.assertEquals(16096, validationRequest.maxCharsPerCellLimit());
123123
}
124124
}
125125

0 commit comments

Comments
 (0)