gliffy 0.0.8 → 0.0.9

checksums.yaml

@@ -1,15 +1,7 @@
 ---
-!binary "U0hBMQ==":
-  metadata.gz: !binary |-
-    YzU3NGY4YjYxMzMzMTg4ZmUwMjBmYWE4Y2E5NWQ3MGUwMzgwNWU1Nw==
-  data.tar.gz: !binary |-
-    YzhkNzViNDI3ZWMyMWFlZWI0NzM5MjI5MjFkZjUzYjk5MzJkNzhmOQ==
+SHA1:
+  metadata.gz: d08e7fc8875943cfbb8dc56185f6162cc3ea967b
+  data.tar.gz: df6d8c71405d5fc7ad14523bda869bd45ed20f5e
 SHA512:
-  metadata.gz: !binary |-
-    YmU2NDFiMWYwZGZiMDRiMmU3YjZhNDE2MjY3OWI2MzYwOTVmMzE0NWI3MjYz
-    YTEzM2NkYTgzY2E0ZDIwOWUwNGI5NjE3Nzk2NDIxMTA4ZTcyZDI0YzE0NTBk
-    OTg2OTAwN2ZkNzVkMjM4ZmYyMmIzYzE2OTVmNzBjMTNlYTE1ZjM=
-  data.tar.gz: !binary |-
-    NTE2NDVlNzQ5MTBlNDVhYTA3ZTQ0ZGJkOTMyYTQ4Yzk0NmMxY2NiMjkzZDA3
-    ZjNjYmJhZTA3ZmM2NGMwNDFlYjE2ZjJhMWE2OTVhY2M3ZmExYjAyYWJjNDQy
-    Y2M2MGFlMjgxODViNzVhYmIzNDRkZWJjYWYyNzVhMzgzZGE2NWE=
+  metadata.gz: 2ca1cd73e086d17d17638300d2440d64c88b30035be88da258e803adcabaf084cf2c26d83bf51189ee3b85ee2f546256b714e8857babbe88003ee04ceab85f08
+  data.tar.gz: 6d204415352c176034d4c1a69ff7e5db4fef0b51a05058355d4919c95e46b9c69afd270fe95d423fac776ef97e3c23a1ec7726dcd145295941261c5e63400b86

data/LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) [year] [fullname]
+Copyright (c) 2013 Konstantin Burnaev
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of
 this software and associated documentation files (the "Software"), to deal in

data/README.md

@@ -24,6 +24,9 @@ Basic usage
 
     doc.delete
 
+    doc.public?
+    doc.public = false
+
 #### Download document as PNG
 
     doc.png.full
@@ -47,3 +50,22 @@ Basic usage
     root.folders[1].path
 
     folder.delete
+
+    folder.users
+    folder.grant_access(user)
+    folder.revoke_access(user)
+
+### Users
+
+    account.users
+
+    account.users[0].username
+    account.users[1].email
+
+    account.create_user("john-smith")
+
+    user.email = "new-email@test.com"
+    user.password = "new-password"
+    user.admin = true
+
+    user.accessible_folders

data/lib/gliffy.rb

@@ -11,6 +11,7 @@ require 'gliffy/document/presentation/png'
 require 'gliffy/document/presentation/svg'
 require 'gliffy/document/presentation/xml'
 require 'gliffy/folder'
+require 'gliffy/user'
 
 require 'gliffy/oauth/helper'
 

data/lib/gliffy/account.rb

@@ -24,6 +24,10 @@ module Gliffy
       @root ||= load_root
     end
 
+    def users
+      @users ||= load_users
+    end
+
     def document(document_id)
       response = api.get("/accounts/#{id}/documents/#{document_id}/meta-data.xml",
                          :action => 'get')
@@ -34,6 +38,30 @@ module Gliffy
       )
     end
 
