postfix_admin 0.1.1 → 0.1.2

data/spec/doveadm_spec.rb

@@ -0,0 +1,35 @@
+require File.expand_path(File.dirname(__FILE__) + "/spec_helper")
+require 'postfix_admin/doveadm'
+
+describe PostfixAdmin::Doveadm do
+  describe "schemes" do
+    it "return Array" do
+      PostfixAdmin::Doveadm.schemes.class.should == Array
+    end
+
+    it "return supported schemes" do
+      PostfixAdmin::Doveadm.schemes.should == %W!CRYPT MD5 MD5-CRYPT SHA SHA1 SHA256 SHA512 SMD5 SSHA SSHA256 SSHA512 PLAIN CLEAR CLEARTEXT PLAIN-TRUNC CRAM-MD5 SCRAM-SHA-1 HMAC-MD5 DIGEST-MD5 PLAIN-MD4 PLAIN-MD5 LDAP-MD5 LANMAN NTLM OTP SKEY RPA!
+    end
+  end
+
+  describe "password" do
+    it "CRAM-MD5" do
+      PostfixAdmin::Doveadm.password('password', 'CRAM-MD5').should == CRAM_MD5_PASS
+      PostfixAdmin::Doveadm.password('dovecot', 'CRAM-MD5').should == '2dc40f88a4c2142c3b10cc4b4d11382a648f600301b78a4070172782192898d6'
+    end
+
+    it "SHA256" do
+      PostfixAdmin::Doveadm.password('password', 'SHA256').should == 'XohImNooBHFR0OVvjcYpJ3NgPQ1qq73WKhHvch0VQtg='
+      PostfixAdmin::Doveadm.password('dovecot', 'SHA256').should == 'KN7aHmDsiQ/Ko+HzLzHcKoPqkjk7bditnD433YQYhcs='
+    end
+
+    it "MD5-CRYPT" do
+      PostfixAdmin::Doveadm.password('password', 'MD5-CRYPT').should =~ EX_MD5_CRYPT
+      PostfixAdmin::Doveadm.password('dovecot', 'MD5-CRYPT').should =~ EX_MD5_CRYPT
+    end
+
+    it "unknown scheme raise error" do
+      lambda{ PostfixAdmin::Doveadm.password('password', 'UNKNOWN-SCHEME') }.should raise_error Error
+    end
+  end
+end

data/spec/models_spec.rb

@@ -1,6 +1,13 @@
 require File.expand_path(File.dirname(__FILE__) + "/spec_helper")
 require 'postfix_admin/models'
 
+describe PostfixAdmin do
+  it "flag_str" do
+    PostfixAdmin.flag_str(true).should == "YES"
+    PostfixAdmin.flag_str(false).should == "NO"
+  end
+end
+
 describe PostfixAdmin::Admin do
   before do
     db_initialize
@@ -12,6 +19,14 @@ describe PostfixAdmin::Admin do
     Admin.exist?('unknown@example.com').should === false
   end
 
+  it "active" do
+    Admin.find('admin@example.com').active.should === true
+    Admin.find('all@example.com').active.should === true
+    non_active_admin = create_admin('non_active_admin@example.com', false)
+
+    Admin.find('non_active_admin@example.com').active.should === false
+  end
+
   it "#super_admin?" do
     Admin.find('admin@example.com').super_admin?.should === false
     Admin.find('all@example.com').super_admin?.should === true
@@ -25,7 +40,7 @@ describe PostfixAdmin::Admin do
 
     it "should not delete 'ALL' domain" do
       Admin.find('all@example.com').super_admin = false
-      Domain.exist?('ALL').should be_true
+      Domain.exist?('ALL').should be true
     end
 
     it "disable super admin flag" do
@@ -69,6 +84,14 @@ describe PostfixAdmin::Domain do
     Domain.exist?('unknown.example.com').should === false
   end
 
+  it "active" do
+    Domain.find('example.com').active.should == true
+    Domain.find('example.org').active.should == true
+
+    create_domain('non-active.example.com', false)
+    Domain.find('non-active.example.com').active.should == false
+  end
+
   describe "#num_total_aliases and .num_total_aliases" do
     it "when only alias@example.com" do
       Domain.num_total_aliases.should be(1)
@@ -103,6 +126,22 @@ describe PostfixAdmin::Mailbox do
     db_initialize
   end
 
+  it "active" do
+    Mailbox.find('user@example.com').active.should == true
+    domain = Domain.find('example.com')
+    domain.mailboxes << create_mailbox('non_active_user@example.com', nil, false)
+    domain.aliases   << create_mailbox_alias('non_active_user@example.com', false)
+    domain.save
+
+    Mailbox.find('non_active_user@example.com').active.should == false
+  end
+
+  it "can use long maildir" do
+    domain = Domain.find('example.com')
+    domain.mailboxes << create_mailbox('long_maildir_user@example.com', 'looooooooooooong_path/example.com/long_maildir_user@example.com/')
+    domain.save.should == true
+  end
+
   describe ".exist?" do
     it "returns true for exist account (mailbox)" do
       Mailbox.exist?('user@example.com').should === true
@@ -123,6 +162,16 @@ describe PostfixAdmin::Alias do
     db_initialize
   end
 
+  it "active" do
+    domain = Domain.find('example.com')
+    domain.aliases << create_alias('non_active_alias@example.com', false)
+    domain.save
+
+    Alias.find('user@example.com').active.should == true
+    Alias.find('alias@example.com').active.should == true
+    Alias.find('non_active_alias@example.com').active.should == false
+  end
+
   describe ".exist?" do
     it "returns true when exist alias and account" do
       Alias.exist?('user@example.com').should === true

data/spec/runner_spec.rb

@@ -1,3 +1,5 @@
+# -*- coding: utf-8 -*-
+
 require File.expand_path(File.dirname(__FILE__) + "/spec_helper")
 require 'postfix_admin/runner'
 
@@ -15,17 +17,21 @@ describe PostfixAdmin::Runner do
     capture(:stdout){ Runner.start(["summary", "example.com"]) }.should =~ /\[Summary of example.com\]/
   end
 
+  it "schemes" do
+    capture(:stdout){ Runner.start(["schemes"]) }.should =~ /CLEARTEXT/
+  end
+
   describe "show" do
     it "shows information of example.com" do
      capture(:stdout){ Runner.start(["show"]) }.should =~ /example.com\s+1\s+\/\s+30\s+1\s+\/\s+30\s+100/
     end
 
     it "shows information of admin@example.com" do
-      capture(:stdout){ Runner.start(["show"]) }.should =~ /admin@example.com\s+1\s+password/
+      capture(:stdout){ Runner.start(["show"]) }.should =~ /admin@example.com\s+1\s+YES/
     end
 
     it "show the detail of example.com" do
-      capture(:stdout){ Runner.start(["show", "example.com"]) }.should =~ /user@example.com\s+100\s+password/
+      capture(:stdout){ Runner.start(["show", "example.com"]) }.should =~ /user@example.com\s+100\s+YES/
     end
 
     it "when no admins, no aliases and no addresses" do
@@ -36,10 +42,18 @@ describe PostfixAdmin::Runner do
       out.should =~ /No aliases/
     end
 
+    it "shows information of an admin" do
+      capture(:stdout){  Runner.start(["show", "admin@example.com"]) }.should =~ /admin@example.com/
+    end
+
     it "shows information of an account" do
       capture(:stdout){  Runner.start(["show", "user@example.com"]) }.should =~ /user@example.com/
     end
 
+    it "shows information of an alias" do
+      capture(:stdout){  Runner.start(["show", "alias@example.com"]) }.should =~ /alias@example.com/
+    end
+
     it "when no domains" do
       capture(:stdout){ Runner.start(['delete_domain', 'example.com']) }.should =~ EX_DELETED
       capture(:stdout){ Runner.start(['delete_domain', 'example.org']) }.should =~ EX_DELETED
@@ -83,6 +97,7 @@ describe PostfixAdmin::Runner do
   describe "account_passwd" do
     it "can change password of an account" do
       capture(:stdout){ Runner.start(['account_passwd', 'user@example.com', 'new_password']) }.should =~ /successfully changed/
+      Mailbox.find('user@example.com').password.should == CRAM_MD5_NEW_PASS
     end
 
     it "can not use too short password (< 5)" do
@@ -110,8 +125,35 @@ describe PostfixAdmin::Runner do
   end
 
   describe "add_admin" do
+    before do
+      @args = ['add_admin', 'admin@example.jp', 'password']
+    end
+
     it "can add an new admin" do
-      capture(:stdout){ Runner.start(['add_admin', 'admin@example.jp', 'password']) }.should =~ EX_REGISTERED
+      capture(:stdout){ Runner.start(@args) }.should =~ EX_REGISTERED
+    end
+
+    describe "scheme option" do
+      it "--scheme does not show error" do
+        capture(:stderr){ Runner.start(@args + ['--scheme', 'CRAM-MD5']) }.should == ""
+        Admin.find('admin@example.jp').password.should == CRAM_MD5_PASS
+      end
+
+      it "--shceme can resister admin" do
+        capture(:stdout){ Runner.start(@args + ['--scheme', 'CRAM-MD5']) }.should =~ EX_REGISTERED
+      end
+
+      it "-s does not show error" do
+        capture(:stderr){ Runner.start(@args + ['-s', 'CRAM-MD5']) }.should == ""
+      end
+
+      it "-s can resister admin" do
+        capture(:stdout){ Runner.start(@args + ['-s', 'CRAM-MD5']) }.should =~ EX_REGISTERED
+      end
+
+      it "-s require argument" do
+        capture(:stderr){ Runner.start(@args + ['-s']) }.should =~ /Specify password scheme/
+      end
     end
 
     it "can use long password" do
@@ -119,11 +161,11 @@ describe PostfixAdmin::Runner do
     end
 
     it "--super option" do
-      capture(:stdout){ Runner.start(['add_admin', 'admin@example.jp', 'password', '--super']) }.should =~ /registered as a super admin/
+      capture(:stdout){ Runner.start(@args + ['--super']) }.should =~ /registered as a super admin/
     end
 
-    it "-s (--super) option" do
-      capture(:stdout){ Runner.start(['add_admin', 'admin@example.jp', 'password', '-s']) }.should =~ /registered as a super admin/
+    it "-S (--super) option" do
+      capture(:stdout){ Runner.start(@args + ['-S']) }.should =~ /registered as a super admin/
     end
   end
 
@@ -133,11 +175,11 @@ describe PostfixAdmin::Runner do
     end
 
     it "can edit limitations of domain" do
-      capture(:stdout){ Runner.start(['edit_domain', 'example.com', '--aliases', '40', '--mailboxes', '40', '--maxquota', '400']) }.should =~ /Successfully updated/
+      capture(:stdout){ Runner.start(['edit_domain', 'example.com', '--aliases', '40', '--mailboxes', '40', '--maxquota', '400']) }.should =~ EX_UPDATED
     end
 
     it "aliases options -a, -m, -q" do
-      capture(:stdout){ Runner.start(['edit_domain', 'example.com', '-a', '40', '-m', '40', '-m', '400']) }.should =~ /Successfully updated/
+      capture(:stdout){ Runner.start(['edit_domain', 'example.com', '-a', '40', '-m', '40', '-m', '400']) }.should =~ EX_UPDATED
     end
 
     it "can not use unknown domain" do
@@ -146,18 +188,45 @@ describe PostfixAdmin::Runner do
   end
 
   describe "edit_account" do
+    before do
+      @args = ['edit_account', 'user@example.com']
+    end
+
     it "when no options, shows usage" do
-      capture(:stderr){ Runner.start(['edit_account', 'user@example.com']) }.should =~ /Use one or more options/
+      capture(:stderr){ Runner.start(@args) }.should =~ /Use one or more options/
     end
 
     it "can edit quota limitation" do
-      output = capture(:stdout){ Runner.start(['edit_account', 'user@example.com', '--quota', '50'])}
-      output.should =~ /Successfully updated/
+      output = capture(:stdout){ Runner.start(@args + ['--quota', '50'])}
+      output.should =~ EX_UPDATED
       output.should =~ /Quota/
     end
 
     it "can use alias -q option" do
-      capture(:stdout){ Runner.start(['edit_account', 'user@example.com', '-q', '50'])}.should =~ /Successfully updated/
+      capture(:stdout){ Runner.start(@args + ['-q', '50'])}.should =~ EX_UPDATED
+    end
+
+    it "-q option require an argment" do
+      capture(:stderr){ Runner.start(@args + ['-q'])}.should_not == ""
+    end
+
+    it "can update name using --name option" do
+      capture(:stdout){ Runner.start(@args + ['--name', 'Hitoshi Kurokawa'])}.should =~ EX_UPDATED
+      Mailbox.find('user@example.com').name.should == 'Hitoshi Kurokawa'
+    end
+
+    it "can update name using -n option" do
+      capture(:stdout){ Runner.start(@args + ['-n', 'Hitoshi Kurokawa'])}.should =~ EX_UPDATED
+      Mailbox.find('user@example.com').name.should == 'Hitoshi Kurokawa'
+    end
+
+    it "-n option supports Japanese" do
+      capture(:stdout){ Runner.start(@args + ['-n', '黒川　仁'])}.should =~ EX_UPDATED
+      Mailbox.find('user@example.com').name.should == '黒川　仁'
+    end
+
+    it "-n option require an argument" do
+      capture(:stderr){ Runner.start(@args + ['-n'])}.should_not == ""
     end
   end
 
@@ -178,8 +247,68 @@ describe PostfixAdmin::Runner do
     capture(:stdout){ Runner.start(['delete_account', 'user2@example.com']) }.should =~ EX_DELETED
   end
 
-  it "add_account can use long passwrod" do
-    capture(:stdout){ Runner.start(['add_account', 'user2@example.com', '9c5e77f2da26fc03e9fa9e13ccd77aeb50c85539a4d90b70812715aea9ebda1d']) }.should =~ EX_REGISTERED
+  describe "add_account" do
+    before do
+      @user = 'user2@example.com'
+      @args = ['add_account', @user, 'password']
+      @name = 'Hitoshi Kurokawa'
+    end
+
+    it "default scheme (CRAM-MD5) is applied" do
+      capture(:stdout){ Runner.start(@args) }.should =~ /scheme: CRAM-MD5/
+      Mailbox.find('user2@example.com').password.should == CRAM_MD5_PASS
+    end
+
+    it "add_account can use long password" do
+      capture(:stdout){ Runner.start(['add_account', 'user2@example.com', '9c5e77f2da26fc03e9fa9e13ccd77aeb50c85539a4d90b70812715aea9ebda1d']) }.should =~ EX_REGISTERED
+    end
+
+    describe "name option" do
+      it "--name options does not raise error" do
+        capture(:stderr){ Runner.start(@args + ['--name', @name]) }.should == ""
+      end
+
+      it "-n options does not raise error" do
+        capture(:stderr){ Runner.start(@args + ['-n', @name]) }.should == ""
+      end
+
+      it "require an argument" do
+        capture(:stderr){ Runner.start(@args + ['-n']) }.should_not == ""
+      end
+
+      it "can change full name" do
+        Runner.start(@args + ['-n', @name])
+        Mailbox.find(@user).name.should == @name
+      end
+
+      it "can use Japanese" do
+        capture(:stderr){ Runner.start(@args + ['-n', '黒川　仁']) }.should == ""
+        Mailbox.find(@user).name.should == '黒川　仁'
+      end
+    end
+
+    describe "scheme" do
+      it "--scheme require argument" do
+        capture(:stderr){ Runner.start(@args + ['--scheme']) }.should =~ /Specify password scheme/
+    end
+
+      it "can use CRAM-MD5 using --scheme" do
+        capture(:stdout){ Runner.start(@args + ['--scheme', 'CRAM-MD5']) }.should =~ EX_REGISTERED
+        Mailbox.find('user2@example.com').password.should == CRAM_MD5_PASS
+      end
+
+      it "can use CRAM-MD5 using -s" do
+        capture(:stdout){ Runner.start(@args + ['-s', 'CRAM-MD5']) }.should =~ EX_REGISTERED
+        Mailbox.find('user2@example.com').password.should == CRAM_MD5_PASS
+      end
+
+      it "can use MD5-CRYPT using -s" do
+        result = capture(:stdout){ Runner.start(@args + ['-s', 'MD5-CRYPT']) }
+        result.should =~ EX_REGISTERED
+        result.should =~ /scheme: MD5-CRYPT/
+        Mailbox.find('user2@example.com').password.should =~ EX_MD5_CRYPT
+      end
+    end
   end
 
   it "add and delete methods" do
@@ -191,4 +320,16 @@ describe PostfixAdmin::Runner do
     lambda { Runner.start(['add_account', 'user2@example.net', 'password']) }.should_not raise_error
     lambda { Runner.start(['delete_domain', 'example.net']) }.should_not raise_error
   end
+
+  describe "dump" do
+    it "does not raise error" do
+      capture(:stderr){ Runner.start(['dump']) }.should == ""
+    end
+
+    it "all data" do
+      result = capture(:stdout){ Runner.start(['dump']) }
+      result.should =~ /Domains/
+    end
+  end
+
 end

data/spec/spec_helper.rb

@@ -4,9 +4,11 @@ $:.unshift(File.join(File.dirname(__FILE__), "..", "lib"))
 require 'postfix_admin'
 require 'postfix_admin/cli'
 
-
 include PostfixAdmin
 
+# CRAM-MD5
+SAMPLE_PASSWORD = "9186d855e11eba527a7a52ca82b313e180d62234f0acc9051b527243d41e2740"
+
 # [fixtures]
 # Domain:
 #  ALL
@@ -28,6 +30,7 @@ def config_initialize
 end
 
 def db_clear
