Merge branch 'rails3' of gitlab.com:participa/noosfero into rails3

Evandro Jr
2 parents 441fbae9 0988fe9d
Showing 12 changed files with 124 additions and 64 deletions Show diff stats
app/controllers/public/account_controller.rb
app/controllers/public/profile_controller.rb
app/models/user.rb
app/views/content_viewer/_article_toolbar.html.erb
app/views/user/mailer/activation_code.html.erb
features/edit_profile.feature
features/login.feature
features/profile_domain.feature
features/signup.feature
features/step_definitions/web_steps.rb
plugins/video/lib/video_block.rb
test/functional/profile_controller_test.rb
@@ -17,6 +17,8 @@ class AccountController &lt; ApplicationController
     @user = User.find_by_activation_code(params[:activation_code]) if params[:activation_code]
     if @user and @user.activate
       @message = _("Your account has been activated, now you can log in!")
+      check_redirection
+      session[:join] = params[:join] unless params[:join].blank?
       render :action => 'login', :userlogin => @user.login
     else
       session[:notice] = _("It looks like you're trying to activate an account. Perhaps have already activated this account?")
@@ -35,6 +37,7 @@ class AccountController &lt; ApplicationController
     self.current_user ||= User.authenticate(params[:user][:login], params[:user][:password], environment) if params[:user]
     if logged_in?
+      check_join_in_community(self.current_user)
       if params[:remember_me] == "1"
         self.current_user.remember_me
         cookies[:auth_token] = { :value => self.current_user.remember_token , :expires => self.current_user.remember_token_expires_at }
@@ -91,6 +94,7 @@ class AccountController &lt; ApplicationController
           if session[:may_be_a_bot]
             return false unless verify_recaptcha :model=>@user, :message=>_('Captcha (the human test)')
           end
+          @user.community_to_join = session[:join]
           @user.signup!
           owner_role = Role.find_by_name('owner')
           @user.person.affiliate(@user.person, [owner_role]) if owner_role
@@ -101,6 +105,7 @@ class AccountController &lt; ApplicationController
           end
           if @user.activated?
             self.current_user = @user
+            check_join_in_community(@user)
             go_to_signup_initial_page
           else
             @register_pending = true
@@ -388,12 +393,6 @@ class AccountController &lt; ApplicationController
   end
   def go_to_initial_page
-    if params[:redirection]
-      session[:return_to] = @user.return_to
-      @user.return_to = nil
-      @user.save
-    end
-
     if params[:return_to]
       redirect_to params[:return_to]
     elsif environment.enabled?('allow_change_of_redirection_after_login')
@@ -444,4 +443,19 @@ class AccountController &lt; ApplicationController
       redirect_back_or_default(default)
     end
   end
+
+  def check_redirection
+    unless params[:redirection].blank?
+      session[:return_to] = @user.return_to
+      @user.update_attributes(:return_to => nil)
+    end
+  end
+
+  def check_join_in_community(user)
+    profile_to_join = session[:join]
+    unless profile_to_join.blank?
+     environment.profiles.find_by_identifier(profile_to_join).add_member(user.person)
+     session.delete(:join)
+    end
+  end
 end
@@ -3,7 +3,7 @@ class ProfileController &lt; PublicController
   needs_profile
   before_filter :check_access_to_profile, :except => [:join, :join_not_logged, :index, :add]
   before_filter :store_location, :only => [:join, :join_not_logged, :report_abuse, :send_mail]
-  before_filter :login_required, :only => [:add, :join, :join_not_logged, :leave, :unblock, :leave_scrap, :remove_scrap, :remove_activity, :view_more_activities, :view_more_network_activities, :report_abuse, :register_report, :leave_comment_on_activity, :send_mail]
+  before_filter :login_required, :only => [:add, :join, :leave, :unblock, :leave_scrap, :remove_scrap, :remove_activity, :view_more_activities, :view_more_network_activities, :report_abuse, :register_report, :leave_comment_on_activity, :send_mail]
   helper TagsHelper
@@ -97,21 +97,12 @@ class ProfileController &lt; PublicController
   end
   def join_not_logged
-    if request.post?
-      profile.add_member(user)
-      session[:notice] = _('%s administrator still needs to accept you as member.') % profile.name if profile.closed?
-      redirect_to_previous_location
+    session[:join] = profile.identifier
+
+    if user
+      redirect_to :controller => 'profile', :action => 'join'
     else
