serverspec-extended-types 0.0.1 → 0.0.2

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YjhlYTA0NzY0NDE2NGRlOTJkMTcwNGQ3NTcwOTI3NTQwZWQ4OTE0ZQ==
+    N2QzMDBhYzA5MjU0M2RkOTA3MGU1ZTgxNDFlMzg3ZWE4YWQyOWQ2OQ==
   data.tar.gz: !binary |-
-    MDNiOTI1NDg4NzRjY2RhNGQ0YjRjNmQ4OGY2ZTNlMmI4MmFkMjFlMQ==
+    OWYxYTE2NGM5NmExZWNhY2JlYWZjNmZhYThlNDdjZTFiMzM4YTZjNA==
 SHA512:
   metadata.gz: !binary |-
-    NThiNGVmZTM5NWY5Y2EyMDNjMzg3ZmVkNWVjN2RhYTZhMzQwMmMzNzQxOGFi
-    OGQwNDQ1ZTMyNWY5MGFiNTU0MWE5NWVkZDk1OWUzNTNjMDRiNDNkMDU1MjBi
-    Njc5YWJkNjg3MzlmNWVjYjQzZTQ1Nzc0MjI3ZTgwZTc2ODVkNDQ=
+    NTk1ZTA4NzQ4NzQ4YTM0YmQ4MTE0ZTQ5ZTEwNjgxZThhNzExODk0MGM4MDg0
+    MjcyN2NkNDRhMTJlNmE1ZGE4YmUxNTJmZGE3NDY2ZjQyM2E0YmEyZDA0Nzlk
+    ZDE2YjEzYjJlMTVjNTU5YWUyN2YzNzIwZjBjMzFhZmJhYWFlZGM=
   data.tar.gz: !binary |-
-    MDJkMjhlOWUyMDg3ZTczZTA4M2Q5ZGVjOGE5Y2Y0NzJiODNjYjE1M2NiMDI3
-    ZDY3ODA4NzlkNzVmOTU2ZDVmNjAxZjg1OTUwYzVhOGU5ZWM0Yzk2MmIwODdk
-    MWFkMjE3NjRhYzljOGE4MTdhZjYwNzhhOTIyMzNjMDE5ODYyZjE=
+    ZTczY2FlZTQxZWUxNDkwNDVmMzA3MjA4MmU2YzIxMzdjZTQ3ODFjZTg2ODYx
+    YWMxNzU4ODE2Y2UxMTE0NWY0NTE5MjhiNWQwYjJhOTAyYTE2NTcwNzdjYzk4
+    NzgxYTZhYWZmMmRkNDQ1YjBmZTQ0NTFjZWFlZTY2M2ZkYTJlNmY=

data/CHANGES.md

@@ -3,3 +3,7 @@
 ## 0.0.1 2015-03-14 Jason Antman <jason@jasonantman.com>
 
 * Initial release with bitlbee, http_get and virtualenv types
+
+## 0.0.2 2015-03-14 Jason Antman <jason@jasonantman.com>
+
+* Minor bugfix in Serverspec::Types::Virtualenv#virtualenv?

data/README.md

@@ -217,9 +217,9 @@ Test whether this appears to be a working venv
 
 Tests performed:
 
-* venv_path/bin/pip executable by root?
-* venv_path/bin/python executable by root?
-* venv_path/bin/activate executable by root?
+* venv_path/bin/pip executable by owner?
+* venv_path/bin/python executable by owner?
+* venv_path/bin/activate readable by owner?
 * 'export VIRTUAL_ENV' in venv_path/bin/activate?
 
 ## Contributing

data/lib/serverspec_extended_types/version.rb

@@ -1,3 +1,3 @@
 module ServerspecExtendedTypes
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/serverspec_extended_types/virtualenv.rb

@@ -15,9 +15,9 @@ module Serverspec::Type
     # Test whether this appears to be a working venv
     #
     # Tests performed:
-    # - venv_path/bin/pip executable by root?
-    # - venv_path/bin/python executable by root?
-    # - venv_path/bin/activate executable by root?
+    # - venv_path/bin/pip executable by owner?
+    # - venv_path/bin/python executable by owner?
+    # - venv_path/bin/activate readable by owner?
     # - 'export VIRTUAL_ENV' in venv_path/bin/activate?
     #
     # @example
