215f09541a
For the includers of HyperTagMethods, this changes the authorization code and object_tag member validation code to look at the project_id column for the object, instead of looking a row with the project and object in the access_tag table. This removes all calls to associate_with_project, other than those for Account. It removes the projects association for the includers of HyperTagMethods, and adds a project association to the models that didn't already have one, since there is only a single project for each object now. Most HyperTagMethods code is inlined into Account, since it is only user of the code now. Temporarily, other models will still include HyperTagMethods for the before_destroy hook, but eventually it will go away completely. The associations in Projects that previous used access_tag as a join table, are changed from many_to_many to one_to_many, except for Account (which still uses the join table). Project#has_resources now needs separate queries for all of the resource classes to see if there any associated objects. This causes a lot of fallout in the specs, but unfortunately that is unavoidable due the extensive use of projects.first in the specs to get the related project for the objects, as well as the extensive use of associate_with_project.
278 lines
10 KiB
Ruby
278 lines
10 KiB
Ruby
# frozen_string_literal: true
|
||
|
||
require_relative "../../spec_helper"
|
||
|
||
RSpec.describe Clover, "load-balancer" do
|
||
let(:user) { create_account }
|
||
|
||
let(:project) { project_with_default_policy(user) }
|
||
|
||
let(:lb) do
|
||
ps = Prog::Vnet::SubnetNexus.assemble(project.id, name: "subnet-1", location: LocationNameConverter.to_internal_name(TEST_LOCATION))
|
||
dz = DnsZone.create_with_id(name: "test-dns-zone", project_id: project.id)
|
||
cert = Prog::Vnet::CertNexus.assemble("test-host-name", dz.id).subject
|
||
lb = Prog::Vnet::LoadBalancerNexus.assemble(ps.id, name: "lb-1", src_port: 80, dst_port: 80).subject
|
||
lb.add_cert(cert)
|
||
lb
|
||
end
|
||
|
||
describe "unauthenticated" do
|
||
it "cannot perform authenticated operations" do
|
||
[
|
||
[:get, "/project/#{project.ubid}/load-balancer"],
|
||
[:post, "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/lb-1"],
|
||
[:delete, "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}"],
|
||
[:get, "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}"],
|
||
[:post, "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}/attach-vm", {vm_id: "vm-1"}],
|
||
[:post, "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}/detach-vm", {vm_id: "vm-1"}],
|
||
[:get, "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/_#{lb.ubid}"]
|
||
].each do |method, path, body|
|
||
send(method, path, body)
|
||
|
||
expect(last_response).to have_api_error(401, "Please login to continue")
|
||
end
|
||
end
|
||
end
|
||
|
||
describe "authenticated" do
|
||
before do
|
||
login_api(user.email)
|
||
lb_project = Project.create_with_id(name: "default")
|
||
allow(Config).to receive(:load_balancer_service_project_id).and_return(lb_project.id)
|
||
end
|
||
|
||
describe "list" do
|
||
it "empty" do
|
||
get "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer"
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(JSON.parse(last_response.body)["items"]).to eq([])
|
||
end
|
||
|
||
it "success single" do
|
||
lb
|
||
|
||
get "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer"
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(JSON.parse(last_response.body)["items"].length).to eq(1)
|
||
end
|
||
|
||
it "success multiple" do
|
||
lb
|
||
Prog::Vnet::LoadBalancerNexus.assemble(lb.private_subnet.id, name: "lb-2", src_port: 80, dst_port: 80).subject
|
||
|
||
get "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer"
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(JSON.parse(last_response.body)["items"].length).to eq(2)
|
||
end
|
||
end
|
||
|
||
describe "id" do
|
||
it "success" do
|
||
get "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/_#{lb.ubid}"
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(JSON.parse(last_response.body)["name"]).to eq("lb-1")
|
||
end
|
||
|
||
it "not found" do
|
||
get "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/_invalid"
|
||
|
||
expect(last_response).to have_api_error(404, "Sorry, we couldn’t find the resource you’re looking for.")
|
||
end
|
||
end
|
||
|
||
describe "create" do
|
||
it "success" do
|
||
ps = Prog::Vnet::SubnetNexus.assemble(project.id, name: "subnet-1", location: "hetzner-fsn1").subject
|
||
post "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/lb1", {
|
||
private_subnet_id: ps.ubid,
|
||
stack: LoadBalancer::Stack::IPV4,
|
||
src_port: "80", dst_port: "80",
|
||
health_check_endpoint: "/up", algorithm: "round_robin",
|
||
health_check_protocol: "http"
|
||
}.to_json
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(JSON.parse(last_response.body)["name"]).to eq("lb1")
|
||
end
|
||
|
||
it "invalid private_subnet_id" do
|
||
post "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/lb1", {
|
||
private_subnet_id: "invalid",
|
||
stack: LoadBalancer::Stack::IPV6,
|
||
src_port: "80", dst_port: "80",
|
||
health_check_endpoint: "/up", algorithm: "round_robin",
|
||
health_check_protocol: "http"
|
||
}.to_json
|
||
|
||
expect(last_response).to have_api_error(400, "Validation failed for following fields: private_subnet_id")
|
||
end
|
||
end
|
||
|
||
describe "delete" do
|
||
it "success" do
|
||
delete "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}"
|
||
|
||
expect(last_response.status).to eq(204)
|
||
end
|
||
|
||
it "not found" do
|
||
delete "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/invalid_name"
|
||
|
||
expect(last_response.status).to eq(204)
|
||
end
|
||
end
|
||
|
||
describe "get" do
|
||
it "success" do
|
||
get "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}"
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(JSON.parse(last_response.body)["name"]).to eq("lb-1")
|
||
end
|
||
|
||
it "not found" do
|
||
get "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/invalid"
|
||
|
||
expect(last_response).to have_api_error(404, "Sorry, we couldn’t find the resource you’re looking for.")
|
||
end
|
||
end
|
||
|
||
describe "update" do
|
||
let(:vm) {
|
||
nic = Nic.create_with_id(name: "nic-1", private_subnet_id: lb.private_subnet.id, mac: "00:00:00:00:00:01", private_ipv4: "1.1.1.1", private_ipv6: "2001:db8::1")
|
||
vm = create_vm
|
||
nic.update(vm_id: vm.id)
|
||
vm.update(project_id: project.id)
|
||
vm
|
||
}
|
||
|
||
it "success" do
|
||
patch "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}", {
|
||
src_port: "80", dst_port: "80",
|
||
health_check_endpoint: "/up", algorithm: "round_robin", vms: []
|
||
}.to_json
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(JSON.parse(last_response.body)["name"]).to eq("lb-1")
|
||
end
|
||
|
||
it "not found" do
|
||
patch "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/invalid", {
|
||
src_port: "80", dst_port: "80",
|
||
health_check_endpoint: "/up", algorithm: "round_robin", vms: []
|
||
}.to_json
|
||
|
||
expect(last_response).to have_api_error(404, "Sorry, we couldn’t find the resource you’re looking for.")
|
||
end
|
||
|
||
it "missing required parameters" do
|
||
patch "/project/#{project.ubid}/location/#{lb.private_subnet.display_location}/load-balancer/#{lb.name}", {}.to_json
|
||
|
||
expect(last_response).to have_api_error(400, "Validation failed for following fields: body")
|
||
end
|
||
|
||
it "updates vms" do
|
||
patch "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}", {
|
||
src_port: "80", dst_port: "80", health_check_endpoint: "/up", algorithm: "round_robin", vms: [vm.ubid]
|
||
}.to_json
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(JSON.parse(last_response.body)["vms"].length).to eq(1)
|
||
end
|
||
|
||
it "detaches vms" do
|
||
lb.add_vm(vm)
|
||
|
||
patch "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}", {
|
||
src_port: "80", dst_port: "80", health_check_endpoint: "/up", algorithm: "round_robin", vms: []
|
||
}.to_json
|
||
|
||
expect(last_response.status).to eq(200)
|
||
expect(lb.reload.vms.count).to eq(0)
|
||
expect(JSON.parse(last_response.body)["vms"]).to eq([])
|
||
end
|
||
|
||
it "invalid vm" do
|
||
patch "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}", {
|
||
src_port: "80", dst_port: "80", health_check_endpoint: "/up", algorithm: "round_robin", vms: ["invalid"]
|
||
}.to_json
|
||
|
||
expect(last_response).to have_api_error(400, "Validation failed for following fields: vms")
|
||
end
|
||
|
||
it "vm already attached to a different load balancer" do
|
||
lb2 = Prog::Vnet::LoadBalancerNexus.assemble(lb.private_subnet.id, name: "lb-2", src_port: 80, dst_port: 80).subject
|
||
dz = DnsZone.create_with_id(name: "test-dns-zone2", project_id: lb2.private_subnet.project_id)
|
||
cert = Prog::Vnet::CertNexus.assemble("test-host-name", dz.id).subject
|
||
lb2.add_cert(cert)
|
||
lb2.add_vm(vm)
|
||
|
||
patch "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}", {
|
||
src_port: "80", dst_port: "80", health_check_endpoint: "/up", algorithm: "round_robin", vms: [vm.ubid]
|
||
}.to_json
|
||
|
||
expect(last_response).to have_api_error(400)
|
||
end
|
||
|
||
it "vm already attached to the same load balancer" do
|
||
lb.add_vm(vm)
|
||
|
||
patch "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}", {
|
||
src_port: "80", dst_port: "80", health_check_endpoint: "/up", algorithm: "round_robin", vms: [vm.ubid]
|
||
}.to_json
|
||
|
||
expect(last_response.status).to eq(200)
|
||
end
|
||
end
|
||
|
||
describe "attach-vm" do
|
||
let(:vm) {
|
||
nic = Nic.create_with_id(name: "nic-1", private_subnet_id: lb.private_subnet.id, mac: "00:00:00:00:00:01", private_ipv4: "1.1.1.1", private_ipv6: "2001:db8::1")
|
||
vm = create_vm
|
||
nic.update(vm_id: vm.id)
|
||
vm
|
||
}
|
||
|
||
it "success" do
|
||
vm.update(project_id: project.id)
|
||
post "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}/attach-vm", {vm_id: vm.ubid}.to_json
|
||
|
||
expect(last_response.status).to eq(200)
|
||
end
|
||
|
||
it "not existing vm" do
|
||
post "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}/attach-vm", {vm_id: "invalid"}.to_json
|
||
|
||
expect(last_response).to have_api_error(400, "Validation failed for following fields: vm_id")
|
||
end
|
||
end
|
||
|
||
describe "detach-vm" do
|
||
let(:vm) {
|
||
nic = Nic.create_with_id(name: "nic-1", private_subnet_id: lb.private_subnet.id, mac: "00:00:00:00:00:01", private_ipv4: "1.1.1.1", private_ipv6: "2001:db8::1")
|
||
vm = create_vm
|
||
nic.update(vm_id: vm.id)
|
||
vm
|
||
}
|
||
|
||
it "success" do
|
||
vm.update(project_id: project.id)
|
||
lb.add_vm(vm)
|
||
|
||
post "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}/detach-vm", {vm_id: vm.ubid}.to_json
|
||
|
||
expect(last_response.status).to eq(200)
|
||
end
|
||
|
||
it "not existing vm" do
|
||
post "/project/#{project.ubid}/location/#{TEST_LOCATION}/load-balancer/#{lb.name}/detach-vm", {vm_id: "invalid"}.to_json
|
||
|
||
expect(last_response).to have_api_error(400, "Validation failed for following fields: vm_id")
|
||
end
|
||
end
|
||
end
|
||
end
|