Commit 79ade41e authored by Jan Beckmann's avatar Jan Beckmann Committed by Rémy Coutable

Fix static analysis and test cases

parent bb0f525a
......@@ -105,7 +105,7 @@ export default class Search {
getProjectsData(term) {
return new Promise((resolve) => {
if (this.groupId) {
Api.groupProjects(this.groupId, term, resolve);
Api.groupProjects(this.groupId, term, {}, resolve);
} else {
Api.projects(term, {
order_by: 'id',
......
......@@ -59,9 +59,11 @@ feature 'Group merge requests page' do
describe 'new merge request dropdown' do
let(:project_with_merge_requests_disabled) { create(:project, :merge_requests_disabled, group: group) }
before { visit path }
before do
visit path
end
it 'shows projects only with merge requests feature enabled', js: true do
it 'shows projects only with merge requests feature enabled', :js do
find('.new-project-item-link').click
page.within('.select2-results') do
......@@ -70,5 +72,4 @@ feature 'Group merge requests page' do
end
end
end
end
......@@ -242,7 +242,7 @@ describe('Api', () => {
},
]);
Api.groupProjects(groupId, query, response => {
Api.groupProjects(groupId, query, {}, response => {
expect(response.length).toBe(1);
expect(response[0].name).toBe('test');
done();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment