wagn 1.18.6 → 1.19.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fb5bf101e2cb0e4ff563ad4dbd19bf3bb438f58f
-  data.tar.gz: dcd6763cdd24f0d6ec5765ca0945f15fa8c3746e
+  metadata.gz: 9f5def697ca9071dc1eb8fbb857b6f3b9f7b9d62
+  data.tar.gz: 18c4adb2308e3b2793d98daaf210d02b746ac804
 SHA512:
-  metadata.gz: d9b74edbafb538e92f19d2f4205fd6c86e62c720b8fdb191efe93472e959b85da8a3eb2783709ed2a2520e6d6404273b0fca76d7ee6762fe03d782c1590423b0
-  data.tar.gz: 5b45b232e7b9cb661984cb87a0c7f0511c2e242eb1286240019bd18edd4950243997383e1bd1aabb187537db3a89abab298939a49ea6520040de91505ea76263
+  metadata.gz: 18bed7fa63837c98a67740861e9d9461774faddc0052a1f915ea2438c7524ef9b607283ae86c002725b22151aa9a5167ba1b2b60f04c644bd5610e916624b38a
+  data.tar.gz: d8df098265a6e02e6e3e1b98afa07dfea4e32ee1ec9609a429bfceaa75ded1777c43e5af756817f03ac42b31247b6577534454bf939b6a45933bd74175c9e821

data/Guardfile

@@ -2,10 +2,10 @@
 # More info at https://github.com/guard/guard#readme
 
 # guard :rspec, cmd: 'bundle exec env COVERAGE=false rspec' do
-guard :rspec, cmd: 'bundle exec wagn rspec' do
+guard :rspec, cmd: "bundle exec wagn rspec" do
   watch(%r{^spec/.+_spec\.rb$})
   watch(%r{^lib/(.+)\.rb$})     { |m| "spec/lib/#{m[1]}_spec.rb" }
-  watch('spec/spec_helper.rb')  { 'spec' }
+  watch("spec/spec_helper.rb")  { "spec" }
 
   # Rails example
   watch(%r{^app/(.+)\.rb$})                           { |m| "spec/#{m[1]}_spec.rb" }

data/bin/wagn

@@ -1,9 +1,9 @@
 #!/usr/bin/env ruby
 
-git_path = File.expand_path('../../../.git', __FILE__)
+git_path = File.expand_path("../../../.git", __FILE__)
 
 if File.exist?(git_path)
-  railties_path = File.expand_path('../../lib', __FILE__)
+  railties_path = File.expand_path("../../lib", __FILE__)
   $LOAD_PATH.unshift(railties_path)
 end
-require 'wagn/cli'
+require "wagn/cli"

data/features/notifications.feature

@@ -1,3 +1,4 @@
+@javascript @delayed-jobs
 Feature: Notifications
   In order for Wagn to be a more effective communication platform
   Users should be able to track changes to Wagn cards from their email

data/features/rules.feature

@@ -1,3 +1,4 @@
+@javascript
 Feature: Set-based Rles
   In order to control settings in an efficient way
   As a Wagneer
@@ -6,6 +7,7 @@ Feature: Set-based Rles
   Background:
     Given I am signed in as Joe Admin
     And I create card "*all+*help" with content "say something spicy"
+    And I wait a sec
 
   Scenario: default setting and plus card override
     Given I create Phrase card "color+*right+*add help" with content "If colorblind, leave blank"

data/features/step_definitions/email_steps.rb

@@ -31,7 +31,7 @@ module EmailHelpers
     # Replace with your a way to find your current email. e.g session -> email
     # last_email_address will return the last email address used by email spec to find an email.
     # Note that last_email_address will be reset after each Scenario.
-    last_email_address || 'fixthis@wagn.org'
+    last_email_address || "fixthis@wagn.org"
   end
 end
 
@@ -189,6 +189,10 @@ Then /^save and open all raw emails$/ do
 end
 
 Then /^(.*) should be notified that "(.*)"$/ do |username, subject|
+  Timeout.timeout(Capybara.default_wait_time) do
+    sleep(0.5) while page.evaluate_script("jQuery.active") != 0
+  end
+  Delayed::Worker.new.work_off
   email = address_for_user username
   begin
     step %("#{email}" should receive 1 email)
@@ -199,10 +203,14 @@ Then /^(.*) should be notified that "(.*)"$/ do |username, subject|
 end
 
 Then /^No notification should be sent$/ do
+  Timeout.timeout(Capybara.default_wait_time) do
+    sleep(0.5) while page.evaluate_script("jQuery.active") != 0
+  end
+  Delayed::Worker.new.work_off
   expect(all_emails).to be_empty
 end
 
 def address_for_user username