+    def create_user(username)
+      if username =~ /\s/ then
+        raise ArgumentError.new(username)
+      end
+
+      if users.select { |u| u.username == username }.length > 0 then
+        raise ArgumentError.new(username)
+      end
+
+      api.create_user(username)
+      users.select { |u| u.username == username }.first
+    end
+
+    # observer callback
+    def update(event, target)
+      case event
+      when :user_deleted
+        @users = @users.delete_if { |element| element == target }
+        target.delete_observer(self)
+      else
+        raise ArgumentError.new(event)
+      end
+    end
+
     private
 
     def initialize(api, params)
@@ -55,5 +83,15 @@ module Gliffy
         response.node("/g:response/g:folders/g:folder")
       )
     end
+
+    def load_users
+      api.get_users(id)
+        .nodes('//g:user')
+        .map { |n| load_user n }
+    end
+
+    def load_user(node)
+      Gliffy::User.load(self, node)
+    end
   end
 end

data/lib/gliffy/api/facade.rb

@@ -45,7 +45,36 @@ module Gliffy
 
       def get_folders(account_id)
         get("/accounts/#{account_id}/folders.xml",
-            :action => 'get')
+            :action => "get")
+      end
+
+      def folders_accessible_to_user(username)
+        get("/accounts/#{account_id}/users/#{username}/folders.xml",
+            :action => "get")
+      end
+
+      def users_with_access_to_folder(path)
+        get("/accounts/#{account_id}/folders/#{escape_path path}/users.xml",
+            :action => "get")
+      end
+
+      def grant_access_to_folder(username, path)
+        post("/accounts/#{account_id}/folders/#{escape_path path}/users/#{username}.xml",
+             :action => "update",
+             :read => "true",
+             :write => "true")
+      end
+
+      def revoke_access_to_folder(username, path)
+        post("/accounts/#{account_id}/folders/#{escape_path path}/users/#{username}.xml",
+             :action => "update",
+             :read => "false",
+             :write => "false")
+      end
+
+      def get_users(account_id)
+        get("/accounts/#{account_id}/users.xml",
+            :action => "get")
       end
 
       def update_document_metadata(document_id, name, shared)
@@ -109,6 +138,38 @@ module Gliffy
              :action => "create")
       end
 
+      def create_user(username)
+        post("/accounts/#{account_id}/users.xml",
+             :action => "create",
+             :userName => username)
+      end
+
+      def update_user(username, email, password, is_admin)
+        params = {
+          :action => "update"
+        }
+
+        if not email.nil?
+          params[:email] = email
+        end
+
+        if not password.nil?
+          params[:password] = password
+        end
+
+        if not is_admin.nil?
+          params[:admin] = is_admin ? "true" : "false"
+        end
+
+        post("/accounts/#{account_id}/users/#{username}.xml",
+             params)
+      end
+
+      def delete_user(username)
+        post("/accounts/#{account_id}/users/#{username}.xml",
+             :action => "delete")
+      end
+
       private
 
       def handle_error(response)

data/lib/gliffy/document.rb

@@ -88,6 +88,11 @@ module Gliffy
       is_public
     end
 
+    def public=(value)
+      api.update_document_metadata(id, nil, value)
+      @is_public = value
+    end
+
     def api
       owner.api
     end

data/lib/gliffy/folder.rb

@@ -31,6 +31,20 @@ module Gliffy
       @is_deleted = false
     end
 
+    def users
+      api.users_with_access_to_folder(path)
+        .nodes("//g:user")
+        .map { |n| Gliffy::User.load(owner, n) }
+    end
+
+    def grant_access(user)
+      api.grant_access_to_folder(user.username, path)
+    end
+
+    def revoke_access(user)
+      api.revoke_access_to_folder(user.username, path)
+    end
+
     def parent=(parent)
       if path != parent.path + "/" + name then
         raise "Invalid parent"

data/lib/gliffy/user.rb

