bpm 0.1.0

data/spec/cli/fetch_spec.rb

@@ -0,0 +1,135 @@
+require "spec_helper"
+
+describe 'bpm fetch' do
+  
+  before do
+    goto_home
+    FileUtils.cp_r(fixtures('hello_world'), '.')
+    set_host
+    start_fake(FakeGemServer.new)
+  end
+
+  it "fetches all dependencies in a project when called without command" do
+    cd home('hello_world')
+    bpm "fetch"
+
+    stdout.read.should include("Fetched dependent packages for hello_world")
+
+    # note: ivory is a soft dependency from core-test
+    %w(spade-0.5.0 core-test-0.4.9 ivory-0.0.1 optparse-1.0.1).each do |package_name|
+      package_name.should be_fetched
+      package_name.should be_unpacked
+    end
+    
+  end
+
+  it "fetches a valid package" do
+    bpm "fetch", "rake"
+
+    stdout.read.should include("Successfully fetched rake (0.8.7)")
+
+    "rake-0.8.7".should be_fetched
+    "rake-0.8.7".should be_unpacked
+  end
+
+  it "fetches a multiple gems" do
+    bpm "fetch", "rake", "builder"
+
+    output = stdout.read
+
+    %w(builder-3.0.0 rake-0.8.7).each do |pkg|
+      output.should include("Successfully fetched #{pkg.sub(/-[^-]*$/,'')} (#{pkg.sub(/^.*-/, '')})")
+      pkg.should be_fetched
+      pkg.should be_unpacked
+    end
+  end
+
+  it "fetches valid gems while ignoring invalid ones" do
+    bpm "fetch", "rake", "fake", :track_stderr => true
+
+    stdout.read.should include("Successfully fetched rake (0.8.7)")
+    stderr.read.should include("Can't find package fake")
+
+    "rake-0.8.7".should be_fetched
+    "rake-0.8.7".should be_unpacked
+    "fake-0".should_not be_fetched
+    "fake-0".should_not be_unpacked
+  end
+
+  it "fails when fetching an invalid gem" do
+    bpm "fetch", "fake", :track_stderr => true
+
+    stderr.read.should include("Can't find package fake")
+    "rake-0.8.7".should_not be_fetched
+    "rake-0.8.7".should_not be_unpacked
+    "fake-0".should_not be_fetched
+    "fake-0".should_not be_unpacked
+  end
+
+  it "fails if bpm can't write to the bpm directory" do
+    FileUtils.mkdir_p bpm_dir
+    FileUtils.chmod 0555, bpm_dir
+
+    bpm "fetch", "rake", :track_stderr => true
+    exit_status.should_not be_success
+
+    "rake-0.8.7".should_not be_fetched
+    "rake-0.8.7".should_not be_unpacked
+  end
+
+  it "fetches gems with a different version" do
+    bpm "fetch", "rake", "-v", "0.8.6"
+
+    stdout.read.should include("Successfully fetched rake (0.8.6)")
+
+    "rake-0.8.7".should_not be_fetched
+    "rake-0.8.7".should_not be_unpacked
+    "rake-0.8.6".should be_fetched
+    "rake-0.8.6".should be_unpacked
+  end
+
+  it "fetches a valid prerelease package" do
+    bpm "fetch", "bundler", "--pre"
+
+    stdout.read.should include("Successfully fetched bundler (1.1.pre)")
+
+    "bundler-1.1.pre".should be_fetched
+    "bundler-1.1.pre".should be_unpacked
+  end
+
+  it "does not fetch the normal package when asking for a prerelease" do
+    bpm "fetch", "rake", "--pre", :track_stderr => true
+
+    stderr.read.should include("Can't find package rake")
+
+    "rake-0.8.7".should_not be_fetched
+    "rake-0.8.7".should_not be_unpacked
+    "rake-0.8.6".should_not be_fetched
+    "rake-0.8.6".should_not be_unpacked
+  end
+
+  it "requires at least one package to fetch (when not in a project)" do
+    bpm "fetch", :track_stderr => true
+    stderr.read.should include("called incorrectly")
+  end
+
+  it "does not make a .gem directory" do
+    bpm "fetch", "rake"
+    wait
+    home(".gem").exist?.should be_false
+  end
+
+  it "fetches package dependencies" do
+    bpm "fetch", "core-test"
+
+    output = stdout.read
+
+    %w(ivory-0.0.1 optparse-1.0.1 core-test-0.4.9).each do |name|
+      output.should include("Successfully fetched #{name.sub(/-[^-]*$/,'')} (#{name.sub(/^.*-/, '')})")
+
+      name.should be_fetched
+      name.should be_unpacked
+    end
+  end
+    
+end

