Fix flaky endpoint discovery cache test (#3093)

alextwoods · web-flow · commit 6610f5feddfd · 2024-08-26T13:25:53.000-07:00
diff --git a/gems/aws-sdk-core/spec/aws/plugins/endpoint_discovery_spec.rb b/gems/aws-sdk-core/spec/aws/plugins/endpoint_discovery_spec.rb
@@ -275,8 +275,12 @@ module Plugins
             ]
           }
         )
+        cache_thread_block = nil
+        allow(Thread).to receive(:new) do |&block|
+          cache_thread_block = block
+        end
         c.operation_not_required(foo: 'foo')
-        sleep(0.1)
+        cache_thread_block.call
         expect(c.api_requests.size).to eq(3)
         c.operation_not_required(foo: 'foo')
         expect(c.api_requests.size).to eq(4)

Original file line number	Diff line number	Diff line change
`@@ -275,8 +275,12 @@ module Plugins`
`275`	`275`	`]`
`276`	`276`	`}`
`277`	`277`	`)`
	`278`	`+ cache_thread_block = nil`
	`279`	`+ allow(Thread).to receive(:new) do \|&block\|`
	`280`	`+ cache_thread_block = block`
	`281`	`+ end`
`278`	`282`	`c.operation_not_required(foo: 'foo')`
`279`		`- sleep(0.1)`
	`283`	`+ cache_thread_block.call`
`280`	`284`	`expect(c.api_requests.size).to eq(3)`
`281`	`285`	`c.operation_not_required(foo: 'foo')`
`282`	`286`	`expect(c.api_requests.size).to eq(4)`