fix bug on removing support

for the Removing support, i don't need to divided removing support into
before/during/after making suggesting?

On Tue, Sep 16, 2014 at 4:46 PM, Oliver Azevedo Barnes <
notifications@github.com> wrote:

> In tests/acceptance/supporting-an-initiative-test.js:
>
> > @@ -19,10 +19,49 @@ test('Successfully', function(){
> >      click( $("a:contains('Support this initiative')") ).then(function() {
> >        expect(currentURL()).to.equal('/initiatives/1');
> >        expect(find('.support').text()).to.equal('Remove support for this initiative');
> > +      expect(find('.new_suggestion').text()).to.equal('Make a suggestion');
> > +      // test on removing support before making suggestion
>
> This is the success scenario for supporting an initiative, so testing
> removal of support here is out of scope. Notice how the comment here is
> really describing another scenario?
>
> I think this test suite can be renamed to a general 'Initiative support'
>
> suite('Initiative Support', {
>
> And then have just two scenarios, 'Adding support' and 'Removing support'.
> Ensuring that the suggestion text field is not present after removing
> support is enough
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oliverbarnes/participate-frontend/pull/50/files#r17601352>
> .
>