data/spec/cli/fetched_spec.rb

@@ -0,0 +1,52 @@
+require "spec_helper"
+
+describe "bpm fetched" do
+  before do
+    goto_home
+    set_host
+    start_fake(FakeGemServer.new)
+  end
+
+  it "lists fetched packages" do
+    bpm "fetch", "rake"
+    wait
+    bpm "fetched"
+
+    output = stdout.read
+    output.should include("rake (0.8.7)")
+    output.should_not include("0.8.6")
+    output.should_not include("builder")
+    output.should_not include("bundler")
+    output.should_not include("highline")
+  end
+
+  it "lists all installed packages from different versions" do
+    bpm "fetch", "rake"
+    wait
+    bpm "fetch", "rake", "-v", "0.8.6"
+    wait
+    bpm "fetched"
+
+    output = stdout.read
+    output.should include("rake (0.8.7, 0.8.6)")
+  end
+
+  it "filters packages when given an argument" do
+    bpm "fetch", "rake"
+    wait
+    bpm "fetch", "builder"
+    wait
+    bpm "fetched", "builder"
+
+    output = stdout.read
+    output.should_not include("rake")
+    output.should include("builder (3.0.0)")
+  end
+
+  it "says it couldn't find any if none found" do
+    bpm "fetched", "rails", :track_stderr => true
+
+    stderr.read.strip.should == 'No packages found matching "rails".'
+    exit_status.should_not be_success
+  end
+end

data/spec/cli/init_spec.rb

