diff --git a/lib/ohai/plugins/darwin/cpu.rb b/lib/ohai/plugins/darwin/cpu.rb index ec8f095b1..9ff64ba50 100644 --- a/lib/ohai/plugins/darwin/cpu.rb +++ b/lib/ohai/plugins/darwin/cpu.rb @@ -1,7 +1,7 @@ # # Author:: Nathan L Smith () -# Author:: Tim Smith () -# Copyright:: Copyright (c) 2013 Opscode, Inc. +# Author:: Tim Smith () +# Copyright:: Copyright (c) 2013-2015 Chef Software, Inc. # License:: Apache License, Version 2.0 # # Licensed under the Apache License, Version 2.0 (the "License"); @@ -22,8 +22,10 @@ collect_data(:darwin) do cpu Mash.new - so = shell_out("sysctl -n hw.physicalcpu") + so = shell_out("sysctl -n hw.packages") cpu[:real] = so.stdout.to_i + so = shell_out("sysctl -n hw.physicalcpu") + cpu[:cores] = so.stdout.to_i so = shell_out("sysctl -n hw.logicalcpu") cpu[:total] = so.stdout.to_i so = shell_out("sysctl -n hw.cpufrequency") diff --git a/spec/unit/plugins/darwin/cpu_spec.rb b/spec/unit/plugins/darwin/cpu_spec.rb index 5ddc6e3a7..fab98b277 100644 --- a/spec/unit/plugins/darwin/cpu_spec.rb +++ b/spec/unit/plugins/darwin/cpu_spec.rb @@ -1,6 +1,6 @@ # # Author:: Nathan L Smith () -# Copyright:: Copyright (c) 2013 Opscode, Inc. +# Copyright:: Copyright (c) 2013-2015 Chef Software, Inc. # License:: Apache License, Version 2.0 # # Licensed under the Apache License, Version 2.0 (the "License"); @@ -23,6 +23,7 @@ before(:each) do @plugin = get_plugin("darwin/cpu") allow(@plugin).to receive(:collect_os).and_return(:darwin) + allow(@plugin).to receive(:shell_out).with("sysctl -n hw.packages").and_return(mock_shell_out(0, "1", "")) allow(@plugin).to receive(:shell_out).with("sysctl -n hw.physicalcpu").and_return(mock_shell_out(0, "4", "")) allow(@plugin).to receive(:shell_out).with("sysctl -n hw.logicalcpu").and_return(mock_shell_out(0, "8", "")) allow(@plugin).to receive(:shell_out).with("sysctl -n hw.cpufrequency").and_return(mock_shell_out(0, "2300000000", "")) @@ -35,12 +36,16 @@ @plugin.run end + it "should set cpu[:cores] to 4" do + expect(@plugin[:cpu][:cores]).to eq(4) + end + it "should set cpu[:total] to 8" do expect(@plugin[:cpu][:total]).to eq(8) end - it "should set cpu[:real] to 4" do - expect(@plugin[:cpu][:real]).to eq(4) + it "should set cpu[:real] to 1" do + expect(@plugin[:cpu][:real]).to eq(1) end it "should set cpu[:mhz] to 2300" do