Skip to content

Commit

Permalink
Fixed more warnings (#125)
Browse files Browse the repository at this point in the history
  • Loading branch information
depryf committed Oct 20, 2023
1 parent ab5035b commit 9e69a68
Show file tree
Hide file tree
Showing 5 changed files with 21 additions and 11 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,12 @@

**Version 3.1**

- Updated Groovy library from version 4.0.8 to version 4.0.15.
- Updated staging client from version 10.1.0 to version 11.0.0.

**Version 3.1**

- Updated Groovy library from version 4.0.8 to version 4.0.13.
- Updated staging client from version 6.3.0 to version 10.1.0.

**Version 3.0**
Expand Down
20 changes: 10 additions & 10 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -24,21 +24,21 @@ repositories {
}

dependencies {
api 'com.imsweb:staging-client-java:10.2.0'
api 'org.apache.groovy:groovy:4.0.13'
api 'com.imsweb:staging-client-java:11.0.0'
api 'org.apache.groovy:groovy:4.0.15'
api 'com.thoughtworks.xstream:xstream:1.4.20'

implementation 'commons-codec:commons-codec:1.16.0'
implementation 'org.apache.commons:commons-lang3:3.12.0'
implementation 'org.apache.commons:commons-lang3:3.13.0'

testImplementation 'junit:junit:4.13.2'
testImplementation 'commons-io:commons-io:2.13.0'
testImplementation 'com.imsweb:layout:5.2'
testImplementation 'com.imsweb:staging-client-java-cs:10.2.0'
testImplementation 'com.imsweb:staging-client-java-eod-public:10.2.0'
testImplementation 'com.imsweb:staging-client-java-tnm:10.2.0'
testImplementation('com.imsweb:validation-edits-seer:022-18') { exclude module: 'validation' }
testImplementation('com.imsweb:validation-edits-naaccr-translated:015-01') { exclude module: 'validation' }
testImplementation 'commons-io:commons-io:2.14.0'
testImplementation 'com.imsweb:layout:5.3'
testImplementation 'com.imsweb:staging-client-java-cs:11.0.0'
testImplementation 'com.imsweb:staging-client-java-eod-public:11.0.0'
testImplementation 'com.imsweb:staging-client-java-tnm:11.0.0'
testImplementation('com.imsweb:validation-edits-seer:023-08') { exclude module: 'validation' }
testImplementation('com.imsweb:validation-edits-naaccr-translated:018-01') { exclude module: 'validation' }
}

// enforce UTF-8, display the compilation warnings
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -186,6 +186,8 @@ public void testValidatorErrorWriteNullValidatorForWriter() throws IOException {
public void testValidatorEmptyData() throws IOException, ConstructionException {
Validator v = ValidationXmlUtils.loadValidatorFromXml(Thread.currentThread().getContextClassLoader().getResource("fake-validator-empty.xml"));

Assert.assertTrue(v.getRules().isEmpty());

ValidationEngine.getInstance().addValidator(new EditableValidator(v));
ValidationEngine.getInstance().deleteValidator(new EditableValidator(v));

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
import static com.imsweb.validation.functions.MetafileContextFunctions.TRIM_LEFT;
import static com.imsweb.validation.functions.MetafileContextFunctions.TRIM_RIGHT;

@SuppressWarnings("java:S5961") // methods too complex
public class MetafileContextFunctionsTest {

private MetafileContextFunctions _functions;
Expand Down Expand Up @@ -1218,6 +1219,7 @@ public void testGEN_ILOOKUP() {
}

@Test
@SuppressWarnings("java:S5778")
public void testGEN_SQLLOOKUP() {

// define the table we are going to use
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -563,7 +563,7 @@ public void testGetEodSchemaId() {
input.clear();
input.put("primarySite", "C530");
input.put("histologicTypeIcdO3", "8000");
Assert.assertNull(_functions.getEodSchemaId(input));
Assert.assertEquals("cervix_9th_2021", _functions.getEodSchemaId(input));
input.put("dateOfDiagnosisYear", "2018");
Assert.assertEquals("cervix", _functions.getEodSchemaId(input));
}
Expand Down

0 comments on commit 9e69a68

Please sign in to comment.