-  card_with_acct = username == 'I' ? Auth.current : Card[username]
+  card_with_acct = username == "I" ? Auth.current : Card[username]
   card_with_acct ? card_with_acct.account.email : username
 end

data/features/step_definitions/history_steps.rb

@@ -1,5 +1,5 @@
 # -*- encoding : utf-8 -*-
 # rubocop:disable Lint/AmbiguousRegexpLiteral
 When /^I expand act (\d+)$/ do |index|
-  all('a.arrow-right')[-index.to_i].click
+  all("a.arrow-right")[-index.to_i].click
 end

data/features/step_definitions/wagn_steps.rb

@@ -1,14 +1,14 @@
 # -*- encoding : utf-8 -*-
 # rubocop:disable Lint/AmbiguousRegexpLiteral
-require 'uri'
-require 'cgi'
-support_paths_file = File.join File.dirname(__FILE__), '..', 'support', 'paths'
+require "uri"
+require "cgi"
+support_paths_file = File.join File.dirname(__FILE__), "..", "support", "paths"
 require File.expand_path support_paths_file
 
 if RUBY_VERSION =~ /^2/
-  require 'byebug'
+  require "byebug"
 else
-  require 'debugger'
+  require "debugger"
 end
 
 Given /^site simulates setup need$/ do
@@ -17,19 +17,19 @@ end
 
 Given /^site stops simulating setup need$/ do
   Card::Auth.simulate_setup_need! false
-  step 'I am signed out'
+  step "I am signed out"
 end
 
 Given /^I am signed in as (.+)$/ do |account_name|
   accounted = Card[account_name]
-  visit '/update/:signin?card[subcards][%2B*email][content]='\
+  visit "/update/:signin?card[subcards][%2B*email][content]="\
     "#{accounted.account.email}&card[subcards][%2B*password][content]=joe_pass"
   # could optimize by specifying simple text success page
 end
 
 Given /^I am signed out$/ do
-  visit '/'
-  step 'I follow "Sign out"' if page.has_content? 'Sign out'
+  visit "/"
+  step 'I follow "Sign out"' if page.has_content? "Sign out"
 end
 
 # Given /^I sign in as (.+)$/ do |account_name|
@@ -62,7 +62,7 @@ When wysiwyg_re do |username, cardname, content|
   signed_in_as(username) do
     visit "/card/edit/#{cardname.to_name.url_key}"
     page.execute_script "$('#main .card-content').val('#{content}')"
-    click_button 'Submit'
+    click_button "Submit"
   end
 end
 
@@ -70,40 +70,55 @@ edit_re = /^(.*) edits? "([^\"]*)" setting (.*) to "([^\"]*)"$/
 When edit_re do |username, cardname, _field, content|
   signed_in_as(username) do
     visit "/card/edit/#{cardname.to_name.url_key}"
-    fill_in 'card[content]', with: content
-    click_button 'Submit'
+    set_content "card[content]", content
+    click_button "Submit"
   end
 end
 
 filling_re = /^(.*) edits? "([^\"]*)" filling in "([^\"]*)"$/
 When filling_re do |_username, cardname, content|
   visit "/card/edit/#{cardname.to_name.url_key}"
-  fill_in 'card[content]', with: content
+  fill_in "card[content]", with: content
 end
 
 When /^(.*) edits? "([^\"]*)" with plusses:/ do |username, cardname, plusses|
   signed_in_as(username) do
     visit "/card/edit/#{cardname.to_name.url_key}"
     plusses.hashes.first.each do |name, content|
-      fill_in "card[subcards][#{cardname}+#{name}][content]", with: content
+      set_content "card[subcards][#{cardname}+#{name}][content]", content
     end
-    click_button 'Submit'
+    click_button "Submit"
   end
 end
 
+def set_content name, content, cardtype=nil
+  Capybara.ignore_hidden_elements = false
+  ace_editors = all(".ace-editor-textarea[name='#{name}']")
+  pm_editors = all(".prosemirror-editor > [name='#{name}']")
+  if ace_editors.present? &&
+     page.evaluate_script("typeof ace != 'undefined'")
+    page.execute_script "ace.edit($('.ace_editor').get(0))"\
+        ".getSession().setValue('#{content}')"
+  elsif pm_editors.present?
+    editor_id = pm_editors.first.first(:xpath, ".//..")[:id]
+    set_prosemirror_content editor_id, content
+  else
+#rescue Selenium::WebDriver::Error::JavascriptError
+    fill_in(name, with: content)
+  end
+  Capybara.ignore_hidden_elements = true
+end
+
+def set_prosemirror_content editor_id, content
+  escaped_quotes = content.gsub("'", "\\'")
+  page.execute_script "getProseMirror('#{editor_id}')"\
+                      ".setContent('#{escaped_quotes}', 'text')"
+end
+
 content_re = /^(.*) creates?\s*a?\s*([^\s]*) card "(.*)" with content "(.*)"$/
 When content_re do |username, cardtype, cardname, content|
   create_card(username, cardtype, cardname, content) do
