Skip to content

Commit

Permalink
Remove test enforcing use of local UAA fork
Browse files Browse the repository at this point in the history
A bugfix for an invitation link expiry issue has been merged
([PR](cloudfoundry/uaa#1128)). Thus we switch
back to upstream and this test is no longer required.
  • Loading branch information
schmie committed Sep 9, 2021
1 parent ddc9dfc commit 19bde12
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions manifests/cf-manifest/spec/manifest/release_versions_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,6 @@ def normalise_version(version)
end

pinned_releases = {
"uaa" => {
local: "0.1.28",
upstream: "75.7.0",
},
}

manifest_releases = manifest_without_vars_store.fetch("releases").map { |release|
Expand All @@ -57,8 +53,6 @@ def normalise_version(version)
pinned_releases.each do |name, pinned_versions|
expect(manifest_releases).to have_key(name), "expected pinned release #{name} for found in manifest"
expect(cf_deployment_releases).to have_key(name), "expected pinned release #{name} for found in cf-deployment"
expect(normalise_version(manifest_releases[name])).to be(normalise_version(pinned_versions[:local])),
"expected #{name} to be using our own built tarball #{pinned_versions[:local]} not #{manifest_releases[name]}"

expect(normalise_version(cf_deployment_releases[name])).to be(normalise_version(pinned_versions[:upstream])),
"expected #{name} upstream to be #{pinned_versions[:upstream]} not #{cf_deployment_releases[name]}. We might need to rebase our forked #{name} release and generate a new tarball, or use the upstream version."
Expand Down

0 comments on commit 19bde12

Please sign in to comment.