Merge pull request #1274 from consul/rubocop
Rubocop - Timezone warnings + .rubocop.yml
This commit is contained in:
56
.rubocop.yml
Normal file
56
.rubocop.yml
Normal file
@@ -0,0 +1,56 @@
|
||||
inherit_from: .rubocop_todo.yml
|
||||
|
||||
AllCops:
|
||||
Include:
|
||||
- '**/Rakefile'
|
||||
- '**/config.ru'
|
||||
Exclude:
|
||||
- 'db/**/*'
|
||||
- 'config/**/*'
|
||||
- 'script/**/*'
|
||||
TargetRubyVersion: 2.3
|
||||
|
||||
Rails:
|
||||
Enabled: true
|
||||
|
||||
Documentation:
|
||||
Enabled: false
|
||||
|
||||
Metrics/LineLength:
|
||||
Max: 140
|
||||
|
||||
Style/IndentationConsistency:
|
||||
EnforcedStyle: rails
|
||||
|
||||
Style/StringLiterals:
|
||||
Enabled: false
|
||||
|
||||
Style/FrozenStringLiteralComment:
|
||||
Enabled: false
|
||||
|
||||
Style/PercentLiteralDelimiters:
|
||||
Enabled: false
|
||||
|
||||
Style/EmptyLinesAroundClassBody:
|
||||
Enabled: false
|
||||
|
||||
Style/EmptyLinesAroundBlockBody:
|
||||
Enabled: false
|
||||
|
||||
Style/EmptyLinesAroundModuleBody:
|
||||
Enabled: false
|
||||
|
||||
Style/SpaceBeforeBlockBraces:
|
||||
Enabled: false
|
||||
|
||||
Style/SpaceInsideBrackets:
|
||||
Enabled: false
|
||||
|
||||
Style/SpaceInsideHashLiteralBraces:
|
||||
Enabled: false
|
||||
|
||||
Style/SpaceInsideBlockBraces:
|
||||
Enabled: false
|
||||
|
||||
Style/TrailingBlankLines:
|
||||
Enabled: false
|
||||
838
.rubocop_todo.yml
Normal file
838
.rubocop_todo.yml
Normal file
@@ -0,0 +1,838 @@
|
||||
# This configuration was generated by
|
||||
# `rubocop --auto-gen-config`
|
||||
# on 2016-11-25 13:34:56 +0100 using RuboCop version 0.45.0.
|
||||
# The point is for the user to remove these configuration records
|
||||
# one by one as the offenses are removed from the code base.
|
||||
# Note that changes in the inspected code, or installation of new
|
||||
# versions of RuboCop, may require this file to be generated again.
|
||||
|
||||
# Offense count: 3
|
||||
Lint/AmbiguousRegexpLiteral:
|
||||
Exclude:
|
||||
- 'app/helpers/verification_helper.rb'
|
||||
- 'spec/features/verification/residence_spec.rb'
|
||||
|
||||
# Offense count: 3
|
||||
# Configuration parameters: AllowSafeAssignment.
|
||||
Lint/AssignmentInCondition:
|
||||
Exclude:
|
||||
- 'app/controllers/management/sessions_controller.rb'
|
||||
- 'app/models/notification.rb'
|
||||
- 'lib/capistrano/template.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AlignWith, SupportedStyles.
|
||||
# SupportedStyles: either, start_of_block, start_of_line
|
||||
Lint/BlockAlignment:
|
||||
Exclude:
|
||||
- 'spec/features/tracks_spec.rb'
|
||||
- 'spec/models/proposal_spec.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AlignWith, SupportedStyles, AutoCorrect.
|
||||
# SupportedStyles: start_of_line, def
|
||||
Lint/DefEndAlignment:
|
||||
Exclude:
|
||||
- 'app/controllers/comments_controller.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: runtime_error, standard_error
|
||||
Lint/InheritException:
|
||||
Exclude:
|
||||
- 'app/controllers/concerns/feature_flags.rb'
|
||||
|
||||
# Offense count: 7
|
||||
Lint/NestedMethodDefinition:
|
||||
Exclude:
|
||||
- 'lib/acts_as_paranoid_aliases.rb'
|
||||
|
||||
# Offense count: 13
|
||||
Lint/ParenthesesAsGroupedExpression:
|
||||
Exclude:
|
||||
- 'spec/features/admin/organizations_spec.rb'
|
||||
- 'spec/features/debates_spec.rb'
|
||||
- 'spec/features/proposals_spec.rb'
|
||||
- 'spec/models/debate_spec.rb'
|
||||
|
||||
# Offense count: 1
|
||||
Lint/UnderscorePrefixedVariableName:
|
||||
Exclude:
|
||||
- 'lib/manager_authenticator.rb'
|
||||
|
||||
# Offense count: 4
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: IgnoreEmptyBlocks, AllowUnusedKeywordArguments.
|
||||
Lint/UnusedBlockArgument:
|
||||
Exclude:
|
||||
- 'app/controllers/admin/spending_proposals_controller.rb'
|
||||
- 'app/models/ahoy/data_source.rb'
|
||||
- 'spec/spec_helper.rb'
|
||||
|
||||
# Offense count: 5
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods.
|
||||
Lint/UnusedMethodArgument:
|
||||
Exclude:
|
||||
- 'app/controllers/organizations/registrations_controller.rb'
|
||||
- 'app/controllers/users/omniauth_callbacks_controller.rb'
|
||||
- 'app/controllers/users/registrations_controller.rb'
|
||||
- 'app/models/abilities/everyone.rb'
|
||||
- 'app/models/abilities/valuator.rb'
|
||||
|
||||
# Offense count: 135
|
||||
Lint/UselessAssignment:
|
||||
Enabled: false
|
||||
|
||||
# Offense count: 41
|
||||
Metrics/AbcSize:
|
||||
Max: 44
|
||||
|
||||
# Offense count: 4
|
||||
# Configuration parameters: CountComments.
|
||||
Metrics/ClassLength:
|
||||
Max: 205
|
||||
|
||||
# Offense count: 5
|
||||
Metrics/CyclomaticComplexity:
|
||||
Max: 8
|
||||
|
||||
# Offense count: 52
|
||||
# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives.
|
||||
# URISchemes: http, https
|
||||
Metrics/LineLength:
|
||||
Max: 307
|
||||
|
||||
# Offense count: 28
|
||||
# Configuration parameters: CountComments.
|
||||
Metrics/MethodLength:
|
||||
Max: 38
|
||||
|
||||
# Offense count: 2
|
||||
# Configuration parameters: CountComments.
|
||||
Metrics/ModuleLength:
|
||||
Max: 193
|
||||
|
||||
# Offense count: 3
|
||||
Metrics/PerceivedComplexity:
|
||||
Max: 11
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Performance/RedundantBlockCall:
|
||||
Exclude:
|
||||
- 'app/mailers/mailer.rb'
|
||||
|
||||
# Offense count: 4
|
||||
# Cop supports --auto-correct.
|
||||
Performance/RedundantMatch:
|
||||
Exclude:
|
||||
- 'app/controllers/valuation/spending_proposals_controller.rb'
|
||||
- 'app/helpers/embed_videos_helper.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: strict, flexible
|
||||
Rails/Date:
|
||||
Exclude:
|
||||
- 'app/controllers/concerns/commentable_actions.rb'
|
||||
- 'app/models/direct_message.rb'
|
||||
|
||||
# Offense count: 4
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: Whitelist.
|
||||
# Whitelist: find_by_sql
|
||||
Rails/DynamicFindBy:
|
||||
Exclude:
|
||||
- 'app/controllers/users/confirmations_controller.rb'
|
||||
- 'app/controllers/users/registrations_controller.rb'
|
||||
- 'spec/features/management/users_spec.rb'
|
||||
|
||||
# Offense count: 4
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: Include.
|
||||
# Include: app/models/**/*.rb
|
||||
Rails/FindBy:
|
||||
Exclude:
|
||||
- 'app/models/setting.rb'
|
||||
- 'app/models/verification/email.rb'
|
||||
- 'app/models/verification/management/email.rb'
|
||||
- 'app/models/verification/residence.rb'
|
||||
|
||||
# Offense count: 24
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: Include.
|
||||
# Include: spec/**/*, test/**/*
|
||||
Rails/HttpPositionalArguments:
|
||||
Exclude:
|
||||
- 'spec/controllers/admin/api/stats_controller_spec.rb'
|
||||
- 'spec/controllers/concerns/has_filters_spec.rb'
|
||||
- 'spec/controllers/concerns/has_orders_spec.rb'
|
||||
- 'spec/controllers/debates_controller_spec.rb'
|
||||
- 'spec/controllers/management/sessions_controller_spec.rb'
|
||||
- 'spec/controllers/pages_controller_spec.rb'
|
||||
- 'spec/controllers/users/registrations_controller_spec.rb'
|
||||
|
||||
# Offense count: 12
|
||||
Rails/OutputSafety:
|
||||
Exclude:
|
||||
- 'app/controllers/spending_proposals_controller.rb'
|
||||
- 'app/helpers/application_helper.rb'
|
||||
- 'app/helpers/text_with_links_helper.rb'
|
||||
- 'app/helpers/users_helper.rb'
|
||||
- 'app/helpers/valuation_helper.rb'
|
||||
|
||||
# Offense count: 6
|
||||
# Cop supports --auto-correct.
|
||||
Rails/PluralizationGrammar:
|
||||
Exclude:
|
||||
- 'spec/features/admin/banners_spec.rb'
|
||||
- 'spec/features/debates_spec.rb'
|
||||
- 'spec/features/proposals_spec.rb'
|
||||
- 'spec/models/residence_spec.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: referer, referrer
|
||||
Rails/RequestReferer:
|
||||
Exclude:
|
||||
- 'app/controllers/users/sessions_controller.rb'
|
||||
|
||||
# Offense count: 11
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: strict, flexible
|
||||
Rails/TimeZone:
|
||||
Exclude:
|
||||
- 'lib/score_calculator.rb'
|
||||
- 'spec/controllers/admin/api/stats_controller_spec.rb'
|
||||
- 'spec/models/ahoy/data_source_spec.rb'
|
||||
|
||||
# Offense count: 7
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: Include.
|
||||
# Include: app/models/**/*.rb
|
||||
Rails/Validation:
|
||||
Exclude:
|
||||
- 'app/models/comment.rb'
|
||||
- 'app/models/spending_proposal.rb'
|
||||
- 'app/models/verification/residence.rb'
|
||||
- 'app/models/verification/sms.rb'
|
||||
|
||||
# Offense count: 9
|
||||
Style/AccessorMethodName:
|
||||
Exclude:
|
||||
- 'app/controllers/application_controller.rb'
|
||||
- 'app/controllers/concerns/commentable_actions.rb'
|
||||
- 'app/controllers/management/proposals_controller.rb'
|
||||
- 'app/controllers/management/spending_proposals_controller.rb'
|
||||
- 'app/controllers/proposals_controller.rb'
|
||||
|
||||
# Offense count: 30
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle, SupportedLastArgumentHashStyles.
|
||||
# SupportedLastArgumentHashStyles: always_inspect, always_ignore, ignore_implicit, ignore_explicit
|
||||
Style/AlignHash:
|
||||
Exclude:
|
||||
- 'spec/features/admin/banners_spec.rb'
|
||||
|
||||
# Offense count: 21
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
|
||||
# SupportedStyles: with_first_parameter, with_fixed_indentation
|
||||
Style/AlignParameters:
|
||||
Exclude:
|
||||
- 'app/helpers/search_helper.rb'
|
||||
- 'app/models/verification/management/email.rb'
|
||||
- 'bin/rspec'
|
||||
- 'bin/spring'
|
||||
- 'spec/features/admin/spending_proposals_spec.rb'
|
||||
- 'spec/features/spending_proposals_spec.rb'
|
||||
- 'spec/features/valuation/spending_proposals_spec.rb'
|
||||
- 'spec/features/verification/verified_user_spec.rb'
|
||||
- 'spec/i18n_spec.rb'
|
||||
- 'spec/models/user_spec.rb'
|
||||
- 'spec/rails_helper.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: always, conditionals
|
||||
Style/AndOr:
|
||||
Exclude:
|
||||
- 'app/helpers/embed_videos_helper.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles, ProceduralMethods, FunctionalMethods, IgnoredMethods.
|
||||
# SupportedStyles: line_count_based, semantic, braces_for_chaining
|
||||
# ProceduralMethods: benchmark, bm, bmbm, create, each_with_object, measure, new, realtime, tap, with_object
|
||||
# FunctionalMethods: let, let!, subject, watch
|
||||
# IgnoredMethods: lambda, proc, it
|
||||
Style/BlockDelimiters:
|
||||
Exclude:
|
||||
- 'spec/features/users_auth_spec.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
Style/BlockEndNewline:
|
||||
Exclude:
|
||||
- 'app/models/banner.rb'
|
||||
- 'spec/features/users_auth_spec.rb'
|
||||
|
||||
# Offense count: 19
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: braces, no_braces, context_dependent
|
||||
Style/BracesAroundHashParameters:
|
||||
Exclude:
|
||||
- 'app/controllers/valuation/spending_proposals_controller.rb'
|
||||
- 'app/models/concerns/searchable.rb'
|
||||
- 'app/models/verification/residence.rb'
|
||||
- 'lib/manager_authenticator.rb'
|
||||
- 'spec/controllers/management/users_controller_spec.rb'
|
||||
- 'spec/features/admin/spending_proposals_spec.rb'
|
||||
- 'spec/lib/manager_authenticator_spec.rb'
|
||||
- 'spec/models/residence_spec.rb'
|
||||
- 'spec/models/user_spec.rb'
|
||||
|
||||
# Offense count: 57
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: nested, compact
|
||||
Style/ClassAndModuleChildren:
|
||||
Enabled: false
|
||||
|
||||
# Offense count: 5
|
||||
Style/ClassVars:
|
||||
Exclude:
|
||||
- 'app/models/concerns/measurable.rb'
|
||||
- 'app/models/organization.rb'
|
||||
- 'app/models/user.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Style/ClosingParenthesisIndentation:
|
||||
Exclude:
|
||||
- 'spec/rails_helper.rb'
|
||||
|
||||
# Offense count: 4
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles, SingleLineConditionsOnly.
|
||||
# SupportedStyles: assign_to_condition, assign_inside_condition
|
||||
Style/ConditionalAssignment:
|
||||
Exclude:
|
||||
- 'app/controllers/comments_controller.rb'
|
||||
- 'app/controllers/management/spending_proposals_controller.rb'
|
||||
- 'app/controllers/spending_proposals_controller.rb'
|
||||
- 'app/controllers/verification/sms_controller.rb'
|
||||
|
||||
# Offense count: 6
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: leading, trailing
|
||||
Style/DotPosition:
|
||||
Exclude:
|
||||
- 'app/models/banner.rb'
|
||||
- 'app/models/tag_cloud.rb'
|
||||
- 'app/models/verification/management/managed_user.rb'
|
||||
|
||||
# Offense count: 1
|
||||
Style/DoubleNegation:
|
||||
Exclude:
|
||||
- 'app/models/flag.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Style/EmptyCaseCondition:
|
||||
Exclude:
|
||||
- 'app/models/concerns/verification.rb'
|
||||
|
||||
# Offense count: 9
|
||||
# Cop supports --auto-correct.
|
||||
Style/EmptyLines:
|
||||
Exclude:
|
||||
- 'app/models/concerns/search_cache.rb'
|
||||
- 'app/models/notification.rb'
|
||||
- 'spec/features/admin/spending_proposals_spec.rb'
|
||||
- 'spec/features/admin/verifications_spec.rb'
|
||||
- 'spec/features/debates_spec.rb'
|
||||
- 'spec/features/registration_form_spec.rb'
|
||||
- 'spec/features/users_auth_spec.rb'
|
||||
- 'spec/features/verification/verified_user_spec.rb'
|
||||
- 'spec/support/verifiable.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Style/EmptyLinesAroundAccessModifier:
|
||||
Exclude:
|
||||
- 'app/controllers/users_controller.rb'
|
||||
|
||||
# Offense count: 29
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AllowForAlignment, ForceEqualSignAlignment.
|
||||
Style/ExtraSpacing:
|
||||
Enabled: false
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
|
||||
# SupportedStyles: consistent, special_for_inner_method_call, special_for_inner_method_call_in_parentheses
|
||||
Style/FirstParameterIndentation:
|
||||
Exclude:
|
||||
- 'app/controllers/users_controller.rb'
|
||||
|
||||
# Offense count: 32
|
||||
# Configuration parameters: MinBodyLength.
|
||||
Style/GuardClause:
|
||||
Enabled: false
|
||||
|
||||
# Offense count: 10
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles, UseHashRocketsWithSymbolValues, PreferHashRocketsForNonAlnumEndingSymbols.
|
||||
# SupportedStyles: ruby19, hash_rockets, no_mixed_keys, ruby19_no_mixed_keys
|
||||
Style/HashSyntax:
|
||||
Exclude:
|
||||
- 'lib/sms_api.rb'
|
||||
- 'spec/factories.rb'
|
||||
|
||||
# Offense count: 6
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: MaxLineLength.
|
||||
Style/IfUnlessModifier:
|
||||
Exclude:
|
||||
- 'app/controllers/annotations_controller.rb'
|
||||
- 'app/controllers/verification/letter_controller.rb'
|
||||
- 'app/controllers/welcome_controller.rb'
|
||||
- 'app/helpers/embed_videos_helper.rb'
|
||||
- 'app/models/proposal.rb'
|
||||
- 'app/models/spending_proposal.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: SupportedStyles, IndentationWidth.
|
||||
# SupportedStyles: special_inside_parentheses, consistent, align_brackets
|
||||
Style/IndentArray:
|
||||
EnforcedStyle: consistent
|
||||
|
||||
# Offense count: 4
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: SupportedStyles, IndentationWidth.
|
||||
# SupportedStyles: special_inside_parentheses, consistent, align_braces
|
||||
Style/IndentHash:
|
||||
EnforcedStyle: consistent
|
||||
|
||||
# Offense count: 3
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: normal, rails
|
||||
Style/IndentationConsistency:
|
||||
Exclude:
|
||||
- 'spec/features/tracks_spec.rb'
|
||||
- 'spec/models/proposal_spec.rb'
|
||||
|
||||
# Offense count: 11
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: Width.
|
||||
Style/IndentationWidth:
|
||||
Exclude:
|
||||
- 'app/controllers/annotations_controller.rb'
|
||||
- 'app/helpers/flags_helper.rb'
|
||||
- 'app/mailers/devise_mailer.rb'
|
||||
- 'app/mailers/mailer.rb'
|
||||
- 'app/models/ahoy/data_source.rb'
|
||||
- 'app/models/comment_notifier.rb'
|
||||
- 'app/models/concerns/search_cache.rb'
|
||||
- 'spec/features/comments/proposals_spec.rb'
|
||||
- 'spec/features/moderation/users_spec.rb'
|
||||
- 'spec/features/tracks_spec.rb'
|
||||
- 'spec/features/verification/verified_user_spec.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: line_count_dependent, lambda, literal
|
||||
Style/Lambda:
|
||||
Exclude:
|
||||
- 'app/models/banner.rb'
|
||||
- 'app/models/direct_message.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Style/MethodCallParentheses:
|
||||
Exclude:
|
||||
- 'app/controllers/management/document_verifications_controller.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: require_parentheses, require_no_parentheses, require_no_parentheses_except_multiline
|
||||
Style/MethodDefParentheses:
|
||||
Exclude:
|
||||
- 'spec/helpers/comments_helper_spec.rb'
|
||||
|
||||
# Offense count: 3
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: symmetrical, new_line, same_line
|
||||
Style/MultilineArrayBraceLayout:
|
||||
Exclude:
|
||||
- 'app/controllers/valuation/spending_proposals_controller.rb'
|
||||
- 'app/helpers/search_helper.rb'
|
||||
|
||||
# Offense count: 1
|
||||
Style/MultilineBlockChain:
|
||||
Exclude:
|
||||
- 'app/controllers/valuation/spending_proposals_controller.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
Style/MultilineBlockLayout:
|
||||
Exclude:
|
||||
- 'app/models/banner.rb'
|
||||
- 'spec/features/users_auth_spec.rb'
|
||||
|
||||
# Offense count: 6
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: symmetrical, new_line, same_line
|
||||
Style/MultilineHashBraceLayout:
|
||||
Exclude:
|
||||
- 'app/controllers/valuation/spending_proposals_controller.rb'
|
||||
- 'app/models/debate.rb'
|
||||
- 'app/models/proposal.rb'
|
||||
- 'app/models/spending_proposal.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
Style/MultilineIfThen:
|
||||
Exclude:
|
||||
- 'app/controllers/management/users_controller.rb'
|
||||
- 'lib/census_api.rb'
|
||||
|
||||
# Offense count: 6
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: symmetrical, new_line, same_line
|
||||
Style/MultilineMethodCallBraceLayout:
|
||||
Exclude:
|
||||
- 'app/controllers/users_controller.rb'
|
||||
- 'app/models/comment.rb'
|
||||
- 'app/models/organization.rb'
|
||||
- 'app/models/user.rb'
|
||||
- 'spec/rails_helper.rb'
|
||||
|
||||
# Offense count: 33
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
|
||||
# SupportedStyles: aligned, indented, indented_relative_to_receiver
|
||||
Style/MultilineMethodCallIndentation:
|
||||
Exclude:
|
||||
- 'app/models/banner.rb'
|
||||
- 'app/models/tag_cloud.rb'
|
||||
- 'app/models/verification/management/managed_user.rb'
|
||||
- 'spec/models/comment_spec.rb'
|
||||
- 'spec/models/debate_spec.rb'
|
||||
- 'spec/models/proposal_spec.rb'
|
||||
- 'spec/models/user_spec.rb'
|
||||
|
||||
# Offense count: 5
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
|
||||
# SupportedStyles: aligned, indented
|
||||
Style/MultilineOperationIndentation:
|
||||
Exclude:
|
||||
- 'app/helpers/users_helper.rb'
|
||||
- 'app/helpers/valuation_helper.rb'
|
||||
- 'app/models/verification/letter.rb'
|
||||
|
||||
# Offense count: 5
|
||||
# Cop supports --auto-correct.
|
||||
Style/MutableConstant:
|
||||
Exclude:
|
||||
- 'app/models/activity.rb'
|
||||
- 'app/models/proposal.rb'
|
||||
- 'lib/tag_sanitizer.rb'
|
||||
- 'lib/wysiwyg_sanitizer.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Style/NegatedIf:
|
||||
Exclude:
|
||||
- 'spec/spec_helper.rb'
|
||||
|
||||
# Offense count: 17
|
||||
# Cop supports --auto-correct.
|
||||
Style/NestedParenthesizedCalls:
|
||||
Exclude:
|
||||
- 'spec/features/debates_spec.rb'
|
||||
- 'spec/features/valuation/spending_proposals_spec.rb'
|
||||
- 'spec/helpers/settings_helper_spec.rb'
|
||||
- 'spec/helpers/verification_helper_spec.rb'
|
||||
|
||||
# Offense count: 12
|
||||
# Cop supports --auto-correct.
|
||||
Style/NumericLiterals:
|
||||
MinDigits: 9
|
||||
|
||||
# Offense count: 19
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AutoCorrect, EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: predicate, comparison
|
||||
Style/NumericPredicate:
|
||||
Exclude:
|
||||
- 'spec/**/*'
|
||||
- 'app/controllers/users_controller.rb'
|
||||
- 'app/controllers/valuation/spending_proposals_controller.rb'
|
||||
- 'app/helpers/banners_helper.rb'
|
||||
- 'app/helpers/debates_helper.rb'
|
||||
- 'app/helpers/votes_helper.rb'
|
||||
- 'app/models/concerns/conflictable.rb'
|
||||
- 'app/models/concerns/taggable.rb'
|
||||
- 'app/models/concerns/verification.rb'
|
||||
- 'app/models/debate.rb'
|
||||
- 'app/models/lock.rb'
|
||||
- 'app/models/user.rb'
|
||||
- 'app/models/verification/management/email.rb'
|
||||
- 'lib/score_calculator.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
Style/ParallelAssignment:
|
||||
Exclude:
|
||||
- 'lib/active_model/dates.rb'
|
||||
- 'spec/support/common_actions.rb'
|
||||
|
||||
# Offense count: 3
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AllowSafeAssignment.
|
||||
Style/ParenthesesAroundCondition:
|
||||
Exclude:
|
||||
- 'app/controllers/proposals_controller.rb'
|
||||
- 'app/models/debate.rb'
|
||||
- 'app/models/proposal.rb'
|
||||
|
||||
# Offense count: 6
|
||||
# Configuration parameters: NamePrefix, NamePrefixBlacklist, NameWhitelist.
|
||||
# NamePrefix: is_, has_, have_
|
||||
# NamePrefixBlacklist: is_, has_, have_
|
||||
# NameWhitelist: is_a?
|
||||
Style/PredicateName:
|
||||
Exclude:
|
||||
- 'spec/**/*'
|
||||
- 'app/controllers/concerns/has_filters.rb'
|
||||
- 'app/controllers/concerns/has_orders.rb'
|
||||
- 'app/helpers/banners_helper.rb'
|
||||
- 'app/helpers/debates_helper.rb'
|
||||
- 'app/models/user.rb'
|
||||
- 'lib/census_api.rb'
|
||||
|
||||
# Offense count: 3
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: compact, exploded
|
||||
Style/RaiseArgs:
|
||||
Exclude:
|
||||
- 'app/controllers/management/base_controller.rb'
|
||||
- 'app/controllers/users/omniauth_callbacks_controller.rb'
|
||||
- 'app/controllers/valuation/spending_proposals_controller.rb'
|
||||
|
||||
# Offense count: 16
|
||||
# Cop supports --auto-correct.
|
||||
Style/RedundantParentheses:
|
||||
Exclude:
|
||||
- 'app/controllers/admin/settings_controller.rb'
|
||||
- 'app/helpers/proposals_helper.rb'
|
||||
- 'spec/features/admin/organizations_spec.rb'
|
||||
- 'spec/features/debates_spec.rb'
|
||||
- 'spec/features/proposals_spec.rb'
|
||||
- 'spec/models/debate_spec.rb'
|
||||
|
||||
# Offense count: 5
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AllowMultipleReturnValues.
|
||||
Style/RedundantReturn:
|
||||
Exclude:
|
||||
- 'app/models/ahoy/data_source.rb'
|
||||
- 'app/models/verification/management/document.rb'
|
||||
- 'lib/capistrano/template.rb'
|
||||
- 'lib/census_api.rb'
|
||||
|
||||
# Offense count: 49
|
||||
# Cop supports --auto-correct.
|
||||
Style/RedundantSelf:
|
||||
Enabled: false
|
||||
|
||||
# Offense count: 3
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles, AllowInnerSlashes.
|
||||
# SupportedStyles: slashes, percent_r, mixed
|
||||
Style/RegexpLiteral:
|
||||
Exclude:
|
||||
- 'app/helpers/embed_videos_helper.rb'
|
||||
- 'spec/customization_engine_spec.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
Style/RescueModifier:
|
||||
Exclude:
|
||||
- 'app/controllers/concerns/commentable_actions.rb'
|
||||
- 'app/controllers/verification/sms_controller.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
Style/SpaceAfterColon:
|
||||
Exclude:
|
||||
- 'spec/models/user_spec.rb'
|
||||
|
||||
# Offense count: 14
|
||||
# Cop supports --auto-correct.
|
||||
Style/SpaceAfterComma:
|
||||
Exclude:
|
||||
- 'app/models/ahoy/data_source.rb'
|
||||
- 'app/models/banner.rb'
|
||||
- 'app/models/concerns/search_cache.rb'
|
||||
- 'app/models/concerns/taggable.rb'
|
||||
- 'app/models/spending_proposal.rb'
|
||||
- 'app/models/user.rb'
|
||||
- 'spec/features/valuation/spending_proposals_spec.rb'
|
||||
- 'spec/lib/census_api_spec.rb'
|
||||
- 'spec/models/abilities/moderator_spec.rb'
|
||||
- 'spec/models/verification/management/email_spec.rb'
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Style/SpaceAfterNot:
|
||||
Exclude:
|
||||
- 'app/models/flag.rb'
|
||||
|
||||
# Offense count: 20
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: space, no_space
|
||||
Style/SpaceAroundEqualsInParameterDefault:
|
||||
Exclude:
|
||||
- 'app/helpers/cache_keys_helper.rb'
|
||||
- 'app/helpers/proposals_helper.rb'
|
||||
- 'app/helpers/spending_proposals_helper.rb'
|
||||
- 'app/helpers/stats_helper.rb'
|
||||
- 'app/helpers/tracks_helper.rb'
|
||||
- 'app/helpers/valuation_helper.rb'
|
||||
- 'app/mailers/devise_mailer.rb'
|
||||
- 'app/models/comment.rb'
|
||||
- 'app/models/tag_cloud.rb'
|
||||
- 'app/models/verification/residence.rb'
|
||||
- 'lib/acts_as_paranoid_aliases.rb'
|
||||
- 'lib/capistrano/template.rb'
|
||||
- 'lib/census_api.rb'
|
||||
- 'lib/manager_authenticator.rb'
|
||||
- 'spec/support/common_actions.rb'
|
||||
|
||||
# Offense count: 66
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AllowForAlignment.
|
||||
Style/SpaceAroundOperators:
|
||||
Enabled: false
|
||||
|
||||
# Offense count: 6
|
||||
# Cop supports --auto-correct.
|
||||
Style/SpaceBeforeComma:
|
||||
Exclude:
|
||||
- 'app/models/proposal.rb'
|
||||
- 'spec/controllers/management/sessions_controller_spec.rb'
|
||||
- 'spec/controllers/management/users_controller_spec.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: AllowForAlignment.
|
||||
Style/SpaceBeforeFirstArg:
|
||||
Exclude:
|
||||
- 'spec/factories.rb'
|
||||
|
||||
# Offense count: 9
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: require_no_space, require_space
|
||||
Style/SpaceInLambdaLiteral:
|
||||
Exclude:
|
||||
- 'app/models/concerns/filterable.rb'
|
||||
- 'app/models/spending_proposal.rb'
|
||||
- 'app/models/user.rb'
|
||||
- 'app/models/verified_user.rb'
|
||||
|
||||
# Offense count: 8
|
||||
# Cop supports --auto-correct.
|
||||
Style/SpaceInsideParens:
|
||||
Exclude:
|
||||
- 'app/models/user.rb'
|
||||
- 'lib/manager_authenticator.rb'
|
||||
- 'spec/features/proposals_spec.rb'
|
||||
- 'spec/models/abilities/moderator_spec.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
Style/SpaceInsidePercentLiteralDelimiters:
|
||||
Exclude:
|
||||
- 'app/models/activity.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: EnforcedStyle, SupportedStyles.
|
||||
# SupportedStyles: single_quotes, double_quotes
|
||||
Style/StringLiteralsInInterpolation:
|
||||
Exclude:
|
||||
- 'app/controllers/stats_controller.rb'
|
||||
- 'app/models/proposal.rb'
|
||||
|
||||
# Offense count: 9
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: IgnoredMethods.
|
||||
# IgnoredMethods: respond_to, define_method
|
||||
Style/SymbolProc:
|
||||
Exclude:
|
||||
- 'app/controllers/admin/settings_controller.rb'
|
||||
- 'app/controllers/notifications_controller.rb'
|
||||
- 'app/controllers/organizations/registrations_controller.rb'
|
||||
- 'lib/manager_authenticator.rb'
|
||||
- 'spec/factories.rb'
|
||||
|
||||
# Offense count: 2
|
||||
# Cop supports --auto-correct.
|
||||
Style/TrailingWhitespace:
|
||||
Exclude:
|
||||
- 'app/controllers/admin/api/stats_controller.rb'
|
||||
|
||||
# Offense count: 31
|
||||
# Cop supports --auto-correct.
|
||||
Style/UnneededInterpolation:
|
||||
Exclude:
|
||||
- 'app/controllers/users/omniauth_callbacks_controller.rb'
|
||||
- 'spec/factories.rb'
|
||||
- 'spec/features/campaigns_spec.rb'
|
||||
- 'spec/features/management/managed_users_spec.rb'
|
||||
- 'spec/features/management/proposals_spec.rb'
|
||||
- 'spec/features/management/spending_proposals_spec.rb'
|
||||
- 'spec/models/residence_spec.rb'
|
||||
- 'spec/models/spending_proposal_spec.rb'
|
||||
|
||||
# Offense count: 13
|
||||
# Configuration parameters: SupportedStyles.
|
||||
# SupportedStyles: snake_case, normalcase, non_integer
|
||||
Style/VariableNumber:
|
||||
EnforcedStyle: normalcase
|
||||
|
||||
# Offense count: 9
|
||||
# Cop supports --auto-correct.
|
||||
# Configuration parameters: SupportedStyles, WordRegex.
|
||||
# SupportedStyles: percent, brackets
|
||||
Style/WordArray:
|
||||
EnforcedStyle: percent
|
||||
MinSize: 3
|
||||
|
||||
# Offense count: 1
|
||||
# Cop supports --auto-correct.
|
||||
Style/ZeroLengthPredicate:
|
||||
Exclude:
|
||||
- 'app/models/concerns/verification.rb'
|
||||
@@ -1,8 +1,8 @@
|
||||
# Personalización
|
||||
|
||||
Puedes modificar consul y ponerle tu propia imagen, para esto debes primero hacer un fork de https://github.com/consul/consul creando un repositorio nuevo en Github. Puedes usar otro servicio como Gitlab, pero no te olvides de poner el enlace en el footer a tu repositorio en cumplimiento con la licencia de este proyecto (GPL Affero 3).
|
||||
Puedes modificar consul y ponerle tu propia imagen, para esto debes primero hacer un fork de https://github.com/consul/consul creando un repositorio nuevo en Github. Puedes usar otro servicio como Gitlab, pero no te olvides de poner el enlace en el footer a tu repositorio en cumplimiento con la licencia de este proyecto (GPL Affero 3).
|
||||
|
||||
Hemos creado una estructura específica donde puedes sobreescribir y personalizar la aplicación para que puedas actualizar sin que tengas problemas al hacer merge y se sobreescriban por error tus cambios. Intentamos que Consul sea una aplicación Ruby on Rails lo más plain vanilla posible para facilitar el acceso de nuevas desarrolladoras.
|
||||
Hemos creado una estructura específica donde puedes sobreescribir y personalizar la aplicación para que puedas actualizar sin que tengas problemas al hacer merge y se sobreescriban por error tus cambios. Intentamos que Consul sea una aplicación Ruby on Rails lo más plain vanilla posible para facilitar el acceso de nuevas desarrolladoras.
|
||||
|
||||
## Ficheros y directorios especiales
|
||||
|
||||
@@ -21,16 +21,16 @@ Aparte de estos directorios también cuentas con ciertos ficheros para:
|
||||
* Gemfile_custom
|
||||
* config/application.custom.rb
|
||||
|
||||
### Internacionalización
|
||||
### Internacionalización
|
||||
|
||||
Si quieres modificar algún texto de la web deberías encontrarlos en los ficheros formato YML disponibles en *config/locales/*. Puedes leer la [guía de internacionalización](http://guides.rubyonrails.org/i18n.html) de Ruby on Rails sobre como funciona este sistema.
|
||||
Si quieres modificar algún texto de la web deberías encontrarlos en los ficheros formato YML disponibles en *config/locales/*. Puedes leer la [guía de internacionalización](http://guides.rubyonrails.org/i18n.html) de Ruby on Rails sobre como funciona este sistema.
|
||||
|
||||
Las adaptaciones los debes poner en el directorio *config/locales/custom/*, recomendamos poner solo los textos que quieras personalizar. Por ejemplo si quieres personalizar el texto de "Ayuntamiento de Madrid, 2016" que se encuentra en el footer en todas las páginas, primero debemos ubicar en que plantilla se encuentra (app/views/layouts/_footer.html.erb), vemos que en el código pone lo siguiente:
|
||||
Las adaptaciones los debes poner en el directorio *config/locales/custom/*, recomendamos poner solo los textos que quieras personalizar. Por ejemplo si quieres personalizar el texto de "Ayuntamiento de Madrid, 2016" que se encuentra en el footer en todas las páginas, primero debemos ubicar en que plantilla se encuentra (app/views/layouts/_footer.html.erb), vemos que en el código pone lo siguiente:
|
||||
```
|
||||
<%= t("layouts.footer.copyright", year: Time.now.year) %>
|
||||
<%= t("layouts.footer.copyright", year: Time.current.year) %>
|
||||
```
|
||||
|
||||
Y que en el fichero config/locales/es.yml sigue esta estructura (solo ponemos lo relevante para este caso):
|
||||
Y que en el fichero config/locales/es.yml sigue esta estructura (solo ponemos lo relevante para este caso):
|
||||
|
||||
```
|
||||
es:
|
||||
@@ -40,11 +40,11 @@ es:
|
||||
|
||||
```
|
||||
|
||||
Si creamos el fichero config/locales/custom/es.yml y modificamos "Ayuntamiento de Madrid" por el nombre de la organización que se este haciendo la modificación. Recomendamos directamente copiar los ficheros config/locales/ e ir revisando y corrigiendo las que querramos, borrando las líneas que no querramos traducir.
|
||||
Si creamos el fichero config/locales/custom/es.yml y modificamos "Ayuntamiento de Madrid" por el nombre de la organización que se este haciendo la modificación. Recomendamos directamente copiar los ficheros config/locales/ e ir revisando y corrigiendo las que querramos, borrando las líneas que no querramos traducir.
|
||||
|
||||
### Imágenes
|
||||
|
||||
Si quieres sobreescribir alguna imagen debes primero fijarte el nombre que tiene, por defecto se encuentran en *app/assets/images*. Por ejemplo si quieres modificar *app/assets/images/logo_header.png* debes poner otra con ese mismo nombre en el directorio app/assets/images/custom. Los iconos que seguramente quieras modificar son:
|
||||
Si quieres sobreescribir alguna imagen debes primero fijarte el nombre que tiene, por defecto se encuentran en *app/assets/images*. Por ejemplo si quieres modificar *app/assets/images/logo_header.png* debes poner otra con ese mismo nombre en el directorio app/assets/images/custom. Los iconos que seguramente quieras modificar son:
|
||||
|
||||
* apple-touch-icon-200.png
|
||||
* icon_home.png
|
||||
@@ -59,7 +59,7 @@ Si quieres modificar el HTML de alguna página puedes hacerlo copiando el HTML d
|
||||
|
||||
### CSS
|
||||
|
||||
Si quieres cambiar algun selector CSS (de las hojas de estilo) puedes hacerlo en el fichero *app/assets/stylesheets/custom.scss*. Por ejemplo si quieres cambiar el color del header (.top-links) puedes hacerlo agregando:
|
||||
Si quieres cambiar algun selector CSS (de las hojas de estilo) puedes hacerlo en el fichero *app/assets/stylesheets/custom.scss*. Por ejemplo si quieres cambiar el color del header (.top-links) puedes hacerlo agregando:
|
||||
|
||||
```
|
||||
.top-links {
|
||||
@@ -67,11 +67,11 @@ Si quieres cambiar algun selector CSS (de las hojas de estilo) puedes hacerlo en
|
||||
}
|
||||
```
|
||||
|
||||
Usamos un preprocesador de CSS, [SASS, con la sintaxis SCSS](http://sass-lang.com/guide).
|
||||
Usamos un preprocesador de CSS, [SASS, con la sintaxis SCSS](http://sass-lang.com/guide).
|
||||
|
||||
### Javascript
|
||||
|
||||
Si quieres agregar código Javascript puedes hacerlo en el fichero *app/assets/javascripts/custom.js". Por ejemplo si quieres que salga una alerta puedes poner lo siguiente:
|
||||
Si quieres agregar código Javascript puedes hacerlo en el fichero *app/assets/javascripts/custom.js". Por ejemplo si quieres que salga una alerta puedes poner lo siguiente:
|
||||
|
||||
```
|
||||
$(function(){
|
||||
@@ -81,9 +81,9 @@ $(function(){
|
||||
|
||||
### Modelos
|
||||
|
||||
Si quieres agregar modelos nuevos, o modificar o agregar métodos a uno ya existente puedes hacerlo en *app/models/custom*. En el caso de los modelos antiguos debes primero hacer un require de la dependencia.
|
||||
Si quieres agregar modelos nuevos, o modificar o agregar métodos a uno ya existente puedes hacerlo en *app/models/custom*. En el caso de los modelos antiguos debes primero hacer un require de la dependencia.
|
||||
|
||||
Por ejemplo en el caso del Ayuntamiento de Madrid se requiere comprobar que el código postal durante la verificación sigue un cierto formato (empieza con 280). Esto se realiza creando este fichero en *app/models/custom/verification/residence.rb*:
|
||||
Por ejemplo en el caso del Ayuntamiento de Madrid se requiere comprobar que el código postal durante la verificación sigue un cierto formato (empieza con 280). Esto se realiza creando este fichero en *app/models/custom/verification/residence.rb*:
|
||||
|
||||
```
|
||||
require_dependency Rails.root.join('app', 'models', 'verification', 'residence').to_s
|
||||
@@ -123,10 +123,10 @@ No olvides poner los tests relevantes en *spec/models/custom*, siguiendo con el
|
||||
require 'rails_helper'
|
||||
|
||||
describe Verification::Residence do
|
||||
|
||||
|
||||
let(:residence) { build(:verification_residence, document_number: "12345678Z") }
|
||||
|
||||
describe "verification" do
|
||||
describe "verification" do
|
||||
|
||||
describe "postal code" do
|
||||
it "should be valid with postal codes starting with 280" do
|
||||
@@ -162,7 +162,7 @@ TODO
|
||||
|
||||
### Gemfile
|
||||
|
||||
Para agregar librerías (gems) nuevas puedes hacerlo en el fichero *Gemfile_custom*. Por ejemplo si quieres agregar la gema [rails-footnotes](https://github.com/josevalim/rails-footnotes) debes hacerlo agregandole
|
||||
Para agregar librerías (gems) nuevas puedes hacerlo en el fichero *Gemfile_custom*. Por ejemplo si quieres agregar la gema [rails-footnotes](https://github.com/josevalim/rails-footnotes) debes hacerlo agregandole
|
||||
|
||||
```
|
||||
gem 'rails-footnotes', '~> 4.0'
|
||||
@@ -170,9 +170,9 @@ gem 'rails-footnotes', '~> 4.0'
|
||||
|
||||
Y siguiendo el flujo clásico en Ruby on Rails (bundle install y seguir con los pasos específicos de la gema en la documentación)
|
||||
|
||||
### application.rb
|
||||
### application.rb
|
||||
|
||||
Cuando necesites extender o modificar el *config/application.rb* puedes hacerlo a través del fichero *config/application_custom.rb*. Por ejemplo si quieres modificar el idioma por defecto al inglés pondrías lo siguiente:
|
||||
Cuando necesites extender o modificar el *config/application.rb* puedes hacerlo a través del fichero *config/application_custom.rb*. Por ejemplo si quieres modificar el idioma por defecto al inglés pondrías lo siguiente:
|
||||
|
||||
|
||||
```
|
||||
@@ -198,15 +198,15 @@ TODO
|
||||
|
||||
TODO
|
||||
|
||||
## Actualizar
|
||||
## Actualizar
|
||||
|
||||
Te recomendamos que agregues el remote de consul para facilitar este proceso de merge:
|
||||
Te recomendamos que agregues el remote de consul para facilitar este proceso de merge:
|
||||
|
||||
```
|
||||
$ git remote add consul https://github.com/consul/consul
|
||||
$ git remote add consul https://github.com/consul/consul
|
||||
```
|
||||
|
||||
Con esto puedes actualizarte con
|
||||
Con esto puedes actualizarte con
|
||||
|
||||
```
|
||||
git checkout -b consul_update
|
||||
|
||||
@@ -522,6 +522,3 @@ DEPENDENCIES
|
||||
unicorn (~> 5.1.0)
|
||||
web-console (= 3.3.0)
|
||||
whenever
|
||||
|
||||
BUNDLED WITH
|
||||
1.13.3
|
||||
|
||||
@@ -41,7 +41,7 @@ class DebatesController < ApplicationController
|
||||
end
|
||||
|
||||
def mark_featured
|
||||
@debate.update_attribute(:featured_at, Time.now)
|
||||
@debate.update_attribute(:featured_at, Time.current)
|
||||
redirect_to request.query_parameters.merge(action: :index)
|
||||
end
|
||||
|
||||
@@ -55,4 +55,4 @@ class DebatesController < ApplicationController
|
||||
Debate
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -8,8 +8,8 @@ class Management::UsersController < Management::BaseController
|
||||
@user = User.new(user_params)
|
||||
@user.skip_password_validation = true
|
||||
@user.terms_of_service = '1'
|
||||
@user.residence_verified_at = Time.now
|
||||
@user.verified_at = Time.now
|
||||
@user.residence_verified_at = Time.current
|
||||
@user.verified_at = Time.current
|
||||
|
||||
if @user.save then
|
||||
render :show
|
||||
|
||||
@@ -38,7 +38,7 @@ class ProposalsController < ApplicationController
|
||||
end
|
||||
|
||||
def retire
|
||||
if valid_retired_params? && @proposal.update(retired_params.merge(retired_at: Time.now))
|
||||
if valid_retired_params? && @proposal.update(retired_params.merge(retired_at: Time.current))
|
||||
redirect_to proposal_path(@proposal), notice: t('proposals.notice.retired')
|
||||
else
|
||||
render action: :retire_form
|
||||
@@ -107,4 +107,4 @@ class ProposalsController < ApplicationController
|
||||
@proposal_successfull_exists = Proposal.successfull.exists?
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -23,6 +23,6 @@ class StatsController < ApplicationController
|
||||
private
|
||||
|
||||
def daily_cache(key, &block)
|
||||
Rails.cache.fetch("public_stats/#{Time.now.strftime("%Y-%m-%d")}/#{key}", &block)
|
||||
Rails.cache.fetch("public_stats/#{Time.current.strftime("%Y-%m-%d")}/#{key}", &block)
|
||||
end
|
||||
end
|
||||
|
||||
@@ -6,7 +6,7 @@ class Verification::EmailController < ApplicationController
|
||||
|
||||
def show
|
||||
if Verification::Email.find(current_user, params[:email_verification_token])
|
||||
current_user.update(verified_at: Time.now)
|
||||
current_user.update(verified_at: Time.current)
|
||||
redirect_to account_path, notice: t('verification.email.show.flash.success')
|
||||
else
|
||||
redirect_to verified_user_path, alert: t('verification.email.show.alert.failure')
|
||||
|
||||
@@ -29,7 +29,7 @@ class Verification::LetterController < ApplicationController
|
||||
def update
|
||||
@letter = Verification::Letter.new(letter_params.merge(user: current_user, verify: true))
|
||||
if @letter.valid?
|
||||
current_user.update(verified_at: Time.now)
|
||||
current_user.update(verified_at: Time.current)
|
||||
redirect_to account_path, notice: t('verification.letter.update.flash.success')
|
||||
else
|
||||
Lock.increase_tries(@letter.user) if @letter.user
|
||||
@@ -56,4 +56,4 @@ class Verification::LetterController < ApplicationController
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -31,7 +31,7 @@ class Verification::SmsController < ApplicationController
|
||||
ahoy.track(:level_2_user, user_id: current_user.id) rescue nil
|
||||
|
||||
if VerifiedUser.phone?(current_user)
|
||||
current_user.update(verified_at: Time.now)
|
||||
current_user.update(verified_at: Time.current)
|
||||
end
|
||||
|
||||
redirect_to_next_path
|
||||
@@ -69,4 +69,4 @@ class Verification::SmsController < ApplicationController
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -12,9 +12,9 @@ class Banner < ActiveRecord::Base
|
||||
validates :post_started_at, presence: true
|
||||
validates :post_ended_at, presence: true
|
||||
|
||||
scope :with_active, -> {where("post_started_at <= ?", Time.now).
|
||||
where("post_ended_at >= ?", Time.now) }
|
||||
scope :with_active, -> {where("post_started_at <= ?", Time.current).
|
||||
where("post_ended_at >= ?", Time.current) }
|
||||
|
||||
scope :with_inactive,-> {where("post_started_at > ? or post_ended_at < ?", Time.now, Time.now) }
|
||||
scope :with_inactive,-> {where("post_started_at > ? or post_ended_at < ?", Time.current, Time.current) }
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -13,7 +13,7 @@ module Flaggable
|
||||
end
|
||||
|
||||
def ignore_flag
|
||||
update(ignored_flag_at: Time.now)
|
||||
update(ignored_flag_at: Time.current)
|
||||
end
|
||||
|
||||
end
|
||||
|
||||
@@ -4,7 +4,7 @@ class Lock < ActiveRecord::Base
|
||||
before_save :set_locked_until
|
||||
|
||||
def locked?
|
||||
locked_until > Time.now
|
||||
locked_until > Time.current
|
||||
end
|
||||
|
||||
def set_locked_until
|
||||
@@ -12,7 +12,7 @@ class Lock < ActiveRecord::Base
|
||||
end
|
||||
|
||||
def lock_time
|
||||
Time.now + (2**tries).minutes
|
||||
Time.current + (2**tries).minutes
|
||||
end
|
||||
|
||||
def too_many_tries?
|
||||
|
||||
@@ -14,11 +14,11 @@ class Organization < ActiveRecord::Base
|
||||
scope :rejected, -> { where.not(rejected_at: nil).where("(organizations.verified_at IS NULL or organizations.verified_at < rejected_at)") }
|
||||
|
||||
def verify
|
||||
update(verified_at: Time.now)
|
||||
update(verified_at: Time.current)
|
||||
end
|
||||
|
||||
def reject
|
||||
update(rejected_at: Time.now)
|
||||
update(rejected_at: Time.current)
|
||||
end
|
||||
|
||||
def verified?
|
||||
|
||||
@@ -9,9 +9,9 @@ class ProposalNotification < ActiveRecord::Base
|
||||
|
||||
def minimum_interval
|
||||
return true if proposal.try(:notifications).blank?
|
||||
if proposal.notifications.last.created_at > (Time.now - Setting[:proposal_notification_minimum_interval_in_days].to_i.days).to_datetime
|
||||
if proposal.notifications.last.created_at > (Time.current - Setting[:proposal_notification_minimum_interval_in_days].to_i.days).to_datetime
|
||||
errors.add(:title, I18n.t('activerecord.errors.models.proposal_notification.attributes.minimum_interval.invalid', interval: Setting[:proposal_notification_minimum_interval_in_days]))
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -114,7 +114,7 @@ class SpendingProposal < ActiveRecord::Base
|
||||
|
||||
def send_unfeasible_email
|
||||
Mailer.unfeasible_spending_proposal(self).deliver_later
|
||||
update(unfeasible_email_sent_at: Time.now)
|
||||
update(unfeasible_email_sent_at: Time.current)
|
||||
end
|
||||
|
||||
def reason_for_not_being_votable_by(user)
|
||||
|
||||
@@ -69,7 +69,7 @@ class User < ActiveRecord::Base
|
||||
oauth_email: oauth_email,
|
||||
password: Devise.friendly_token[0,20],
|
||||
terms_of_service: '1',
|
||||
confirmed_at: oauth_email_confirmed ? DateTime.now : nil
|
||||
confirmed_at: oauth_email_confirmed ? DateTime.current : nil
|
||||
)
|
||||
end
|
||||
|
||||
@@ -158,7 +158,7 @@ class User < ActiveRecord::Base
|
||||
|
||||
def erase(erase_reason = nil)
|
||||
self.update(
|
||||
erased_at: Time.now,
|
||||
erased_at: Time.current,
|
||||
erase_reason: erase_reason,
|
||||
username: nil,
|
||||
email: nil,
|
||||
@@ -258,4 +258,4 @@ class User < ActiveRecord::Base
|
||||
validator.validate(self)
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -17,7 +17,7 @@ class Verification::Letter
|
||||
end
|
||||
|
||||
def letter_requested!
|
||||
user.update(letter_requested_at: Time.now, letter_verification_code: generate_verification_code)
|
||||
user.update(letter_requested_at: Time.current, letter_verification_code: generate_verification_code)
|
||||
end
|
||||
|
||||
def validate_existing_user
|
||||
|
||||
@@ -40,7 +40,7 @@ class Verification::Management::Document
|
||||
end
|
||||
|
||||
def verify
|
||||
user.update(verified_at: Time.now) if user?
|
||||
user.update(verified_at: Time.current) if user?
|
||||
end
|
||||
|
||||
end
|
||||
|
||||
@@ -27,8 +27,8 @@ class Verification::Management::Email
|
||||
|
||||
user.update(document_type: document_type,
|
||||
document_number: document_number,
|
||||
residence_verified_at: Time.now,
|
||||
level_two_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
level_two_verified_at: Time.current,
|
||||
email_verification_token: plain_token)
|
||||
|
||||
Mailer.email_verification(user, email, encrypted_token, document_type, document_number).deliver_later
|
||||
|
||||
@@ -31,7 +31,7 @@ class Verification::Residence
|
||||
geozone: self.geozone,
|
||||
date_of_birth: date_of_birth.to_datetime,
|
||||
gender: gender,
|
||||
residence_verified_at: Time.now)
|
||||
residence_verified_at: Time.current)
|
||||
end
|
||||
|
||||
def allowed_age
|
||||
|
||||
@@ -42,7 +42,7 @@
|
||||
<div class="subfooter row">
|
||||
<div class="small-12 medium-8 column">
|
||||
<p>
|
||||
<%= t("layouts.footer.copyright", year: Time.now.year) %> |
|
||||
<%= t("layouts.footer.copyright", year: Time.current.year) %> |
|
||||
<%= link_to t("layouts.footer.more_info"), page_path('more_information') %> |
|
||||
<%= link_to t("layouts.footer.privacy"), page_path('privacy') %> |
|
||||
<%= link_to t("layouts.footer.conditions"), page_path('conditions') %> |
|
||||
|
||||
@@ -41,7 +41,7 @@ puts "Creating Users"
|
||||
def create_user(email, username = Faker::Name.name)
|
||||
pwd = '12345678'
|
||||
puts " #{username}"
|
||||
User.create!(username: username, email: email, password: pwd, password_confirmation: pwd, confirmed_at: Time.now, terms_of_service: "1")
|
||||
User.create!(username: username, email: email, password: pwd, password_confirmation: pwd, confirmed_at: Time.current, terms_of_service: "1")
|
||||
end
|
||||
|
||||
admin = create_user('admin@consul.dev', 'admin')
|
||||
@@ -54,10 +54,10 @@ valuator = create_user('valuator@consul.dev', 'valuator')
|
||||
valuator.create_valuator
|
||||
|
||||
level_2 = create_user('leveltwo@consul.dev', 'level 2')
|
||||
level_2.update(residence_verified_at: Time.now, confirmed_phone: Faker::PhoneNumber.phone_number, document_number: "2222222222", document_type: "1" )
|
||||
level_2.update(residence_verified_at: Time.current, confirmed_phone: Faker::PhoneNumber.phone_number, document_number: "2222222222", document_type: "1" )
|
||||
|
||||
verified = create_user('verified@consul.dev', 'verified')
|
||||
verified.update(residence_verified_at: Time.now, confirmed_phone: Faker::PhoneNumber.phone_number, document_type: "1", verified_at: Time.now, document_number: "3333333333")
|
||||
verified.update(residence_verified_at: Time.current, confirmed_phone: Faker::PhoneNumber.phone_number, document_type: "1", verified_at: Time.current, document_number: "3333333333")
|
||||
|
||||
(1..10).each do |i|
|
||||
org_name = Faker::Company.name
|
||||
@@ -82,10 +82,10 @@ end
|
||||
user = create_user("user#{i}@consul.dev")
|
||||
level = [1, 2, 3].sample
|
||||
if level >= 2
|
||||
user.update(residence_verified_at: Time.now, confirmed_phone: Faker::PhoneNumber.phone_number, document_number: Faker::Number.number(10), document_type: "1" )
|
||||
user.update(residence_verified_at: Time.current, confirmed_phone: Faker::PhoneNumber.phone_number, document_number: Faker::Number.number(10), document_type: "1" )
|
||||
end
|
||||
if level == 3
|
||||
user.update(verified_at: Time.now, document_number: Faker::Number.number(10) )
|
||||
user.update(verified_at: Time.current, document_number: Faker::Number.number(10) )
|
||||
end
|
||||
end
|
||||
|
||||
@@ -118,7 +118,7 @@ tags = Faker::Lorem.words(25)
|
||||
description = "<p>#{Faker::Lorem.paragraphs.join('</p><p>')}</p>"
|
||||
debate = Debate.create!(author: author,
|
||||
title: Faker::Lorem.sentence(3).truncate(60),
|
||||
created_at: rand((Time.now - 1.week) .. Time.now),
|
||||
created_at: rand((Time.current - 1.week) .. Time.current),
|
||||
description: description,
|
||||
tag_list: tags.sample(3).join(','),
|
||||
geozone: Geozone.reorder("RANDOM()").first,
|
||||
@@ -133,7 +133,7 @@ tags = ActsAsTaggableOn::Tag.where(kind: 'category')
|
||||
description = "<p>#{Faker::Lorem.paragraphs.join('</p><p>')}</p>"
|
||||
debate = Debate.create!(author: author,
|
||||
title: Faker::Lorem.sentence(3).truncate(60),
|
||||
created_at: rand((Time.now - 1.week) .. Time.now),
|
||||
created_at: rand((Time.current - 1.week) .. Time.current),
|
||||
description: description,
|
||||
tag_list: tags.sample(3).join(','),
|
||||
geozone: Geozone.reorder("RANDOM()").first,
|
||||
@@ -155,7 +155,7 @@ tags = Faker::Lorem.words(25)
|
||||
responsible_name: Faker::Name.name,
|
||||
external_url: Faker::Internet.url,
|
||||
description: description,
|
||||
created_at: rand((Time.now - 1.week) .. Time.now),
|
||||
created_at: rand((Time.current - 1.week) .. Time.current),
|
||||
tag_list: tags.sample(3).join(','),
|
||||
geozone: Geozone.reorder("RANDOM()").first,
|
||||
terms_of_service: "1")
|
||||
@@ -175,7 +175,7 @@ tags = Faker::Lorem.words(25)
|
||||
responsible_name: Faker::Name.name,
|
||||
external_url: Faker::Internet.url,
|
||||
description: description,
|
||||
created_at: rand((Time.now - 1.week) .. Time.now),
|
||||
created_at: rand((Time.current - 1.week) .. Time.current),
|
||||
tag_list: tags.sample(3).join(','),
|
||||
geozone: Geozone.reorder("RANDOM()").first,
|
||||
terms_of_service: "1",
|
||||
@@ -195,7 +195,7 @@ tags = ActsAsTaggableOn::Tag.where(kind: 'category')
|
||||
responsible_name: Faker::Name.name,
|
||||
external_url: Faker::Internet.url,
|
||||
description: description,
|
||||
created_at: rand((Time.now - 1.week) .. Time.now),
|
||||
created_at: rand((Time.current - 1.week) .. Time.current),
|
||||
tag_list: tags.sample(3).join(','),
|
||||
geozone: Geozone.reorder("RANDOM()").first,
|
||||
terms_of_service: "1")
|
||||
@@ -209,7 +209,7 @@ puts "Commenting Debates"
|
||||
author = User.reorder("RANDOM()").first
|
||||
debate = Debate.reorder("RANDOM()").first
|
||||
Comment.create!(user: author,
|
||||
created_at: rand(debate.created_at .. Time.now),
|
||||
created_at: rand(debate.created_at .. Time.current),
|
||||
commentable: debate,
|
||||
body: Faker::Lorem.sentence)
|
||||
end
|
||||
@@ -221,7 +221,7 @@ puts "Commenting Proposals"
|
||||
author = User.reorder("RANDOM()").first
|
||||
proposal = Proposal.reorder("RANDOM()").first
|
||||
Comment.create!(user: author,
|
||||
created_at: rand(proposal.created_at .. Time.now),
|
||||
created_at: rand(proposal.created_at .. Time.current),
|
||||
commentable: proposal,
|
||||
body: Faker::Lorem.sentence)
|
||||
end
|
||||
@@ -233,7 +233,7 @@ puts "Commenting Comments"
|
||||
author = User.reorder("RANDOM()").first
|
||||
parent = Comment.reorder("RANDOM()").first
|
||||
Comment.create!(user: author,
|
||||
created_at: rand(parent.created_at .. Time.now),
|
||||
created_at: rand(parent.created_at .. Time.current),
|
||||
commentable_id: parent.commentable_id,
|
||||
commentable_type: parent.commentable_type,
|
||||
body: Faker::Lorem.sentence,
|
||||
@@ -299,7 +299,7 @@ tags = Faker::Lorem.words(10)
|
||||
title: Faker::Lorem.sentence(3).truncate(60),
|
||||
external_url: Faker::Internet.url,
|
||||
description: description,
|
||||
created_at: rand((Time.now - 1.week) .. Time.now),
|
||||
created_at: rand((Time.current - 1.week) .. Time.current),
|
||||
geozone: [geozone, nil].sample,
|
||||
feasible: feasible,
|
||||
feasible_explanation: feasible_explanation,
|
||||
@@ -353,8 +353,8 @@ Proposal.last(3).each do |proposal|
|
||||
image: ["banner-img banner-img-one", "banner-img banner-img-two",
|
||||
"banner-img banner-img-three"].sample,
|
||||
target_url: Rails.application.routes.url_helpers.proposal_path(proposal),
|
||||
post_started_at: rand((Time.now - 1.week) .. (Time.now - 1.day)),
|
||||
post_ended_at: rand((Time.now - 1.day) .. (Time.now + 1.week)),
|
||||
created_at: rand((Time.now - 1.week) .. Time.now))
|
||||
post_started_at: rand((Time.current - 1.week) .. (Time.current - 1.day)),
|
||||
post_ended_at: rand((Time.current - 1.day) .. (Time.current + 1.week)),
|
||||
created_at: rand((Time.current - 1.week) .. Time.current))
|
||||
puts " #{banner.title}"
|
||||
end
|
||||
|
||||
@@ -1,7 +1,7 @@
|
||||
# coding: utf-8
|
||||
# Default admin user (change password after first deploy to a server!)
|
||||
if Administrator.count == 0 && !Rails.env.test?
|
||||
admin = User.create!(username: 'admin', email: 'admin@consul.dev', password: '12345678', password_confirmation: '12345678', confirmed_at: Time.now, terms_of_service: "1")
|
||||
admin = User.create!(username: 'admin', email: 'admin@consul.dev', password: '12345678', password_confirmation: '12345678', confirmed_at: Time.current, terms_of_service: "1")
|
||||
admin.create_administrator
|
||||
end
|
||||
|
||||
@@ -80,4 +80,4 @@ Setting['banner-img.banner-img-three'] = "Banner image 3"
|
||||
|
||||
# Proposal notifications
|
||||
Setting['proposal_notification_minimum_interval_in_days'] = 3
|
||||
Setting['direct_message_max_per_day'] = 3
|
||||
Setting['direct_message_max_per_day'] = 3
|
||||
|
||||
@@ -5,7 +5,7 @@ module ActsAsParanoidAliases
|
||||
|
||||
def hide
|
||||
return false if hidden?
|
||||
update_attribute(:hidden_at, Time.now)
|
||||
update_attribute(:hidden_at, Time.current)
|
||||
after_hide
|
||||
end
|
||||
|
||||
@@ -21,7 +21,7 @@ module ActsAsParanoidAliases
|
||||
end
|
||||
|
||||
def confirm_hide
|
||||
update_attribute(:confirmed_hide_at, Time.now)
|
||||
update_attribute(:confirmed_hide_at, Time.current)
|
||||
end
|
||||
|
||||
def restore(opts={})
|
||||
|
||||
@@ -21,7 +21,7 @@ class EmailDigest
|
||||
end
|
||||
|
||||
def mark_as_emailed
|
||||
notifications.update_all(emailed_at: Time.now)
|
||||
notifications.update_all(emailed_at: Time.current)
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -11,7 +11,7 @@ module ScoreCalculator
|
||||
score = ups - downs
|
||||
offset = Math.log([score.abs, 1].max, 10) * (ups / [total, 1].max)
|
||||
sign = score <=> 0
|
||||
seconds = ((date || Time.now) - EPOC).to_f
|
||||
seconds = ((date || Time.current) - EPOC).to_f
|
||||
|
||||
(((offset * sign) + (seconds/TIME_UNIT)) * 10000000).round
|
||||
end
|
||||
|
||||
@@ -7,7 +7,7 @@ FactoryGirl.define do
|
||||
|
||||
password 'judgmentday'
|
||||
terms_of_service '1'
|
||||
confirmed_at { Time.now }
|
||||
confirmed_at { Time.current }
|
||||
|
||||
trait :incomplete_verification do
|
||||
after :create do |user|
|
||||
@@ -16,7 +16,7 @@ FactoryGirl.define do
|
||||
end
|
||||
|
||||
trait :level_two do
|
||||
residence_verified_at Time.now
|
||||
residence_verified_at Time.current
|
||||
unconfirmed_phone "611111111"
|
||||
confirmed_phone "611111111"
|
||||
sms_confirmation_code "1234"
|
||||
@@ -25,17 +25,17 @@ FactoryGirl.define do
|
||||
end
|
||||
|
||||
trait :level_three do
|
||||
verified_at Time.now
|
||||
verified_at Time.current
|
||||
document_type "1"
|
||||
document_number
|
||||
end
|
||||
|
||||
trait :hidden do
|
||||
hidden_at Time.now
|
||||
hidden_at Time.current
|
||||
end
|
||||
|
||||
trait :with_confirmed_hide do
|
||||
confirmed_hide_at Time.now
|
||||
confirmed_hide_at Time.current
|
||||
end
|
||||
end
|
||||
|
||||
@@ -86,7 +86,7 @@ FactoryGirl.define do
|
||||
factory :lock do
|
||||
user
|
||||
tries 0
|
||||
locked_until Time.now
|
||||
locked_until Time.current
|
||||
end
|
||||
|
||||
factory :verified_user do
|
||||
@@ -101,15 +101,15 @@ FactoryGirl.define do
|
||||
association :author, factory: :user
|
||||
|
||||
trait :hidden do
|
||||
hidden_at Time.now
|
||||
hidden_at Time.current
|
||||
end
|
||||
|
||||
trait :with_ignored_flag do
|
||||
ignored_flag_at Time.now
|
||||
ignored_flag_at Time.current
|
||||
end
|
||||
|
||||
trait :with_confirmed_hide do
|
||||
confirmed_hide_at Time.now
|
||||
confirmed_hide_at Time.current
|
||||
end
|
||||
|
||||
trait :flagged do
|
||||
@@ -146,15 +146,15 @@ FactoryGirl.define do
|
||||
association :author, factory: :user
|
||||
|
||||
trait :hidden do
|
||||
hidden_at Time.now
|
||||
hidden_at Time.current
|
||||
end
|
||||
|
||||
trait :with_ignored_flag do
|
||||
ignored_flag_at Time.now
|
||||
ignored_flag_at Time.current
|
||||
end
|
||||
|
||||
trait :with_confirmed_hide do
|
||||
confirmed_hide_at Time.now
|
||||
confirmed_hide_at Time.current
|
||||
end
|
||||
|
||||
trait :flagged do
|
||||
@@ -212,15 +212,15 @@ FactoryGirl.define do
|
||||
sequence(:body) { |n| "Comment body #{n}" }
|
||||
|
||||
trait :hidden do
|
||||
hidden_at Time.now
|
||||
hidden_at Time.current
|
||||
end
|
||||
|
||||
trait :with_ignored_flag do
|
||||
ignored_flag_at Time.now
|
||||
ignored_flag_at Time.current
|
||||
end
|
||||
|
||||
trait :with_confirmed_hide do
|
||||
confirmed_hide_at Time.now
|
||||
confirmed_hide_at Time.current
|
||||
end
|
||||
|
||||
trait :flagged do
|
||||
@@ -269,11 +269,11 @@ FactoryGirl.define do
|
||||
sequence(:name) { |n| "org#{n}" }
|
||||
|
||||
trait :verified do
|
||||
verified_at Time.now
|
||||
verified_at Time.current
|
||||
end
|
||||
|
||||
trait :rejected do
|
||||
rejected_at Time.now
|
||||
rejected_at Time.current
|
||||
end
|
||||
end
|
||||
|
||||
@@ -296,13 +296,13 @@ FactoryGirl.define do
|
||||
|
||||
factory :ahoy_event, :class => Ahoy::Event do
|
||||
id { SecureRandom.uuid }
|
||||
time DateTime.now
|
||||
time DateTime.current
|
||||
sequence(:name) {|n| "Event #{n} type"}
|
||||
end
|
||||
|
||||
factory :visit do
|
||||
id { SecureRandom.uuid }
|
||||
started_at DateTime.now
|
||||
started_at DateTime.current
|
||||
end
|
||||
|
||||
factory :campaign do
|
||||
@@ -326,8 +326,8 @@ FactoryGirl.define do
|
||||
style {["banner-style-one", "banner-style-two", "banner-style-three"].sample}
|
||||
image {["banner.banner-img-one", "banner.banner-img-two", "banner.banner-img-three"].sample}
|
||||
target_url {["/proposals", "/debates" ].sample}
|
||||
post_started_at Time.now - 7.days
|
||||
post_ended_at Time.now + 7.days
|
||||
post_started_at Time.current - 7.days
|
||||
post_ended_at Time.current + 7.days
|
||||
end
|
||||
|
||||
factory :proposal_notification do
|
||||
|
||||
@@ -13,40 +13,40 @@ feature 'Admin banners magement' do
|
||||
target_url: "http://www.url.com",
|
||||
style: "banner-style.banner-one",
|
||||
image: "banner-img.banner-one",
|
||||
post_started_at: (Time.now + 4.days),
|
||||
post_ended_at: (Time.now + 10.days))
|
||||
post_started_at: (Time.current + 4.days),
|
||||
post_ended_at: (Time.current + 10.days))
|
||||
|
||||
@banner2 = create(:banner, title: "Banner number two",
|
||||
description: "This is the text of banner number two and is not longer active",
|
||||
target_url: "http://www.url.com",
|
||||
style: "banner-style.banner-two",
|
||||
image: "banner-img.banner-two",
|
||||
post_started_at: (Time.now - 10.days),
|
||||
post_ended_at: (Time.now - 3.days))
|
||||
post_started_at: (Time.current - 10.days),
|
||||
post_ended_at: (Time.current - 3.days))
|
||||
|
||||
@banner3 = create(:banner, title: "Banner number three",
|
||||
description: "This is the text of banner number three and has style banner-three",
|
||||
target_url: "http://www.url.com",
|
||||
style: "banner-style.banner-three",
|
||||
image: "banner-img.banner-three",
|
||||
post_started_at: (Time.now - 1.days),
|
||||
post_ended_at: (Time.now + 10.days))
|
||||
post_started_at: (Time.current - 1.days),
|
||||
post_ended_at: (Time.current + 10.days))
|
||||
|
||||
@banner4 = create(:banner, title: "Banner number four",
|
||||
description: "This is the text of banner number four and has style banner-one",
|
||||
target_url: "http://www.url.com",
|
||||
style: "banner-style.banner-one",
|
||||
image: "banner-img.banner-one",
|
||||
post_started_at: (DateTime.now - 10.days),
|
||||
post_ended_at: (DateTime.now + 10.days))
|
||||
post_started_at: (DateTime.current - 10.days),
|
||||
post_ended_at: (DateTime.current + 10.days))
|
||||
|
||||
@banner5 = create(:banner, title: "Banner number five",
|
||||
description: "This is the text of banner number five and has style banner-two",
|
||||
target_url: "http://www.url.com",
|
||||
style: "banner-style.banner-one",
|
||||
image: "banner-img.banner-one",
|
||||
post_started_at: (DateTime.now - 10.days),
|
||||
post_ended_at: (DateTime.now + 10.days))
|
||||
post_started_at: (DateTime.current - 10.days),
|
||||
post_ended_at: (DateTime.current + 10.days))
|
||||
end
|
||||
|
||||
scenario 'Index show active banners' do
|
||||
@@ -87,8 +87,8 @@ feature 'Admin banners magement' do
|
||||
fill_in 'banner_title', with: 'Such banner'
|
||||
fill_in 'banner_description', with: 'many text wow link'
|
||||
fill_in 'banner_target_url', with: 'https://www.url.com'
|
||||
last_week = Time.now - 7.days
|
||||
next_week = Time.now + 7.days
|
||||
last_week = Time.current - 7.days
|
||||
next_week = Time.current + 7.days
|
||||
fill_in 'post_started_at', with: last_week.strftime("%d/%m/%Y")
|
||||
fill_in 'post_ended_at', with: next_week.strftime("%d/%m/%Y")
|
||||
|
||||
@@ -108,8 +108,8 @@ feature 'Admin banners magement' do
|
||||
target_url: 'http://www.url.com',
|
||||
style: 'banner-style.banner-one',
|
||||
image: 'banner-img.banner-one',
|
||||
post_started_at: (Time.now + 4.days),
|
||||
post_ended_at: (Time.now + 10.days))
|
||||
post_started_at: (Time.current + 4.days),
|
||||
post_ended_at: (Time.current + 10.days))
|
||||
|
||||
visit admin_root_path
|
||||
|
||||
@@ -145,8 +145,8 @@ feature 'Admin banners magement' do
|
||||
target_url: 'http://www.url.com',
|
||||
style: 'banner-style.banner-one',
|
||||
image: 'banner-img.banner-one',
|
||||
post_started_at: (Time.now + 4.days),
|
||||
post_ended_at: (Time.now + 10.days))
|
||||
post_started_at: (Time.current + 4.days),
|
||||
post_ended_at: (Time.current + 10.days))
|
||||
|
||||
visit admin_root_path
|
||||
|
||||
@@ -162,4 +162,4 @@ feature 'Admin banners magement' do
|
||||
expect(page).to_not have_content 'Ugly banner'
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -51,7 +51,7 @@ feature 'Incomplete verifications' do
|
||||
end
|
||||
|
||||
scenario "Phone not given" do
|
||||
incompletely_verified_user = create(:user, residence_verified_at: Time.now, unconfirmed_phone: nil)
|
||||
incompletely_verified_user = create(:user, residence_verified_at: Time.current, unconfirmed_phone: nil)
|
||||
|
||||
visit admin_verifications_path
|
||||
|
||||
@@ -61,7 +61,7 @@ feature 'Incomplete verifications' do
|
||||
end
|
||||
|
||||
scenario "SMS code not confirmed" do
|
||||
incompletely_verified_user = create(:user, residence_verified_at: Time.now,
|
||||
incompletely_verified_user = create(:user, residence_verified_at: Time.current,
|
||||
unconfirmed_phone: "611111111",
|
||||
sms_confirmation_code: "1234",
|
||||
confirmed_phone: nil)
|
||||
|
||||
@@ -62,9 +62,9 @@ feature 'Commenting debates' do
|
||||
end
|
||||
|
||||
scenario 'Comment order' do
|
||||
c1 = create(:comment, :with_confidence_score, commentable: debate, cached_votes_up: 100, cached_votes_total: 120, created_at: Time.now - 2)
|
||||
c2 = create(:comment, :with_confidence_score, commentable: debate, cached_votes_up: 10, cached_votes_total: 12, created_at: Time.now - 1)
|
||||
c3 = create(:comment, :with_confidence_score, commentable: debate, cached_votes_up: 1, cached_votes_total: 2, created_at: Time.now)
|
||||
c1 = create(:comment, :with_confidence_score, commentable: debate, cached_votes_up: 100, cached_votes_total: 120, created_at: Time.current - 2)
|
||||
c2 = create(:comment, :with_confidence_score, commentable: debate, cached_votes_up: 10, cached_votes_total: 12, created_at: Time.current - 1)
|
||||
c3 = create(:comment, :with_confidence_score, commentable: debate, cached_votes_up: 1, cached_votes_total: 2, created_at: Time.current)
|
||||
|
||||
visit debate_path(debate, order: :most_voted)
|
||||
|
||||
@@ -83,10 +83,10 @@ feature 'Commenting debates' do
|
||||
end
|
||||
|
||||
scenario 'Creation date works differently in roots and in child comments, even when sorting by confidence_score' do
|
||||
old_root = create(:comment, commentable: debate, created_at: Time.now - 10)
|
||||
new_root = create(:comment, commentable: debate, created_at: Time.now)
|
||||
old_child = create(:comment, commentable: debate, parent_id: new_root.id, created_at: Time.now - 10)
|
||||
new_child = create(:comment, commentable: debate, parent_id: new_root.id, created_at: Time.now)
|
||||
old_root = create(:comment, commentable: debate, created_at: Time.current - 10)
|
||||
new_root = create(:comment, commentable: debate, created_at: Time.current)
|
||||
old_child = create(:comment, commentable: debate, parent_id: new_root.id, created_at: Time.current - 10)
|
||||
new_child = create(:comment, commentable: debate, parent_id: new_root.id, created_at: Time.current)
|
||||
|
||||
visit debate_path(debate, order: :most_voted)
|
||||
|
||||
@@ -418,7 +418,7 @@ feature 'Commenting debates' do
|
||||
|
||||
feature 'Voting comments' do
|
||||
background do
|
||||
@manuela = create(:user, verified_at: Time.now)
|
||||
@manuela = create(:user, verified_at: Time.current)
|
||||
@pablo = create(:user)
|
||||
@debate = create(:debate)
|
||||
@comment = create(:comment, commentable: @debate)
|
||||
|
||||
@@ -62,9 +62,9 @@ feature 'Commenting proposals' do
|
||||
end
|
||||
|
||||
scenario 'Comment order' do
|
||||
c1 = create(:comment, :with_confidence_score, commentable: proposal, cached_votes_up: 100, cached_votes_total: 120, created_at: Time.now - 2)
|
||||
c2 = create(:comment, :with_confidence_score, commentable: proposal, cached_votes_up: 10, cached_votes_total: 12, created_at: Time.now - 1)
|
||||
c3 = create(:comment, :with_confidence_score, commentable: proposal, cached_votes_up: 1, cached_votes_total: 2, created_at: Time.now)
|
||||
c1 = create(:comment, :with_confidence_score, commentable: proposal, cached_votes_up: 100, cached_votes_total: 120, created_at: Time.current - 2)
|
||||
c2 = create(:comment, :with_confidence_score, commentable: proposal, cached_votes_up: 10, cached_votes_total: 12, created_at: Time.current - 1)
|
||||
c3 = create(:comment, :with_confidence_score, commentable: proposal, cached_votes_up: 1, cached_votes_total: 2, created_at: Time.current)
|
||||
|
||||
visit proposal_path(proposal, order: :most_voted)
|
||||
|
||||
@@ -83,10 +83,10 @@ feature 'Commenting proposals' do
|
||||
end
|
||||
|
||||
scenario 'Creation date works differently in roots and in child comments, when sorting by confidence_score' do
|
||||
old_root = create(:comment, commentable: proposal, created_at: Time.now - 10)
|
||||
new_root = create(:comment, commentable: proposal, created_at: Time.now)
|
||||
old_child = create(:comment, commentable: proposal, parent_id: new_root.id, created_at: Time.now - 10)
|
||||
new_child = create(:comment, commentable: proposal, parent_id: new_root.id, created_at: Time.now)
|
||||
old_root = create(:comment, commentable: proposal, created_at: Time.current - 10)
|
||||
new_root = create(:comment, commentable: proposal, created_at: Time.current)
|
||||
old_child = create(:comment, commentable: proposal, parent_id: new_root.id, created_at: Time.current - 10)
|
||||
new_child = create(:comment, commentable: proposal, parent_id: new_root.id, created_at: Time.current)
|
||||
|
||||
visit proposal_path(proposal, order: :most_voted)
|
||||
|
||||
@@ -407,7 +407,7 @@ feature 'Commenting proposals' do
|
||||
feature 'Voting comments' do
|
||||
|
||||
background do
|
||||
@manuela = create(:user, verified_at: Time.now)
|
||||
@manuela = create(:user, verified_at: Time.current)
|
||||
@pablo = create(:user)
|
||||
@proposal = create(:proposal)
|
||||
@comment = create(:comment, commentable: @proposal)
|
||||
|
||||
@@ -422,9 +422,9 @@ feature 'Debates' do
|
||||
end
|
||||
|
||||
scenario 'Debates are ordered by newest', :js do
|
||||
create(:debate, title: 'Best', created_at: Time.now)
|
||||
create(:debate, title: 'Medium', created_at: Time.now - 1.hour)
|
||||
create(:debate, title: 'Worst', created_at: Time.now - 1.day)
|
||||
create(:debate, title: 'Best', created_at: Time.current)
|
||||
create(:debate, title: 'Medium', created_at: Time.current - 1.hour)
|
||||
create(:debate, title: 'Worst', created_at: Time.current - 1.day)
|
||||
|
||||
visit debates_path
|
||||
click_link 'newest'
|
||||
@@ -808,7 +808,7 @@ feature 'Debates' do
|
||||
scenario "Reorder results maintaing search", :js do
|
||||
debate1 = create(:debate, title: "Show you got", cached_votes_up: 10, created_at: 1.week.ago)
|
||||
debate2 = create(:debate, title: "Show what you got", cached_votes_up: 1, created_at: 1.month.ago)
|
||||
debate3 = create(:debate, title: "Show you got", cached_votes_up: 100, created_at: Time.now)
|
||||
debate3 = create(:debate, title: "Show you got", cached_votes_up: 100, created_at: Time.current)
|
||||
debate4 = create(:debate, title: "Do not display", cached_votes_up: 1, created_at: 1.week.ago)
|
||||
|
||||
visit debates_path
|
||||
@@ -1032,7 +1032,7 @@ feature 'Debates' do
|
||||
admin = create(:administrator)
|
||||
login_as(admin.user)
|
||||
|
||||
debate1 = create(:debate, featured_at: Time.now)
|
||||
debate1 = create(:debate, featured_at: Time.current)
|
||||
debate2 = create(:debate)
|
||||
|
||||
visit debates_path
|
||||
|
||||
@@ -170,9 +170,9 @@ feature 'Moderate comments' do
|
||||
end
|
||||
|
||||
scenario "sorting comments" do
|
||||
create(:comment, body: "Flagged comment", created_at: Time.now - 1.day, flags_count: 5)
|
||||
create(:comment, body: "Flagged newer comment", created_at: Time.now - 12.hours, flags_count: 3)
|
||||
create(:comment, body: "Newer comment", created_at: Time.now)
|
||||
create(:comment, body: "Flagged comment", created_at: Time.current - 1.day, flags_count: 5)
|
||||
create(:comment, body: "Flagged newer comment", created_at: Time.current - 12.hours, flags_count: 3)
|
||||
create(:comment, body: "Newer comment", created_at: Time.current)
|
||||
|
||||
visit moderation_comments_path(order: 'newest')
|
||||
|
||||
|
||||
@@ -177,9 +177,9 @@ feature 'Moderate debates' do
|
||||
end
|
||||
|
||||
scenario "sorting debates" do
|
||||
create(:debate, title: "Flagged debate", created_at: Time.now - 1.day, flags_count: 5)
|
||||
create(:debate, title: "Flagged newer debate", created_at: Time.now - 12.hours, flags_count: 3)
|
||||
create(:debate, title: "Newer debate", created_at: Time.now)
|
||||
create(:debate, title: "Flagged debate", created_at: Time.current - 1.day, flags_count: 5)
|
||||
create(:debate, title: "Flagged newer debate", created_at: Time.current - 12.hours, flags_count: 3)
|
||||
create(:debate, title: "Newer debate", created_at: Time.current)
|
||||
|
||||
visit moderation_debates_path(order: 'created_at')
|
||||
|
||||
|
||||
@@ -169,9 +169,9 @@ feature 'Moderate proposals' do
|
||||
end
|
||||
|
||||
scenario "sorting proposals" do
|
||||
create(:proposal, title: "Flagged proposal", created_at: Time.now - 1.day, flags_count: 5)
|
||||
create(:proposal, title: "Flagged newer proposal", created_at: Time.now - 12.hours, flags_count: 3)
|
||||
create(:proposal, title: "Newer proposal", created_at: Time.now)
|
||||
create(:proposal, title: "Flagged proposal", created_at: Time.current - 1.day, flags_count: 5)
|
||||
create(:proposal, title: "Flagged newer proposal", created_at: Time.current - 12.hours, flags_count: 3)
|
||||
create(:proposal, title: "Newer proposal", created_at: Time.current)
|
||||
|
||||
visit moderation_proposals_path(order: 'created_at')
|
||||
|
||||
|
||||
@@ -480,7 +480,7 @@ feature 'Proposals' do
|
||||
scenario 'Index do not list retired proposals by default' do
|
||||
create_featured_proposals
|
||||
not_retired = create(:proposal)
|
||||
retired = create(:proposal, retired_at: Time.now)
|
||||
retired = create(:proposal, retired_at: Time.current)
|
||||
|
||||
visit proposals_path
|
||||
|
||||
@@ -494,7 +494,7 @@ feature 'Proposals' do
|
||||
scenario 'Index has a link to retired proposals list' do
|
||||
create_featured_proposals
|
||||
not_retired = create(:proposal)
|
||||
retired = create(:proposal, retired_at: Time.now)
|
||||
retired = create(:proposal, retired_at: Time.current)
|
||||
|
||||
visit proposals_path
|
||||
|
||||
@@ -514,8 +514,8 @@ feature 'Proposals' do
|
||||
end
|
||||
|
||||
scenario 'Retired proposals index has links to filter by retired_reason' do
|
||||
unfeasible = create(:proposal, retired_at: Time.now, retired_reason: 'unfeasible')
|
||||
duplicated = create(:proposal, retired_at: Time.now, retired_reason: 'duplicated')
|
||||
unfeasible = create(:proposal, retired_at: Time.current, retired_reason: 'unfeasible')
|
||||
duplicated = create(:proposal, retired_at: Time.current, retired_reason: 'duplicated')
|
||||
|
||||
visit proposals_path(retired: 'all')
|
||||
|
||||
@@ -660,9 +660,9 @@ feature 'Proposals' do
|
||||
scenario 'Proposals are ordered by newest', :js do
|
||||
create_featured_proposals
|
||||
|
||||
create(:proposal, title: 'Best proposal', created_at: Time.now)
|
||||
create(:proposal, title: 'Medium proposal', created_at: Time.now - 1.hour)
|
||||
create(:proposal, title: 'Worst proposal', created_at: Time.now - 1.day)
|
||||
create(:proposal, title: 'Best proposal', created_at: Time.current)
|
||||
create(:proposal, title: 'Medium proposal', created_at: Time.current - 1.hour)
|
||||
create(:proposal, title: 'Worst proposal', created_at: Time.current - 1.day)
|
||||
|
||||
visit proposals_path
|
||||
click_link 'newest'
|
||||
@@ -1172,7 +1172,7 @@ feature 'Proposals' do
|
||||
scenario "Reorder results maintaing search", :js do
|
||||
proposal1 = create(:proposal, title: "Show you got", cached_votes_up: 10, created_at: 1.week.ago)
|
||||
proposal2 = create(:proposal, title: "Show what you got", cached_votes_up: 1, created_at: 1.month.ago)
|
||||
proposal3 = create(:proposal, title: "Show you got", cached_votes_up: 100, created_at: Time.now)
|
||||
proposal3 = create(:proposal, title: "Show you got", cached_votes_up: 100, created_at: Time.current)
|
||||
proposal4 = create(:proposal, title: "Do not display", cached_votes_up: 1, created_at: 1.week.ago)
|
||||
|
||||
visit proposals_path
|
||||
|
||||
@@ -286,7 +286,7 @@ feature 'Users' do
|
||||
end
|
||||
|
||||
scenario 'Sign in, admin with password expired' do
|
||||
user = create(:user, password_changed_at: Time.now - 1.year)
|
||||
user = create(:user, password_changed_at: Time.current - 1.year)
|
||||
admin = create(:administrator, user: user)
|
||||
|
||||
login_as(admin.user)
|
||||
@@ -304,7 +304,7 @@ feature 'Users' do
|
||||
end
|
||||
|
||||
scenario 'Sign in, admin without password expired' do
|
||||
user = create(:user, password_changed_at: Time.now - 360.days)
|
||||
user = create(:user, password_changed_at: Time.current - 360.days)
|
||||
admin = create(:administrator, user: user)
|
||||
|
||||
login_as(admin.user)
|
||||
@@ -314,7 +314,7 @@ feature 'Users' do
|
||||
end
|
||||
|
||||
scenario 'Sign in, user with password expired' do
|
||||
user = create(:user, password_changed_at: Time.now - 1.year)
|
||||
user = create(:user, password_changed_at: Time.current - 1.year)
|
||||
|
||||
login_as(user)
|
||||
visit root_path
|
||||
@@ -323,7 +323,7 @@ feature 'Users' do
|
||||
end
|
||||
|
||||
scenario 'Admin with password expired trying to use same password' do
|
||||
user = create(:user, password_changed_at: Time.now - 1.year, password: '123456789')
|
||||
user = create(:user, password_changed_at: Time.current - 1.year, password: '123456789')
|
||||
admin = create(:administrator, user: user)
|
||||
|
||||
login_as(admin.user)
|
||||
@@ -339,4 +339,4 @@ feature 'Users' do
|
||||
expect(page).to have_content "must be different than the current password."
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -4,7 +4,7 @@ feature 'Verify email' do
|
||||
|
||||
scenario 'Verify' do
|
||||
user = create(:user,
|
||||
residence_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
document_number: '12345678Z',
|
||||
document_type: 'dni')
|
||||
|
||||
@@ -34,7 +34,7 @@ feature 'Verify email' do
|
||||
end
|
||||
|
||||
scenario "Errors on token verification" do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
|
||||
login_as(user)
|
||||
visit email_path(email_verification_token: "1234")
|
||||
@@ -44,7 +44,7 @@ feature 'Verify email' do
|
||||
|
||||
scenario "Errors on sending confirmation email" do
|
||||
user = create(:user,
|
||||
residence_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
document_number: '12345678Z',
|
||||
document_type: 'dni')
|
||||
|
||||
@@ -62,4 +62,4 @@ feature 'Verify email' do
|
||||
|
||||
expect(page).to have_content "There was a problem with sending an email to your account"
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
@@ -3,7 +3,7 @@ require 'rails_helper'
|
||||
feature 'Verify Letter' do
|
||||
|
||||
scenario 'Request a letter' do
|
||||
user = create(:user, residence_verified_at: Time.now,
|
||||
user = create(:user, residence_verified_at: Time.current,
|
||||
confirmed_phone: "611111111")
|
||||
|
||||
login_as(user)
|
||||
@@ -20,7 +20,7 @@ feature 'Verify Letter' do
|
||||
end
|
||||
|
||||
scenario 'Go to office instead of send letter' do
|
||||
user = create(:user, residence_verified_at: Time.now,
|
||||
user = create(:user, residence_verified_at: Time.current,
|
||||
confirmed_phone: "611111111")
|
||||
|
||||
login_as(user)
|
||||
@@ -40,7 +40,7 @@ feature 'Verify Letter' do
|
||||
end
|
||||
|
||||
scenario "Deny access unless verified phone/email" do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
|
||||
login_as(user)
|
||||
visit new_letter_path
|
||||
@@ -52,7 +52,7 @@ feature 'Verify Letter' do
|
||||
context "Code verification" do
|
||||
|
||||
scenario "Valid verification user logged in" do
|
||||
user = create(:user, residence_verified_at: Time.now,
|
||||
user = create(:user, residence_verified_at: Time.current,
|
||||
confirmed_phone: "611111111",
|
||||
letter_verification_code: "123456")
|
||||
|
||||
@@ -69,7 +69,7 @@ feature 'Verify Letter' do
|
||||
end
|
||||
|
||||
scenario "Valid verification of user failing to add trailing zeros" do
|
||||
user = create(:user, residence_verified_at: Time.now,
|
||||
user = create(:user, residence_verified_at: Time.current,
|
||||
confirmed_phone: "611111111",
|
||||
letter_verification_code: "012345")
|
||||
|
||||
@@ -86,7 +86,7 @@ feature 'Verify Letter' do
|
||||
end
|
||||
|
||||
scenario "Valid verification user not logged in" do
|
||||
user = create(:user, residence_verified_at: Time.now,
|
||||
user = create(:user, residence_verified_at: Time.current,
|
||||
confirmed_phone: "611111111",
|
||||
letter_verification_code: "123456")
|
||||
|
||||
@@ -110,7 +110,7 @@ feature 'Verify Letter' do
|
||||
end
|
||||
|
||||
scenario "Error messages on verification" do
|
||||
user = create(:user, residence_verified_at: Time.now,
|
||||
user = create(:user, residence_verified_at: Time.current,
|
||||
confirmed_phone: "611111111")
|
||||
|
||||
visit edit_letter_path
|
||||
@@ -122,7 +122,7 @@ feature 'Verify Letter' do
|
||||
end
|
||||
|
||||
scenario '6 tries allowed' do
|
||||
user = create(:user, residence_verified_at: Time.now,
|
||||
user = create(:user, residence_verified_at: Time.current,
|
||||
confirmed_phone: "611111111",
|
||||
letter_verification_code: "123456")
|
||||
|
||||
|
||||
@@ -3,7 +3,7 @@ require 'rails_helper'
|
||||
feature 'SMS Verification' do
|
||||
|
||||
scenario 'Verify' do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
login_as(user)
|
||||
|
||||
visit new_sms_path
|
||||
@@ -21,7 +21,7 @@ feature 'SMS Verification' do
|
||||
end
|
||||
|
||||
scenario 'Errors on phone number' do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
login_as(user)
|
||||
|
||||
visit new_sms_path
|
||||
@@ -32,7 +32,7 @@ feature 'SMS Verification' do
|
||||
end
|
||||
|
||||
scenario 'Errors on verification code' do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
login_as(user)
|
||||
|
||||
visit new_sms_path
|
||||
@@ -58,7 +58,7 @@ feature 'SMS Verification' do
|
||||
end
|
||||
|
||||
scenario '5 tries allowed' do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
login_as(user)
|
||||
|
||||
visit new_sms_path
|
||||
@@ -77,4 +77,4 @@ feature 'SMS Verification' do
|
||||
expect(current_path).to eq(account_path)
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -3,7 +3,7 @@ require 'rails_helper'
|
||||
feature 'Verification path' do
|
||||
|
||||
scenario "User is an organization" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
create(:organization, user: user)
|
||||
|
||||
login_as(user)
|
||||
@@ -13,7 +13,7 @@ feature 'Verification path' do
|
||||
end
|
||||
|
||||
scenario "User is verified" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
|
||||
login_as(user)
|
||||
visit verification_path
|
||||
@@ -23,8 +23,8 @@ feature 'Verification path' do
|
||||
end
|
||||
|
||||
scenario "User requested a letter" do
|
||||
user = create(:user, confirmed_phone: "623456789", residence_verified_at: Time.now,
|
||||
letter_requested_at: Time.now, letter_verification_code: "666")
|
||||
user = create(:user, confirmed_phone: "623456789", residence_verified_at: Time.current,
|
||||
letter_requested_at: Time.current, letter_verification_code: "666")
|
||||
|
||||
login_as(user)
|
||||
visit verification_path
|
||||
@@ -33,7 +33,7 @@ feature 'Verification path' do
|
||||
end
|
||||
|
||||
scenario "User is level two verified" do
|
||||
user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666666666")
|
||||
user = create(:user, residence_verified_at: Time.current, confirmed_phone: "666666666")
|
||||
|
||||
login_as(user)
|
||||
visit verification_path
|
||||
@@ -42,7 +42,7 @@ feature 'Verification path' do
|
||||
end
|
||||
|
||||
scenario "User received a verification sms" do
|
||||
user = create(:user, residence_verified_at: Time.now, unconfirmed_phone: "666666666", sms_confirmation_code: "666")
|
||||
user = create(:user, residence_verified_at: Time.current, unconfirmed_phone: "666666666", sms_confirmation_code: "666")
|
||||
|
||||
login_as(user)
|
||||
visit verification_path
|
||||
@@ -51,7 +51,7 @@ feature 'Verification path' do
|
||||
end
|
||||
|
||||
scenario "User received verification email" do
|
||||
user = create(:user, residence_verified_at: Time.now, email_verification_token: "1234")
|
||||
user = create(:user, residence_verified_at: Time.current, email_verification_token: "1234")
|
||||
|
||||
login_as(user)
|
||||
visit verification_path
|
||||
@@ -64,7 +64,7 @@ feature 'Verification path' do
|
||||
end
|
||||
|
||||
scenario "User has verified residence" do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
|
||||
login_as(user)
|
||||
visit verification_path
|
||||
@@ -86,7 +86,7 @@ feature 'Verification path' do
|
||||
end
|
||||
|
||||
scenario "A verified user can not access verification pages" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
|
||||
login_as(user)
|
||||
|
||||
@@ -99,4 +99,4 @@ feature 'Verification path' do
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -4,7 +4,7 @@ feature 'Verified users' do
|
||||
|
||||
scenario "Verified emails" do
|
||||
user = create(:user,
|
||||
residence_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
document_number: '12345678Z')
|
||||
|
||||
create(:verified_user,
|
||||
@@ -29,7 +29,7 @@ feature 'Verified users' do
|
||||
|
||||
scenario "Verified phones" do
|
||||
user = create(:user,
|
||||
residence_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
document_number: '12345678Z')
|
||||
|
||||
create(:verified_user,
|
||||
@@ -54,7 +54,7 @@ feature 'Verified users' do
|
||||
|
||||
scenario "No emails or phones" do
|
||||
user = create(:user,
|
||||
residence_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
document_number: '12345678Z')
|
||||
|
||||
create(:verified_user,
|
||||
@@ -72,7 +72,7 @@ feature 'Verified users' do
|
||||
|
||||
scenario "Select a verified email" do
|
||||
user = create(:user,
|
||||
residence_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
document_number: '12345678Z')
|
||||
|
||||
verified_user = create(:verified_user,
|
||||
@@ -92,7 +92,7 @@ feature 'Verified users' do
|
||||
|
||||
scenario "Select a verified phone" do
|
||||
user = create(:user,
|
||||
residence_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
document_number: '12345678Z')
|
||||
|
||||
verified_user = create(:verified_user,
|
||||
@@ -111,7 +111,7 @@ feature 'Verified users' do
|
||||
|
||||
scenario "Continue without selecting any verified information" do
|
||||
user = create(:user,
|
||||
residence_verified_at: Time.now,
|
||||
residence_verified_at: Time.current,
|
||||
document_number: '12345678Z')
|
||||
|
||||
create(:verified_user,
|
||||
@@ -127,7 +127,7 @@ feature 'Verified users' do
|
||||
end
|
||||
|
||||
scenario "No verified information" do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
|
||||
login_as(user)
|
||||
visit verified_user_path
|
||||
@@ -135,4 +135,4 @@ feature 'Verified users' do
|
||||
expect(current_path).to eq(new_sms_path)
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -3,7 +3,7 @@ require 'rails_helper'
|
||||
feature 'Votes' do
|
||||
|
||||
background do
|
||||
@manuela = create(:user, verified_at: Time.now)
|
||||
@manuela = create(:user, verified_at: Time.current)
|
||||
@pablo = create(:user)
|
||||
end
|
||||
|
||||
|
||||
@@ -45,7 +45,7 @@ feature "Welcome screen" do
|
||||
end
|
||||
|
||||
scenario 'it is not shown to level-2 users' do
|
||||
user = create(:user, residence_verified_at: Time.now, confirmed_phone: "123")
|
||||
user = create(:user, residence_verified_at: Time.current, confirmed_phone: "123")
|
||||
|
||||
login_through_form_as(user)
|
||||
|
||||
@@ -53,7 +53,7 @@ feature "Welcome screen" do
|
||||
end
|
||||
|
||||
scenario 'it is not shown to level-3 users' do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
|
||||
login_through_form_as(user)
|
||||
|
||||
|
||||
@@ -51,7 +51,7 @@ describe EmailDigest do
|
||||
user = create(:user)
|
||||
|
||||
proposal_notification = create(:proposal_notification)
|
||||
notification = create(:notification, notifiable: proposal_notification, user: user, emailed_at: Time.now)
|
||||
notification = create(:notification, notifiable: proposal_notification, user: user, emailed_at: Time.current)
|
||||
|
||||
email_digest = EmailDigest.new(user)
|
||||
expect(email_digest.pending_notifications?).to_not be
|
||||
@@ -85,7 +85,7 @@ describe EmailDigest do
|
||||
user = create(:user)
|
||||
|
||||
proposal_notification = create(:proposal_notification)
|
||||
create(:notification, notifiable: proposal_notification, user: user, emailed_at: Time.now)
|
||||
create(:notification, notifiable: proposal_notification, user: user, emailed_at: Time.current)
|
||||
|
||||
reset_mailer
|
||||
email_digest = EmailDigest.new(user)
|
||||
@@ -124,4 +124,4 @@ describe EmailDigest do
|
||||
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
@@ -90,7 +90,7 @@ describe "Abilities::Common" do
|
||||
describe "when level 2 verified" do
|
||||
let(:own_spending_proposal) { create(:spending_proposal, author: user) }
|
||||
let(:own_direct_message) { create(:direct_message, sender: user) }
|
||||
before{ user.update(residence_verified_at: Time.now, confirmed_phone: "1") }
|
||||
before{ user.update(residence_verified_at: Time.current, confirmed_phone: "1") }
|
||||
|
||||
it { should be_able_to(:vote, Proposal) }
|
||||
it { should be_able_to(:vote_featured, Proposal) }
|
||||
@@ -108,7 +108,7 @@ describe "Abilities::Common" do
|
||||
describe "when level 3 verified" do
|
||||
let(:own_spending_proposal) { create(:spending_proposal, author: user) }
|
||||
let(:own_direct_message) { create(:direct_message, sender: user) }
|
||||
before{ user.update(verified_at: Time.now) }
|
||||
before{ user.update(verified_at: Time.current) }
|
||||
|
||||
it { should be_able_to(:vote, Proposal) }
|
||||
it { should be_able_to(:vote_featured, Proposal) }
|
||||
|
||||
@@ -129,12 +129,12 @@ describe Debate do
|
||||
end
|
||||
|
||||
it "should be true for level two verified users" do
|
||||
user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111")
|
||||
user = create(:user, residence_verified_at: Time.current, confirmed_phone: "666333111")
|
||||
expect(debate.votable_by?(user)).to be true
|
||||
end
|
||||
|
||||
it "should be true for level three verified users" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
expect(debate.votable_by?(user)).to be true
|
||||
end
|
||||
|
||||
@@ -166,24 +166,24 @@ describe Debate do
|
||||
|
||||
describe "from level two verified users" do
|
||||
it "should register vote" do
|
||||
user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111")
|
||||
user = create(:user, residence_verified_at: Time.current, confirmed_phone: "666333111")
|
||||
expect {debate.register_vote(user, 'yes')}.to change{debate.reload.votes_for.size}.by(1)
|
||||
end
|
||||
|
||||
it "should not increase anonymous votes counter " do
|
||||
user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111")
|
||||
user = create(:user, residence_verified_at: Time.current, confirmed_phone: "666333111")
|
||||
expect {debate.register_vote(user, 'yes')}.to_not change{debate.reload.cached_anonymous_votes_total}
|
||||
end
|
||||
end
|
||||
|
||||
describe "from level three verified users" do
|
||||
it "should register vote" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
expect {debate.register_vote(user, 'yes')}.to change{debate.reload.votes_for.size}.by(1)
|
||||
end
|
||||
|
||||
it "should not increase anonymous votes counter " do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
expect {debate.register_vote(user, 'yes')}.to_not change{debate.reload.cached_anonymous_votes_total}
|
||||
end
|
||||
end
|
||||
@@ -225,7 +225,7 @@ describe Debate do
|
||||
end
|
||||
|
||||
describe '#hot_score' do
|
||||
let(:now) { Time.now }
|
||||
let(:now) { Time.current }
|
||||
|
||||
it "increases for newer debates" do
|
||||
old = create(:debate, :with_hot_score, created_at: now - 1.day)
|
||||
@@ -611,7 +611,7 @@ describe Debate do
|
||||
|
||||
it "should be able to reorder by created_at after searching" do
|
||||
recent = create(:debate, title: 'stop corruption', cached_votes_up: 1, created_at: 1.week.ago)
|
||||
newest = create(:debate, title: 'stop corruption', cached_votes_up: 2, created_at: Time.now)
|
||||
newest = create(:debate, title: 'stop corruption', cached_votes_up: 2, created_at: Time.current)
|
||||
oldest = create(:debate, title: 'stop corruption', cached_votes_up: 3, created_at: 1.month.ago)
|
||||
|
||||
results = Debate.search('stop corruption')
|
||||
|
||||
@@ -9,17 +9,17 @@ describe Organization do
|
||||
expect(subject.verified?).to be false
|
||||
end
|
||||
it "is true when verified_at? exists" do
|
||||
subject.verified_at = Time.now
|
||||
subject.verified_at = Time.current
|
||||
expect(subject.verified?).to be true
|
||||
end
|
||||
it "is false when the organization was verified and then rejected" do
|
||||
subject.verified_at = Time.now
|
||||
subject.rejected_at = Time.now + 1
|
||||
subject.verified_at = Time.current
|
||||
subject.rejected_at = Time.current + 1
|
||||
expect(subject.verified?).to be false
|
||||
end
|
||||
it "is true when the organization was rejected and then verified" do
|
||||
subject.rejected_at = Time.now
|
||||
subject.verified_at = Time.now + 1
|
||||
subject.rejected_at = Time.current
|
||||
subject.verified_at = Time.current + 1
|
||||
expect(subject.verified?).to be true
|
||||
end
|
||||
end
|
||||
@@ -29,17 +29,17 @@ describe Organization do
|
||||
expect(subject.rejected?).to be false
|
||||
end
|
||||
it "is true when rejected_at? exists" do
|
||||
subject.rejected_at = Time.now
|
||||
subject.rejected_at = Time.current
|
||||
expect(subject.rejected?).to be true
|
||||
end
|
||||
it "is true when the organization was verified and then rejected" do
|
||||
subject.verified_at = Time.now
|
||||
subject.rejected_at = Time.now + 1
|
||||
subject.verified_at = Time.current
|
||||
subject.rejected_at = Time.current + 1
|
||||
expect(subject.rejected?).to be true
|
||||
end
|
||||
it "is false when the organization was rejected and then verified" do
|
||||
subject.rejected_at = Time.now
|
||||
subject.verified_at = Time.now + 1
|
||||
subject.rejected_at = Time.current
|
||||
subject.verified_at = Time.current + 1
|
||||
expect(subject.rejected?).to be false
|
||||
end
|
||||
end
|
||||
|
||||
@@ -166,12 +166,12 @@ describe Proposal do
|
||||
let(:proposal) { create(:proposal) }
|
||||
|
||||
it "should be true for level two verified users" do
|
||||
user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111")
|
||||
user = create(:user, residence_verified_at: Time.current, confirmed_phone: "666333111")
|
||||
expect(proposal.votable_by?(user)).to be true
|
||||
end
|
||||
|
||||
it "should be true for level three verified users" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
expect(proposal.votable_by?(user)).to be true
|
||||
end
|
||||
|
||||
@@ -186,14 +186,14 @@ describe Proposal do
|
||||
|
||||
describe "from level two verified users" do
|
||||
it "should register vote" do
|
||||
user = create(:user, residence_verified_at: Time.now, confirmed_phone: "666333111")
|
||||
user = create(:user, residence_verified_at: Time.current, confirmed_phone: "666333111")
|
||||
expect {proposal.register_vote(user, 'yes')}.to change{proposal.reload.votes_for.size}.by(1)
|
||||
end
|
||||
end
|
||||
|
||||
describe "from level three verified users" do
|
||||
it "should register vote" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
expect {proposal.register_vote(user, 'yes')}.to change{proposal.reload.votes_for.size}.by(1)
|
||||
end
|
||||
end
|
||||
@@ -206,7 +206,7 @@ describe Proposal do
|
||||
end
|
||||
|
||||
it "should not register vote for archived proposals" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
archived_proposal = create(:proposal, :archived)
|
||||
|
||||
expect {archived_proposal.register_vote(user, 'yes')}.to change{proposal.reload.votes_for.size}.by(0)
|
||||
@@ -230,7 +230,7 @@ describe Proposal do
|
||||
end
|
||||
|
||||
describe '#hot_score' do
|
||||
let(:now) { Time.now }
|
||||
let(:now) { Time.current }
|
||||
|
||||
it "increases for newer proposals" do
|
||||
old = create(:proposal, :with_hot_score, created_at: now - 1.day)
|
||||
@@ -267,7 +267,7 @@ describe Proposal do
|
||||
|
||||
it "increases with votes" do
|
||||
previous = proposal.hot_score
|
||||
5.times { proposal.register_vote(create(:user, verified_at: Time.now), true) }
|
||||
5.times { proposal.register_vote(create(:user, verified_at: Time.current), true) }
|
||||
expect(previous).to be < proposal.reload.hot_score
|
||||
end
|
||||
|
||||
@@ -317,7 +317,7 @@ describe Proposal do
|
||||
|
||||
it "increases with like" do
|
||||
previous = proposal.confidence_score
|
||||
5.times { proposal.register_vote(create(:user, verified_at: Time.now), true) }
|
||||
5.times { proposal.register_vote(create(:user, verified_at: Time.current), true) }
|
||||
expect(previous).to be < proposal.confidence_score
|
||||
end
|
||||
end
|
||||
@@ -612,7 +612,7 @@ describe Proposal do
|
||||
|
||||
it "should be able to reorder by created_at after searching" do
|
||||
recent = create(:proposal, title: 'stop corruption', cached_votes_up: 1, created_at: 1.week.ago)
|
||||
newest = create(:proposal, title: 'stop corruption', cached_votes_up: 2, created_at: Time.now)
|
||||
newest = create(:proposal, title: 'stop corruption', cached_votes_up: 2, created_at: Time.current)
|
||||
oldest = create(:proposal, title: 'stop corruption', cached_votes_up: 3, created_at: 1.month.ago)
|
||||
|
||||
results = Proposal.search('stop corruption')
|
||||
@@ -794,7 +794,7 @@ describe Proposal do
|
||||
|
||||
describe "retired" do
|
||||
let!(:proposal1) { create(:proposal) }
|
||||
let!(:proposal2) { create(:proposal, retired_at: Time.now) }
|
||||
let!(:proposal2) { create(:proposal, retired_at: Time.current) }
|
||||
|
||||
it "retired? is true" do
|
||||
expect(proposal1.retired?).to eq false
|
||||
|
||||
@@ -274,10 +274,10 @@ describe User do
|
||||
# Subdomains are also accepted
|
||||
|
||||
Setting['email_domain_for_officials'] = 'officials.madrid.es'
|
||||
user1 = create(:user, email: "john@officials.madrid.es", confirmed_at: Time.now)
|
||||
user2 = create(:user, email: "john@yes.officials.madrid.es", confirmed_at: Time.now)
|
||||
user3 = create(:user, email: "john@unofficials.madrid.es", confirmed_at: Time.now)
|
||||
user4 = create(:user, email: "john@example.org", confirmed_at: Time.now)
|
||||
user1 = create(:user, email: "john@officials.madrid.es", confirmed_at: Time.current)
|
||||
user2 = create(:user, email: "john@yes.officials.madrid.es", confirmed_at: Time.current)
|
||||
user3 = create(:user, email: "john@unofficials.madrid.es", confirmed_at: Time.current)
|
||||
user4 = create(:user, email: "john@example.org", confirmed_at: Time.current)
|
||||
|
||||
expect(user1.has_official_email?).to eq(true)
|
||||
expect(user2.has_official_email?).to eq(true)
|
||||
@@ -332,7 +332,7 @@ describe User do
|
||||
it "returns users that have not been erased" do
|
||||
user1 = create(:user, erased_at: nil)
|
||||
user2 = create(:user, erased_at: nil)
|
||||
user3 = create(:user, erased_at: Time.now)
|
||||
user3 = create(:user, erased_at: Time.current)
|
||||
|
||||
expect(User.active).to include(user1)
|
||||
expect(User.active).to include(user2)
|
||||
|
||||
@@ -25,7 +25,7 @@ module CommonActions
|
||||
end
|
||||
|
||||
def login_as_authenticated_manager
|
||||
login, user_key, date = "JJB042", "31415926", Time.now.strftime("%Y%m%d%H%M%S")
|
||||
login, user_key, date = "JJB042", "31415926", Time.current.strftime("%Y%m%d%H%M%S")
|
||||
allow_any_instance_of(ManagerAuthenticator).to receive(:auth).and_return({login: login, user_key: user_key, date: date}.with_indifferent_access)
|
||||
visit management_sign_in_path(login: login, clave_usuario: user_key, fecha_conexion: date)
|
||||
end
|
||||
|
||||
@@ -4,7 +4,7 @@ shared_examples_for "verifiable" do
|
||||
describe "#scopes" do
|
||||
describe "#level_three_verified" do
|
||||
it "returns level three verified users" do
|
||||
user1 = create(:user, verified_at: Time.now)
|
||||
user1 = create(:user, verified_at: Time.current)
|
||||
user2 = create(:user, verified_at: nil)
|
||||
|
||||
expect(model.level_three_verified).to include(user1)
|
||||
@@ -14,10 +14,10 @@ shared_examples_for "verifiable" do
|
||||
|
||||
describe "#level_two_verified" do
|
||||
it "returns level two verified users" do
|
||||
user1 = create(:user, confirmed_phone: "123456789", residence_verified_at: Time.now)
|
||||
user1 = create(:user, confirmed_phone: "123456789", residence_verified_at: Time.current)
|
||||
user2 = create(:user, confirmed_phone: "123456789", residence_verified_at: nil)
|
||||
user3 = create(:user, confirmed_phone: nil, residence_verified_at: Time.now)
|
||||
user4 = create(:user, level_two_verified_at: Time.now)
|
||||
user3 = create(:user, confirmed_phone: nil, residence_verified_at: Time.current)
|
||||
user4 = create(:user, level_two_verified_at: Time.current)
|
||||
|
||||
expect(model.level_two_verified).to include(user1)
|
||||
expect(model.level_two_verified).to_not include(user2)
|
||||
@@ -28,11 +28,11 @@ shared_examples_for "verifiable" do
|
||||
|
||||
describe "#level_two_or_three_verified" do
|
||||
it "returns level two or three verified users" do
|
||||
user1 = create(:user, confirmed_phone: "123456789", residence_verified_at: Time.now)
|
||||
user2 = create(:user, verified_at: Time.now)
|
||||
user1 = create(:user, confirmed_phone: "123456789", residence_verified_at: Time.current)
|
||||
user2 = create(:user, verified_at: Time.current)
|
||||
user3 = create(:user, confirmed_phone: "123456789", residence_verified_at: nil)
|
||||
user4 = create(:user, confirmed_phone: nil, residence_verified_at: Time.now)
|
||||
user5 = create(:user, level_two_verified_at: Time.now)
|
||||
user4 = create(:user, confirmed_phone: nil, residence_verified_at: Time.current)
|
||||
user5 = create(:user, level_two_verified_at: Time.current)
|
||||
|
||||
expect(model.level_two_or_three_verified).to include(user1)
|
||||
expect(model.level_two_or_three_verified).to include(user2)
|
||||
@@ -46,9 +46,9 @@ shared_examples_for "verifiable" do
|
||||
it "returns unverified users" do
|
||||
user1 = create(:user, verified_at: nil, confirmed_phone: nil)
|
||||
user2 = create(:user, verified_at: nil, residence_verified_at: nil, confirmed_phone: "123456789")
|
||||
user3 = create(:user, verified_at: nil, residence_verified_at: Time.now, confirmed_phone: nil)
|
||||
user4 = create(:user, verified_at: Time.now, residence_verified_at: Time.now, confirmed_phone: "123456789")
|
||||
user5 = create(:user, level_two_verified_at: Time.now)
|
||||
user3 = create(:user, verified_at: nil, residence_verified_at: Time.current, confirmed_phone: nil)
|
||||
user4 = create(:user, verified_at: Time.current, residence_verified_at: Time.current, confirmed_phone: "123456789")
|
||||
user5 = create(:user, level_two_verified_at: Time.current)
|
||||
|
||||
expect(model.unverified).to include(user1)
|
||||
expect(model.unverified).to include(user2)
|
||||
@@ -63,9 +63,9 @@ shared_examples_for "verifiable" do
|
||||
user1 = create(:user, verified_at: nil, confirmed_phone: nil)
|
||||
create(:failed_census_call, user: user1)
|
||||
|
||||
user2 = create(:user, verified_at: nil, residence_verified_at: Time.now, unconfirmed_phone: nil)
|
||||
user2 = create(:user, verified_at: nil, residence_verified_at: Time.current, unconfirmed_phone: nil)
|
||||
user3 = create(:user, verified_at: nil, confirmed_phone: nil)
|
||||
user4 = create(:user, verified_at: Time.now, residence_verified_at: Time.now, unconfirmed_phone: "123456789", confirmed_phone: "123456789")
|
||||
user4 = create(:user, verified_at: Time.current, residence_verified_at: Time.current, unconfirmed_phone: "123456789", confirmed_phone: "123456789")
|
||||
|
||||
|
||||
expect(model.incomplete_verification).to include(user1)
|
||||
@@ -78,7 +78,7 @@ shared_examples_for "verifiable" do
|
||||
|
||||
describe "#methods" do
|
||||
it "residence_verified? is true only if residence_verified_at" do
|
||||
user = create(:user, residence_verified_at: Time.now)
|
||||
user = create(:user, residence_verified_at: Time.current)
|
||||
expect(user.residence_verified?).to eq(true)
|
||||
|
||||
user = create(:user, residence_verified_at: nil)
|
||||
@@ -94,13 +94,13 @@ shared_examples_for "verifiable" do
|
||||
end
|
||||
|
||||
it "level_two_verified? is true if manually set, or if residence_verified_at and confirmed_phone" do
|
||||
user = create(:user, level_two_verified_at: Time.now)
|
||||
user = create(:user, level_two_verified_at: Time.current)
|
||||
expect(user.level_two_verified?).to eq(true)
|
||||
|
||||
user = create(:user, confirmed_phone: "123456789", residence_verified_at: Time.now)
|
||||
user = create(:user, confirmed_phone: "123456789", residence_verified_at: Time.current)
|
||||
expect(user.level_two_verified?).to eq(true)
|
||||
|
||||
user = create(:user, confirmed_phone: nil, residence_verified_at: Time.now)
|
||||
user = create(:user, confirmed_phone: nil, residence_verified_at: Time.current)
|
||||
expect(user.level_two_verified?).to eq(false)
|
||||
|
||||
user = create(:user, confirmed_phone: "123456789", residence_verified_at: nil)
|
||||
@@ -108,7 +108,7 @@ shared_examples_for "verifiable" do
|
||||
end
|
||||
|
||||
it "level_three_verified? is true only if verified_at" do
|
||||
user = create(:user, verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current)
|
||||
expect(user.level_three_verified?).to eq(true)
|
||||
|
||||
user = create(:user, verified_at: nil)
|
||||
@@ -119,7 +119,7 @@ shared_examples_for "verifiable" do
|
||||
user = create(:user, verified_at: nil, confirmed_phone: nil)
|
||||
expect(user.unverified?).to eq(true)
|
||||
|
||||
user = create(:user, verified_at: Time.now, confirmed_phone: "123456789", residence_verified_at: Time.now)
|
||||
user = create(:user, verified_at: Time.current, confirmed_phone: "123456789", residence_verified_at: Time.current)
|
||||
expect(user.unverified?).to eq(false)
|
||||
end
|
||||
|
||||
@@ -146,13 +146,13 @@ shared_examples_for "verifiable" do
|
||||
end
|
||||
|
||||
it "verification_letter_sent? is true only if user has letter_requested_at and letter_verification_code" do
|
||||
user = create(:user, letter_requested_at: Time.now, letter_verification_code: "666")
|
||||
user = create(:user, letter_requested_at: Time.current, letter_verification_code: "666")
|
||||
expect(user.verification_letter_sent?).to eq(true)
|
||||
|
||||
user = create(:user, letter_requested_at: nil, letter_verification_code: "666")
|
||||
expect(user.verification_letter_sent?).to eq(false)
|
||||
|
||||
user = create(:user, letter_requested_at: Time.now, letter_verification_code: nil)
|
||||
user = create(:user, letter_requested_at: Time.current, letter_verification_code: nil)
|
||||
expect(user.verification_letter_sent?).to eq(false)
|
||||
|
||||
user = create(:user, letter_requested_at: nil, letter_verification_code: nil)
|
||||
@@ -160,4 +160,4 @@ shared_examples_for "verifiable" do
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user