Skip to content

Conversation

PhilETaylor
Copy link
Contributor

@PhilETaylor PhilETaylor commented May 4, 2017

Closing right away as using Github as a placeholder to share these changes.

(Replicate #15673 in an isolated test and unit test)

@mbabker
Copy link
Contributor

mbabker commented May 4, 2017

Reopening, similar to #15642

@PhilETaylor

This comment was marked as abuse.

@photodude
Copy link
Contributor

For what it's worth, it's stalling out when it runs the testCleanWithDefaultBlackList with the data set in your PR.

@wilsonge
Copy link
Contributor

@PhilETaylor can you integrate your unit test here into #15966 please

@rdeutz
Copy link
Contributor

rdeutz commented May 12, 2017

Doesn't makes sense to have this in unit tests because it brings the test suite into a infinitive loop

@rdeutz
Copy link
Contributor

rdeutz commented May 12, 2017

ah this here worked: https://travis-ci.org/photodude/filter/jobs/231328081

@PhilETaylor

This comment was marked as abuse.

@PhilETaylor

This comment was marked as abuse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants