? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
4 May 2017

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

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

avatar PhilETaylor PhilETaylor - open - 4 May 2017
avatar PhilETaylor PhilETaylor - change - 4 May 2017
Status New Pending
avatar PhilETaylor PhilETaylor - change - 4 May 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-05-04 09:15:50
Closed_By PhilETaylor
Labels Added: ? ?
avatar PhilETaylor PhilETaylor - close - 4 May 2017
avatar joomla-cms-bot joomla-cms-bot - change - 4 May 2017
Category Unit Tests
avatar mbabker
mbabker - comment - 4 May 2017

Reopening, similar to #15642

avatar mbabker mbabker - change - 4 May 2017
Status Closed New
Closed_Date 2017-05-04 09:15:50
Closed_By PhilETaylor
avatar mbabker mbabker - change - 4 May 2017
Status New Pending
avatar mbabker mbabker - reopen - 4 May 2017
avatar mbabker mbabker - change - 4 May 2017
Title
Replicate #15673 in an isolated test and unit test
Add test case replicating JFilterInput timeout issue
avatar mbabker mbabker - edited - 4 May 2017
avatar PhilETaylor
PhilETaylor - comment - 4 May 2017

ok thanks - its completely basic - not intended to be fit for use at all at the moment, just gives an example of the issue, in its broken state, and a test script for whoever is better at mb* stuff than me

avatar PhilETaylor PhilETaylor - edited - 4 May 2017
avatar photodude
photodude - comment - 4 May 2017

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

avatar wilsonge
wilsonge - comment - 12 May 2017

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

avatar rdeutz
rdeutz - comment - 12 May 2017

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

avatar rdeutz
rdeutz - comment - 12 May 2017
avatar PhilETaylor
PhilETaylor - comment - 12 May 2017

Im merging the test with his remote branch now...

avatar PhilETaylor
PhilETaylor - comment - 12 May 2017

ok I failed to merge it to ggppdk:patch-30 :(

I tested locally and the unit tests run/pass when applied like this #15978

avatar PhilETaylor PhilETaylor - change - 12 May 2017
The description was changed
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-05-12 10:33:31
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 12 May 2017

Add a Comment

Login with GitHub to post a comment