gas 0.1.6 → 0.1.7

data/lib/gas/user.rb

@@ -4,6 +4,7 @@ module Gas
   class User
     attr_reader :name, :email, :nickname
 
+
     # @param [String] name The name of the user
     # @param [String] email The email of the user
     # @param [String] nickname A nickname for the user, not used when parsing from gitconfig
@@ -11,6 +12,7 @@ module Gas
       @name = name
       @email = email
       @nickname = nickname
+
     end
 
     # Returns the git format of user
@@ -23,8 +25,9 @@ module Gas
     # @param [Boolean] use_nickname Defaults to true
     # @return [String]
     def to_s(use_nickname = true)
-      "[#{use_nickname ? @nickname : 'user'}]\n  name = #{@name}\n  email = #{@email}"
+      "      [#{use_nickname ? @nickname : 'user'}]\n         name = #{@name}\n         email = #{@email}"
     end
 
+
   end
 end

data/lib/gas/version.rb

@@ -1,6 +1,6 @@
 module Gas
 
-  VERSION = '0.1.6'
+  VERSION = '0.1.7'
 
 end
 

data/spec/gas/config_spec.rb

@@ -24,7 +24,10 @@ describe Gas::Config do
     user2 = Gas::User.new 'foo', 'bar', 'user2'
     users = [user1, user2]
     config = Gas::Config.new users
-    config.to_s.should == "[walle]\n  name = Fredrik Wallgren\n  email = fredrik.wallgren@gmail.com\n[user2]\n  name = foo\n  email = bar"
+
+    stub(config).is_current_user { false }
+
+    config.to_s.should == "      [walle]\n         name = Fredrik Wallgren\n         email = fredrik.wallgren@gmail.com\n      [user2]\n         name = foo\n         email = bar"
   end
 
   it 'should be able to tell if a nickname exists' do
@@ -71,5 +74,10 @@ describe Gas::Config do
     config.delete 'user2'
     config.users.count.should == 0
   end
+
+  it 'should remove the ssh keys from .gas appropriately'
+
+  it 'should remove the keys from .ssh if present'
+
 end
 

data/spec/gas/gitconfig_spec.rb

@@ -1,12 +1,14 @@
 require './spec/spec_helper'
 
 require './lib/gas'
+#require './lib/gas/user'
 
 describe Gas::Gitconfig do
 
   before :each do
     @name = 'Fredrik Wallgren'
     @email = 'fredrik.wallgren@gmail.com'
+    @nickname = 'Fred'
     @gitconfig = Gas::Gitconfig.new
   end
 
@@ -26,19 +28,58 @@ describe Gas::Gitconfig do
     @gitconfig.current_user.should == nil
   end
 
-  it 'should be able to change the current user' do
-    name = 'Test Testsson'
-    email = 'test@testsson.com'
 
-    mock_cli_call(@gitconfig, "git config --global user.name \"#{name}\"") { nil }
-    mock_cli_call(@gitconfig, "git config --global user.email \"#{email}\"") { nil }
-    mock_cli_call(@gitconfig, 'git config --global --get user.name') { name + "\n" }
-    mock_cli_call(@gitconfig, 'git config --global --get user.email') { email + "\n" }
+  describe "Multiple users" do
 