@@ -0,0 +1,57 @@
+module Gliffy
+  class User
+    include Observable
+
+    attr_reader :username, :email
+
+    def self.load(owner, node)
+      Gliffy::User.new(
+        owner,
+        node.string('g:username'),
+        node.string('g:email'),
+      )
+    end
+
+    def initialize(owner, username, email)
+      @owner = owner
+      @username = username
+      @email = email
+    end
+
+    def accessible_folders
+      api.folders_accessible_to_user(username)
+        .nodes("//g:folders/g:folder")
+        .map { |n| Gliffy::Folder.load(owner, n) }
+    end
+
+    def delete
+      api.delete_user(username)
+
+      changed
+      notify_observers :user_deleted, self
+    end
+
+    def email=(email)
+      api.update_user(username, email, nil, nil)
+      @email = email
+    end
+
+    def password=(value)
+      api.update_user(username, nil, value, nil)
+    end
+
+    def admin=(value)
+      api.update_user(username, nil, nil, value)
+    end
+
+    private
+
+    def api
+      owner.api
+    end
+
+    def owner
+      @owner
+    end
+  end
+end

data/spec/fixtures/folder-users.xml

@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<response xmlns="http://www.gliffy.com" success="true">
+    <users>
+        <user id="209">
+            <username>barney</username>
+            <email>barney@BurnsODyne.apiuser.gliffy.com</email>
+        </user>
+        <user id="205">
+            <username>bart</username>
+            <email>bart@BurnsODyne.apiuser.gliffy.com</email>
+        </user>
+        <user id="210">
+            <username>discostu</username>
+            <email>discostu@BurnsODyne.apiuser.gliffy.com</email>
+        </user>
+        <user id="204">
+            <username>homer</username>
+            <email>homer@BurnsODyne.apiuser.gliffy.com</email>
+        </user>
+    </users>
+</response>

data/spec/fixtures/user-folders.xml

@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<response xmlns="http://www.gliffy.com" success="true">
+    <folders>
+        <folder is-default="true">
+            <name>ROOT</name>
+            <path>ROOT</path>
+            <folder is-default="false">
+                <name>Tulacca Camp</name>
+                <path>ROOT/Tulacca Camp</path>
+            </folder>
+            <folder is-default="false">
+                <name>Simpsons Family</name>
+                <path>ROOT/Simpsons Family</path>
+                <folder is-default="false">
+                    <name>Homer's Secrets</name>
+                    <path>ROOT/Simpsons Family/Homer's Secrets</path>
+                </folder>
+            </folder>
+        </folder>
+    </folders>
+</response>

data/spec/fixtures/user-list.xml

@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<response xmlns="http://www.gliffy.com" success="true">
+    <users>
+        <user id="209">
+            <username>barney</username>
+            <email>barney@BurnsODyne.apiuser.gliffy.com</email>
+        </user>
+        <user id="205">
+            <username>bart</username>
+            <email>bart@BurnsODyne.apiuser.gliffy.com</email>
+        </user>
+        <user id="210">
+            <username>discostu</username>
+            <email>discostu@BurnsODyne.apiuser.gliffy.com</email>
+        </user>
+        <user id="204">
+            <username>homer</username>
+            <email>homer@BurnsODyne.apiuser.gliffy.com</email>
+        </user>
+    </users>
+</response>

data/spec/lib/gliffy/account_spec.rb

@@ -93,7 +93,7 @@ describe Gliffy::Account do
     subject(:root_folder) { account.root }
 
     it "is named ROOT" do
-      expect(root_folder.name).to eq "ROOT" 
+      expect(root_folder.name).to eq "ROOT"
     end
 
     it "has ROOT path" do
@@ -104,4 +104,116 @@ describe Gliffy::Account do
       expect(root_folder.owner).to be account
     end
   end
+
+  it "has a list of users" do
+    api
+      .stub(:get_users)
+      .and_return(Gliffy::API::Response.new(fixture("user-list")))
+
+    expect(account).to respond_to :users
+    expect(account.users).to be_instance_of Array
+    expect(account.users.length).to eq 4
+
+    account.users.each do |u|
+      expect(u).to be_instance_of Gliffy::User
+    end
+  end
+
+  it "delegates the task of fetching user list to API" do
+    api
+      .stub(:get_users)
+      .and_return(Gliffy::API::Response.new(fixture("user-list")))
+
+    account.users
+
+    expect(api).to have_received(:get_users)
+      .with(account_id)
+  end
+
+  it "allows to create a new user" do
+    expect(account).to respond_to :create_user
+  end
+
+  context "when creating new user" do
+    let(:username) { "USER" }
+    let(:user1) { double(Gliffy::User, :username => "A") }
+    let(:user2) { double(Gliffy::User, :username => "B") }
+    let(:user) { double(Gliffy::User, :username => username) }
+
+    before :each do
+      api.stub(:create_user)
+
+      account.stub(:users)
+        .and_return([user1, user2],
+                    [user1, user, user2])
+    end
+
+    it "calls REST API" do
+      account.create_user username
+
+      expect(api).to have_received(:create_user)
+        .with(username)
+    end
+
+    it "returns this user" do
+      new_user = account.create_user username
+      expect(new_user).to be user
+    end
+
+    context "when username contains a space" do
+      let(:username) { "US ER" }
+
+      it "throws an exception" do
+        expect { account.create_user username }.to raise_error
+      end
+    end
+
+    context "when username is already taken" do
+      let(:username) { "USER" }
+      let(:user) { double(Gliffy::User, { :username => username }) }
+
+      before :each do
+        account.stub(:users).and_return([user])
+      end
+
+      it "throws an exception" do
+        expect { account.create_user username }.to raise_error
+      end
+    end
+  end
+
+  context "when notified that user has been deleted" do
+    let(:user) do
+      api.stub(:get_users)
+        .and_return(Gliffy::API::Response.new(fixture("user-list")))
+
+      account.users[1]
+    end
+
+    before :each do
+      user.stub(:delete_observer).and_call_original
+    end
+
+    it "updates user list" do
+      original_length = account.users.length
+
+      account.update(:user_deleted, user)
+
+      expect(account.users.length).to eq original_length - 1
+      expect(account.users).to_not include user
+    end
+
+    it "stops listening to this user's' events" do
+      account.update(:user_deleted, user)
+
+      expect(user).to have_received(:delete_observer)
+        .with(account)
+    end
+  end
+
+  context "when receives an unknown event" do
+    it "throws an exception" do
+      expect { account.update(:unknown, double(Object)) }.to raise_error ArgumentError
+    end
+  end
 end

data/spec/lib/gliffy/api/facade_spec.rb

@@ -67,6 +67,44 @@ shared_examples_for "an API facade" do
     end
   end
 
+  it "allows to load a list of folders accessible to an user" do
+    expect(facade).to respond_to :folders_accessible_to_user
+  end
+
+  context "when loading list of folders accessible to an user" do
+    let(:username) { "USERNAME" }
+
+    before :each do
+      facade.stub(:get)
+      facade.folders_accessible_to_user username
+    end
+
+    it "sends GET request" do
+      expect(facade).to have_received(:get)
+        .with("/accounts/#{account_id}/users/#{username}/folders.xml",
+              hash_including(:action => "get"))
+    end
+  end
+
+  it "allows to load a list of users with access to a folder" do
+    expect(facade).to respond_to :users_with_access_to_folder
+  end
+
+  context "when loading list of users with access to a folder" do
+    let(:path) { "PATH" }
+
+    before :each do
+      facade.stub(:get)
+      facade.users_with_access_to_folder path
+    end
+
+    it "sends GET request" do
+      expect(facade).to have_received(:get)
+        .with("/accounts/#{account_id}/folders/#{path}/users.xml",
+              hash_including(:action => "get"))
+    end
+  end
+
   context "when loading a list of folders" do
     it "wraps own 'get' method" do
       account_id = 99
@@ -270,6 +308,128 @@ shared_examples_for "an API facade" do
     end
   end
 
+  it "has provides access to folder list" do
+    expect(facade).to respond_to :get_users
+  end
+
+  context "when loading user list" do
+    let(:response) { double(Gliffy::API::Response) }
+
+    it "sends GET request and returns its result" do
+      facade.stub(:get).and_return(response)
+
+      expect(facade.get_users(account_id)).to be response
+
+      expect(facade).to have_received(:get)
+        .with("/accounts/#{account_id}/users.xml",
+              hash_including(:action => "get"))
+    end
+  end
+
+  it "allows to create a new user" do
+    expect(facade).to respond_to :create_user
+  end
+
+  context "when creating user" do
+    let(:username) { "USER" }
+    let(:email) { "test@test.com" }
+
+    before :each do
+      facade.stub(:post)
+    end
+
+    it "sends POST request to API" do
+      facade.create_user username
+
+      expect(facade).to have_received(:post)
+        .with("/accounts/#{account_id}/users.xml",
+              hash_including(:action => "create" ))
+    end
+  end
+
+  it "allows to delete existing user" do
+    expect(facade).to respond_to :delete_user
+  end
+
+  context "when deleting user" do
+    let(:username) { "USER" }
+
+    before :each do
+      facade.stub(:post)
+    end
+
+    it "sends POST request to API" do
+      facade.delete_user username
+
+      expect(facade).to have_received(:post)
+        .with("/accounts/#{account_id}/users/#{username}.xml",
+              hash_including(:action => "delete"))
+    end
+  end
+
+  it "allows to update existing user" do
+    expect(facade).to respond_to :update_user
+  end
+
+  context "when updating user" do
+    let(:username) { "testuser" }
+    let(:new_email) { "new-email@test.com" }
+    let(:new_password) { "password" }
+    let(:new_admin) { true }
+
+    before :each do
+      facade.stub(:post)
+
+      facade.update_user(username, new_email, new_password, new_admin)
+    end
+
+    it "sends POST request to API" do
+      expect(facade).to have_received(:post)
+        .with("/accounts/#{account_id}/users/#{username}.xml",
+              hash_including(:action => "update"))
+    end
+
+    context "when all data is given" do
+      it "updates all fields" do
+        expect(facade).to have_received(:post)
+          .with(anything(),
+                hash_including(:email => new_email,
+                               :password => new_password,
+                               :admin => new_admin ? "true" : "false"))
+      end
+    end
+
+    context "when no email is given" do
+      let(:new_email) { nil }
+
+      it "doesnt try to update email" do
+        expect(facade).to have_received(:post)
+          .with(anything(),
+                hash_not_including(:email))
+      end
+    end
+
+    context "when no password is given" do
+      let(:new_password) { nil }
+
+      it "doesnt try to update password" do
+        expect(facade).to have_received(:post)
+          .with(anything(),
+                hash_not_including(:password))
+      end
+    end
+
+    context "when no admin flag is given" do
+      let(:new_admin) { nil }
+
+      it "doesnt try to update admin flag" do
+        expect(facade).to have_received(:post)
+          .with(anything(),
+                hash_not_including(:admin))
+      end
+    end
+  end
+
   context "when POST request returns an error" do
     let(:response) { Gliffy::API::Response.new(fixture("error-401")) }
 
@@ -301,6 +461,50 @@ shared_examples_for "an API facade" do
       expect { facade.get("/random_url", {}) }.to raise_error(Gliffy::API::Error)
     end
   end
