From 1b8de3ee359ecfd82a791777d45a5944266e385f Mon Sep 17 00:00:00 2001 From: echarp Date: Sat, 13 Feb 2016 17:24:23 +0100 Subject: [PATCH] Bundle update --- Gemfile | 7 +- Gemfile.lock | 145 +++++++++++----------- app/controllers/events_controller.rb | 13 +- app/controllers/moderations_controller.rb | 6 +- app/controllers/orgas_controller.rb | 6 +- app/controllers/stats_controller.rb | 2 +- app/controllers/tags_controller.rb | 2 - app/models/event.rb | 2 +- config/application.rb | 3 +- db/seeds.rb | 56 +++++---- deploy/before_restart | 2 +- test/controllers/orgas_controller_test.rb | 6 +- 12 files changed, 122 insertions(+), 128 deletions(-) diff --git a/Gemfile b/Gemfile index b64eb36c..38b836c2 100644 --- a/Gemfile +++ b/Gemfile @@ -69,6 +69,10 @@ group :development do gem 'sqlite3' gem 'webrick' gem 'quiet_assets' + + # Access an IRB console on exception pages or by using <%= console %> in views + gem 'web-console', '~> 2.0' + gem 'guard-migrate', require: false gem 'guard-bundler', require: false gem 'guard-rubocop', require: false @@ -90,9 +94,6 @@ group :development, :test do # console gem 'byebug' - # Access an IRB console on exception pages or by using <%= console %> in views - gem 'web-console', '~> 2.0' - # Spring speeds up development by keeping your application running in the # background. Read more: https://github.com/rails/spring gem 'spring' diff --git a/Gemfile.lock b/Gemfile.lock index 0ac69682..bf493a96 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -1,6 +1,6 @@ GIT remote: git://github.com/Compass/compass-rails.git - revision: a1c890628d61f62e61474de2c00dde2658251196 + revision: 5b16c6c6ef8ad28f84e61a460602592fea5fc810 specs: compass-rails (3.0.0) compass (~> 1.0.0) @@ -25,21 +25,21 @@ GIT GEM remote: https://rubygems.org/ specs: - actionmailer (4.2.5) - actionpack (= 4.2.5) - actionview (= 4.2.5) - activejob (= 4.2.5) + actionmailer (4.2.5.1) + actionpack (= 4.2.5.1) + actionview (= 4.2.5.1) + activejob (= 4.2.5.1) mail (~> 2.5, >= 2.5.4) rails-dom-testing (~> 1.0, >= 1.0.5) - actionpack (4.2.5) - actionview (= 4.2.5) - activesupport (= 4.2.5) + actionpack (4.2.5.1) + actionview (= 4.2.5.1) + activesupport (= 4.2.5.1) rack (~> 1.6) rack-test (~> 0.6.2) rails-dom-testing (~> 1.0, >= 1.0.5) rails-html-sanitizer (~> 1.0, >= 1.0.2) - actionview (4.2.5) - activesupport (= 4.2.5) + actionview (4.2.5.1) + activesupport (= 4.2.5.1) builder (~> 3.1) erubis (~> 2.7.0) rails-dom-testing (~> 1.0, >= 1.0.5) @@ -61,17 +61,17 @@ GEM sass-rails activeadmin_pagedown (0.0.2) activeadmin (>= 0.5.0) - activejob (4.2.5) - activesupport (= 4.2.5) + activejob (4.2.5.1) + activesupport (= 4.2.5.1) globalid (>= 0.3.0) - activemodel (4.2.5) - activesupport (= 4.2.5) + activemodel (4.2.5.1) + activesupport (= 4.2.5.1) builder (~> 3.1) - activerecord (4.2.5) - activemodel (= 4.2.5) - activesupport (= 4.2.5) + activerecord (4.2.5.1) + activemodel (= 4.2.5.1) + activesupport (= 4.2.5.1) arel (~> 6.0) - activesupport (4.2.5) + activesupport (4.2.5.1) i18n (~> 0.7) json (~> 1.7, >= 1.7.7) minitest (~> 5.1) @@ -82,15 +82,13 @@ GEM activesupport (>= 3.0.0) arel (6.0.3) ast (2.2.0) - astrolabe (1.3.1) - parser (~> 2.2) bcrypt (3.1.10) binding_of_caller (0.7.2) debug_inspector (>= 0.0.1) bourbon (4.2.6) sass (~> 3.4) thor (~> 0.19) - brakeman (3.1.4) + brakeman (3.1.5) erubis (~> 2.6) fastercsv (~> 1.5) haml (>= 3.0, < 5.0) @@ -104,7 +102,7 @@ GEM terminal-table (~> 1.4) buftok (0.2.0) builder (3.2.2) - byebug (8.2.1) + byebug (8.2.2) chunky_png (1.3.5) coderay (1.1.0) coffee-rails (4.1.1) @@ -128,7 +126,7 @@ GEM sass (>= 3.2, < 3.5) concurrent-ruby (1.0.0) debug_inspector (0.0.2) - devise (3.5.3) + devise (3.5.6) bcrypt (~> 3.0) orm_adapter (~> 0.1) railties (>= 3.2.6, < 5) @@ -138,7 +136,7 @@ GEM devise-i18n (0.12.1) differ (0.1.2) docile (1.1.5) - domain_name (0.5.25) + domain_name (0.5.20160128) unf (>= 0.0.5, < 1.0.0) em-websocket (0.5.1) eventmachine (>= 0.12.9) @@ -147,7 +145,7 @@ GEM activemodel equalizer (0.0.10) erubis (2.7.0) - eventmachine (1.0.8) + eventmachine (1.0.9.1) execjs (2.6.0) faraday (0.9.2) multipart-post (>= 1.2, < 3) @@ -158,8 +156,8 @@ GEM formatador (0.2.5) formtastic (3.1.3) actionpack (>= 3.2.13) - formtastic_i18n (0.4.1) - geocoder (1.2.14) + formtastic_i18n (0.5.0) + geocoder (1.3.0) globalid (0.3.6) activesupport (>= 4.1.0) guard (2.13.0) @@ -179,7 +177,7 @@ GEM guard (~> 2.2) guard-compat (~> 1.1) guard-compat (1.2.1) - guard-livereload (2.5.1) + guard-livereload (2.5.2) em-websocket (~> 0.5) guard (~> 2.8) guard-compat (~> 1.0) @@ -215,7 +213,7 @@ GEM haml (~> 4.0.0) nokogiri (~> 1.6.0) ruby_parser (~> 3.5) - http (0.9.8) + http (1.0.2) addressable (~> 2.3) http-cookie (~> 1.0) http-form_data (~> 1.0.1) @@ -231,10 +229,10 @@ GEM has_scope (~> 0.6.0.rc) railties (>= 3.2, < 5) responders - jbuilder (2.4.0) + jbuilder (2.4.1) activesupport (>= 3.0.0, < 5.1) multi_json (~> 1.2) - jquery-rails (4.0.5) + jquery-rails (4.1.0) rails-dom-testing (~> 1.0) railties (>= 4.2.0) thor (>= 0.14, < 2.0) @@ -251,9 +249,9 @@ GEM railties (>= 3.1) leaflet-rails (0.7.4) libv8 (3.16.14.13) - listen (3.0.5) + listen (3.0.6) rb-fsevent (>= 0.9.3) - rb-inotify (>= 0.9) + rb-inotify (>= 0.9.7) loofah (2.0.3) nokogiri (>= 1.5.9) lumberjack (1.0.10) @@ -266,25 +264,25 @@ GEM method_source (0.8.2) mime-types (2.99) mini_portile2 (2.0.0) - minitest (5.8.3) + minitest (5.8.4) modernizr-rails (2.7.1) multi_json (1.11.2) multipart-post (2.0.0) mysql2 (0.3.20) naught (1.1.0) - nenv (0.2.0) - nokogiri (1.6.7.1) + nenv (0.3.0) + nokogiri (1.6.7.2) mini_portile2 (~> 2.0.0.rc2) notiffany (0.0.8) nenv (~> 0.1) shellany (~> 0.0) orm_adapter (0.5.0) - paper_trail (4.0.1) + paper_trail (4.0.2) activerecord (>= 3.0, < 6.0) activesupport (>= 3.0, < 6.0) request_store (~> 1.1) - parser (2.2.3.0) - ast (>= 1.1, < 3.0) + parser (2.3.0.5) + ast (~> 2.2) polyamorous (1.3.0) activerecord (>= 3.0) powerpack (0.1.1) @@ -299,16 +297,16 @@ GEM rack rack-test (0.6.3) rack (>= 1.0) - rails (4.2.5) - actionmailer (= 4.2.5) - actionpack (= 4.2.5) - actionview (= 4.2.5) - activejob (= 4.2.5) - activemodel (= 4.2.5) - activerecord (= 4.2.5) - activesupport (= 4.2.5) + rails (4.2.5.1) + actionmailer (= 4.2.5.1) + actionpack (= 4.2.5.1) + actionview (= 4.2.5.1) + activejob (= 4.2.5.1) + activemodel (= 4.2.5.1) + activerecord (= 4.2.5.1) + activesupport (= 4.2.5.1) bundler (>= 1.3.0, < 2.0) - railties (= 4.2.5) + railties (= 4.2.5.1) sprockets-rails rails-deprecated_sanitizer (1.0.3) activesupport (>= 4.2.0.alpha) @@ -316,18 +314,18 @@ GEM activesupport (>= 4.2.0.beta, < 5.0) nokogiri (~> 1.6.0) rails-deprecated_sanitizer (>= 1.0.1) - rails-html-sanitizer (1.0.2) + rails-html-sanitizer (1.0.3) loofah (~> 2.0) rails-i18n (4.0.8) i18n (~> 0.7) railties (~> 4.0) - railties (4.2.5) - actionpack (= 4.2.5) - activesupport (= 4.2.5) + railties (4.2.5.1) + actionpack (= 4.2.5.1) + activesupport (= 4.2.5.1) rake (>= 0.8.7) thor (>= 0.18.1, < 2.0) - rainbow (2.0.0) - rake (10.4.2) + rainbow (2.1.0) + rake (10.5.0) ransack (1.7.0) actionpack (>= 3.0) activerecord (>= 3.0) @@ -335,30 +333,29 @@ GEM i18n polyamorous (~> 1.2) rb-fsevent (0.9.7) - rb-inotify (0.9.5) + rb-inotify (0.9.7) ffi (>= 0.5.0) - rdoc (4.2.1) + rdoc (4.2.2) json (~> 1.4) redcarpet (3.3.4) ref (2.0.0) request_store (1.3.0) responders (2.1.1) railties (>= 4.2.0, < 5.1) - rubocop (0.35.1) - astrolabe (~> 1.3) - parser (>= 2.2.3.0, < 3.0) + rubocop (0.37.2) + parser (>= 2.3.0.4, < 3.0) powerpack (~> 0.1) rainbow (>= 1.99.1, < 3.0) ruby-progressbar (~> 1.7) - tins (<= 1.6.0) + unicode-display_width (~> 0.3) ruby-progressbar (1.7.5) ruby2ruby (2.2.0) ruby_parser (~> 3.1) sexp_processor (~> 4.0) - ruby_parser (3.7.2) + ruby_parser (3.7.3) sexp_processor (~> 4.1) safe_yaml (1.0.4) - sass (3.4.20) + sass (3.4.21) sass-rails (5.0.4) railties (>= 4.0.0, < 5.0) sass (~> 3.1) @@ -370,12 +367,12 @@ GEM rdoc (~> 4.0) select2-rails (3.5.9.3) thor (~> 0.14) - sexp_processor (4.6.0) + sexp_processor (4.6.1) shellany (0.0.1) - simple_calendar (2.0.3) + simple_calendar (2.0.6) rails (>= 3.0) simple_oauth (0.3.1) - simplecov (0.11.1) + simplecov (0.11.2) docile (~> 1.1.0) json (~> 1.8) simplecov-html (~> 0.10.0) @@ -384,11 +381,11 @@ GEM temple (~> 0.7.3) tilt (>= 1.3.3, < 2.1) slop (3.6.0) - spring (1.6.1) + spring (1.6.3) sprockets (3.5.2) concurrent-ruby (~> 1.0) rack (> 1, < 3) - sprockets-rails (3.0.0) + sprockets-rails (3.0.1) actionpack (>= 4.0) activesupport (>= 4.0) sprockets (>= 3.0.0) @@ -404,19 +401,18 @@ GEM thor (0.19.1) thread_safe (0.3.5) tilt (2.0.2) - tins (1.6.0) - tinymce-rails (4.3.2) + tinymce-rails (4.3.3) railties (>= 3.1.1) tinymce-rails-langs (4.20140129) tinymce-rails (~> 4.0) turbolinks (2.5.3) coffee-rails - twitter (5.15.0) + twitter (5.16.0) addressable (~> 2.3) buftok (~> 0.2.0) equalizer (= 0.0.10) faraday (~> 0.9.0) - http (>= 0.4, < 0.10) + http (~> 1.0) http_parser.rb (~> 0.6.0) json (~> 1.8) memoizable (~> 0.4.0) @@ -429,10 +425,11 @@ GEM json (>= 1.8.0) unf (0.1.4) unf_ext - unf_ext (0.0.7.1) - warden (1.2.4) + unf_ext (0.0.7.2) + unicode-display_width (0.3.1) + warden (1.2.6) rack (>= 1.0) - web-console (2.2.1) + web-console (2.3.0) activemodel (>= 4.0) binding_of_caller (>= 0.7.2) railties (>= 4.0) diff --git a/app/controllers/events_controller.rb b/app/controllers/events_controller.rb index e9e79e6f..af27e40c 100644 --- a/app/controllers/events_controller.rb +++ b/app/controllers/events_controller.rb @@ -88,11 +88,8 @@ class EventsController < ApplicationController # Use callbacks to share common setup or constraints between actions def set_event - if params[:secret].present? - @event = Event.where secret: params[:secret] - else - @event = Event.moderated - end + @event = Event.moderated + @event = Event.where secret: params[:secret] if params[:secret].present? @event = @event.find params[:id] end @@ -104,9 +101,9 @@ class EventsController < ApplicationController # through def event_params params.require(:event) - .permit :lock_version, :title, :start_time, :end_time, :description, - :place_name, :address, :city, :region_id, :locality, :url, - :contact, :submitter, :tags + .permit :lock_version, :title, :start_time, :end_time, :description, + :place_name, :address, :city, :region_id, :locality, :url, + :contact, :submitter, :tags end def locked diff --git a/app/controllers/moderations_controller.rb b/app/controllers/moderations_controller.rb index 75698e52..6d340967 100644 --- a/app/controllers/moderations_controller.rb +++ b/app/controllers/moderations_controller.rb @@ -64,9 +64,9 @@ class ModerationsController < ApplicationController # through. def moderation_params params.require(:event) - .permit :lock_version, :title, :start_time, :end_time, :description, - :place_name, :address, :city, :region_id, :locality, :url, - :contact, :submitter, :tags + .permit :lock_version, :title, :start_time, :end_time, :description, + :place_name, :address, :city, :region_id, :locality, :url, + :contact, :submitter, :tags end # Useful to manage absolute url in mails diff --git a/app/controllers/orgas_controller.rb b/app/controllers/orgas_controller.rb index 82ebd4b7..fb4661c7 100644 --- a/app/controllers/orgas_controller.rb +++ b/app/controllers/orgas_controller.rb @@ -89,9 +89,9 @@ class OrgasController < ApplicationController # through def orga_params params.require(:orga) - .permit :lock_version, :kind_id, :name, :description, :city, :department, - :region_id, :url, :diaspora, :feed, :contact, :submitter, :tags, - :active + .permit :lock_version, :kind_id, :name, :description, :city, + :department, :region_id, :url, :diaspora, :feed, :contact, + :submitter, :tags, :active end # Check that you can only edit an existing event if you know its secret diff --git a/app/controllers/stats_controller.rb b/app/controllers/stats_controller.rb index 041b25bc..5c0da9e7 100644 --- a/app/controllers/stats_controller.rb +++ b/app/controllers/stats_controller.rb @@ -12,7 +12,7 @@ class StatsController < ApplicationController def set_local @regions = Event.joins(:region).group(:region_id, year_grouping).count @city_events = Event.group(:city).having('count(city) > 3') - .order('count(city) desc').count + .order('count(city) desc').count end def year_grouping diff --git a/app/controllers/tags_controller.rb b/app/controllers/tags_controller.rb index c6984654..6cb98cd9 100644 --- a/app/controllers/tags_controller.rb +++ b/app/controllers/tags_controller.rb @@ -9,7 +9,6 @@ class TagsController < InheritedResources::Base .pluck(:tags).map(&:split).flatten .group_by { |i| i } .map { |k, v| [k, v.size] } - .reject { |_k, v| v <= 3 } .sort respond_to do |format| @@ -28,7 +27,6 @@ class TagsController < InheritedResources::Base .pluck(:tags).map(&:split).flatten .group_by { |i| i } .map { |k, v| [k, v.size] } - .reject { |_k, v| v <= 3 } .sort respond_to do |format| diff --git a/app/models/event.rb b/app/models/event.rb index 32e7ebdf..f8428dc6 100644 --- a/app/models/event.rb +++ b/app/models/event.rb @@ -115,7 +115,7 @@ class Event < ActiveRecord::Base host: ActionMailer::Base.default_url_options[:host]) tweet = "#{self} #{url}" - if (tweet.size >= 140) + if tweet.size >= 140 tweet = "#{tweet[0, tweet.rindex(/\s/, 140 - url.size)]} #{url}" end tweet diff --git a/config/application.rb b/config/application.rb index 19dfde6a..19849b2d 100644 --- a/config/application.rb +++ b/config/application.rb @@ -38,8 +38,7 @@ module AgendaDuLibreRails # config.quiet_assets = false config.sass.preferred_syntax = :sass - config.action_dispatch.default_headers - .merge! 'X-Frame-Options' => 'ALLOWALL' + config.action_dispatch.default_headers['X-Frame-Options'] = 'ALLOWALL' config.active_record.raise_in_transactional_callbacks = true end diff --git a/db/seeds.rb b/db/seeds.rb index 37f6ad18..2227ab6a 100644 --- a/db/seeds.rb +++ b/db/seeds.rb @@ -25,17 +25,18 @@ Kind.create name: 'provider', icon: 'tty' Kind.create name: 'institution', icon: 'institution' # rubocop:disable Metrics/LineLength -I18n::Backend::ActiveRecord::Translation.create([ - { locale: 'fr', key: 'mail_suffix', value: '[AdL] ' }, +I18n::Backend::ActiveRecord::Translation.create( + [ + { locale: 'fr', key: 'mail_suffix', value: '[AdL] ' }, - { locale: 'fr', key: 'layouts.application.subtitle', - value: 'Les événements du Libre en France' }, + { locale: 'fr', key: 'layouts.application.subtitle', + value: 'Les événements du Libre en France' }, - { locale: 'fr', key: 'events.new.subtitle', - value: "Cette page permet de soumettre un événement dans l'Agenda du Libre. Celui-ci n'apparaîtra pas automatiquement dans l'Agenda, il sera tout d'abord validé par un modérateur. Un courrier électronique vous sera envoyé à l'adresse e-mail de contact donnée ci-dessous lorsque l'événement aura été modéré." }, + { locale: 'fr', key: 'events.new.subtitle', + value: "Cette page permet de soumettre un événement dans l'Agenda du Libre. Celui-ci n'apparaîtra pas automatiquement dans l'Agenda, il sera tout d'abord validé par un modérateur. Un courrier électronique vous sera envoyé à l'adresse e-mail de contact donnée ci-dessous lorsque l'événement aura été modéré." }, - { locale: 'fr', key: 'events.new.advises', - value: "# Recommandations importantes + { locale: 'fr', key: 'events.new.advises', + value: "# Recommandations importantes Ces quelques recommandations vous permettront de proposer un événement qui sera validé plus rapidement, et dont la lisibilité sera meilleure pour les utilisateurs de l'Agenda du Libre. @@ -55,28 +56,28 @@ L'équipe de modération se réserve le droit de modifier la description de l'é Si vous soumettez souvent un événement régulier dans l'Agenda du Libre, vous pouvez automatiser cette procédure à l'aide d'un [script que nous vous proposons](/adl-submit.py). " }, - { locale: 'fr', key: 'events.form.title_helper', - value: 'Décrivez en moins de 5 mots votre événement, sans y indiquer le lieu, la ville ou la date' }, + { locale: 'fr', key: 'events.form.title_helper', + value: 'Décrivez en moins de 5 mots votre événement, sans y indiquer le lieu, la ville ou la date' }, - { locale: 'fr', key: 'events.form.description_helper', - value: '**Décrivez de la manière la plus complète possible votre événement.**' }, + { locale: 'fr', key: 'events.form.description_helper', + value: '**Décrivez de la manière la plus complète possible votre événement.**' }, - { locale: 'fr', key: 'events.form.url_helper', - value: "Lien **direct** vers une page donnant plus d'informations sur l'événement" }, + { locale: 'fr', key: 'events.form.url_helper', + value: "Lien **direct** vers une page donnant plus d'informations sur l'événement" }, - { locale: 'fr', key: 'events.form.contact_helper', - value: '*Adresse e-mail de contact. Elle sera affichée de manière peu compréhensible par les spammeurs.*' }, + { locale: 'fr', key: 'events.form.contact_helper', + value: '*Adresse e-mail de contact. Elle sera affichée de manière peu compréhensible par les spammeurs.*' }, - { locale: 'fr', key: 'events.form.submitter_helper', - value: "*Adresse e-mail du soumetteur de l'événement. Elle ne sera utilisée que par les modérateurs pour contacter la personne ayant proposé l'événement, pour l'informer de sa validation ou de son rejet. Si cette adresse n'est pas présente, l'adresse de contact sera utilisée*" }, + { locale: 'fr', key: 'events.form.submitter_helper', + value: "*Adresse e-mail du soumetteur de l'événement. Elle ne sera utilisée que par les modérateurs pour contacter la personne ayant proposé l'événement, pour l'informer de sa validation ou de son rejet. Si cette adresse n'est pas présente, l'adresse de contact sera utilisée*" }, - { locale: 'fr', key: 'events.form.tags_helper', - value: "*Tags pour l'événement. Les tags sont séparés par des espaces. Un tag ne peut contenir que des lettres minuscules, des chiffres et des tirets.* + { locale: 'fr', key: 'events.form.tags_helper', + value: "*Tags pour l'événement. Les tags sont séparés par des espaces. Un tag ne peut contenir que des lettres minuscules, des chiffres et des tirets.* *Dans les tags, indiquez le nom de la ou des associations organisatrices. N'indiquez pas le nom de la ville ou de la région.*" }, - { locale: 'fr', key: 'application.infos.content', - value: "## Informations + { locale: 'fr', key: 'application.infos.content', + value: "## Informations ### Table des matières @@ -360,16 +361,16 @@ Bien que le logiciel faisant fonctionner l'Agenda du Libre soit spécifique, d'a #### 05/06/2005 * Lancement du site" }, - { locale: 'fr', key: 'application.contact.content', - value: "## Contact + { locale: 'fr', key: 'application.contact.content', + value: "## Contact Pour contacter les modérateurs du site *Agenda du Libre*: * Par courrier électronique, à l'adresse moderateurs@agendadulibre.org * Par [IRC](http://fr.wikipedia.org/wiki/IRC), sur le canal `#agendadulibre` du réseau *Freenode*" }, - { locale: 'fr', key: 'application.rules.content', - value: "## Recommandations sur la modération + { locale: 'fr', key: 'application.rules.content', + value: "## Recommandations sur la modération Tous les événements de l'Agenda du Libre passent par une phase de modération, qui permet de s'assurer que les événements recensés rentrent bien dans la [ligne éditoriale](#ligne) et que la [qualité de la description des événements](#qualite) reste bonne. @@ -409,4 +410,5 @@ Si des informations manquent, les récupérer sur le site de l'événement si el Ces recommandations de modération sont à discuter et à améliorer au fur et à mesure de la vie du site. Nous pouvons en discuter via la liste `moderateurs@agendadulibre.org`. " } -]) + ] +) diff --git a/deploy/before_restart b/deploy/before_restart index 7b0a4739..84122a73 100755 --- a/deploy/before_restart +++ b/deploy/before_restart @@ -35,7 +35,7 @@ if File.file? 'Rakefile' .split("\0") task = 'assets:precompile' - if changed_assets.size > 0 + unless changed_assets.empty? run "#{rake_cmd} #{task} RAILS_ENV=#{RAILS_ENV} RAILS_GROUPS=assets" end end diff --git a/test/controllers/orgas_controller_test.rb b/test/controllers/orgas_controller_test.rb index ca730ba1..d30c319e 100644 --- a/test/controllers/orgas_controller_test.rb +++ b/test/controllers/orgas_controller_test.rb @@ -57,7 +57,7 @@ class OrgasControllerTest < ActionController::TestCase # Necessary to have the proper paper_trail version @orga.update_attributes name: 'My Title' - patch :update, id: @orga, orga: { name: @orga.name } + patch :update, id: @orga, secret: @orga.secret, orga: { name: @orga.name } assert_empty assigns(:orga).errors.messages assert_redirected_to assigns(:orga) @@ -71,10 +71,10 @@ class OrgasControllerTest < ActionController::TestCase end test 'should not update orga without proper secret' do - patch :update, id: @orga, secret: @orga.secret, orga: { + patch :update, id: @orga, orga: { name: 'hello world' } - assert_redirected_to assigns(:orga) + assert_redirected_to :new_user_session end end