markap14 commented on a change in pull request #3930: NIFI-6851: Added
FlowFileAcquisitionMethod to fetch/receive methods i…
URL: https://github.com/apache/nifi/pull/3930#discussion_r357752200
##########
File path:
nifi-nar-bundles/nifi-provenance-repository-bundle/nifi-persistent-provenance-repository/src/test/groovy/org/apache/nifi/provenance/EncryptedSchemaRecordReaderWriterTest.groovy
##########
@@ -273,10 +273,10 @@ class EncryptedSchemaRecordReaderWriterTest extends
AbstractTestRecordReaderWrit
RecordReader incompatibleReader = new
EventIdFirstSchemaRecordReader(efis, encryptedJournalFile.getName(),
incompatibleTocReader, MAX_ATTRIBUTE_SIZE)
logger.info("Generated standard reader (attempting to read encrypted
file): ${incompatibleReader}")
- def msg = shouldFail(EOFException) {
+ def msg = shouldFail(IOException) {
ProvenanceEventRecord encryptedEvent =
incompatibleReader.nextRecord()
}
logger.expected(msg)
- assert msg =~ "EOFException: Failed to read field"
+ assert msg =~ "IOException: Invalid Boolean value found when reading
'Repetition' of field"
Review comment:
This change is not clear to me. The behavior of the test seems to have
changed here. The test was testing for a very specific failure. This appears to
have changed the failure that the test expects...
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
