Project

General

Profile

Actions

Bug #11131

closed

Intermittent Puppet class cache test with timestamps

Added by Dominic Cleal over 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Tests
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

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')

Related issues 1 (0 open1 closed)

Related to Smart Proxy - Bug #10941: puppet classes cache not working rightClosed06/26/2015Actions
Actions #1

Updated by Dominic Cleal over 9 years ago

  • Related to Bug #10941: puppet classes cache not working right added
Actions #2

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 ()
Actions #3

Updated by Dominic Cleal over 9 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #4

Updated by Dominic Cleal over 9 years ago

  • Translation missing: en.field_release set to 62
Actions #5

Updated by Dominic Cleal over 9 years ago

  • Related to Bug #11229: Import classes does not find any new classes added
Actions #6

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.

Actions #7

Updated by Dominic Cleal over 9 years ago

  • Related to deleted (Bug #11229: Import classes does not find any new classes)
Actions

Also available in: Atom PDF