-      if user.memberships.include?(profile)
-        session[:notice] = _('You are already a member of %s.') % profile.name
-        redirect_to profile.url
-        return
-      end
-      if request.xhr?
-        render :layout => false
-      else
-        redirect_to profile.url
-      end
+      redirect_to :controller => '/account', :action => 'login'
     end
   end
@@ -63,6 +63,9 @@ class User &lt; ActiveRecord::Base
     self.person.preferred_domain && self.person.preferred_domain.name || self.environment.default_hostname(true)
   end
+  # virtual attribute used to stash which community to join on signup or login
+  attr_accessor :community_to_join
+
   class Mailer < ActionMailer::Base
     def activation_email_notify(user)
       user_email = "#{user.login}@#{user.email_domain}"
@@ -85,7 +88,8 @@ class User &lt; ActiveRecord::Base
         :activation_code => user.activation_code,
         :environment => user.environment.name,
         :url => user.environment.top_url,
-        :redirection => (true if user.return_to)
+        :redirection => (true if user.return_to),
+        :join => (user.community_to_join if user.community_to_join)
     end
     def signup_welcome_email(user)
@@ -10,7 +10,7 @@
     <% if @page != profile.home_page && !@page.has_posts?  && @page.allow_delete?(user) && !remove_content_button(:delete)%>
       <% content = content_tag( 'span', _('Delete') ) %>
-      <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'destroy', :id => @page}) %>
+      <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'destroy', :id => @page.id}) %>
       <% options = {:method => :post, :confirm => delete_article_message(@page)} %>
       <%= expirable_button @page, :delete, content, url, options %>
     <% end %>
@@ -19,9 +19,9 @@
       <% content = content_tag( 'span', _('Spread this') ) %>
       <% url = nil %>
       <% if profile.kind_of?(Person) %>
-        <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish', :id => @page }) %>
+        <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish', :id => @page.id }) %>
       <% elsif profile.kind_of?(Community) && environment.portal_community %>
-        <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish_on_portal_community', :id => @page }) %>
+        <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish_on_portal_community', :id => @page.id }) %>
       <% end %>
       <%= expirable_button @page, :spread, content, url if url %>
     <% end %>
 <%= _('Hi, %{recipient}!') % { :recipient => @recipient } %>
-<%= word_wrap(_('Welcome to %{environment}! To activate your account, follow the link: %{activation_url}') % { :environment => @environment, :activation_url => @url + url_for(:controller => :account, :action => :activate, :activation_code => @activation_code, :redirection => @redirection) }) %>
+<%= word_wrap(_('Welcome to %{environment}! To activate your account, follow the link: %{activation_url}') % { :environment => @environment, :activation_url => @url + url_for(:controller => :account, :action => :activate, :activation_code => @activation_code, :redirection => @redirection, :join => @join) }) %>
 <%= _("Greetings,") %>
@@ -12,7 +12,7 @@ Feature: edit profile
       | birth_date |
     When I go to joao's control panel
     And I follow "Edit Profile"
-    And I select "1980-11-15" from "profile_data_birth_date"
+    And I fill in "Birth date" with "1980-11-15"
     And I press "Save"
     Then I should not see "Birth date is invalid"
     And I should not see "Birth date is mandatory"
@@ -23,7 +23,7 @@ Feature: edit profile
       | birth_date |
     When I go to joao's control panel
     And I follow "Edit Profile"
-    And I select "1980-11-15" from "profile_data_birth_date"
+    And I fill in "Birth date" with "1980-11-15"
     And I press "Save"
     Then I should not see "Birth date is invalid"
     And I should not see "Birth date is mandatory"
@@ -207,3 +207,18 @@ Feature: login
       | Password | 123456 |
     When I press "Log in"
     Then I should be on joaosilva's control panel
+
+  Scenario: join community on login
+    Given the following users
+      | login | name |
+      | mariasilva | Maria Silva |
+    And the following communities
+       | name           | identifier    | owner     |
+       | Free Software  | freesoftware  | mariasilva |
+    And I am on /freesoftware
+    When I follow "Join"
+    And I fill in the following:
+      | Username / Email | joaosilva |
+      | Password | 123456 |
+    And I press "Log in"
+    Then "Joao Silva" should be a member of "Free Software"
@@ -9,11 +9,11 @@ Feature: domain for profile
       | joaosilva | Joao Silva |
     And the following communities
       | identifier       | name             | domain    |
-      | sample-community | Sample Community | localhost |
+      | sample-community | Sample Community | 127.0.0.1 |
     And the following blocks
       | owner            | type              |
       | joaosilva        | ProfileInfoBlock  |
-    And the environment domain is "127.0.0.1"
+    And the environment domain is "localhost"
     And "Joao Silva" is environment admin
     And "Joao Silva" is admin of "Sample Community"
@@ -29,8 +29,7 @@ Feature: domain for profile
   @selenium
   Scenario: access user control panel
     Given I am logged in as "joaosilva"
-    When I go to the homepage
-    And I follow "joaosilva"
+    When I follow "joaosilva"
     And I go to sample-community's homepage
     And I follow "Login"
     And I fill in "joaosilva" for "Username"
@@ -39,18 +38,13 @@ Feature: domain for profile
     And I follow "Control panel" within "div#user"
     Then I should see "Joao Silva" within "span.control-panel-title"
-  # Looking for page title is problematic on selenium since it considers the
-  # title to be invibible. Checkout some information about this:
-  #   * https://github.com/jnicklas/capybara/issues/863
-  #   * https://github.com/jnicklas/capybara/pull/953
   @selenium
   Scenario: access user page
     Given I am logged in as "joaosilva"
-    When I go to the homepage
-    And I follow "joaosilva"
+    When I follow "joaosilva"
     Then I should be on joaosilva's profile
     And I should see "Joao Silva" within any "h1"
-    #And the page title should be "Joao Silva"
+    And the page title should be "Joao Silva"
   Scenario: access community by domain
     Given I go to the search communities page
@@ -64,7 +58,7 @@ Feature: domain for profile
     Given I am on sample-community's homepage
     When I go to /something-that-does-not-exist
     And I follow "Go to the home page"
-    Then the page title should be "Sample Community - Colivre.net"
+    Then the page title should be "Sample Community"
   Scenario: Go to environment homepage after clicking on home button on not found page
     Given I am on the homepage
@@ -72,8 +66,3 @@ Feature: domain for profile
     And I follow "Go to the home page"
     Then I should be on the homepage
     And the page title should be "Colivre.net"
-
-  @selenium
-  Scenario: Compose link to administration with environment domain
-    Given I am logged in as "joaosilva"
-    Then I should see "Administration" linking to "http://127.0.0.1.*/admin"
@@ -250,3 +250,51 @@ Feature: signup
     And I fill in "Password" with "secret"
     And I press "Log in"
     Then I should be on the homepage
