Actions
Bug #11131
closedIntermittent Puppet class cache test with timestamps
Description
I noticed this in the test output from #10941 just after I pushed, but it looks like we have some tests that use Time.now in the expectation and then Time.now (or new) in the tested code, so it's possible for it to run over the second boundary. This results in the following, where the timestamps are slightly different:
unexpected invocation: Proxy::Puppet::PuppetCache.write_to_cache({'./test/fixtures/modules_include' => {'testinclude' => {'timestamp' => 1436970178, 'manifest' => [[#<Proxy::Puppet::PuppetClass:0x4aac8b0>], [#<Proxy::Puppet::PuppetClass:0x4aac5b8>]]}}}, 'example_env') unsatisfied expectations: - expected exactly once, not yet invoked: Proxy::Puppet::PuppetCache.write_to_cache({'./test/fixtures/modules_include' => {'testinclude' => {'timestamp' => 1436970177, 'manifest' => [[#<Proxy::Puppet::PuppetClass:0x4abcb48>], [#<Proxy::Puppet::PuppetClass:0x4abca58>]]}}}, 'example_env')
Updated by Dominic Cleal over 9 years ago
- Related to Bug #10941: puppet classes cache not working right added
Updated by The Foreman Bot over 9 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/smart-proxy/pull/302 added
- Pull request deleted (
)
Updated by Dominic Cleal over 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset f11f708911618a9692e6cee91fff34921fab1ffe.
Updated by Dominic Cleal over 9 years ago
- Translation missing: en.field_release set to 62
Updated by Dominic Cleal over 9 years ago
- Related to Bug #11229: Import classes does not find any new classes added
Updated by Dominic Cleal over 9 years ago
- Translation missing: en.field_release changed from 62 to 35
Bumping to 1.9 due to regression found.
Updated by Dominic Cleal over 9 years ago
- Related to deleted (Bug #11229: Import classes does not find any new classes)
Actions