Project

General

Profile

Bug #10684

ComputeOrchestrationTest::only physical interfaces are matched test failures

Added by Dominic Cleal about 8 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Urgent
Assignee:
Category:
Tests
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Since #10664 was merged, test_develop is failing with:

  1) Failure:
test_0001_matching fog attributes only for physical interfaces(ComputeOrchestrationTest::only physical interfaces are matched) [/var/lib/workspace/workspace/test_develop/database/postgresql/ruby/1.9.3/app/models/concerns/orchestration/compute.rb:295]:
unexpected invocation: #<Mock:vm>.select_nic([], #<Nic::Base:0xd10f4c0>)
unsatisfied expectations:
- expected exactly once, invoked twice: #<Mock:vm>.select_nic(any_parameters)
satisfied expectations:
- allowed any number of times, not yet invoked: #<Foreman::Model::Libvirt:0xccf8d50>.provided_attributes(any_parameters)
- expected never, not yet invoked: #<Mock:vm>.select_nic([], #<Nic::Base:0xd1ec820>)
- allowed any number of times, invoked once: #<Mock:vm>.interfaces(any_parameters)
- allowed any number of times, invoked once: #<Host::Managed:0xd162f08>.validate_foreman_attr(any_parameters)

  2) Failure:
test_0002_adding only physical interfaces(ComputeOrchestrationTest::only physical interfaces are matched) [/var/lib/workspace/workspace/test_develop/database/postgresql/ruby/1.9.3/test/unit/orchestration/compute_test.rb:40]:
<1> expected but was
<2>.

http://ci.theforeman.org/job/test_develop/1328/


Related issues

Related to Foreman - Bug #10664: Virtual interfaces does not work with compute resourcesClosed2015-06-01

Associated revisions

Revision ee55ec21 (diff)
Added by Marek Hulán about 8 years ago

Fixes #10684 - re-adding missing code

History

#1 Updated by Dominic Cleal about 8 years ago

  • Related to Bug #10664: Virtual interfaces does not work with compute resources added

#2 Updated by The Foreman Bot about 8 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2424 added
  • Pull request deleted ()

#3 Updated by Marek Hulán about 8 years ago

  • Assignee set to Marek Hulán

#4 Updated by Dominic Cleal about 8 years ago

  • Legacy Backlogs Release (now unused) set to 35

#5 Updated by Marek Hulán about 8 years ago

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

Also available in: Atom PDF