-    normal_textarea_types = %w(
-      JavaScript CoffeeScript HTML CSS SCSS Search
-    )
-    if !normal_textarea_types.include?(cardtype) ||
-       !page.evaluate_script("typeof ace != 'undefined'")
-      fill_in('card[content]', with: content)
-    else
-      page.execute_script "ace.edit($('.ace_editor').get(0))"\
-        ".getSession().setValue('#{content}')"
-    end
+    set_content "card[content]", content, cardtype
   end
 end
 
@@ -116,7 +131,7 @@ plus_re = /^(.*) creates?\s*([^\s]*) card "([^"]*)" with plusses:$/
 When plus_re do |username, cardtype, cardname, plusses|
   create_card(username, cardtype, cardname) do
     plusses.hashes.first.each do |name, content|
-      fill_in "card[subcards][+#{name}][content]", with: content
+      set_content "card[subcards][+#{name}][content]", content, cardtype
     end
   end
 end
@@ -135,19 +150,20 @@ end
 When /^(?:|I )upload the (.+) "(.+)"$/ do |attachment_name, filename|
   script = "$('input[type=file]').css('opacity','1');"
   page.driver.browser.execute_script(script)
-  file = File.join Wagn.gem_root, 'features', 'support', filename
+  file = File.join Wagn.gem_root, "features", "support", filename
   attach_file "card_#{attachment_name}", file
 end
 
 Given /^(.*) (is|am) watching "([^\"]+)"$/ do |user, _verb, cardname|
-  user = Card::Auth.current.name if user == 'I'
+  Delayed::Worker.new.work_off
+  user = Card::Auth.current.name if user == "I"
   signed_in_as user do
     step "the card #{cardname}+#{user}+*follow contains \"[[*always]]\""
   end
 end
 
 Given /^(.*) (is|am) not watching "([^\"]+)"$/ do |user, _verb, cardname|
-  user = Card::Auth.current.name if user == 'I'
+  user = Card::Auth.current.name if user == "I"
   signed_in_as user do
     step "the card #{cardname}+#{user}+*follow contains \"[[*never]]\""
   end
@@ -163,7 +179,7 @@ end
 
 When /^I wait until ajax response done$/ do
   Timeout.timeout(Capybara.default_wait_time) do
-    sleep(0.5) while page.evaluate_script('jQuery.active') != 0
+    sleep(0.5) while page.evaluate_script("jQuery.active") != 0
   end
 end
 
@@ -181,20 +197,20 @@ end
 #   nil
 # end
 
-def create_card username, cardtype, cardname, content=''
+def create_card username, cardtype, cardname, content=""
   signed_in_as(username) do
-    if cardtype == 'Pointer'
+    if cardtype == "Pointer"
       Card.create name: cardname, type: cardtype, content: content
     else
       visit "/card/new?card[name]=#{CGI.escape(cardname)}&type=#{cardtype}"
       yield if block_given?
-      click_button 'Submit'
+      click_button "Submit"
     end
   end
 end
 
 def signed_in_as username
-  sameuser = (username == 'I')
+  sameuser = (username == "I")
   sameuser ||= (Card::Auth.current.key == username.to_name.key)
   was_signed_in = Card::Auth.current_id if Card::Auth.signed_in?
   step "I am signed in as #{username}" unless sameuser
@@ -245,18 +261,22 @@ When /^In (.*) I find button with icon "(.*)" and click it$/ do |section, icon|
   end
 end
 
+Given /^Jobs are dispatched$/ do
+  Delayed::Worker.new.work_off
+end
+
 Then /I submit$/ do
-  click_button 'Submit'
+  click_button "Submit"
 end
 
 When /^I open the main card menu$/ do
   slot = "$('#main .menu-slot .vertical-card-menu.show-on-hover .card-slot')"
   page.execute_script "#{slot}.show()"
-  page.find('#main .menu-slot .card-menu a').click
+  page.find("#main .menu-slot .card-menu a").click
 end
 
 When /^I close the modal window$/ do
-  page.find('.modal-menu .close-modal').click
+  page.find(".modal-menu .close-modal").click
 end
 
 # When /^I pick (.*)$/ do |menu_item|
@@ -264,36 +284,36 @@ end
 
 Then /the card (.*) should contain "([^\"]*)"$/ do |cardname, content|
   visit path_to("card #{cardname}")
-  within scope_of('main card content') do
+  within scope_of("main card content") do
     expect(page).to have_content(content)
   end
 end
 
 Then /the card (.*) should not contain "([^\"]*)"$/ do |cardname, content|
   visit path_to("card #{cardname}")