+
+  @selenium
+  Scenario: join community on signup
+    Given the following users
+      | login | name |
+      | mariasilva | Maria Silva |
+    And the following communities
+       | name           | identifier    | owner     |
+       | Free Software  | freesoftware  | mariasilva |
+    And feature "skip_new_user_email_confirmation" is disabled on environment
+    And I am on /freesoftware
+    When I follow "Join"
+    And I follow "New user"
+    And I fill in the following within ".no-boxes":
+      | e-Mail                | josesilva@example.com |
+      | Username              | josesilva             |
+      | Password              | secret                |
+      | Password confirmation | secret                |
+      | Full name             | José da Silva         |
+    And wait for the captcha signup time
+    And I press "Create my account"
+    And I go to josesilva's confirmation URL
+    And I fill in "Username" with "josesilva"
+    And I fill in "Password" with "secret"
+    And I press "Log in"
+    Then "José da Silva" should be a member of "Free Software"
+
+  @selenium
+  Scenario: join community on direct signup
+    Given the following users
+      | login | name |
+      | mariasilva | Maria Silva |
+    And the following communities
+       | name           | identifier    | owner      |
+       | Free Software  | freesoftware  | mariasilva |
+    And feature "skip_new_user_email_confirmation" is enabled on environment
+    And I am on /freesoftware
+    When I follow "Join"
+    And I follow "New user"
+    And I fill in the following within ".no-boxes":
+      | e-Mail                | josesilva@example.com |
+      | Username              | josesilva             |
+      | Password              | secret                |
+      | Password confirmation | secret                |
+      | Full name             | José da Silva         |
+    And wait for the captcha signup time
+    And I press "Create my account"
+    Then "José da Silva" should be a member of "Free Software"
@@ -118,9 +118,9 @@ end
 Then /^(?:|I )should see "([^"]*)" within any "([^"]*)"?$/ do |text, selector|
   if page.respond_to? :should
-    page.should have_css(selector, :content => text)
+    page.should have_css(selector, :text => text)
   else
-    assert page.has_css?(selector, :content => text)
+    assert page.has_css?(selector, :text => text)
   end
 end
 class VideoBlock < Block
+  attr_accessible :url, :width, :height
+  
   settings_items :url, :type => :string, :default => ""
   settings_items :width, :type => :integer, :default => 400
   settings_items :height, :type => :integer, :default => 315
@@ -474,17 +474,23 @@ class ProfileControllerTest &lt; ActionController::TestCase
     assert_equal "/profile/#{community.identifier}", @request.session[:previous_location]
   end
-  should 'redirect to location before login after join community' do
+  should 'redirect to login after user not logged asks to join a community' do
     community = Community.create!(:name => 'my test community')
-    @request.expects(:referer).returns("/profile/#{community.identifier}/to_go")
-    login_as(profile.identifier)
+    get :join_not_logged, :profile => community.identifier
-    post :join_not_logged, :profile => community.identifier
+    assert_equal community.identifier, @request.session[:join]
+    assert_redirected_to :controller => :account, :action => :login
+  end
-    assert_redirected_to "/profile/#{community.identifier}/to_go"
+  should 'redirect to join after user logged asks to join_not_logged a community' do
+    community = Community.create!(:name => 'my test community')
+
+    login_as(profile.identifier)
+    get :join_not_logged, :profile => community.identifier
-    assert_nil @request.session[:previous_location]
+    assert_equal community.identifier, @request.session[:join]
+    assert_redirected_to :controller => :profile, :action => :join
   end
   should 'show number of published events in index' do
@@ -1186,7 +1192,7 @@ class ProfileControllerTest &lt; ActionController::TestCase
     20.times {comment = fast_create(Comment, :source_id => article, :title => 'a comment', :body => 'lalala', :created_at => Time.now)}
     article.reload
     get :index, :profile => profile.identifier
-    assert_tag 'ul', :attributes => {:class => 'profile-wall-activities-comments'}, :children => {:count => 0 } 
+    assert_tag 'ul', :attributes => {:class => 'profile-wall-activities-comments'}, :children => {:count => 0 }
   end
   should "view more comments paginated" do
@@ -1212,7 +1218,7 @@ class ProfileControllerTest &lt; ActionController::TestCase
     20.times {fast_create(Scrap, :sender_id => profile.id, :receiver_id => profile.id, :scrap_id => scrap.id)}
     profile.reload
     get :index, :profile => profile.identifier
-    assert_tag 'ul', :attributes => {:class => 'profile-wall-activities-comments scrap-replies'}, :children => {:count => 0 } 
+    assert_tag 'ul', :attributes => {:class => 'profile-wall-activities-comments scrap-replies'}, :children => {:count => 0 }
   end
   should "view more replies paginated" do
@@ -1269,15 +1275,6 @@ class ProfileControllerTest &lt; ActionController::TestCase
     assert_tag :tag => 'div', :content => /#{instance_eval(&plugin2.profile_tabs[:content])}/, :attributes => {:id => /#{plugin2.profile_tabs[:id]}/}
   end
-  should 'redirect to profile page when try to request join_not_logged via GET method' do
-    community = Community.create!(:name => 'my test community')
-    login_as(profile.identifier)
-    get :join_not_logged, :profile => community.identifier
-    assert_nothing_raised do
-      assert_redirected_to community.url
-    end
-  end
-
   should 'check different profile from the domain profile' do
     default = Environment.default
     default.domains.create!(:name => 'environment.com')
	@@ -10,7 +10,7 @@		@@ -10,7 +10,7 @@
10		10
11	<% if @page != profile.home_page && !@page.has_posts? && @page.allow_delete?(user) && !remove_content_button(:delete)%>	11	<% if @page != profile.home_page && !@page.has_posts? && @page.allow_delete?(user) && !remove_content_button(:delete)%>
12	<% content = content_tag( 'span', _('Delete') ) %>	12	<% content = content_tag( 'span', _('Delete') ) %>
13	- <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'destroy', :id => @page}) %>	13	+ <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'destroy', :id => @page.id}) %>
14	<% options = {:method => :post, :confirm => delete_article_message(@page)} %>	14	<% options = {:method => :post, :confirm => delete_article_message(@page)} %>
15	<%= expirable_button @page, :delete, content, url, options %>	15	<%= expirable_button @page, :delete, content, url, options %>
16	<% end %>	16	<% end %>
	@@ -19,9 +19,9 @@		@@ -19,9 +19,9 @@
