b77a695b7a
* python: tweak script linking in virtualenv
Instead of making the formula author use a slightly awkward block like
venv.link_scripts(bin) { venv.pip_install buildpath }
avoid exposing this implementation detail and offer the more familiar:
venv.pip_install buildpath, :link_scripts => bin
* Add non-block form and use instead of recursion
* Update 'pip_install' documentation
* Remove obsolete 'link_scripts'
* Add test for 'pip_install' with linking scripts
Also drop no longer relevant (and broken) `link_scripts` test, that
served as a template for the new test.
* Restore compatibility with Ruby 1.8.7
* Replace option hash with 'pip_install_and_link'
* Avoid confusing 'Object#tap' and fix silly bug
* Avoid side effects in mock object parameter check
* Simplify argument check (no need for a block)
100 lines
2.8 KiB
Ruby
100 lines
2.8 KiB
Ruby
require "testing_env"
|
|
require "language/python"
|
|
require "resource"
|
|
|
|
class LanguagePythonTests < Homebrew::TestCase
|
|
def setup
|
|
@dir = Pathname.new(mktmpdir)
|
|
resource = stub("resource", :stage => true)
|
|
formula_bin = @dir/"formula_bin"
|
|
@formula = mock("formula") do
|
|
stubs(:resource).returns(resource)
|
|
stubs(:bin).returns(formula_bin)
|
|
end
|
|
@venv = Language::Python::Virtualenv::Virtualenv.new(@formula, @dir, "python")
|
|
end
|
|
|
|
def teardown
|
|
FileUtils.rm_rf @dir
|
|
end
|
|
|
|
def test_virtualenv_creation
|
|
@formula.expects(:resource).with("homebrew-virtualenv").returns(
|
|
mock("resource", :stage => true)
|
|
)
|
|
@venv.create
|
|
end
|
|
|
|
# or at least doesn't crash the second time
|
|
def test_virtualenv_creation_is_idempotent
|
|
@formula.expects(:resource).with("homebrew-virtualenv").returns(
|
|
mock("resource", :stage => true)
|
|
)
|
|
@venv.create
|
|
FileUtils.mkdir_p @dir/"bin"
|
|
FileUtils.touch @dir/"bin/python"
|
|
@venv.create
|
|
FileUtils.rm @dir/"bin/python"
|
|
end
|
|
|
|
def test_pip_install_accepts_string
|
|
@formula.expects(:system).returns(true).with do |*params|
|
|
params.first == @dir/"bin/pip" && params.last == "foo"
|
|
end
|
|
@venv.pip_install "foo"
|
|
end
|
|
|
|
def test_pip_install_accepts_multiline_string
|
|
@formula.expects(:system).returns(true).with do |*params|
|
|
params.first == @dir/"bin/pip" && params[-2..-1] == ["foo", "bar"]
|
|
end
|
|
@venv.pip_install <<-EOS.undent
|
|
foo
|
|
bar
|
|
EOS
|
|
end
|
|
|
|
def test_pip_install_accepts_array
|
|
@formula.expects(:system).returns(true).with do |*params|
|
|
params.first == @dir/"bin/pip" && params.last == "foo"
|
|
end
|
|
@formula.expects(:system).returns(true).with do |*params|
|
|
params.first == @dir/"bin/pip" && params.last == "bar"
|
|
end
|
|
@venv.pip_install ["foo", "bar"]
|
|
end
|
|
|
|
def test_pip_install_accepts_resource
|
|
res = Resource.new "test"
|
|
res.expects(:stage).yields(nil)
|
|
@formula.expects(:system).returns(true).with do |*params|
|
|
params.first == @dir/"bin/pip" && params.last == Pathname.pwd
|
|
end
|
|
@venv.pip_install res
|
|
end
|
|
|
|
def test_pip_install_and_link_links_scripts
|
|
bin = @dir/"bin"
|
|
bin.mkpath
|
|
dest = @formula.bin
|
|
|
|
refute_predicate bin/"kilroy", :exist?
|
|
refute_predicate dest/"kilroy", :exist?
|
|
|
|
FileUtils.touch bin/"irrelevant"
|
|
bin_before = Dir[bin/"*"]
|
|
FileUtils.touch bin/"kilroy"
|
|
bin_after = Dir[bin/"*"]
|
|
@venv.expects(:pip_install).with("foo")
|
|
Dir.expects(:[]).twice.returns(bin_before, bin_after)
|
|
|
|
@venv.pip_install_and_link "foo"
|
|
|
|
assert_predicate bin/"kilroy", :exist?
|
|
assert_predicate dest/"kilroy", :exist?
|
|
assert_predicate dest/"kilroy", :symlink?
|
|
assert_equal((bin/"kilroy").realpath, (dest/"kilroy").realpath)
|
|
refute_predicate dest/"irrelevant", :exist?
|
|
end
|
|
end
|