-  within scope_of('main card content') do
+  within scope_of("main card content") do
     expect(page).not_to have_content(content)
   end
 end
 
 Then /the card (.*) should point to "([^\"]*)"$/ do |cardname, content|
   visit path_to("card #{cardname}")
-  within scope_of('pointer card content') do
+  within scope_of("pointer card content") do
     expect(page).to have_content(content)
   end
 end
 
 Then /the card (.*) should not point to "([^\"]*)"$/ do |cardname, content|
   visit path_to("card #{cardname}")
-  within scope_of('pointer card content') do
+  within scope_of("pointer card content") do
     expect(page).not_to have_content(content)
   end
 end
 
 Then /^In (.*) I should see "([^\"]*)"$/ do |section, text|
   within scope_of(section) do
-    if text.index('|')
-      expect(text.split('|').any? { |t| have_content(t) }).to be
+    if text.index("|")
+      expect(text.split("|").any? { |t| have_content(t) }).to be
     else
       expect(page).to have_content(text)
     end
@@ -309,17 +329,17 @@ end
 class_re = /^In (.*) I should (not )?see a ([^\"]*) with class "([^\"]*)"$/
 Then class_re do |selection, neg, element, selector|
   # checks for existence of a element with a class in a selection context
-  element = 'a' if element == 'link'
+  element = "a" if element == "link"
   within scope_of(selection) do
     verb = neg ? :should_not : :should
-    page.send(verb, have_css([element, selector].join('.')))
+    page.send(verb, have_css([element, selector].join(".")))
   end
 end
 
 content_re = /^In (.*) I should (not )?see a ([^\"]*) with content "([^\"]*)"$/
 Then content_re do |selection, neg, element, content|
   # checks for existence of a element with a class in a selection context
-  element = 'a' if element == 'link'
+  element = "a" if element == "link"
   within scope_of(selection) do
     verb = neg ? :should_not : :should
     page.send(verb, have_css(element, text: content))
@@ -337,11 +357,11 @@ Then /^"([^"]*)" should be selected for "([^"]*)"$/ do |value, field|
 end
 
 Then /^"([^"]*)" should be signed in$/ do |user|  # "
-  has_css?('#my-card-link', text: user)
+  has_css?("#my-card-link", text: user)
 end
 
 When /^I press enter to search$/ do
-  find('#_keyword').native.send_keys(:return)
+  find("#_keyword").native.send_keys(:return)
 end
 
 ## variants of standard steps to handle """ style quoted args
@@ -374,6 +394,10 @@ Then /^I should see "([^\"]*)" in color (.*)$/ do |text, css_class|
   page.has_css?(".diff-#{css_class}", text: text)
 end
 
+Then /^I should see css class "([^\"]*)"$/ do |css_class|
+  find(css_class)
+end
+
 css_should = /^I should see css class "([^\"]*)" within "(.*)"$/
 Then css_should do |css_class, selector|
   within selector do

data/features/step_definitions/web_steps.rb

@@ -6,9 +6,9 @@
 # instead of editing this one. Cucumber will automatically load all features/**/*.rb
 # files.
 
-require 'uri'
-require 'cgi'
-require File.expand_path(File.join(File.dirname(__FILE__), '..', 'support', 'paths'))
+require "uri"
+require "cgi"
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "support", "paths"))
 
 # Commonly used webrat steps
 # http://github.com/brynary/webrat
@@ -108,17 +108,17 @@ end
 # passed to attach_file() you will get a "Photo file is not one of the allowed file types."
 # error message
 When /^(?:|I )attach the file "([^"]*)" to "([^"]*)"$/ do |path, field|
-  type = path.split('.')[1]
+  type = path.split(".")[1]
 
   case type
-  when 'jpg'
-    type = 'image/jpg'
-  when 'jpeg'
-    type = 'image/jpeg'
-  when 'png'
-    type = 'image/png'
-  when 'gif'
-    type = 'image/gif'
+  when "jpg"
+    type = "image/jpg"
+  when "jpeg"
+    type = "image/jpeg"
+  when "png"
+    type = "image/png"
+  when "gif"
+    type = "image/gif"
   end
 
   attach_file(field, path, type)

data/features/step_definitions/window_steps.rb

@@ -13,14 +13,14 @@ END_TAG
 end
 
 When /I switch to (.+) window$/ do |window|
-  if window == 'first'
+  if window == "first"
     page.driver.browser.switch_to.window(page.driver.browser.window_handles.first)
-  elsif window == 'last'
+  elsif window == "last"
     page.driver.browser.switch_to.window(page.driver.browser.window_handles.last)
   end
 end
 
 When /^I close window$/ do
-  page.execute_script('window.close();')
+  page.execute_script("window.close();")
   page.driver.browser.switch_to.window(page.driver.browser.window_handles.last)
 end