Skip to content

Add test case replicating JFilterInput timeout issue#15810

Closed
PhilETaylor wants to merge 1 commit intojoomla:stagingfrom
PhilETaylor:joomla-cms-15673
Closed

Add test case replicating JFilterInput timeout issue#15810
PhilETaylor wants to merge 1 commit intojoomla:stagingfrom
PhilETaylor:joomla-cms-15673

Conversation

@PhilETaylor
Copy link
Copy Markdown
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
Copy Markdown
Contributor

mbabker commented May 4, 2017

Reopening, similar to #15642

@PhilETaylor

This comment was marked as abuse.

@photodude
Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

@rdeutz
Copy link
Copy Markdown
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
Copy Markdown
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