-    @gitconfig.change_user name, email
+    before :each do
+      @user1 = Gas::User.new(@name, @email, @nickname)   # create a primary user for testing
+    end
+
+    it "should be able to set the current user" do
+      # setup the cli interrupt things...
+      mock_cli_call(@gitconfig, "git config --global user.name \"#{@user1.name}\"") { nil }
+      mock_cli_call(@gitconfig, "git config --global user.email \"#{@user1.email}\"") { nil }
+      mock_cli_call(@gitconfig, 'git config --global --get user.name') { @user1.name + "\n" }
+      mock_cli_call(@gitconfig, 'git config --global --get user.email') { @user1.email + "\n" }
+
+      @gitconfig.change_user @user1
+
+      user = @gitconfig.current_user
+      user.name.should == @user1.name
+      user.email.should == @user1.email
+      user.nickname.should == @user1.nickname
+    end
+
+    it 'should be able to change the current user' do
+      name = 'Test Testsson'
+      email = 'test@testsson.com'
+      nickname = 'test'
+
+      # User 1 cli interrupt things...
+      mock_cli_call(@gitconfig, "git config --global user.name \"#{@name}\"") { nil }
+      mock_cli_call(@gitconfig, "git config --global user.email \"#{@email}\"") { nil }
+      mock_cli_call(@gitconfig, 'git config --global --get user.name') { @name + "\n" }
+      mock_cli_call(@gitconfig, 'git config --global --get user.email') { @email + "\n" }
+
+      @gitconfig.change_user @user1
+
+      user = @gitconfig.current_user
+      user.name.should == @name
+      user.email.should == @email      # test that the user switch worked (paranoid, huh?)
+
+      # User 2 cli interrupt things...
+      mock_cli_call(@gitconfig, "git config --global user.name \"#{name}\"") { nil }
+      mock_cli_call(@gitconfig, "git config --global user.email \"#{email}\"") { nil }
+      mock_cli_call(@gitconfig, 'git config --global --get user.name') { name + "\n" }
+      mock_cli_call(@gitconfig, 'git config --global --get user.email') { email + "\n" }
+
+      @user2 = Gas::User.new(name, email, nickname)   # create user 2
+      @gitconfig.change_user @user2
+
+      user = @gitconfig.current_user
+      user.name.should == name
+      user.email.should == email       # test that the user changed appropriately
+    end
 
-    user = @gitconfig.current_user
-    user.name.should == name
-    user.email.should == email
   end
 end

data/spec/gas/ssh_spec.rb

