Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#12484] - Remove JFilter mock from JInstaller Adapter tests

? ? Pending

User tests: Successful: Unsuccessful:

avatar photodude
photodude
20 Oct 2016

Pull Request for Issue unnecessary code in Unit test .

Summary of Changes

Remove JFilter from JInstaller test setup

We don't need JFilter just hanging out in the JInstaller Adapter tests

Testing Instructions

Review Travis Tests for passing

Documentation Changes Required

None

avatar photodude photodude - open - 20 Oct 2016
avatar photodude photodude - change - 20 Oct 2016
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2016
Labels Added: ? ?
avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2016
Category ⇒ Unit Tests
avatar photodude photodude - change - 20 Oct 2016
Title
…
Remove JFilter mock
Remove JFilter mock from JInstaller Adapter tests
avatar photodude photodude - edited - 20 Oct 2016
avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2016
Labels Removed: ?
avatar brianteeman brianteeman - change - 23 Oct 2016
Labels Added: ?
avatar photodude
photodude - comment - 26 Oct 2016

Bump for merge consideration
@mbabker, @wilsonge, @rdeutz

avatar brianteeman brianteeman - close - 26 Oct 2016
avatar mbabker mbabker - close - 26 Oct 2016
avatar mbabker mbabker - merge - 26 Oct 2016
avatar mbabker mbabker - change - 26 Oct 2016
Status Pending ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2016-10-26 23:52:38
Closed_By ⇒ mbabker
avatar andrepereiradasilva
andrepereiradasilva - comment - 3 Nov 2016

missing 3.7.0 label here

avatar brianteeman brianteeman - change - 3 Nov 2016
The description was changed

Add a Comment

Login with GitHub to post a comment