Merge branch 'add-missed-weight-filter-reset-spec' into 'master'

Added a weight filter reset scenario

Looks like we missed a scenario for testing weight filtering for EE.

I'm not 100% this is the correct implementation, maybe I have to `create(:weight)` but I did this in the web UI so will adjust the spec if they fail.

/cc @brycepj

See merge request !788

spec/features/issues/reset_filters_spec.rb

@@ -9,7 +9,7 @@ feature 'Issues filter reset button', feature: true, js: true do
   let!(:milestone)  { create(:milestone, project: project) }
   let!(:bug)        { create(:label, project: project, name: 'bug')}
   let!(:issue1)     { create(:issue, project: project, milestone: milestone, author: user, assignee: user, title: 'Feature')}
-  let!(:issue2)     { create(:labeled_issue, project: project, labels: [bug], title: 'Bugfix1')}
+  let!(:issue2)     { create(:labeled_issue, project: project, labels: [bug], title: 'Bugfix1', weight: '1')}
 
   before do
     project.team << [user, :developer]
@@ -64,10 +64,20 @@ feature 'Issues filter reset button', feature: true, js: true do
       expect(page).to have_css('.issue', count: 2)
     end
   end
+  
+  context 'when weight filter has been applied' do
+    it 'resets the weight filter' do
+      visit_issues(project, weight: '1')
+      expect(page).to have_css('.issue', count: 1)
+
+      reset_filters
+      expect(page).to have_css('.issue', count: 2)
+    end
+  end
 
   context 'when all filters have been applied' do
     it 'resets all filters' do
-      visit_issues(project, assignee_id: user.id, author_id: user.id, milestone_title: milestone.title, label_name: bug.name, search: 'Bug')
+      visit_issues(project, assignee_id: user.id, author_id: user.id, milestone_title: milestone.title, label_name: bug.name, weight: '1', search: 'Bug')
       expect(page).to have_css('.issue', count: 0)
 
       reset_filters