@@ -0,0 +1,50 @@
+require "spec_helper"
+
+describe "bpm init" do
+
+  before do
+    @project_path = home("new_project")
+    FileUtils.mkdir @project_path
+    cd @project_path
+  end
+
+  it "should create files" do
+    bpm 'init'
+
+    files = %w(new_project.json assets assets/bpm_packages.js assets/bpm_styles.css)
+
+    output = stdout.read.gsub(/\e\[\d+m/,'') #without colors
+
+    files.each do |file|
+      output.should =~ /create\s+#{file}$/
+      File.join(@project_path, file).should exist
+    end
+  end
+
+  it "should fetch dependencies"
+
+  it "should build"
+
+  it "should not overwrite existing files" do
+    File.open("new_project.json", 'w'){|f| f.print "Valuable info!" }
+
+    bpm 'init', '--skip' # skip, since we can't test the prompt
+
+    output = stdout.read.gsub(/\e\[\d+m/,'') # without colors
+
+    output.should =~ /skip\s+new_project.json/
+    File.read("new_project.json").should == "Valuable info!"
+  end
+
+  it "should allow a name to be specified" do
+    bpm 'init', '--name=DifferentName' and wait
+
+    File.join(@project_path, "DifferentName.json").should exist
+    File.join(@project_path, "new_project.json").should_not exist
+  end
+
+  it "should initialize multiple at once"
+
+  it "should not allow a name when initializing multiple"
+
+end

data/spec/cli/list_spec.rb

@@ -0,0 +1,74 @@
+require "spec_helper"
+
+describe "bpm list" do
+  before do
+    goto_home
+    set_host
+    start_fake(FakeGemServer.new)
+  end
+
+  it "lists latest packages by default" do
+    bpm "list"
+
+    output = stdout.read
+    output.should include("builder (3.0.0)")
+    output.should include("rake (0.8.7)")
+  end
+
+  it "lists all packages when given the all argument" do
+    bpm "list", "-a"
+
+    output = stdout.read
+    output.should include("builder (3.0.0)")
+    output.should include("rake (0.8.7, 0.8.6)")
+  end
+
+  it "filters packages when given an argument" do
+    bpm "list", "builder"
+
+    output = stdout.read
+    output.should include("builder (3.0.0)")
+    output.should_not include("rake")
+  end
+
+  it "filters packages when given an argument and shows all versions" do
+    bpm "list", "rake", "-a"
+
+    output = stdout.read
+    output.should include("rake (0.8.7, 0.8.6)")
+    output.should_not include("builder")
+  end
+
+  it "filters multiple packages" do
+    bpm "list", "rake", "highline"
+
+    output = stdout.read
+    output.should include("highline (1.6.1)")
+    output.should include("rake (0.8.7)")
+    output.should_not include("builder")
+  end
+
+  it "shows prerelease packages" do
+    bpm "list", "--prerelease"
+
+    output = stdout.read
+    output.should include("bundler (1.1.pre)")
+    output.should_not include("highline")
+    output.should_not include("rake")
+    output.should_not include("builder")
+  end
+
+  it "says it couldn't find any if none found" do
+    bpm "list", "rails", :track_stderr => true
+
+    stderr.read.strip.should == 'No packages found matching "rails".'
+    exit_status.should_not be_success
+  end
+
+  it "says it couldn't find any if none found matching multiple packages" do
+    bpm "list", "rails", "bake", :track_stderr => true
+
+    stderr.read.strip.should == 'No packages found matching "rails", "bake".'
+    exit_status.should_not be_success
+  end
+end

data/spec/cli/login_spec.rb

@@ -0,0 +1,110 @@
+require "spec_helper"
+
+describe "bpm login" do
+  let(:email)    { "email@example.com" }
+  let(:password) { "secrets" }
+  let(:api_key)  { "deadbeef" }
+  let(:creds)    { bpm_dir("credentials") }
+
+  before do
+    goto_home
+
+    fake = lambda { |env|
+      [200, {"Content-Type" => "text/plain"}, [api_key]]
+    }
+
+    protected_fake = Rack::Auth::Basic.new(fake) do |user, pass|
+      user == email && password == pass
+    end
+
+    set_host
+    start_fake(protected_fake)
+  end
+
+  it "says email that user is logging in as" do
+    bpm "login"
+    input email
+    input password
+    output = stdout.read
+    output.should include("Enter your BPM credentials.")
+    output.should include("Logging in as #{email}...")
+  end
+
+  it "makes a request out for the api key and stores it in BPM_DIR/credentials" do
+    bpm "login"
+    input email
+    input password
+
+    stdout.read.should include("Logged in!")
+    File.exist?(creds).should be_true
+    YAML.load_file(creds)[:bpm_api_key].should == api_key
+    YAML.load_file(creds)[:bpm_email].should == email
+  end
+
+  it "notifies user if bad creds given" do
+    bpm "login", :track_stderr => true
+    input email
+    input "badpassword"
+    sleep 1
+    kill!
+
+    stdout.read.should include("Incorrect email or password.")
+    File.exist?(creds).should be_false
+  end
+
+  it "allows the user to retry if bad creds given" do
+    bpm "login"
+    input "bademail@example.com"
+    input "badpassword"
+
+    input email
+    input password
+
+    output = stdout.read.split("\n").select { |line| line.size > 0 }
+    output[0].should include("Enter your BPM credentials.")
+    output[3].should include("Logging in as bademail@example.com...")
+    output[4].should include("Incorrect email or password.")
+    output[5].should include("Enter your BPM credentials.")
+    output[8].should include("Logging in as #{email}...")
+    output[9].should include("Logged in!")
+
+    File.exist?(creds).should be_true
+    YAML.load_file(creds)[:bpm_api_key].should == api_key
+    YAML.load_file(creds)[:bpm_email].should == email
+  end
+
+  it "will login with credentials provided as cli arguments" do
+    bpm "login", "-u", email, "-p", password
+    output = stdout.read
+    output.should_not include("Enter your BPM credentials.")
+    output.should include("Logging in as #{email}...")
+  end
+  
+  it "will not retry on failure if username and password are provided as cli arguments" do
+    bpm "login", "-u", email, "-p", "badpassword"
+    output = stdout.read
+    output.should_not include("Enter your BPM credentials.")
+    output.should include("Logging in as #{email}...")
+    output.should include("Incorrect email or password.")
+  end
+  
+  it "will only prompt for username if password is provided" do
+    bpm "login", "-p", password
+    input email
+    output = stdout.read
+    output.should include("Enter your BPM credentials.")
+    output.should include("Email:")
+    output.should_not include("Password:")
+    output.should include("Logging in as #{email}...")
+  end
+  
+  it "will only prompt for password if username is provided" do
+    bpm "login", "-u", email
+    input password
+    output = stdout.read
+    output.should include("Enter your BPM credentials.")
+    output.should_not include("Email:")
+    output.should include("Password:")
+    output.should include("Logging in as #{email}...")
+  end
+end

data/spec/cli/new_spec.rb

@@ -0,0 +1,92 @@
+require "spec_helper"
+
+describe "bpm new" do
+
+  describe "normal" do
+
+    before do
+      cd home
+    end
+
+    it "should create files" do
+      bpm 'new', 'BpmTest'
+
+      files = %w(LICENSE README.md index.html app.js BpmTest.json assets assets/bpm_packages.js assets/bpm_styles.css)
+
+      # output without coloration
+      output = stdout.read.gsub(/\e\[\d+m/,'')
+
+      files.each do |file|
+        output.should =~ /create\s+#{file}$/
+        home("bpm_test", file).should exist
+      end
+    end
+
+    it "should fetch dependencies"
+
+    it "should build"
+
+    it "should not generate into existing directory" do
+      FileUtils.mkdir 'bpm_test'
+
+      bpm 'new', 'BpmTest'
+
+      stdout.read.should_not include("create")
+      `ls #{home("bpm_test")}`.should be_empty
+    end
+
+    it "should allow a path to be specified" do
+      bpm 'new', 'BpmTest', '--path=DifferentLocation' and wait
+
+      home("DifferentLocation").should exist
+      home("DifferentLocation", "BpmTest.json").should exist
+      home("DifferentLocation", "DifferentLocation.json").should_not exist
+      home("bpm_test").should_not exist
+    end
+
+  end
+
+  describe "package templates" do
+
+    before do
+      goto_home
+
+      # Install package
+      cd fixtures
+      with_env do
+        BPM::Remote.new.install("custom_generator-1.0.spd", ">= 0", false)
+      end
+      cd home
+    end
+
+    it "should create custom files" do
+      bpm 'new', 'BpmTest', '--package=custom_generator'
+
+      files = %w(lib lib/main.js app.js BpmTest.json assets assets/bpm_packages.js assets/bpm_styles.css)
+
+      # output without coloration
+      output = stdout.read.gsub(/\e\[\d+m/,'')
+
+      files.each do |file|
+        output.should =~ /create\s+#{file}$/
+        home("bpm_test", file).should exist
+      end
+    end
+
+    it "should create custom app.js with explicit generator" do
+      bpm 'new', 'BpmTest', '--package=custom_generator' and wait
+
+      File.read(home("bpm_test", "app.js")).should == "require('BpmTest/main.js')\n"
+    end
+
+    it "should create custom project.json without explicit generator" do
+      bpm 'new', 'BpmTest', '--package=custom_generator' and wait
+
+      File.read(home("bpm_test", "BpmTest.json")).should =~ /"spade": ">= 0"/
+    end
+
+    it "should install the package if it doesn't exist locally"
+
+  end
+
+end