Skip to content

Commit

Permalink
Fix specs
Browse files Browse the repository at this point in the history
  • Loading branch information
Jberlinsky committed Feb 12, 2019
1 parent d86affb commit 3746115
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 32 deletions.
8 changes: 2 additions & 6 deletions test/integration/simple_regional/controls/gcloud.rb
Original file line number Diff line number Diff line change
Expand Up @@ -42,12 +42,8 @@
expect(data['location']).to match(/^.*[1-9]$/)
end

it "uses the public master endpoint" do
expect(data['privateClusterConfig']['enablePrivateEndpoint']).to eq true
end

it "uses public nodes" do
expect(data['privateClusterConfig']['enablePrivateNodes']).to eq true
it "uses public nodes and master endpoint" do
expect(data['privateClusterConfig']).to eq nil
end

it "has the expected addon settings" do
Expand Down
10 changes: 0 additions & 10 deletions test/integration/simple_regional_private/controls/gcloud.rb
Original file line number Diff line number Diff line change
Expand Up @@ -162,16 +162,6 @@
)
)
end

it "has autoupgrade enabled" do
expect(node_pools).to include(
including(
"management" => including(
"autoUpgrade" => true,
),
)
)
end
end
end
end
8 changes: 2 additions & 6 deletions test/integration/simple_zonal/controls/gcloud.rb
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,8 @@
expect(data['locations'].size).to eq 1
end

it "uses the public master endpoint" do
expect(data['privateClusterConfig']['enablePrivateEndpoint']).to eq true
end

it "uses public nodes" do
expect(data['privateClusterConfig']['enablePrivateNodes']).to eq true
it "uses public nodes and master endpoint" do
expect(data['privateClusterConfig']).to eq nil
end

it "has the expected addon settings" do
Expand Down
10 changes: 0 additions & 10 deletions test/integration/simple_zonal_private/controls/gcloud.rb
Original file line number Diff line number Diff line change
Expand Up @@ -166,16 +166,6 @@
)
)
end

it "has autoupgrade enabled" do
expect(node_pools).to include(
including(
"management" => including(
"autoUpgrade" => true,
),
)
)
end
end
end
end

0 comments on commit 3746115

Please sign in to comment.