19	<% content = content_tag( 'span', _('Spread this') ) %>	19	<% content = content_tag( 'span', _('Spread this') ) %>
20	<% url = nil %>	20	<% url = nil %>
21	<% if profile.kind_of?(Person) %>	21	<% if profile.kind_of?(Person) %>
22	- <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish', :id => @page }) %>	22	+ <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish', :id => @page.id }) %>
23	<% elsif profile.kind_of?(Community) && environment.portal_community %>	23	<% elsif profile.kind_of?(Community) && environment.portal_community %>
24	- <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish_on_portal_community', :id => @page }) %>	24	+ <% url = profile.admin_url.merge({ :controller => 'cms', :action => 'publish_on_portal_community', :id => @page.id }) %>
25	<% end %>	25	<% end %>
26	<%= expirable_button @page, :spread, content, url if url %>	26	<%= expirable_button @page, :spread, content, url if url %>
27	<% end %>	27	<% end %>
1	<%= _('Hi, %{recipient}!') % { :recipient => @recipient } %>	1	<%= _('Hi, %{recipient}!') % { :recipient => @recipient } %>
2		2
3	-<%= word_wrap(_('Welcome to %{environment}! To activate your account, follow the link: %{activation_url}') % { :environment => @environment, :activation_url => @url + url_for(:controller => :account, :action => :activate, :activation_code => @activation_code, :redirection => @redirection) }) %>	3	+<%= word_wrap(_('Welcome to %{environment}! To activate your account, follow the link: %{activation_url}') % { :environment => @environment, :activation_url => @url + url_for(:controller => :account, :action => :activate, :activation_code => @activation_code, :redirection => @redirection, :join => @join) }) %>
4		4
5	<%= _("Greetings,") %>	5	<%= _("Greetings,") %>
6		6
	@@ -12,7 +12,7 @@ Feature: edit profile		@@ -12,7 +12,7 @@ Feature: edit profile
12	\| birth_date \|	12	\| birth_date \|
13	When I go to joao's control panel	13	When I go to joao's control panel
14	And I follow "Edit Profile"	14	And I follow "Edit Profile"
15	- And I select "1980-11-15" from "profile_data_birth_date"	15	+ And I fill in "Birth date" with "1980-11-15"
16	And I press "Save"	16	And I press "Save"
17	Then I should not see "Birth date is invalid"	17	Then I should not see "Birth date is invalid"
18	And I should not see "Birth date is mandatory"	18	And I should not see "Birth date is mandatory"
	@@ -23,7 +23,7 @@ Feature: edit profile		@@ -23,7 +23,7 @@ Feature: edit profile
23	\| birth_date \|	23	\| birth_date \|
24	When I go to joao's control panel	24	When I go to joao's control panel
25	And I follow "Edit Profile"	25	And I follow "Edit Profile"
26	- And I select "1980-11-15" from "profile_data_birth_date"	26	+ And I fill in "Birth date" with "1980-11-15"
27	And I press "Save"	27	And I press "Save"
28	Then I should not see "Birth date is invalid"	28	Then I should not see "Birth date is invalid"
29	And I should not see "Birth date is mandatory"	29	And I should not see "Birth date is mandatory"
	@@ -207,3 +207,18 @@ Feature: login		@@ -207,3 +207,18 @@ Feature: login