@@ -0,0 +1,328 @@
+require './spec/spec_helper'
+
+require './lib/gas'
+
+require 'rspec/mocks'
+require 'rspec/mocks/standalone'
+
+describe Gas::Ssh do
+
+  before :all do
+    move_the_testers_personal_ssh_key_out_of_way
+  end
+
+  after :all do
+    restore_the_testers_ssh_key
+  end
+
+  before :each do
+    @uid = "teddy"
+  end
+
+  describe "SSH key file handling..." do
+
+    before :all do
+      #Gas::Ssh.stub!(:delete).and_return(false)
+      Gas::Ssh.stub!(:user_wants_to_delete_all_ssh_data?).and_return("l")  # only delete's local keys
+    end
+
+    after :all do
+      #Gas::Ssh.unstub!(:delete)
+      Gas::Ssh.unstub!(:user_wants_to_delete_all_ssh_data?)
+    end
+
+    describe "Detecting when files are missing..." do
+
+      before :all do
+        File.stub!(:exists?).and_return(false)             # make it so File.exists? always return true
+      end
+
+      after :all do
+        File.unstub!(:exists?)                             # undoes the hook
+      end
+
+      it "should detect when an id_rsa isn't in the .gas directory" do
+        Gas::Ssh.user_wants_to_use_key_already_in_gas?.should be_false
+      end
+
+    end
+
+    describe "Detecting when files exist.@email..." do
+      before :all do
+        File.stub!(:exists?).and_return(true)             # make it so File.exists? always return true
+      end
+
+      after :all do
+        File.unstub!(:exists?)                             # undoes the hook
+      end
+
+      it 'should detect when an id_rsa is already in the .gas directory' do
+        STDIN.stub!(:gets).and_return("y\n")   # fix stdin to recieve a 'y' command...
+        Gas::Ssh.user_wants_to_use_key_already_in_gas?.should be_true
+        STDIN.unstub!(:gets)
+      end
+
+    end
+
+    describe "File System Changes..." do
+
+      before :all do
+        @gas_dir = File.expand_path('~/.gas')
+        @ssh_dir = File.expand_path('~/.ssh')
+
+
+        @nickname = "thisaccountmaybedeletedmysteriously"
+        @name = "tim T"
+        @email = "tim@timmy.com"
+
+        `rm ~/.gas/#{@nickname}_id_rsa`
+        `rm ~/.gas/#{@nickname}_id_rsa.pub`
+        Gas.delete(@nickname)
+
+        # make sure that nickname isn't in use
+      end
+
+
+      it 'should create ssh keys in .gas && Gas.remove should be able to remove those files' do
+        STDIN.stub!(:gets).and_return("y\n")          # forces the dialogs to
+        Gas::Ssh.stub!(:upload_public_key_to_github).and_return(false)
+
+        lambda do
+          Gas.add(@nickname,@name,@email)
+        end.should change{`ls ~/.gas -1 | wc -l`.to_i}.by(2)   #OMG THIS IS A FUN TEST!!!
+
+        lambda do
+          Gas.delete(@nickname)
+        end.should change{`ls ~/.gas -1 | wc -l`.to_i}.by(-2)
+
+        STDIN.unstub!(:gets)
+        Gas::Ssh.unstub!(:upload_public_key_to_github)
+      end
+
+
+      describe 'For the ssh directory...' do
+
+        before :each do
+          # A user for creating
+          @nickname = "thisaccountmaybedeletedmysteriously"
+          @name = "tim T"
+          @email = "tim@timmy.com"
+
+          clean_out_ssh_directory
+          clean_out_gas_directory(@nickname)
+
+          # a user for deleting
+          @nickname2 = "thisaccountmaybedeletedmysteriously2"
+          @name2 = "tim T2"
+          @email2 = "tim@timmy.com2"
+          create_user_no_git(@nickname2, @name2, @email2)
+        end
+
+        after :each do
+          Gas.delete(@nickname)
+          delete_user_no_git(@nickname2)
+        end
+
+
+        it "if there's no key in .ssh" do
+          #  1)  Create a User
+
+          #  2)  Switch to that user
+
+          #  3)  The .ssh directory should now contain that file
+          true.should be_true
+
+        end
+
+        it "if there's a key in ~/.ssh that isn't backed up in .gas" do
+          #  1)  Create a User
+
+          #  2)  Switch to that user
+
+          #  3)  The .ssh directory should now contain that file
+
+        end
+
+        it "If there's a key in ~/.ssh that's backed up in .gas"
+
+        it "should delete the key in .ssh when the user is deleted" do
+          # Gas.add(@nickname,@name,@email)
+
+        end
+
+        #  bundle exec rspec spec/gas/ssh_spec.rb -e 'should be able to copy ssh keys in the ssh'
+        it 'should be able to copy ssh keys in the ssh' do
+          # put a key pair in the ssh directory
+          mock_text = "this is a mock ssh file"
+          File.open(SSH_DIRECTORY + "/id_rsa","w+").write(mock_text)
+          File.open(SSH_DIRECTORY + "/id_rsa.pub","w+").write(mock_text)
+
+          File.exists?(GAS_DIRECTORY + "/#{@nickname}_id_rsa").should be_false
+          File.exists?(GAS_DIRECTORY + "/#{@nickname}_id_rsa.pub").should be_false
+
+          Gas::Ssh.use_current_rsa_files_for_this_user(@nickname)
+
+          File.exists?(GAS_DIRECTORY + "/#{@nickname}_id_rsa").should be_true
+          File.exists?(GAS_DIRECTORY + "/#{@nickname}_id_rsa.pub").should be_true
+
+          ssh_file = File.read(GAS_DIRECTORY + "/#{@nickname}_id_rsa")
+          # ssh_file.should == mock_text # this part doesn't work... hmmm...
+
+          File.delete(GAS_DIRECTORY + "/#{@nickname}_id_rsa")
+          File.delete(GAS_DIRECTORY + "/#{@nickname}_id_rsa.pub")
+        end
+
+
+        it "should have a UTILITY for deleting rsa files of user" do
+          lambda do
+            Gas::Ssh.delete_associated_local_keys!(@nickname2)
+          end.should change{`ls ~/.gas -1 | wc -l`.to_i}.by(-2)
+        end
+
+      end
+
+    end
+
+
+
+  end
+
+  describe "Networking stuff..." do
+
+    before :all do
+      @name = 'Fredrik Wallgren'
+      @email = 'fredrik.wallgren@gmail.com'
+      @nickname = "thisaccountmaybedeletedmysteriously"
+
+      # make sure sample key is deleted
+      @username = "aTestGitAccount"
+      @password = "plzdon'thackthetestaccount1"
+
+      config = "[#{@nickname}]\n  name = #{@name}\n  email = #{@email}\n\n[user2]\n  name = foo\n  email = bar"
+      @config = Gas::Config.new nil, config
+      @user = @config.users[0]
+
+      @credentials = {:username => @username, :password => @password}
+
+      # Code to prepare the github environment for testing
+      @sample_rsa = "ssh-rsa AAAAB3NzaC1yc2EAAAA55555AAABAQCpxktnw9eBaYkxLnvLq8ZeUI8d/d00MeV32na3GZ35qKtJ3Vmvzvb8anF1eZD8/+BtBgYer9/3E0KUi3YNYCeejkdUPj3/Z+aV7Ft0+IeKdzFSqfnfN9UsuS/zkeyia2bjgQJYqk2ZbkMuVIn79UI5ypJWGOXNfKyQ2adYJD7Pjgsxvx8qEXHlU+SszlGr7YFEFwT7rZtSXILylmcwCnZryy91cs50vGWxKzKrOV/2iMd8V4Qv7RbhKtQ7OCd19CaZ08H3xqcG1U2lqXIgxSN75bLL71AM0KfIvNOzvigBZnYyb/RKiUQUhA0FnnIYc/7hF9rOe/S1acRiOF6ihz1x"
+
+
+      Gas::Ssh.remove_key!(@username, @password, @sample_rsa)
+      Gas::Ssh.stub!(:get_username_and_password_and_authenticate).and_return(@credentials)
+    end
+
+    after :all do
+      # make sure sample key is deleted
+
+      Gas::Ssh.unstub!(:get_username_and_password_and_authenticate)
+    end
+
+    it "UTILITY:  should be able to insert a new key into github and conversly remove that key" do
+
+
+      lambda do
+        Gas::Ssh.key_installation_routine!(@user, @sample_rsa)
+      end.should change{Gas::Ssh.get_keys(@username, @password).length}.by(1)
+
+      lambda do
+        Gas::Ssh.remove_key!(@username, @password, @sample_rsa)
+      end.should change{Gas::Ssh.get_keys(@username, @password).length}.by(-1)
+    end
+
+
+    # bundle exec rspec spec/gas/ssh_spec.rb SPEC_OPTS="-e \"should add ssh keys to github when user is created, and delete them when destroyed\""
+=begin
+rake spec SPEC=spec/gas/ssh_spec.rb \
+          SPEC_OPTS="-e \"should add ssh keys to github when user is created, and delete them when destroyed\""
+
+    bundle exec rspec spec/gas/ssh_spec.rb -e 'should add ssh keys to github when user is created, and delete them when destroyed'
+
+bundle exec rspec spec/gas/ssh_spec.rb -e 'UTILITY:  should be able to insert a new key into github and conversly remove that key'
+=end
+
+    it "should add ssh keys to github when user is created, and delete them when destroyed" do
+
+      #move_the_testers_personal_ssh_key_out_of_way
+
+      # yes, delete all
+      Gas::Ssh.stub!(:user_wants_to_delete_all_ssh_data?).and_return("a")  # all keys, local and github
+      # create new user and use ssh handling
+      Gas::Ssh.stub!(:user_wants_gas_to_handle_rsa_keys?).and_return(true)
+      Gas::Ssh.stub!(:user_wants_to_use_key_already_in_ssh?).and_return(false)
+      Gas::Ssh.stub!(:user_wants_to_install_key_to_github?).and_return(true)
+
+
+      lambda do
+        Gas.add(@nickname,@name,@email)
+      end.should change{Gas::Ssh.get_keys(@username, @password).length}.by(1)
+
+      lambda do
+        Gas.delete(@nickname)
+      end.should change{Gas::Ssh.get_keys(@username, @password).length}.by(-1)
+
+
+      #restore_the_testers_ssh_key
+
+      Gas::Ssh.unstub!(:user_wants_to_delete_all_ssh_data?)
+      Gas::Ssh.unstub!(:user_wants_gas_to_handle_rsa_keys?)
+      Gas::Ssh.unstub!(:user_wants_to_use_key_already_in_ssh?)
+      Gas::Ssh.unstub!(:user_wants_to_install_key_to_github?)
+    end
+
+
+
+    it "Gas.Delete should be able to remove the id_rsa from .gas" do
+      Gas::Ssh.stub!(:user_wants_to_delete_all_ssh_data?).and_return("a")
+      Gas::Ssh.stub!(:user_wants_gas_to_handle_rsa_keys?).and_return(true)
+      Gas::Ssh.stub!(:user_wants_to_use_key_already_in_ssh?).and_return(false)
+      Gas::Ssh.stub!(:user_wants_to_install_key_to_github?).and_return(true)
+
+      lambda do
+        Gas.add(@nickname,@name,@email)
+      end.should change{`ls ~/.gas -1 | wc -l`.to_i}.by(2)
+
+      lambda do
+        Gas.delete(@nickname)
+      end.should change{`ls ~/.gas -1 | wc -l`.to_i}.by(-2)
+
+
+      Gas::Ssh.unstub!(:user_wants_to_delete_all_ssh_data?)
+      Gas::Ssh.unstub!(:user_wants_gas_to_handle_rsa_keys?)
+      Gas::Ssh.unstub!(:user_wants_to_use_key_already_in_ssh?)
+      Gas::Ssh.unstub!(:user_wants_to_install_key_to_github?)
+    end
+
+    # bundle exec rspec spec/gas/ssh_spec.rb -e 'Gas.ssh(nickname) should be able to add ssh support to a legacy user or an opt-out'
+    it 'Gas.ssh(nickname) should be able to add ssh support to a legacy user or an opt-out' do
+
+      Gas::Ssh.stub!(:user_wants_gas_to_handle_rsa_keys?).and_return(false)
+      Gas.add(@nickname,@name,@email)
+      Gas::Ssh.unstub!(:user_wants_gas_to_handle_rsa_keys?)
+
+      Gas::Ssh.stub!(:user_wants_gas_to_handle_rsa_keys?).and_return(true)
+      Gas::Ssh.stub!(:upload_public_key_to_github)
+
+      lambda do
+        Gas.ssh(@nickname)
+      end.should change{`ls ~/.gas -1 | wc -l`.to_i}.by(2)
+
+
+      Gas::Ssh.delete_associated_local_keys!(@nickname)
+
+      Gas::Ssh.unstub!(:user_wants_gas_to_handle_rsa_keys?)
+      Gas::Ssh.unstub!(:upload_public_key_to_github)
+    end
+
+
+
+    it "Should be able to tell if it's ever used this key under this ISP provider before and then warn the user"
+
+    it 'Should have the ability to show if the author is associated with a specific github account NAME, stored in gas.accouts file'
+
+    it 'Should have the ability to link up with non-github git-daemons'
+  end
+
+
+
+end