Remove xit test for uninstall group cluster app
This commit is contained in:
parent
abb530a619
commit
eae0fc2bcd
|
@ -144,64 +144,4 @@ describe Groups::Clusters::ApplicationsController do
|
||||||
it_behaves_like 'a secure endpoint'
|
it_behaves_like 'a secure endpoint'
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe 'DELETE destroy' do
|
|
||||||
subject do
|
|
||||||
delete :destroy, params: params.merge(group_id: group)
|
|
||||||
end
|
|
||||||
|
|
||||||
let!(:application) { create(:clusters_applications_cert_managers, :installed, cluster: cluster) }
|
|
||||||
let(:application_name) { application.name }
|
|
||||||
let(:params) { { application: application_name, id: cluster.id } }
|
|
||||||
let(:worker_class) { Clusters::Applications::UninstallWorker }
|
|
||||||
|
|
||||||
describe 'functionality' do
|
|
||||||
let(:user) { create(:user) }
|
|
||||||
|
|
||||||
before do
|
|
||||||
group.add_maintainer(user)
|
|
||||||
sign_in(user)
|
|
||||||
end
|
|
||||||
|
|
||||||
context "when cluster and app exists" do
|
|
||||||
xit "schedules an application update" do
|
|
||||||
expect(worker_class).to receive(:perform_async).with(application.name, application.id).once
|
|
||||||
|
|
||||||
is_expected.to have_http_status(:no_content)
|
|
||||||
|
|
||||||
expect(cluster.application_cert_manager).to be_scheduled
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
context 'when cluster do not exists' do
|
|
||||||
before do
|
|
||||||
cluster.destroy!
|
|
||||||
end
|
|
||||||
|
|
||||||
it { is_expected.to have_http_status(:not_found) }
|
|
||||||
end
|
|
||||||
|
|
||||||
context 'when application is unknown' do
|
|
||||||
let(:application_name) { 'unkwnown-app' }
|
|
||||||
|
|
||||||
it { is_expected.to have_http_status(:not_found) }
|
|
||||||
end
|
|
||||||
|
|
||||||
context 'when application is already scheduled' do
|
|
||||||
before do
|
|
||||||
application.make_scheduled!
|
|
||||||
end
|
|
||||||
|
|
||||||
xit { is_expected.to have_http_status(:bad_request) }
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
describe 'security' do
|
|
||||||
before do
|
|
||||||
allow(worker_class).to receive(:perform_async)
|
|
||||||
end
|
|
||||||
|
|
||||||
it_behaves_like 'a secure endpoint'
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in New Issue