207	\| Password \| 123456 \|	207	\| Password \| 123456 \|
208	When I press "Log in"	208	When I press "Log in"
209	Then I should be on joaosilva's control panel	209	Then I should be on joaosilva's control panel
		210	+
		211	+ Scenario: join community on login
		212	+ Given the following users
		213	+ \| login \| name \|
		214	+ \| mariasilva \| Maria Silva \|
		215	+ And the following communities
		216	+ \| name \| identifier \| owner \|
		217	+ \| Free Software \| freesoftware \| mariasilva \|
		218	+ And I am on /freesoftware
		219	+ When I follow "Join"
		220	+ And I fill in the following:
		221	+ \| Username / Email \| joaosilva \|
		222	+ \| Password \| 123456 \|
		223	+ And I press "Log in"
		224	+ Then "Joao Silva" should be a member of "Free Software"
	@@ -250,3 +250,51 @@ Feature: signup		@@ -250,3 +250,51 @@ Feature: signup
250	And I fill in "Password" with "secret"	250	And I fill in "Password" with "secret"
251	And I press "Log in"	251	And I press "Log in"
252	Then I should be on the homepage	252	Then I should be on the homepage
		253	+
		254	+ @selenium
		255	+ Scenario: join community on signup
		256	+ Given the following users
		257	+ \| login \| name \|
		258	+ \| mariasilva \| Maria Silva \|
		259	+ And the following communities
		260	+ \| name \| identifier \| owner \|
		261	+ \| Free Software \| freesoftware \| mariasilva \|
		262	+ And feature "skip_new_user_email_confirmation" is disabled on environment
		263	+ And I am on /freesoftware
		264	+ When I follow "Join"
		265	+ And I follow "New user"
		266	+ And I fill in the following within ".no-boxes":
		267	+ \| e-Mail \| josesilva@example.com \|
		268	+ \| Username \| josesilva \|
		269	+ \| Password \| secret \|
		270	+ \| Password confirmation \| secret \|
		271	+ \| Full name \| José da Silva \|
		272	+ And wait for the captcha signup time
		273	+ And I press "Create my account"
		274	+ And I go to josesilva's confirmation URL
		275	+ And I fill in "Username" with "josesilva"
		276	+ And I fill in "Password" with "secret"
		277	+ And I press "Log in"
		278	+ Then "José da Silva" should be a member of "Free Software"
		279	+
		280	+ @selenium
		281	+ Scenario: join community on direct signup
		282	+ Given the following users
		283	+ \| login \| name \|
		284	+ \| mariasilva \| Maria Silva \|
		285	+ And the following communities
		286	+ \| name \| identifier \| owner \|
		287	+ \| Free Software \| freesoftware \| mariasilva \|
		288	+ And feature "skip_new_user_email_confirmation" is enabled on environment
		289	+ And I am on /freesoftware
		290	+ When I follow "Join"
		291	+ And I follow "New user"
		292	+ And I fill in the following within ".no-boxes":
		293	+ \| e-Mail \| josesilva@example.com \|
		294	+ \| Username \| josesilva \|
		295	+ \| Password \| secret \|
		296	+ \| Password confirmation \| secret \|
		297	+ \| Full name \| José da Silva \|
		298	+ And wait for the captcha signup time
		299	+ And I press "Create my account"
		300	+ Then "José da Silva" should be a member of "Free Software"
	@@ -118,9 +118,9 @@ end		@@ -118,9 +118,9 @@ end
118		118
119	Then /^(?:\|I )should see "([^"])" within any "([^"])"?$/ do \|text, selector\|	119	Then /^(?:\|I )should see "([^"])" within any "([^"])"?$/ do \|text, selector\|
120	if page.respond_to? :should	120	if page.respond_to? :should
121	- page.should have_css(selector, :content => text)	121	+ page.should have_css(selector, :text => text)
122	else	122	else
123	- assert page.has_css?(selector, :content => text)	123	+ assert page.has_css?(selector, :text => text)
124	end	124	end
125	end	125	end
126		126
1	class VideoBlock < Block	1	class VideoBlock < Block
2		2
		3	+ attr_accessible :url, :width, :height
		4	+
3	settings_items :url, :type => :string, :default => ""	5	settings_items :url, :type => :string, :default => ""
4	settings_items :width, :type => :integer, :default => 400	6	settings_items :width, :type => :integer, :default => 400
5	settings_items :height, :type => :integer, :default => 315	7	settings_items :height, :type => :integer, :default => 315