poise-python 1.1.2 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 776c3bc6bc978dd6a8452e62839af86ea53c0b27
-  data.tar.gz: e1c625316e1f87a11e25dd96887285c9fd9f7728
+  metadata.gz: 6f954d22962ad5f19fbefef8ce992f18bc0f8b73
+  data.tar.gz: 91a35461eaecb05a6e3e5dee93e9b4d551d95403
 SHA512:
-  metadata.gz: 5bc9ed10105ec2fbad61011761331ec83932efdcd13e38e308749e467de44a72a86a3f487d55a1ced0f55bb61737b2a2c385c0ba9e203aecba78c776b5497b18
-  data.tar.gz: a8dd9182bfd9d610511831bb6a1c54ac90a30d79778d5a807a750dded125438d2bcadba5e5849dde8081345450eb751a2fefab8caecaa56ad8dc934338da200d
+  metadata.gz: b108843aca7b5c9f6c4db0f991c88c6179c8b91f406bcb44ae4df0b90ff02106080aa4e53610454f2ce4108c00238b973f119ce4f46cbea88a9eaef47be17c2a
+  data.tar.gz: 60273ff1bc570326fee7b2875210ff4f748dd0620332aeb3bbaa211a83d64ec514c63453f1c216f5444ef5f8f81817c1d286b8837c2f49e9151d3024facb6952

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Poise-Python Changelog
 
+## v1.2.0
+
+* Add support for passing `user` and `group` to `pip_requirements`.
+* Allow passing a virtualenv resource object to the `virtualenv` property.
+* Update PyPy release versions.
+* Make the `python_virtualenv` resource check for `./bin/python` for idempotence
+  instead of the base path.
+* Support for packages with extras in `python_package`.
+* Support for point releases (7.1, 8.1, etc) of Debian in the `system` provider.
+
 ## v1.1.2
 
 * Fix `PythonPackage#response_file_variables` for the Chef 12.6 initializer.

data/README.md

@@ -277,9 +277,11 @@ notifications will only be triggered if a package is actually installed.
 
 * `path` – Path to the requirements file, or a folder containing the
   requirements file. *(name property)*
+* `group` – System group to install the packages.
 * `python` – Name of the `python_runtime` resource to use. If not specified, the
   most recently declared `python_runtime` will be used. Can also be set to the
   full path to a `python` binary.
+* `user` – System user to install the packages.
 * `virtualenv` – Name of the `python_virtualenv` resource to use. This is
   mutually exclusive with the `python` property.
 

data/lib/poise_python/python_command_mixin.rb

@@ -34,7 +34,11 @@ module PoisePython
       # @param name [String] Name of the virtualenv resource.
       # @return [void]
       def virtualenv(name)
-        parent_python("python_virtualenv[#{name}]")
+        if name.is_a?(PoisePython::Resources::PythonVirtualenv::Resource)
+          parent_python(name)
+        else
+          parent_python("python_virtualenv[#{name}]")
+        end
       end
     end
 

data/lib/poise_python/python_providers.rb