+
+  it "allows to grant user access to folder" do
+    expect(facade).to respond_to :grant_access_to_folder
+  end
+
+  context "when user access is granted" do
+    let(:username) { "USERNAME" }
+    let(:path) { "ROOT/FOLDER/SUB" }
+
+    before :each do
+      facade.stub(:post)
+      facade.grant_access_to_folder(username, path)
+    end
+
+    it "sends POST request" do
+      expect(facade).to have_received(:post)
+        .with("/accounts/#{account_id}/folders/#{path}/users/#{username}.xml",
+              hash_including(:action => "update",
+                             :read => "true",
+                             :write => "true"))
+    end
+  end
+
+  it "allows to revoke user access to folder" do
+    expect(facade).to respond_to :revoke_access_to_folder
+  end
+
+  context "when user access is revoked" do
+    let(:username) { "USERNAME" }
+    let(:path) { "ROOT/FOLDER/SUB" }
+
+    before :each do
+      facade.stub(:post)
+      facade.revoke_access_to_folder(username, path)
+    end
+
+    it "sends POST request" do
+      expect(facade).to have_received(:post)
+        .with("/accounts/#{account_id}/folders/#{path}/users/#{username}.xml",
+              hash_including(:action => "update",
+                             :read => "false",
+                             :write => "false"))
+    end
+  end
 end
 
 describe Gliffy::API::Facade do

data/spec/lib/gliffy/document_spec.rb

@@ -121,7 +121,7 @@ describe Gliffy::Document do
   end
 
   context "when renamed" do
-    let (:new_name) { "NEW DOCUMENT NAME" }
+    let(:new_name) { "NEW DOCUMENT NAME" }
     before :each do
       api.stub(:update_document_metadata)
       document.rename new_name
@@ -137,6 +137,29 @@ describe Gliffy::Document do
     end
   end
 
+  it "can be made public or private" do
+    expect(document).to respond_to :public=
+  end
+
+  context "when public state changes" do
+    let(:new_shared) { false }
+
+    before :each do
+      api.stub(:update_document_metadata)
+
+      document.public = new_shared
+    end
+
+    it "calls REST API" do
+      expect(api).to have_received(:update_document_metadata)
+        .with(document_id, nil, new_shared)
+    end
+
+    it "updates local object" do
+      expect(document.public?).to eq new_shared
+    end
+  end
+
   it "can be moved" do
     expect(document).to respond_to :move
   end

data/spec/lib/gliffy/folder_spec.rb

@@ -29,6 +29,32 @@ describe Gliffy::Folder do
     expect(folder).to respond_to :documents
   end
 
+  it "has a list of users with access to this folder" do
+    expect(folder).to respond_to :users
+  end
+
+  describe "user list" do
+    before :each do
+      api.stub(:users_with_access_to_folder)
+        .and_return(Gliffy::API::Response.new(fixture("folder-users")))
+    end
+
+    subject(:users) { folder.users }
+
+    it "is an array" do
+      expect(users).to be_instance_of Array
+    end
+
+    it "has correct length" do
+      expect(users.length).to eq 4
+    end
+
+    it "contains correct user objects" do
+      expect(users[0].username).to eq "barney"
+      expect(users[3].username).to eq "homer"
+    end
+  end
+
   describe "root folder" do
     it "knows it is root" do
       expect(folder.root?).to be_true
@@ -86,7 +112,7 @@ describe Gliffy::Folder do
 
     it { should respond_to :length }
     it { should respond_to :[] }
-    it "has corrent length" do 
+    it "has corrent length" do
       expect(children.length).to eq 4
     end
   end
@@ -217,7 +243,7 @@ describe Gliffy::Folder do
         .with(folder.path + "/" + folder_name)
     end
 
-    it "returns a new folder" do
+    it "returns new folder" do
       new_folder = folder.create_folder(folder_name)
       expect(new_folder).to be_instance_of Gliffy::Folder
     end
@@ -392,4 +418,41 @@ describe Gliffy::Folder do
       expect(folder.deleted?).to be_true
     end
   end
+
+  describe "access rights" do
+    let(:username) { "USERNAME" }
+    let(:user) { double(Gliffy::User, :username => username ) }
+
+    it "can be granted" do
+      expect(folder).to respond_to :grant_access
+    end
+
+    context "when granting" do
+      before :each do
+        api.stub(:grant_access_to_folder)
+        folder.grant_access(user)
+      end
+
+      it "calls API" do
+        expect(api).to have_received(:grant_access_to_folder)
+          .with(username, folder.path)
+      end
+    end
+
+    it "can be revoked" do
+      expect(folder).to respond_to :revoke_access
+    end
+
+    context "when revoking" do
+      before :each do
+        api.stub(:revoke_access_to_folder)
+        folder.revoke_access(user)
+      end
+
+      it "calls API" do
+        expect(api).to have_received(:revoke_access_to_folder)
+          .with(username, folder.path)
+      end
+    end
+  end
 end