@@ -32,9 +32,9 @@ module Serverspec::Type
       python_path = ::File.join(@name, 'bin', 'python')
       act_path = ::File.join(@name, 'bin', 'activate')
       cmd = "grep -q 'export VIRTUAL_ENV' #{act_path}"
-      @runner.check_file_is_executable(pip_path, 'root') and
-        @runner.check_file_is_executable(python_path, 'root') and
-        @runner.check_file_is_executable(act_path, 'root') and
+      @runner.check_file_is_executable(pip_path, 'owner') and
+        @runner.check_file_is_executable(python_path, 'owner') and
+        @runner.check_file_is_readable(act_path, 'owner') and
         @runner.run_command(cmd).exit_status.to_i == 0
     end
 

data/spec/types/http_get_spec.rb

@@ -54,7 +54,7 @@ describe 'http_get()' do
       stub_const('ENV', ENV.to_hash.merge('TARGET_HOST' => 'myhost'))
       conn = double
       headers = double
-      expect(headers).to receive(:[]=).with(:user_agent, 'Serverspec::Type::Http_Get/0.0.1 (https://github.com/jantman/serverspec-extended-types)')
+      expect(headers).to receive(:[]=).with(:user_agent, %r"Serverspec::Type::Http_Get/\d+\.\d+\.\d+ \(https://github.com/jantman/serverspec-extended-types\)")
       expect(conn).to receive(:headers).and_return(headers)
       expect(headers).to receive(:[]=).with(:Host, 'hostheader')
       expect(conn).to receive(:headers).and_return(headers)

data/spec/types/virtualenv_spec.rb

@@ -16,36 +16,36 @@ describe 'Serverspec::Type#virtualenv' do
   context '#virtualenv?' do
     it 'checks all files' do
       v = virtualenv('/foo/bar')
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'root').and_return(true)
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/python', 'root').and_return(true)
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/activate', 'root').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'owner').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/python', 'owner').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_readable).once.ordered.with('/foo/bar/bin/activate', 'owner').and_return(true)
       res = Specinfra::CommandResult.new({:stdout => '', :stderr => '', :exit_signal => nil, :exit_status => 0 })
       expect(v.instance_variable_get(:@runner)).to receive(:run_command).once.ordered.with("grep -q 'export VIRTUAL_ENV' /foo/bar/bin/activate").and_return(res)
       expect(v.virtualenv?).to eq true
     end
     it 'returns false with missing pip' do
       v = virtualenv('/foo/bar')
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'root').and_return(false)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'owner').and_return(false)
       expect(v.virtualenv?).to eq false
     end
     it 'returns false with missing python' do
       v = virtualenv('/foo/bar')
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'root').and_return(true)
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/python', 'root').and_return(false)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'owner').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/python', 'owner').and_return(false)
       expect(v.virtualenv?).to eq false
     end
     it 'returns false with missing activate' do
       v = virtualenv('/foo/bar')
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'root').and_return(true)
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/python', 'root').and_return(true)
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/activate', 'root').and_return(false)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'owner').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/python', 'owner').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_readable).once.ordered.with('/foo/bar/bin/activate', 'owner').and_return(false)
       expect(v.virtualenv?).to eq false
     end
     it 'returns false with invalid activate' do
       v = virtualenv('/foo/bar')
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'root').and_return(true)
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/python', 'root').and_return(true)
-      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/activate', 'root').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/pip', 'owner').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_executable).once.ordered.with('/foo/bar/bin/python', 'owner').and_return(true)
+      expect(v.instance_variable_get(:@runner)).to receive(:check_file_is_readable).once.ordered.with('/foo/bar/bin/activate', 'owner').and_return(true)
       res = Specinfra::CommandResult.new({:stdout => '', :stderr => '', :exit_signal => nil, :exit_status => 254 })
       expect(v.instance_variable_get(:@runner)).to receive(:run_command).once.ordered.with("grep -q 'export VIRTUAL_ENV' /foo/bar/bin/activate").and_return(res)
       expect(v.virtualenv?).to eq false

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: serverspec-extended-types
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Jason Antman