Skip to content

NIFI-2098 Adding ignore annotation to testManyFilesOpened unit test#570

Closed
JPercivall wants to merge 1 commit intoapache:0.xfrom
JPercivall:NIFI-2098
Closed

NIFI-2098 Adding ignore annotation to testManyFilesOpened unit test#570
JPercivall wants to merge 1 commit intoapache:0.xfrom
JPercivall:NIFI-2098

Conversation

@JPercivall
Copy link
Contributor

No description provided.

@markap14
Copy link
Contributor

@JPercivall I think this unit test is actually quite helpful to have. It can add a little while to the build process, but it ensures that we don't accidentally hold open the file handles that we are not supposed to hold open. I think the System.out.println() calls need to be replaced with logger calls but don't think we should Ignore the test. If we want to avoid running it too often, we should probably look at making it an integration test instead.

@JPercivall JPercivall closed this Aug 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants