postfix_admin 0.0.2 → 0.1.0

data/postfix_admin.gemspec

@@ -5,12 +5,13 @@ Gem::Specification.new do |gem|
   gem.add_dependency 'thor'
   gem.add_dependency 'data_mapper'
   gem.add_dependency 'dm-mysql-adapter'
+  gem.add_development_dependency 'dm-sqlite-adapter'
 
   gem.authors       = ["Hitoshi Kurokawa"]
   gem.email         = ["hitoshi@nextseed.jp"]
   gem.description   = %q{Command Line Tools of PostfixAdmin}
   gem.summary       = gem.description
-  gem.homepage      = ""
+  gem.homepage      = "https://github.com/krhitoshi/postfix_admin"
 
   gem.files         = `git ls-files`.split($\)
   gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }

data/spec/base_spec.rb

@@ -0,0 +1,218 @@
+require File.expand_path(File.dirname(__FILE__) + "/spec_helper")
+require 'postfix_admin/base'
+
+describe PostfixAdmin::Base do
+  before do
+    db_initialize
+    @base = PostfixAdmin::Base.new({'database' => 'sqlite::memory:'})
+  end
+
+  it "DEFAULT_CONFIG" do
+    PostfixAdmin::Base::DEFAULT_CONFIG.should == {
+        'database'  => 'mysql://postfix:password@localhost/postfix',
+        'aliases'   => 30,
+        'mailboxes' => 30,
+        'maxquota'  => 100
+    }
+  end
+
+  it "#address_split" do
+    @base.address_split('user@example.com').should == ['user', 'example.com']
+  end
+
+  it "#new without config" do
+    lambda { PostfixAdmin::Base.new }.should raise_error(ArgumentError)
+  end
+
+  it "#new without database config" do
+    lambda { PostfixAdmin::Base.new({}) }.should raise_error(ArgumentError)
+  end
+
+  it "Default configuration should be correct" do
+    @base.config[:aliases].should == 30
+    @base.config[:mailboxes].should == 30
+    @base.config[:maxquota].should == 100
+    @base.config[:mailbox_quota].should == 100 * 1024 * 1000
+  end
+
+  it "#domain_exist?" do
+    Domain.exist?('example.com').should be_true
+  end
+
+  it "#alias_exist?" do
+    Alias.exist?('user@example.com').should be_true
+    Alias.exist?('unknown@example.com').should be_false
+  end
+
+  it "#mailbox_exist?" do
+    Mailbox.exist?('user@example.com').should be_true
+    Mailbox.exist?('unknown@example.com').should be_false
+  end
+
+  it "#admin_exist?" do
+    Admin.exist?('admin@example.com').should be_true
+    Admin.exist?('unknown_admin@example.com').should be_false
+  end
+
+  describe "#add_domain" do
+    it "can add a new domain" do
+      num_domains = Domain.count
+      @base.add_domain('example.net')
+      (Domain.count - num_domains).should be(1)
+    end
+
+    it "can not add exist domain" do
+      lambda{ @base.add_domain('example.com') }.should raise_error Error
+    end
+
+    it "can not add invalid domain" do
+      lambda{ @base.add_domain('localhost') }.should raise_error Error
+    end
+  end
+
+  describe "#add_account" do
+    it "can add a new account" do
+      num_mailboxes = Mailbox.count
+      num_aliases   = Alias.count
+      @base.add_account('new_user@example.com', 'password')
+      (Mailbox.count - num_mailboxes).should be(1)
+        (Alias.count - num_aliases).should be(1)
+    end
+
+    it "can not add account which hsas invalid address" do
+      lambda{ @base.add_account('invalid.example.com', 'password') }.should raise_error Error
+    end
+
+    it "can not add account for unknown domain" do
+      lambda{ @base.add_account('user@unknown.example.com', 'password') }.should raise_error Error
+    end
+
+    it "can not add account which has same address as exist mailbox" do
+      lambda{ @base.add_account('user@example.com', 'password') }.should raise_error Error
+    end
+
+    it "can not add account which has same address as exist alias" do
+      lambda{ @base.add_account('alias@example.com', 'password') }.should raise_error Error
+    end
+  end
+
+  describe "#add_admin" do
+    it "can add an new admin" do
+      num_admins = Admin.count
+      @base.add_admin('admin@example.net', 'password')
+      Admin.exist?('admin@example.net').should be_true
+      (Admin.count - num_admins).should be(1)
+    end
+
+    it "can not add exist admin" do
+      lambda{ @base.add_admin('admin@example.com', 'password') }.should raise_error Error
+    end
+  end
+
+  describe "#add_admin_domain" do
+    it "#add_admin_domain" do
+      @base.add_admin_domain('admin@example.com', 'example.org')
+      Admin.find('admin@example.com').domains.find do |domain|
+        domain.domain_name == 'example.org'
+      end.should be_true
+    end
+
+    it "can not add unknown domain for an admin" do
+      lambda{ @base.add_admin_domain('admin@example.com', 'unknown.example.com') }.should raise_error Error
+    end
+
+    it "can not add domain for unknown admin" do
+      lambda{ @base.add_admin_domain('unknown_admin@example.com', 'example.net') }.should raise_error Error
+    end
+
+    it "can not add a domain which the admin has already privileges for" do
+      lambda{ @base.add_admin_domain('admin@example.com', 'example.com') }.should raise_error Error
+    end
+  end
+
+  describe "#add_alias" do
+    it "can add a new alias" do
+      num_aliases   = Alias.count
+      lambda { @base.add_alias('new_alias@example.com', 'goto@example.jp') }.should_not raise_error
+      (Alias.count - num_aliases).should be(1)
+      Alias.exist?('new_alias@example.com').should be_true
+    end
+
+    it "can not add an alias which has a same name as a mailbox" do
+      lambda { @base.add_alias('user@example.com', 'goto@example.jp') }.should raise_error Error
+    end
+
+    it "can not add an alias which has a sama name as other alias" do
+      @base.add_alias('new_alias@example.com', 'goto@example.jp')
+      lambda { @base.add_alias('new_alias@example.com', 'goto@example.jp') }.should raise_error Error
+    end
+
+    it "can not add an alias of unknown domain" do
+      lambda { @base.add_alias('new_alias@unknown.example.com', 'goto@example.jp') }.should raise_error Error
+    end
+  end
+
+  describe "#delete_alias" do
+    it "can delete an alias" do
+      lambda{ @base.delete_alias('alias@example.com') }.should_not raise_error
+      Alias.exist?('alias@example.com').should be_false
+    end
+
+    it "can not delete mailbox" do
+      lambda{ @base.delete_alias('user@example.com') }.should raise_error Error
+    end
+
+    it "can not delete unknown alias" do
+      lambda{ @base.delete_alias('unknown@example.com') }.should raise_error Error
+    end
+  end
+
+  describe "#delete_domain" do
+    before do
+      @base.add_account('user2@example.com', 'password')
+      @base.add_account('user3@example.com', 'password')
+
+      @base.add_alias('alias2@example.com', 'goto2@example.jp')
+      @base.add_alias('alias3@example.com', 'goto3@example.jp')
+    end
+
+    it "can delete a domain" do
+      lambda{ @base.delete_domain('example.com') }.should_not raise_error
+
+      Domain.exist?('example.com').should be_false
+      Admin.exist?('admin@example.com').should be_false
+
+      Alias.all(:domain_name => 'example.com').count.should be(0)
+      Mailbox.all(:domain_name => 'example.com').count.should be(0)
+
+      DomainAdmin.all(:username => 'admin@example.com', :domain_name => 'example.com').count.should be(0)
+    end
+
+    it "can not delete unknown domain" do
+      lambda{ @base.delete_domain('unknown.example.com') }.should raise_error Error
+    end
+  end
+
+  describe "#delete_admin" do
+    it "can delete an admin" do
+      lambda{ @base.delete_admin('admin@example.com') }.should_not raise_error
+      Admin.exist?('admin@example.com').should be_false
+    end
+
+    it "can not delete unknown admin" do
+      lambda{ @base.delete_admin('unknown_admin@example.com') }.should raise_error Error
+    end
+  end
+
+  describe "#delete_account" do
+    it "can delete an account" do
+      lambda{ @base.delete_account('user@example.com') }.should_not raise_error
+      Mailbox.exist?('user@example.com').should be_false
+      Alias.exist?('user@example.com').should be_false
+    end
+
+    it "can not delete unknown account" do
+      lambda{ @base.delete_account('unknown@example.com') }.should raise_error Error
+    end
+  end
+end

data/spec/cli_spec.rb

@@ -0,0 +1,165 @@
+
+require 'postfix_admin/cli'
+
+describe PostfixAdmin::CLI do
+  before do
+    db_initialize
+    @cli = PostfixAdmin::CLI.new
+  end
+
+  it "#show_domain" do
+    lambda { @cli.show_domain }.should_not raise_error
+    capture(:stdout){ @cli.show_domain }.should_not =~ /ALL/
+  end
+
+  it "#show_summary" do
+    lambda { @cli.show_summary }.should_not raise_error
+    lambda { @cli.show_summary('unknown.example.com') }.should raise_error Error
+  end
+
+  it "#show_admin" do
+    lambda { @cli.show_admin }.should_not raise_error
+  end
+
+  it "#show_address" do
+    lambda { @cli.show_address('example.com') }.should_not raise_error
+    lambda { @cli.show_address('unknown.example.com') }.should raise_error Error
+  end
+
+  it "#show_admin_domain" do
+    lambda { @cli.show_admin_domain('admin@example.com') }.should_not raise_error
+  end
+
+  it "#show_alias" do
+    lambda { @cli.show_alias('example.com') }.should_not raise_error
+    lambda { @cli.show_alias('unknown.example.com') }.should raise_error Error
+  end
+
+  describe "#super_admin" do
+    it "enables super admin flag of an admin" do
+      lambda{ @cli.super_admin('admin@example.com', false) }.should_not raise_error
+      Admin.find('admin@example.com').super_admin?.should be_true
+    end
+
+    it "disable super admin flag of an admin" do
+      lambda{ @cli.super_admin('all@example.com', true) }.should_not raise_error
+      Admin.find('all@example.com').super_admin?.should be_false
+    end
+
+    it "can not user for unknown admin" do
+      lambda{ @cli.super_admin('unknown_admin@example.com', false) }.should raise_error Error
+    end
+  end
+
+  it "#change_admin_password" do
+    lambda { @cli.change_admin_password('admin@example.com', 'new_password') }.should_not raise_error
+    Admin.find('admin@example.com').password.should == 'new_password'
+    lambda { @cli.change_admin_password('unknown_admin@example.com', 'new_password') }.should raise_error Error
+
+    lambda { @cli.change_admin_password('admin@example.com', '1234') }.should raise_error ArgumentError
+  end
+
+  it "#change_account_password" do
+    lambda { @cli.change_account_password('user@example.com', 'new_password') }.should_not raise_error
+    Mailbox.find('user@example.com').password.should == 'new_password'
+    lambda { @cli.change_account_password('unknown@example.com', 'new_password') }.should raise_error Error
+    lambda { @cli.change_account_password('user@example.com', '1234') }.should raise_error ArgumentError
+  end
+
+  describe "#add_admin" do
+    it "can add a new admin" do
+      lambda { @cli.add_admin('new_admin@example.com', 'password') }.should_not raise_error
+      Admin.exist?('new_admin@example.com').should be_true
+    end
+
+    it "can not add exist admin" do
+      lambda { @cli.add_admin('admin@example.com', 'password') }.should raise_error Error
+    end
+
+    it "does not allow too short password (<5)" do
+      lambda { @cli.add_admin('admin@example.com', '1234') }.should raise_error ArgumentError
+    end
+  end
+
+  describe "#delete_admin" do
+    it "can delete an admin" do
+      lambda { @cli.delete_admin('admin@example.com') }.should_not raise_error
+      Admin.exist?('admin@example.com').should be_false
+    end
+
+    it "can not delete unknown admin" do
+      lambda { @cli.delete_admin('unknown_admin@example.com') }.should raise_error Error
+    end
+  end
+
+  it "#add_alias and #delete_alias" do
+    lambda { @cli.add_alias('user@example.com', 'goto@example.jp') }.should raise_error
+    lambda { @cli.delete_alias('user@example.com') }.should raise_error
+    lambda { @cli.delete_alias('unknown@example.com') }.should raise_error
+
+    lambda { @cli.add_alias('new_alias@example.com', 'goto@example.jp') }.should_not raise_error
+    Alias.exist?('new_alias@example.com').should be_true
+
+    lambda { @cli.delete_alias('new_alias@example.com') }.should_not raise_error
+    Alias.exist?('new_alias@example.com').should be_false
+  end
+
+  describe "#add_account" do
+    it "can add an account" do
+      lambda { @cli.add_account('new_user@example.com', 'password') }.should_not raise_error
+      Mailbox.exist?('new_user@example.com').should be_true
+      Alias.exist?('new_user@example.com').should be_true
+    end
+
+    it "can not add account of unknown domain" do
+      lambda { @cli.add_account('user@unknown.example.com', 'password') }.should raise_error Error
+    end
+
+    it "does not allow too short password (<5)" do
+      lambda { @cli.add_account('new_user@example.com', '1234') }.should raise_error ArgumentError
+    end
+  end
+
+  describe "#delete_accont" do
+    it "can delete an account" do
+      lambda { @cli.delete_account('user@example.com') }.should_not raise_error
+      Mailbox.exist?('user@example.com').should be_false
+      Alias.exist?('user@example.com').should be_false
+    end
+
+    it "can not delete unknown account" do
+      lambda { @cli.delete_account('unknown@example.com') }.should raise_error Error
+    end
+  end
+
+  it "add and delete methods" do
+    lambda { @cli.add_domain('example.net') }.should_not raise_error
+
+    lambda { @cli.add_admin('admin@example.net', 'password') }.should_not raise_error
+    lambda { @cli.add_admin_domain('admin@example.net', 'example.net') }.should_not raise_error
+
+    @cli.add_admin('admin2@example.net', 'password')
+    @cli.add_admin_domain('admin2@example.net', 'example.net')
+
+    @cli.add_admin('common@example.net', 'password')
+    @cli.add_admin_domain('common@example.net', 'example.com')
+    @cli.add_admin_domain('common@example.net', 'example.net')
+    lambda { @cli.delete_admin('common@example.net') }.should_not raise_error
+    Admin.exist?('common@example.net').should be_false
+
+    @cli.add_admin('common@example.net', 'password')
+    @cli.add_admin_domain('common@example.net', 'example.com')
+    @cli.add_admin_domain('common@example.net', 'example.net')
+
+    lambda { @cli.add_account('user1@example.net', 'password') }.should_not raise_error
+    lambda { @cli.add_account('user2@example.net', 'password') }.should_not raise_error
+
+    lambda { @cli.delete_domain('example.net') }.should_not raise_error
+    Admin.exist?('admin@example.net').should be_false
+    Admin.exist?('admin2@example.net').should be_false
+
+    Admin.exist?('common@example.net').should be_true
+    lambda { @cli.delete_admin('common@example.net') }.should_not raise_error
+    Admin.exist?('common@example.net').should be_false
+  end
+end

data/spec/models_spec.rb

@@ -0,0 +1,136 @@
+require File.expand_path(File.dirname(__FILE__) + "/spec_helper")
+require 'postfix_admin/models'
+
+describe PostfixAdmin::Admin do
+  before do
+    db_initialize
+  end
+
+  it ".exist?" do
+    Admin.exist?('admin@example.com').should === true
+    Admin.exist?('all@example.com').should === true
+    Admin.exist?('unknown@example.com').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
+  end
+
+  describe "#super_admin=" do
+    it "enable super admin flag" do
+      lambda{ Admin.find('all@example.com').super_admin = false }.should_not raise_error
+      Admin.find('all@example.com').super_admin?.should === false
+    end
+
+    it "should not delete 'ALL' domain" do
+      Admin.find('all@example.com').super_admin = false
+      Domain.exist?('ALL').should be_true
+    end
+
+    it "disable super admin flag" do
+      lambda{ Admin.find('admin@example.com').super_admin = true }.should_not raise_error
+      Admin.find('admin@example.com').super_admin?.should === true
+    end
+  end
+
+  describe "#has_domain?" do
+    it "returns true when the admin has privileges for the domain" do
+      Admin.find('admin@example.com').has_domain?('example.com').should === true
+    end
+
+    it "returns false when the admin does not have privileges for the domain" do
+      Admin.find('admin@example.com').has_domain?('example.org').should === false
+    end
+
+    it "returns false when unknown domain" do
+      Admin.find('admin@example.com').has_domain?('unknown.example.com').should === false
+    end
+
+    it "returns true when super admin and exist domain" do
+      Admin.find('all@example.com').has_domain?('example.com').should === true
+    end
+
+    it "returns false when super admin and unknown domain" do
+      Admin.find('all@example.com').has_domain?('unknown.example.com').should === false
+    end
+  end
+end
+
+describe PostfixAdmin::Domain do
+  before do
+    db_initialize
+    @base = PostfixAdmin::Base.new({'database' => 'sqlite::memory:'})
+  end
+
+  it ".exist?" do
+    Domain.exist?('example.com').should === true
+    Domain.exist?('example.org').should === true
+    Domain.exist?('unknown.example.com').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)
+      Domain.find('example.com').num_total_aliases.should be(1)
+    end
+
+    it "should increase one if you add an alias" do
+      @base.add_alias('new_alias@example.com', 'goto@example.jp')
+      Domain.num_total_aliases.should be(2)
+      Domain.find('example.com').num_total_aliases.should be(2)
+    end
+
+    it "should not increase if you add an account" do
+      @base.add_account('user2@example.com', 'password')
+      Domain.num_total_aliases.should be(1)
+      Domain.find('example.com').num_total_aliases.should be(1)
+    end
+
+    it ".num_total_aliases should not increase if you add an account and an aliase for other domain" do
+      @base.add_account('user@example.org', 'password')
+      Domain.num_total_aliases.should be(1)
+      Domain.find('example.com').num_total_aliases.should be(1)
+      @base.add_alias('new_alias@example.org', 'goto@example.jp')
+      Domain.num_total_aliases.should be(2)
+      Domain.find('example.com').num_total_aliases.should be(1)
+    end
+  end
+end
+
+describe PostfixAdmin::Mailbox do
+  before do
+    db_initialize
+  end
+
+  describe ".exist?" do
+    it "returns true for exist account (mailbox)" do
+      Mailbox.exist?('user@example.com').should === true
+    end
+
+    it "returns false for alias" do
+      Mailbox.exist?('alias@example.com').should === false
+    end
+
+    it "returns false for unknown account (mailbox)" do
+      Mailbox.exist?('unknown@unknown.example.com').should === false
+    end
+  end
+end
+
+describe PostfixAdmin::Alias do
+  before do
+    db_initialize
+  end
+
+  describe ".exist?" do
+    it "returns true when exist alias and account" do
+      Alias.exist?('user@example.com').should === true
+      Alias.exist?('alias@example.com').should === true
+    end
+
+    it "returns false when unknown alias" do
+      Alias.exist?('unknown@unknown.example.com').should === false
+    end
+  end
+end