@@ -31,6 +31,7 @@ module PoisePython
     autoload :Base, 'poise_python/python_providers/base'
 
     Chef::Platform::ProviderPriorityMap.instance.priority(:python_runtime, [
+      PoisePython::PythonProviders::Dummy,
       PoisePython::PythonProviders::PortablePyPy3,
       PoisePython::PythonProviders::PortablePyPy,
       PoisePython::PythonProviders::Scl,

data/lib/poise_python/python_providers/dummy.rb

@@ -27,9 +27,18 @@ module PoisePython
     class Dummy < Base
       provides(:dummy)
 
+      # Enable by default on ChefSpec.
+      #
+      # @api private
+      def self.provides_auto?(node, _resource)
+        node.platform?('chefspec')
+      end
+
+      # Manual overrides for dummy data.
+      #
+      # @api private
       def self.default_inversion_options(node, resource)
         super.merge({
-          # Manual overrides for dummy data.
           python_binary: ::File.join('', 'python'),
           python_environment: nil,
         })

data/lib/poise_python/python_providers/portable_pypy.rb

@@ -26,7 +26,7 @@ module PoisePython
       provides(:portable_pypy)
       include PoiseLanguages::Static(
         name: 'pypy',
-        versions: %w{2.6 2.5.1 2.5 2.4 2.3.1 2.3 2.2.1 2.2 2.1 2.0.2},
+        versions: %w{4.0.1 2.6.1 2.5.1 2.5 2.4 2.3.1 2.3 2.2.1 2.2 2.1 2.0.2},
         machines: %w{linux-i686 linux-x86_64},
         url: 'https://bitbucket.org/squeaky/portable-pypy/downloads/pypy-%{version}-%{kernel}_%{machine}-portable.tar.bz2'
       )

data/lib/poise_python/python_providers/system.rb

@@ -28,9 +28,9 @@ module PoisePython
       provides(:system)
       packages('python', {
         debian: {
-          '8' => %w{python3.4 python2.7},
-          '7' => %w{python3.2 python2.7 python2.6},
-          '6' => %w{python3.1 python2.6 python2.5},
+          '~> 8.0' => %w{python3.4 python2.7},
+          '~> 7.0' => %w{python3.2 python2.7 python2.6},
+          '~> 6.0' => %w{python3.1 python2.6 python2.5},
         },
         ubuntu: {
           '14.04' => %w{python3.4 python2.7},

data/lib/poise_python/resources/pip_requirements.rb

@@ -44,6 +44,14 @@ module PoisePython
         #   requirements file.
         #   @return [String]
         attribute(:path, kind_of: String, name_attribute: true)
+        # @!attribute group
+        #   System group to install the package.
+        #   @return [String, Integer, nil]
+        attribute(:group, kind_of: [String, Integer, NilClass])
+        # @!attribute user
+        #   System user to install the package.
+        #   @return [String, Integer, nil]
+        attribute(:user, kind_of: [String, Integer, NilClass])
       end
 
       # The default provider for `pip_requirements`.
@@ -80,7 +88,7 @@ module PoisePython
           cmd << '--upgrade' if upgrade
           cmd << '--requirement'
           cmd << requirements_path
-          output = python_shell_out!(cmd).stdout
+          output = python_shell_out!(cmd, user: new_resource.user, group: new_resource.group).stdout
           if output.include?('Successfully installed')
             new_resource.updated_by_last_action(true)
           end

data/lib/poise_python/resources/python_package.rb

@@ -91,7 +91,6 @@ EOH
           Poise::Helpers::ChefspecMatchers.create_matcher(:python_package, action)
         end
 
-
         # @!attribute group
         #   System group to install the package.
         #   @return [String, Integer, nil]
@@ -165,7 +164,7 @@ EOH
             version_data[name][:current] = current
           end
           # Check for newer candidates.
-          outdated = pip_outdated(pip_requirements(resource.package_name, version)).stdout
+          outdated = pip_outdated(pip_requirements(resource.package_name, version, parse: true)).stdout
           parse_pip_outdated(outdated).each do |name, candidate|
             # Merge candidates in to the existing versions.
             version_data[name][:candidate] = candidate
@@ -176,13 +175,13 @@ EOH
             @candidate_version = []
             versions = []
             [resource.package_name].flatten.each do |name|
-              ver = version_data[name.downcase]
+              ver = version_data[parse_package_name(name).downcase]
               versions << ver[:current]
               @candidate_version << ver[:candidate]
             end
             resource.version(versions)
           else
-            ver = version_data[resource.package_name.downcase]
+            ver = version_data[parse_package_name(resource.package_name).downcase]
             resource.version(ver[:current])
             @candidate_version = ver[:candidate]
           end
@@ -223,11 +222,16 @@ EOH
         # @param name [String, Array<String>] Name or names for the packages.
         # @param version [String, Array<String>] Version or versions for the
         #   packages.
+        # @param parse [Boolean] Use parsed package names.
         # @return [Array<String>]
-        def pip_requirements(name, version)
+        def pip_requirements(name, version, parse: false)
           [name].flatten.zip([version].flatten).map do |n, v|
+            n = parse_package_name(n) if parse
             v = v.to_s.strip
-            if v.empty?
+            if n =~ /:\/\//
+              # Probably a URI.
+              n
+            elsif v.empty?
               # No version requirement, send through unmodified.
               n
             elsif v =~ /^\d/
@@ -322,6 +326,25 @@ EOH
           end
         end
 
+        # Regexp for package URLs.
+        PACKAGE_NAME_URL = /:\/\/.*?#egg=(.*)$/
+
+        # Regexp for extras.
+        PACKAGE_NAME_EXTRA = /^(.*?)\[.*?\]$/
+
+        # Find the underlying name from a pip input sequence.
+        #
+        # @param raw_name [String] Raw package name.
+        # @return [String]
+        def parse_package_name(raw_name)
+          case raw_name
+          when PACKAGE_NAME_URL, PACKAGE_NAME_EXTRA
+            $1
+          else
+            raw_name
+          end
+        end
+
       end
     end
   end

data/lib/poise_python/resources/python_runtime_test.rb

@@ -40,7 +40,7 @@ module PoisePython
         attribute(:path, kind_of: String, default: lazy { default_path })
 
         def default_path
-          ::File.join('', 'root', "python_test_#{name}")
+          ::File.join('', 'opt', "python_test_#{name}")
         end
       end
 
@@ -58,7 +58,9 @@ module PoisePython
         def action_run
           notifying_block do
             # Top level directory for this test.
-            directory new_resource.path
+            directory new_resource.path do
+              mode '777'
+            end
 
             # Install and log the version.
             python_runtime new_resource.name do
@@ -128,6 +130,34 @@ EOH
             end
             test_import('requests')
             test_import('six')
+
+            # Create a non-root user and test installing with it.
+            test_user = "py#{new_resource.name}"
+            test_home = ::File.join('', 'home', test_user)
+            group test_user do
+              system true
+            end
+            user test_user do
+              comment "Test user for python_runtime_test #{new_resource.name}"
+              gid test_user
+              home test_home
+              shell '/bin/false'
+              system true
+            end
+            directory test_home do
+              mode '700'
+              group test_user
+              user test_user
+            end
+            test_venv = python_virtualenv ::File.join(test_home, 'env') do
+              python new_resource.name
+              user test_user
+            end
+            python_package 'attrs' do
+              user test_user
+              virtualenv test_venv
+            end
+            test_import('attrs', 'attr', python: nil, virtualenv: test_venv, user: test_user)
           end
         end
 
@@ -157,7 +187,7 @@ EOH
           end
         end
 
-        def test_import(name, path=name, python: new_resource.name, virtualenv: nil)
+        def test_import(name, path=name, python: new_resource.name, virtualenv: nil, user: nil)
           # Only queue up this resource once, the ivar is just for tracking.
           @python_import_test ||= file ::File.join(new_resource.path, 'import_version.py') do
             user 'root'
@@ -175,6 +205,7 @@ EOH
 
           python_execute "#{@python_import_test.path} #{name} #{::File.join(new_resource.path, "import_#{path}")}" do
             python python if python
+            user user if user
             virtualenv virtualenv if virtualenv
           end
         end

data/lib/poise_python/resources/python_virtualenv.rb

@@ -112,7 +112,7 @@ module PoisePython
         private
 
         def install_python
-          return if ::File.exist?(new_resource.path)
+          return if ::File.exist?(python_binary)
 
           cmd = python_shell_out(%w{-m venv -h})
           if cmd.error?

data/lib/poise_python/version.rb

@@ -16,5 +16,5 @@
 
 
 module PoisePython
-  VERSION = '1.1.2'
+  VERSION = '1.2.0'
 end

data/test/integration/default/serverspec/default_spec.rb

@@ -20,7 +20,7 @@ set :backend, :exec
 # Set up the shared example for python_runtime_test.
 RSpec.shared_examples 'a python_runtime_test' do |python_name, version=nil|
   let(:python_name) { python_name }
-  let(:python_path) { File.join('', 'root', "python_test_#{python_name}") }
+  let(:python_path) { File.join('', 'opt', "python_test_#{python_name}") }
   # Helper for all the file checks.
   def self.assert_file(rel_path, should_exist=true, &block)
     describe rel_path do

data/test/spec/python_providers/portable_pypy3_spec.rb

@@ -24,6 +24,7 @@ describe PoisePython::PythonProviders::PortablePyPy3 do
   step_into(:python_runtime)
   recipe do
     python_runtime 'test' do
+      provider_no_auto 'dummy'
       version node['poise_python_version']
       virtualenv_version false
     end

data/test/spec/python_providers/portable_pypy_spec.rb

@@ -24,6 +24,7 @@ describe PoisePython::PythonProviders::PortablePyPy do
   step_into(:python_runtime)
   recipe do
     python_runtime 'test' do
+      provider_no_auto 'dummy'
       version node['poise_python_version']
       virtualenv_version false
     end
@@ -37,7 +38,7 @@ describe PoisePython::PythonProviders::PortablePyPy do
 
   context 'with version pypy' do
     let(:python_version) { 'pypy' }
-    it_behaves_like 'portablepypy provider', 'pypy-2.6'
+    it_behaves_like 'portablepypy provider', 'pypy-4.0.1'
   end # /context with version pypy
 
   context 'with version pypy-2.4' do
@@ -53,6 +54,6 @@ describe PoisePython::PythonProviders::PortablePyPy do
       end
     end
 
-    it { is_expected.to uninstall_poise_languages_static('/opt/pypy-2.6') }
+    it { is_expected.to uninstall_poise_languages_static('/opt/pypy-4.0.1') }
   end # /context action :uninstall
 end

data/test/spec/python_providers/scl_spec.rb

@@ -24,6 +24,7 @@ describe PoisePython::PythonProviders::Scl do
   step_into(:python_runtime)
   recipe do
     python_runtime 'test' do
+      provider_no_auto 'dummy'
       version node['poise_python_version']
       virtualenv_version false
     end

data/test/spec/python_providers/system_spec.rb

@@ -25,14 +25,15 @@ describe PoisePython::PythonProviders::System do
   step_into(:python_runtime)
   recipe do
     python_runtime 'test' do
+      provider_no_auto 'dummy'
       version node['poise_python_version']
       virtualenv_version false
     end
   end
 
-  shared_examples_for 'system provider' do |candidates, pkg|
+  shared_examples_for 'system provider' do |candidates=nil, pkg|
     it { expect(python_runtime.provider_for_action(:install)).to be_a described_class }
-    it { expect(system_package_candidates).to eq candidates }
+    it { expect(system_package_candidates).to eq candidates } if candidates
     it { is_expected.to install_poise_languages_system(pkg) }
     it do
       expect_any_instance_of(described_class).to receive(:install_system_packages)
@@ -64,6 +65,23 @@ describe PoisePython::PythonProviders::System do
     it_behaves_like 'system provider', %w{python2.3 python23 python}, 'python2.3'
   end # /context with version 2.3
 
+  context 'on Debian 8.1' do
+    before { chefspec_options.update(platform: 'debian', version: '8.1') }
+    it_behaves_like 'system provider', 'python3.4'
+  end # /context on Debian 8.1
+
+  context 'on CentOS 7' do
+    before { chefspec_options.update(platform: 'centos', version: '7.0') }
+    recipe do
+      python_runtime 'test' do
+        provider :system
+        version node['poise_python_version']
+        virtualenv_version false
+      end
+    end
+    it_behaves_like 'system provider', 'python'
+  end # /context on Debian 8.1
+
   context 'action :uninstall' do
     recipe do
       python_runtime 'test' do

data/test/spec/resources/pip_requirements_spec.rb

@@ -17,19 +17,20 @@
 require 'spec_helper'
 
 describe PoisePython::Resources::PipRequirements do
-  let(:pip_cmd) { }
+  let(:pip_cmd) { %w{-m pip.__main__ install --requirement /test/requirements.txt} }
   let(:pip_output) { '' }
+  let(:pip_user) { nil }
+  let(:pip_group) { nil }
   step_into(:pip_requirements)
   before do
     allow(File).to receive(:directory?).and_return(false)
     allow(File).to receive(:directory?).with('/test').and_return(true)
   end
   before do
-    expect_any_instance_of(PoisePython::Resources::PipRequirements::Provider).to receive(:python_shell_out!).with(pip_cmd).and_return(double(stdout: pip_output))
+    expect_any_instance_of(PoisePython::Resources::PipRequirements::Provider).to receive(:python_shell_out!).with(pip_cmd, {user: pip_user, group: pip_group}).and_return(double(stdout: pip_output))
   end
 
   context 'with a directory' do
-    let(:pip_cmd) { %w{-m pip.__main__ install --requirement /test/requirements.txt} }
     recipe do
       pip_requirements '/test'
     end
@@ -46,6 +47,28 @@ describe PoisePython::Resources::PipRequirements do
     it { is_expected.to install_pip_requirements('/test/reqs.txt') }
   end # /context with a file
 
+  context 'with a user' do
+    let(:pip_user) { 'testuser' }
+    recipe do
+      pip_requirements '/test' do
+        user 'testuser'
+      end
+    end
+
+    it { is_expected.to install_pip_requirements('/test') }
+  end # /context with a user
+
+  context 'with a group' do
+    let(:pip_group) { 'testgroup' }
+    recipe do
+      pip_requirements '/test' do
+        group 'testgroup'
+      end
+    end
+
+    it { is_expected.to install_pip_requirements('/test') }
+  end # /context with a group
+
   context 'action :upgrade' do
     let(:pip_cmd) { %w{-m pip.__main__ install --upgrade --requirement /test/requirements.txt} }
     recipe do
@@ -58,7 +81,6 @@ describe PoisePython::Resources::PipRequirements do
   end # /context action :upgrade
 
   context 'with output' do
-    let(:pip_cmd) { %w{-m pip.__main__ install --requirement /test/requirements.txt} }
     let(:pip_output) { 'Successfully installed' }
     recipe do
       pip_requirements '/test'

data/test/spec/resources/python_package_spec.rb

@@ -21,7 +21,157 @@ describe PoisePython::Resources::PythonPackage do
   end # /describe PoisePython::Resources::PythonPackage::Resource
 
   describe PoisePython::Resources::PythonPackage::Provider do
-    let(:test_provider) { described_class.new(nil, nil) }
+    let(:test_resource) { nil }
+    let(:test_provider) { described_class.new(test_resource, chef_run.run_context) }
+    def stub_cmd(cmd, **options)
+      output_options = {error?: options.delete(:error?) || false, stdout: options.delete(:stdout) || '', stderr: options.delete(:stderr) || ''}
+      expect(test_provider).to receive(:python_shell_out!).with(cmd, options).and_return(double("python #{cmd} return", **output_options))
+    end
+
+    describe '#load_current_resource' do
+      let(:package_name) { nil }
+      let(:package_version) { nil }
+      let(:test_resource) { PoisePython::Resources::PythonPackage::Resource.new(package_name, chef_run.run_context).tap {|r| r.version(package_version) if package_version } }
+      let(:candidate_version) { subject; test_provider.candidate_version }
+      subject { test_provider.load_current_resource }
+
+      context 'with package_name foo' do
+        let(:package_name) { 'foo' }
+        before do
+          stub_cmd(%w{-m pip.__main__ list}, stdout: '')
+          stub_cmd(%w{- list --outdated foo}, input: kind_of(String), stdout: "foo (Current: 0 Latest: 1.0.0 [wheel])\n")
+        end
+
+        its(:version) { is_expected.to be nil }
+        it { expect(candidate_version).to eq '1.0.0' }
+      end # /context with package_name foo
+
+      context 'with package_name ["foo", "bar"]' do
+        let(:package_name) { %w{foo bar} }
+        before do
+          stub_cmd(%w{-m pip.__main__ list}, stdout: '')
+          stub_cmd(%w{- list --outdated foo bar}, input: kind_of(String), stdout: "foo (Current: 0 Latest: 1.0.0 [wheel])\nbar (Current: 0 Latest: 2.0.0 [wheel])\n")
+        end
+
+        its(:version) { is_expected.to eq [nil, nil] }
+        it { expect(candidate_version).to eq %w{1.0.0 2.0.0} }
+      end # /context with package_name ["foo", "bar"]
+
+      context 'with a package with extras' do
+        let(:package_name) { 'foo[bar]' }
+        before do
+          stub_cmd(%w{-m pip.__main__ list}, stdout: '')
+          stub_cmd(%w{- list --outdated foo}, input: kind_of(String), stdout: "foo (Current: 0 Latest: 1.0.0 [wheel])\n")
+        end
+
+        its(:version) { is_expected.to be nil }
+        it { expect(candidate_version).to eq '1.0.0' }
+      end # /context with a package with extras
+    end # /describe #load_current_resource
+
+    describe 'actions' do
+      let(:package_name) { nil }
+      let(:current_version) { nil }
+      let(:candidate_version) { nil }
+      let(:test_resource) { PoisePython::Resources::PythonPackage::Resource.new(package_name, chef_run.run_context) }
+      subject { test_provider.run_action }
+      before do
+        current_version = self.current_version
+        candidate_version = self.candidate_version
+        allow(test_provider).to receive(:load_current_resource) do
+          current_resource = double('current_resource', package_name: package_name, version: current_version)
+          test_provider.instance_eval do
+            @current_resource = current_resource
+            @candidate_version = candidate_version
+          end
+        end
+      end
+
+      describe 'action :install' do
+        before { test_provider.action = :install }
+
+        context 'with package_name foo' do
+          let(:package_name) { 'foo' }
+          let(:candidate_version) { '1.0.0' }
+          it do
+            stub_cmd(%w{-m pip.__main__ install foo==1.0.0})
+            subject
+          end
+        end # /context with package_name foo
+
+        context 'with package_name ["foo", "bar"]' do
+          let(:package_name) { %w{foo bar} }
+          let(:candidate_version) { %w{1.0.0 2.0.0} }
+          it do
+            stub_cmd(%w{-m pip.__main__ install foo==1.0.0 bar==2.0.0})
+            subject
+          end
+        end # /context with package_name ["foo", "bar"]
+
+        context 'with options' do
+          let(:package_name) { 'foo' }
+          let(:candidate_version) { '1.0.0' }
+          before { test_resource.options('--editable') }
+          it do
+            stub_cmd('-m pip.__main__ install --editable foo\\=\\=1.0.0')
+            subject
+          end
+        end # /context with options
+
+        context 'with a package with extras' do
+          let(:package_name) { 'foo[bar]' }
+          let(:candidate_version) { '1.0.0' }
+          it do
+            stub_cmd(%w{-m pip.__main__ install foo[bar]==1.0.0})
+            subject
+          end
+        end # /context with a package with extras
+      end # /describe action :install
+
+      describe 'action :upgrade' do
+        before { test_provider.action = :upgrade }
+
+        context 'with package_name foo' do
+          let(:package_name) { 'foo' }
+          let(:candidate_version) { '1.0.0' }
+          it do
+            stub_cmd(%w{-m pip.__main__ install --upgrade foo==1.0.0})
+            subject
+          end
+        end # /context with package_name foo
+
+        context 'with package_name ["foo", "bar"]' do
+          let(:package_name) { %w{foo bar} }
+          let(:candidate_version) { %w{1.0.0 2.0.0} }
+          it do
+            stub_cmd(%w{-m pip.__main__ install --upgrade foo==1.0.0 bar==2.0.0})
+            subject
+          end
+        end # /context with package_name ["foo", "bar"]
+      end # /describe action :upgrade
+
+      describe 'action :remove' do
+        before { test_provider.action = :remove }
+
+        context 'with package_name foo' do
+          let(:package_name) { 'foo' }
+          let(:current_version) { '1.0.0' }
+          it do
+            stub_cmd(%w{-m pip.__main__ uninstall --yes foo})
+            subject
+          end
+        end # /context with package_name foo
+
+        context 'with package_name ["foo", "bar"]' do
+          let(:package_name) { %w{foo bar} }
+          let(:current_version) { %w{1.0.0 2.0.0} }
+          it do
+            stub_cmd(%w{-m pip.__main__ uninstall --yes foo bar})
+            subject
+          end
+        end # /context with package_name ["foo", "bar"]
+      end # /describe action :remove
+    end # /describe actions
 
     describe '#parse_pip_outdated' do
       let(:text) { '' }
@@ -41,6 +191,17 @@ Fabric (Current: 1.9.1 Latest: 1.10.2 [wheel])
 EOH
         it { is_expected.to eq({'boto' => '2.38.0', 'botocore' => '1.1.1', 'certifi' => '2015.4.28', 'cffi' => '1.1.2', 'fabric' => '1.10.2'}) }
       end # /context with standard content
+
+      context 'with malformed content' do
+        let(:text) { <<-EOH }
+boto (Current: 2.25.0 Latest: 2.38.0 [wheel])
+botocore (Current: 0.56.0 Latest: 1.1.1 [wheel])
+certifi (Current: 14.5.14 Latest: 2015.4.28 [wheel])
+cffi (Current: 0.8.1 Latest: 1.1.2 [sdist])
+Fabric (Future: 1.9.1 [wheel])
+EOH
+        it { is_expected.to eq({'boto' => '2.38.0', 'botocore' => '1.1.1', 'certifi' => '2015.4.28', 'cffi' => '1.1.2'}) }
+      end # /context with malformed content
     end # /describe #parse_pip_outdated
 
     describe '#parse_pip_list' do
@@ -60,6 +221,16 @@ flake8 (2.1.0.dev0)
 EOH
         it { is_expected.to eq({'eventlet' => '0.12.1', 'fabric' => '1.9.1', 'fabric-rundeck' => '1.2', 'flake8' => '2.1.0.dev0'}) }
       end # /context with standard content
+
+      context 'with malformed content' do
+        let(:text) { <<-EOH }
+eventlet (0.12.1)
+Fabric (1.9.1)
+fabric-rundeck (1.2, /Users/coderanger/src/bal/fabric-rundeck)
+flake 8 (2.1.0.dev0)
+EOH
+        it { is_expected.to eq({'eventlet' => '0.12.1', 'fabric' => '1.9.1', 'fabric-rundeck' => '1.2'}) }
+      end # /context with malformed content
     end # /describe #parse_pip_list
   end # /describe PoisePython::Resources::PythonPackage::Provider
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: poise-python
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.2.0
 platform: ruby
 authors:
 - Noah Kantrowitz
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-22 00:00:00.000000000 Z
+date: 2016-01-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: halite