data/spec/lib/gliffy/user_spec.rb

@@ -0,0 +1,144 @@
+# -*- coding: utf-8-unix -*-
+require 'spec_helper'
+
+describe Gliffy::User do
+  let(:account_id) { 100 }
+  let(:api) { double(Gliffy::API::Facade) }
+  let(:account) { double(Gliffy::Account, :api => api, :id => account_id) }
+
+  subject(:user) do
+    Gliffy::User.load(
+      account,
+      Gliffy::API::Response.new(
+        fixture('user-list')
+      ).node("//g:users/g:user[1]")
+    )
+  end
+
+  it "has a username" do
+    expect(user).to respond_to :username
+    expect(user.username).to eq "barney"
+  end
+
+  it "has an email" do
+    expect(user).to respond_to :email
+    expect(user.email).to eq "barney@BurnsODyne.apiuser.gliffy.com"
+  end
+
+  it "has a list of folders accessible to it" do
+    expect(user).to respond_to :accessible_folders
+  end
+
+  describe "list of accessible folders" do
+    before :each do
+      api.stub(:folders_accessible_to_user)
+        .and_return(Gliffy::API::Response.new(fixture("user-folders")))
+    end
+
+    it "is a list of folder objects" do
+      expect(user.accessible_folders).to be_instance_of Array
+      expect(user.accessible_folders.length).to eq 1
+      expect(user.accessible_folders[0].name).to eq "ROOT"
+      expect(user.accessible_folders[0].folders.length).to eq 2
+    end
+
+    context "when list of accessible is loaded" do
+      before :each do
+        user.accessible_folders
+      end
+
+      it "is fetched from API" do
+        expect(api).to have_received(:folders_accessible_to_user)
+          .with(user.username)
+      end
+    end
+  end
+
+  it "can be deleted" do
+    expect(user).to respond_to :delete
+  end
+
+  context "when being deleted" do
+    let(:observer) { double(Object) }
+
+    before :each do
+      api.stub(:delete_user)
+
+      observer.stub(:update)
+      user.add_observer(observer)
+
+      user.delete
+    end
+
+    it "calls API" do
+      expect(api).to have_received(:delete_user)
+        .with(user.username)
+    end
+
+    it "notifies observers" do
+      expect(observer).to have_received(:update)
+        .with(:user_deleted, user)
+    end
+  end
+
+  it "allows to update email" do
+    expect(user).to respond_to :email=
+  end
+
+  context "when email is updated" do
+    let(:new_email) { "new-email@test.com" }
+
+    before :each do
+      api.stub(:update_user)
+
+      user.email = new_email
+    end
+
+    it "sends a request to API" do
+      expect(api).to have_received(:update_user)
+        .with(user.username, new_email, nil, nil)
+    end
+
+    it "updates local email value" do
+      expect(user.email).to eq new_email
+    end
+  end
+
+  it "allows to update password" do
+    expect(user).to respond_to :password=
+  end
+
+  context "when password is updated" do
+    let(:new_password) { "new-password" }
+
+    before :each do
+      api.stub(:update_user)
+
+      user.password = new_password
+    end
+
+    it "sends a request to API" do
+      expect(api).to have_received(:update_user)
+        .with(user.username, nil, new_password, nil)
+    end
+  end
+
+  it "allows to grant or revoke admin priviletes" do
+    expect(user).to respond_to :admin=
+  end
+
+  context "when admin flag is updated" do
+    let(:new_admin) { true }
+
+    before :each do
+      api.stub(:update_user)
+
+      user.admin = new_admin
+    end
+
+    it "sends a request to API" do
+      expect(api).to have_received(:update_user)
+        .with(user.username, nil, nil, new_admin)
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -6,7 +6,7 @@ SimpleCov.command_name "test:units"
 SimpleCov.start do
     add_filter '/spec/'
     add_filter '/lib/gliffy/oauth/'