+  ::PostfixAdmin::Config.all.destroy
   DomainAdmin.all.destroy
   Mailbox.all.destroy
   Alias.all.destroy
@@ -35,7 +38,7 @@ def db_clear
   Admin.all.destroy
 end
 
-def create_domain(domain_name)
+def create_domain(domain_name, active=true)
   domain = Domain.new
   domain.attributes = {
     :domain_name  => domain_name,
@@ -43,69 +46,88 @@ def create_domain(domain_name)
     :maxaliases   => 30,
     :maxmailboxes => 30,
     :maxquota     => 100,
+    :active       => active
   }
   domain.save
 end
 
-def db_initialize
-  db_clear
+def create_alias_base(address, goto, active)
+  Alias.new.attributes = {
+    :address  => address,
+    :goto     => goto,
+    :active   => active
+  }
+end
 
-  create_domain('ALL')
-  create_domain('example.com')
-  create_domain('example.org')
+def create_alias(address, active=true)
+  create_alias_base(address, 'goto@example.jp', active)
+end
+
+def create_mailbox_alias(address, active=true)
+  create_alias_base(address, address, active)
+end
+
+def create_mailbox(address, in_path=nil, active=true)
+  path = in_path || "#{address.split('@').last}/#{address}/"
+  Mailbox.new.attributes = {
+    :username => address,
+    :password => SAMPLE_PASSWORD,
+    :name     => '',
+    :maildir  => path,
+    :quota    => 100 * KB_TO_MB,
+    # :local_part => user,
+    :active  => active
+  }
+end
 
-  username = "admin@example.com"
+def create_admin(username, active=true)
   admin = Admin.new
   admin.attributes = {
     :username => username,
-    :password => 'password',
+    :password => SAMPLE_PASSWORD,
+    :active => active
   }
   admin.save
+  admin
+end
 
-  domain = Domain.find('example.com')
-  domain.admins << admin
-  domain.save
+class ::PostfixAdmin::Mailbox
+  property :local_part, String
+end
+
+def db_initialize
+  db_clear
 
-  all_admin = Admin.new
-  all_admin.attributes = {
-    :username => 'all@example.com',
-    :password => 'password',
+  config = ::PostfixAdmin::Config.new
+  config.attributes = {
+    :id    => 1,
+    :name  => "version",
+    :value => "740"
   }
-  all_admin.save
+  config.save
+
+  create_domain('ALL')
+  create_domain('example.com')
+  create_domain('example.org')
 
+  all_admin = create_admin('all@example.com')
   all_domain = Domain.find('ALL')
   all_domain.admins << all_admin
-  all_domain.save
-
-  address = "user@example.com"
-  mail_alias = Alias.new
-  mail_alias.attributes = {
-    :address  => address,
-    :goto     => address,
-  }
-  domain.aliases << mail_alias
 
-  forward = Alias.new
-  forward.attributes = {
-    :address  => 'alias@example.com',
-    :goto     => 'goto@example.jp',
-  }
-  domain.aliases << forward
+  unless  all_domain.save
+    raise "Could not save all_domain"
+  end
 
-  domain.save
+  admin = create_admin('admin@example.com')
+  domain = Domain.find('example.com')
+  domain.admins    << admin
+  domain.aliases   << create_alias('alias@example.com')
+  domain.aliases   << create_alias('user@example.com')
+  domain.mailboxes << create_mailbox('user@example.com')
 
-  path = "example.com/user@example.com/"
-  mailbox = Mailbox.new
-  mailbox.attributes = {
-    :username => address,
-    :password => 'password',
-    :name     => '',
-    :maildir  => path,
-    :quota    => 100 * KB_TO_MB,
-    # :local_part => user,
-  }
-  domain.mailboxes << mailbox
-  domain.save
+  unless domain.save
+    raise "Could not save domain"
+  end
 end
 
 DataMapper.setup(:default, 'sqlite::memory:')
@@ -126,8 +148,12 @@ module PostfixAdmin
   end
 end
 
+CRAM_MD5_PASS = '9186d855e11eba527a7a52ca82b313e180d62234f0acc9051b527243d41e2740'
+CRAM_MD5_NEW_PASS = '820de4c70957274d41111c5fbcae4c87240c9f047fc56f3e720f103571be6cbc'
 EX_DELETED    = /successfully deleted/
 EX_REGISTERED = /successfully registered/
+EX_UPDATED    = /Successfully updated/
+EX_MD5_CRYPT  = /^\$1\$[\.\/0-9A-Za-z]{8}\$[\.\/0-9A-Za-z]{22}$/
 
 RSpec.configure do |config|
   config.before do