Commit f6057437 authored by Matija Čupić's avatar Matija Čupić

Rename spec local variable to avoid shadowing

parent 42d87f90
...@@ -16,10 +16,7 @@ describe Projects::ClustersController do ...@@ -16,10 +16,7 @@ describe Projects::ClustersController do
context 'when project has one or more clusters' do context 'when project has one or more clusters' do
let(:project) { create(:project) } let(:project) { create(:project) }
let(:clusters) { create_list(:cluster, 2, :provided_by_gcp, projects: [project]) } let(:clusters) { create_list(:cluster, 2, :provided_by_gcp, projects: [project]) }
let(:inactive_cluster) { create(:cluster, :provided_by_gcp, projects: [project]) }
before do
clusters.last.enabled = false
end
it 'lists available clusters' do it 'lists available clusters' do
go go
...@@ -56,16 +53,16 @@ describe Projects::ClustersController do ...@@ -56,16 +53,16 @@ describe Projects::ClustersController do
context 'when only enabled clusters are requested' do context 'when only enabled clusters are requested' do
it 'returns only enabled clusters' do it 'returns only enabled clusters' do
get :index, namespace_id: project.namespace, project_id: project, scope: 'active' get :index, namespace_id: project.namespace, project_id: project, scope: 'active'
clusters = assigns(:clusters) view_clusters = assigns(:clusters)
expect(clusters.all? { |cluster| cluster.enabled == true }).to eq(true) expect(view_clusters.all? { |cluster| cluster.enabled == true }).to eq(true)
end end
end end
context 'when only disabled clusters are requested' do context 'when only disabled clusters are requested' do
it 'returns only disabled clusters' do it 'returns only disabled clusters' do
get :index, namespace_id: project.namespace, project_id: project, scope: 'inactive' get :index, namespace_id: project.namespace, project_id: project, scope: 'inactive'
clusters = assigns(:clusters) view_clusters = assigns(:clusters)
expect(clusters.all? { |cluster| cluster.enabled == false }).to eq(true) expect(view_clusters.all? { |cluster| cluster.enabled == false }).to eq(true)
end end
end end
end end
......
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