diff options
author | Jeremiah Snapp <jeremiah@chef.io> | 2021-09-07 16:20:17 -0400 |
---|---|---|
committer | Jeremiah Snapp <jeremiah@chef.io> | 2021-09-07 16:20:17 -0400 |
commit | 551c9463359579128778b427607e9cc79cbfeda8 (patch) | |
tree | 46897c990a74f432a72a6f8b34e0deac34220fcb | |
parent | 636a3d47c9dbed16165d339a1e80716fabfba21d (diff) | |
download | chef-551c9463359579128778b427607e9cc79cbfeda8.tar.gz |
Add a delay in a few tests for AIX 7.2
For some reason our temporary AIX 7.2 system does not correctly report
group membership immediately after changes have been made.
Adding a 2 second delay for this platform is enough
to get correct results.
We hope to remove this delay after we get more
permanent AIX 7.2 systems in our CI pipeline.
reference: reference: https://github.com/chef/release-engineering/issues/1617
Signed-off-by: Jeremiah Snapp <jeremiah@chef.io>
-rw-r--r-- | spec/functional/resource/group_spec.rb | 4 | ||||
-rw-r--r-- | spec/functional/resource/link_spec.rb | 8 |
2 files changed, 12 insertions, 0 deletions
diff --git a/spec/functional/resource/group_spec.rb b/spec/functional/resource/group_spec.rb index eb5ff59d00..87953455b9 100644 --- a/spec/functional/resource/group_spec.rb +++ b/spec/functional/resource/group_spec.rb @@ -44,6 +44,10 @@ describe Chef::Resource::Group, :requires_root_or_running_windows do members.shift # Get rid of GroupMembership: string members.include?(user) else + # TODO For some reason our temporary AIX 7.2 system does not correctly report group membership immediately after changes have been made. + # Adding a 2 second delay for this platform is enough to get correct results. + # We hope to remove this delay after we get more permanent AIX 7.2 systems in our CI pipeline. reference: https://github.com/chef/release-engineering/issues/1617 + sleep 2 if aix? && (ohai[:platform_version] == "7.2") Etc.getgrnam(group_name).mem.include?(user) end end diff --git a/spec/functional/resource/link_spec.rb b/spec/functional/resource/link_spec.rb index 734897aaa4..4637896fd7 100644 --- a/spec/functional/resource/link_spec.rb +++ b/spec/functional/resource/link_spec.rb @@ -345,9 +345,17 @@ describe Chef::Resource::Link do let(:test_user) { "test-link-user" } before do user(test_user).run_action(:create) + # TODO For some reason our temporary AIX 7.2 system does not correctly report user existence immediately after changes have been made. + # Adding a 2 second delay for this platform is enough to get correct results. + # We hope to remove this delay after we get more permanent AIX 7.2 systems in our CI pipeline. reference: https://github.com/chef/release-engineering/issues/1617 + sleep 2 if aix? && (ohai[:platform_version] == "7.2") end after do user(test_user).run_action(:remove) + # TODO For some reason our temporary AIX 7.2 system does not correctly report user existence immediately after changes have been made. + # Adding a 2 second delay for this platform is enough to get correct results. + # We hope to remove this delay after we get more permanent AIX 7.2 systems in our CI pipeline. reference: https://github.com/chef/release-engineering/issues/1617 + sleep 2 if aix? && (ohai[:platform_version] == "7.2") end before(:each) do resource.owner(test_user) |