-    coverage_dir '/reports/coverage'
+    coverage_dir './reports/coverage'
     minimum_coverage 100
     refuse_coverage_drop
 end

metadata

@@ -1,125 +1,139 @@
 --- !ruby/object:Gem::Specification
 name: gliffy
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
 platform: ruby
 authors:
 - Konstantin Burnaev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-13 00:00:00.000000000 Z
+date: 2013-11-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oauth
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '2.14'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '2.14'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: http_logger
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: guard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rb-readline
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: ci_reporter
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: A simple Gliffy REST API wrapper.
@@ -128,40 +142,45 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- lib/gliffy/api/error.rb
-- lib/gliffy/api/facade.rb
-- lib/gliffy/api/response.rb
+- lib/gliffy/folder.rb
 - lib/gliffy/document/presentation.rb
-- lib/gliffy/document/presentation/xml.rb
 - lib/gliffy/document/presentation/png.rb
+- lib/gliffy/document/presentation/xml.rb
 - lib/gliffy/document/presentation/svg.rb
-- lib/gliffy/oauth/helper.rb
+- lib/gliffy/api/error.rb
+- lib/gliffy/api/facade.rb
+- lib/gliffy/api/response.rb
+- lib/gliffy/account.rb
 - lib/gliffy/api.rb
-- lib/gliffy/folder.rb
+- lib/gliffy/user.rb
 - lib/gliffy/document.rb
-- lib/gliffy/account.rb
+- lib/gliffy/oauth/helper.rb
 - lib/gliffy.rb
 - spec/support/document_presentation_shared_examples.rb
+- spec/fixtures/documents-empty.xml
+- spec/fixtures/token.xml
 - spec/fixtures/folder.xml
-- spec/fixtures/document.xml
-- spec/fixtures/account.xml
 - spec/fixtures/error-401.xml
+- spec/fixtures/account.xml
+- spec/fixtures/document.xml
+- spec/fixtures/user-list.xml
+- spec/fixtures/user-folders.xml
+- spec/fixtures/folder-users.xml
 - spec/fixtures/documents.xml
-- spec/fixtures/documents-empty.xml
-- spec/fixtures/token.xml
-- spec/lib/gliffy/account_spec.rb
-- spec/lib/gliffy/api/facade_spec.rb
-- spec/lib/gliffy/api/response_spec.rb
-- spec/lib/gliffy/api/error_spec.rb
+- spec/spec_helper.rb
+- spec/lib/gliffy/user_spec.rb
+- spec/lib/gliffy/api_spec.rb
 - spec/lib/gliffy/document/presentation_spec.rb
 - spec/lib/gliffy/document/presentation/xml_spec.rb
-- spec/lib/gliffy/document/presentation/svg_spec.rb
 - spec/lib/gliffy/document/presentation/png_spec.rb
-- spec/lib/gliffy/folder_spec.rb
+- spec/lib/gliffy/document/presentation/svg_spec.rb
+- spec/lib/gliffy/api/response_spec.rb
+- spec/lib/gliffy/api/facade_spec.rb
+- spec/lib/gliffy/api/error_spec.rb
 - spec/lib/gliffy/document_spec.rb
-- spec/lib/gliffy/api_spec.rb
+- spec/lib/gliffy/account_spec.rb
+- spec/lib/gliffy/folder_spec.rb
 - spec/lib/gliffy_spec.rb
-- spec/spec_helper.rb
 - README.md
 - LICENSE
 homepage: https://github.com/bkon/gliffy
@@ -174,17 +193,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.1.5
+rubygems_version: 2.1.11
 signing_key: 
 specification_version: 4
 summary: Gliffy API client