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.
72 lines
2.7 KiB
Ruby
72 lines
2.7 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require_relative "../spec_helper"
|
|
|
|
RSpec.describe DnsZone do
|
|
subject(:dns_zone) {
|
|
described_class.create_with_id(
|
|
project_id: "00000000-0000-0000-0000-000000000000",
|
|
name: "example.com"
|
|
)
|
|
}
|
|
|
|
let(:sshable) { instance_double(Sshable) }
|
|
let(:vm) { instance_double(Vm, sshable: sshable) }
|
|
let(:dns_server) { instance_double(DnsServer, id: "00000000-0000-0000-0000-000000000000", vms: [vm]) }
|
|
|
|
before do
|
|
allow(dns_zone).to receive(:dns_servers).and_return([dns_server])
|
|
allow(dns_zone).to receive(:incr_refresh_dns_servers)
|
|
end
|
|
|
|
context "when inserting new record" do
|
|
it "creates record in database" do
|
|
dns_zone.insert_record(record_name: "test", type: "A", ttl: 10, data: "1.2.3.4")
|
|
|
|
expect(dns_zone.records.count).to eq(1)
|
|
end
|
|
end
|
|
|
|
context "when deleting a record" do
|
|
before do
|
|
DnsRecord.create_with_id(dns_zone_id: dns_zone.id, name: "test1.", type: "A", ttl: 10, data: "1.2.3.4")
|
|
DnsRecord.create_with_id(dns_zone_id: dns_zone.id, name: "test1.", type: "A", ttl: 10, data: "5.6.7.8")
|
|
DnsRecord.create_with_id(dns_zone_id: dns_zone.id, name: "test1.", type: "TXT", ttl: 10, data: "5.6.7.8")
|
|
DnsRecord.create_with_id(dns_zone_id: dns_zone.id, name: "test2.", type: "A", ttl: 10, data: "5.6.7.8")
|
|
end
|
|
|
|
it "raises error if data is passed but type is not" do
|
|
expect {
|
|
dns_zone.delete_record(record_name: "test1", data: "1.2.3.4")
|
|
}.to raise_error RuntimeError, "Type needs to be specified if data is specified!"
|
|
end
|
|
|
|
it "deletes all matching records from database when only record_name is passed" do
|
|
dns_zone.delete_record(record_name: "test1")
|
|
expect(dns_zone.records_dataset.where(:tombstoned).count).to eq(3)
|
|
end
|
|
|
|
it "does not insert new tombstoned records for existing tombstoned records" do
|
|
4.times do
|
|
dns_zone.delete_record(record_name: "test1")
|
|
end
|
|
expect(dns_zone.records_dataset.where(:tombstoned).count).to eq(12)
|
|
end
|
|
|
|
it "deletes all matching records from database when record_name and type are passed" do
|
|
dns_zone.delete_record(record_name: "test1", type: "A")
|
|
expect(dns_zone.records_dataset.where(:tombstoned).count).to eq(2)
|
|
end
|
|
|
|
it "deletes all matching records from database when record_name, type and data are passed" do
|
|
dns_zone.delete_record(record_name: "test1", type: "A", data: "1.2.3.4")
|
|
expect(dns_zone.records_dataset.where(:tombstoned).count).to eq(1)
|
|
end
|
|
end
|
|
|
|
it "returns record_name with dot" do
|
|
expect(dns_zone.add_dot_if_missing("pg-name.postgres.ubicloud.com")).to eq("pg-name.postgres.ubicloud.com.")
|
|
expect(dns_zone.add_dot_if_missing("pg-name.postgres.ubicloud.com.")).to eq("pg-name.postgres.ubicloud.com.")
|
|
end
|
|
end
|