solidus_reviews 1.2.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6fc709261f34b4979904d11cceeb17b22a57be36089f37f77dea475262506c24
-  data.tar.gz: 9a3b6b6f70b0867534f634426da005d312db19e21d5633bb9e5ca2e8ab6ba3ec
+  metadata.gz: f17ce08e72bf0a578195dda4763786e80ca5ce9b2f9126db7790f3f1571efa10
+  data.tar.gz: 7ffc6b1fd506cb9d21d5922f5fb049e1574d884ee184e7070a1ae76b3e3eb88a
 SHA512:
-  metadata.gz: 8c4eaa7f6e301a830b196bd194c897a670a8c193607fe98542890625ed6a3d19b54c7ce8b8df9678ba45bc3db114b0a358c58d053a93be0e438b4fa6354939fe
-  data.tar.gz: e2d8efba5a9ba5602318c40e1bcccb16291f49a68c577e76acf2db42b37e879bb9adfcbc56367b47dbfa84092964bdfe199e7aacb47b85c071c94d134c7906e4
+  metadata.gz: 959becf7cbddb80453025ac7601fa6aa25349849707455839040587612a505c014df75469d529249daba670414bf24e886b355da0426eb60ba3102bc45be38dc
+  data.tar.gz: d1e7b71a6aad1548bbedef2d40bbbd986ed39b20c1b6133d8ae73decb7cf7c135923c449cb26ce5d221a2604d8d578867ddba9695d116840ea05718705baacfc

data/.circleci/config.yml

@@ -16,12 +16,18 @@ jobs:
     executor: solidusio_extensions/mysql
     steps:
       - solidusio_extensions/run-tests
+  lint-code:
+    executor: solidusio_extensions/sqlite-memory
+    steps:
+      - solidusio_extensions/lint-code
 
 workflows:
   "Run specs on supported Solidus versions":
     jobs:
       - run-specs-with-postgres
       - run-specs-with-mysql
+      - lint-code
+
   "Weekly run specs against master":
     triggers:
       - schedule:

data/.gem_release.yml

@@ -0,0 +1,5 @@
+bump:
+  recurse: false
+  file: 'lib/solidus_reviews/version.rb'
+  message: Bump SolidusReviews to %{version}
+  tag: true

data/.gitignore

@@ -1,13 +1,20 @@
 *.gem
-*.swp
-.bundle
-.idea
+\#*
+*~
+.#*
 .DS_Store
+.idea
+.project
+.sass-cache
+coverage
 Gemfile.lock
-pkg/*
+tmp
+nbproject
+pkg
+*.swp
 spec/dummy
-coverage
+spec/examples.txt
+/sandbox
 .rvmrc
 .ruby-version
 .ruby-gemset
-spec/examples.txt

data/.rspec

@@ -1,2 +1,2 @@
 --color
--f documentation
+--require spec_helper

data/.rubocop.yml

@@ -1,322 +1,4 @@
-# Relaxed.Ruby.Style
+inherit_from: .rubocop_todo.yml
 
-AllCops:
-  Exclude:
-    - 'vendor/**/*'
-    - '**/vendor/**/*'
-    - '*/spec/dummy/**/*'
-    - 'sandbox/**/*'
-    - '**/templates/**/*'
-  TargetRubyVersion: 2.2
-
-# Sometimes I believe this reads better
-# This also causes spacing issues on multi-line fixes
-Style/BracesAroundHashParameters:
-  Enabled: false
-
-# We use class vars and will have to continue doing so for compatability
-Style/ClassVars:
-  Enabled: false
-
-# We need these names for backwards compatability
-Naming/PredicateName:
-  Enabled: false
-
-Naming/AccessorMethodName:
-  Enabled: false
-
-# This has been used for customization
-Style/MutableConstant:
-  Enabled: false
-
-Style/ClassAndModuleChildren:
-  Enabled: false
-
-Style/EmptyElse:
-  Enabled: false
-
-Style/GuardClause:
-  Enabled: false
-
-Style/Next:
-  Enabled: false
-
-Style/NumericPredicate:
-  Enabled: false
-
-Style/WordArray:
-  Enabled: false
-
-Style/ConditionalAssignment:
-  Enabled: false
-
-Performance/Count:
-  Enabled: false
-
-Style/RaiseArgs:
-  Enabled: false
-
-Naming/BinaryOperatorParameterName:
-  Enabled: false
-
-# We can use good judgement here
-Style/RegexpLiteral:
-  Enabled: false
-
-# Unicode comments are useful
-Style/AsciiComments:
-  Enabled: false
-
-Layout/EndAlignment:
-  Enabled: false
-
-Layout/ElseAlignment:
-  Enabled: false
-
-Layout/IndentationWidth:
-  Enabled: false
-
-Layout/AlignParameters:
-  Enabled: false
-
-Layout/ClosingParenthesisIndentation:
-  Enabled: false
-
-Layout/MultilineMethodCallIndentation:
-  Enabled: false
-
-Layout/IndentArray:
-  Enabled: false
-
-Layout/IndentHash:
-  Enabled: false
-
-Layout/AlignHash:
-  Enabled: false
-
-Style/TrailingCommaInArguments:
-  Enabled: false
-
-Style/TrailingCommaInArrayLiteral:
-  Enabled: false
-
-Style/TrailingCommaInHashLiteral:
-  Enabled: false
-
-# Symbol Arrays are ok and the %i syntax widely unknown
-Style/SymbolArray:
-  Enabled: false
-
-Rails/DynamicFindBy:
-  Whitelist:
-    - find_by_param
-    - find_by_param!
-
-# We use a lot of
-#
-#     expect {
-#       something
-#     }.to { happen }
-#
-# syntax in the specs files.
-Lint/AmbiguousBlockAssociation:
-  Exclude:
-    - '*/spec/**/*'
-    - 'spec/**/*' # For the benefit of apps that inherit from this config
-
-# We use eval to add common_spree_dependencies into the Gemfiles of each of our gems
-Security/Eval:
-  Exclude:
-    - 'Gemfile'
-    - 'common_spree_dependencies.rb'
-    - '*/Gemfile'
-
-Naming/VariableNumber:
-  Enabled: false
-
-# Write empty methods as you wish.
-Style/EmptyMethod:
-  Enabled: false
-
-# From http://relaxed.ruby.style/
-
-Style/Alias:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylealias
-
-Style/BeginBlock:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylebeginblock
-
-Style/BlockDelimiters:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styleblockdelimiters
-
-Style/Documentation:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styledocumentation
-
-Layout/DotPosition:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styledotposition
-
-Style/DoubleNegation:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styledoublenegation
-
-Style/EndBlock:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styleendblock
-
-Style/FormatString:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styleformatstring
-
-Style/IfUnlessModifier:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styleifunlessmodifier
-
-Style/Lambda:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylelambda
-
-Style/ModuleFunction:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylemodulefunction
-
-Style/MultilineBlockChain:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylemultilineblockchain
-
-Style/NegatedIf:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylenegatedif
-
-Style/NegatedWhile:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylenegatedwhile
-
-Style/ParallelAssignment:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styleparallelassignment
-
-Style/PercentLiteralDelimiters:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylepercentliteraldelimiters
-
-Style/PerlBackrefs:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#styleperlbackrefs
-
-Style/Semicolon:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylesemicolon
-
-Style/SignalException:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylesignalexception
-
-Style/SingleLineBlockParams:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylesinglelineblockparams
-
-Style/SingleLineMethods:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylesinglelinemethods
-
-Layout/SpaceBeforeBlockBraces:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylespacebeforeblockbraces
-
-Layout/SpaceInsideParens:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylespaceinsideparens
-
-Style/SpecialGlobalVars:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylespecialglobalvars
-
-Style/StringLiterals:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylestringliterals
-
-Style/SymbolProc:
-  Enabled: false
-
-Style/TernaryParentheses:
-  Enabled: false
-
-Style/WhileUntilModifier:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#stylewhileuntilmodifier
-
-Lint/AmbiguousRegexpLiteral:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#lintambiguousregexpliteral
-
-Lint/AssignmentInCondition:
-  Enabled: false
-  StyleGuide: http://relaxed.ruby.style/#lintassignmentincondition
-
-Metrics/AbcSize:
-  Enabled: false
-
-Metrics/BlockNesting:
-  Enabled: false
-
-Metrics/ClassLength:
-  Enabled: false
-
-Metrics/ModuleLength:
-  Enabled: false
-
-Metrics/BlockLength:
-  Enabled: false
-
-Metrics/CyclomaticComplexity:
-  Enabled: false
-
-Metrics/LineLength:
-  Enabled: false
-
-Metrics/MethodLength:
-  Enabled: false
-
-Metrics/ParameterLists:
-  Enabled: false
-
-Metrics/PerceivedComplexity:
-  Enabled: false
-
-Bundler/OrderedGems:
-  Enabled: false
-
-Style/NumericLiterals:
-  Enabled: false
-
-Style/FrozenStringLiteralComment:
-  Enabled: true
-  EnforcedStyle: always
-
-# json.() is idiomatic in jbuilder files
-Style/LambdaCall:
-  Enabled: false
-
-Naming/UncommunicativeMethodParamName:
-  AllowedNames:
-    - id
-    - to
-    - _
-
-# Rubocop doesn't understand side-effects
-Style/IdenticalConditionalBranches:
-  Enabled: false
-
-Naming/MemoizedInstanceVariableName:
-  Enabled: false
-
-Lint/UselessComparison:
-  Enabled: false
-
-Lint/HandleExceptions:
-  Enabled: false
+require:
+  - solidus_dev_support/rubocop

data/.rubocop_todo.yml

@@ -0,0 +1,197 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2020-09-11 09:54:47 UTC using RuboCop version 0.87.1.
+# 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: 1
+# Cop supports --auto-correct.
+Lint/RedundantCopDisableDirective:
+  Exclude:
+    - 'lib/generators/solidus_reviews/install/install_generator.rb'
+
+# Offense count: 1
+# Configuration parameters: EnforcedStyleForLeadingUnderscores.
+# SupportedStylesForLeadingUnderscores: disallowed, required, optional
+Naming/MemoizedInstanceVariableName:
+  Exclude:
+    - 'app/controllers/spree/feedback_reviews_controller.rb'
+
+# Offense count: 2
+# Configuration parameters: MinNameLength, AllowNamesEndingInNumbers, AllowedNames, ForbiddenNames.
+# AllowedNames: io, id, to, by, on, in, at, ip, db, os, pp
+Naming/MethodParameterName:
+  Exclude:
+    - 'app/helpers/spree/reviews_helper.rb'
+
+# Offense count: 17
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: snake_case, normalcase, non_integer
+Naming/VariableNumber:
+  Exclude:
+    - 'spec/controllers/spree/admin/feedback_reviews_controller_spec.rb'
+    - 'spec/models/feedback_review_spec.rb'
+    - 'spec/models/product_spec.rb'
+    - 'spec/models/review_spec.rb'
+
+# Offense count: 3
+RSpec/AnyInstance:
+  Exclude:
+    - 'spec/controllers/spree/admin/reviews_controller_spec.rb'
+    - 'spec/controllers/spree/reviews_controller_spec.rb'
+
+# Offense count: 37
+# Configuration parameters: Prefixes.
+# Prefixes: when, with, without
+RSpec/ContextWording:
+  Exclude:
+    - 'spec/controllers/spree/api/reviews_controller_spec.rb'
+    - 'spec/controllers/spree/reviews_controller_spec.rb'
+    - 'spec/features/admin_spec.rb'
+    - 'spec/features/reviews_spec.rb'
+    - 'spec/helpers/review_helper_spec.rb'
+    - 'spec/models/feedback_review_spec.rb'
+    - 'spec/models/product_spec.rb'
+    - 'spec/models/review_spec.rb'
+    - 'spec/models/reviews_ability_spec.rb'
+
+# Offense count: 6
+# Configuration parameters: CustomTransform, IgnoreMethods, SpecSuffixOnly.
+RSpec/FilePath:
+  Exclude:
+    - 'spec/helpers/review_helper_spec.rb'
+    - 'spec/models/feedback_review_spec.rb'
+    - 'spec/models/product_spec.rb'
+    - 'spec/models/review_spec.rb'
+    - 'spec/models/reviews_ability_spec.rb'
+    - 'spec/models/reviews_configuration_spec.rb'
+
+# Offense count: 12
+# Configuration parameters: AssignmentOnly.
+RSpec/InstanceVariable:
+  Exclude:
+    - 'spec/controllers/spree/reviews_controller_spec.rb'
+
+# Offense count: 6
+RSpec/LetSetup:
+  Exclude:
+    - 'spec/controllers/spree/admin/feedback_reviews_controller_spec.rb'
+    - 'spec/models/product_spec.rb'
+
+# Offense count: 4
+# Configuration parameters: .
+# SupportedStyles: have_received, receive
+RSpec/MessageSpies:
+  EnforcedStyle: receive
+
+# Offense count: 67
+RSpec/MultipleExpectations:
+  Max: 8
+
+# Offense count: 3
+# Configuration parameters: AllowSubject.
+RSpec/MultipleMemoizedHelpers:
+  Max: 7
+
+# Offense count: 94
+# Configuration parameters: IgnoreSharedExamples.
+RSpec/NamedSubject:
+  Exclude:
+    - 'spec/controllers/spree/api/feedback_reviews_controller_spec.rb'
+    - 'spec/controllers/spree/api/reviews_controller_spec.rb'
+    - 'spec/models/reviews_configuration_spec.rb'
+
+# Offense count: 8
+RSpec/NestedGroups:
+  Max: 4
+
+# Offense count: 3
+RSpec/StubbedMock:
+  Exclude:
+    - 'spec/controllers/spree/admin/reviews_controller_spec.rb'
+    - 'spec/controllers/spree/reviews_controller_spec.rb'
+
+# Offense count: 1
+RSpec/UnspecifiedException:
+  Exclude:
+    - 'spec/controllers/spree/feedback_reviews_controller_spec.rb'
+
+# Offense count: 2
+# Configuration parameters: IgnoreNameless, IgnoreSymbolicNames.
+RSpec/VerifiedDoubles:
+  Exclude:
+    - 'spec/models/reviews_ability_spec.rb'
+
+# Offense count: 1
+# Configuration parameters: Include.
+# Include: app/models/**/*.rb
+Rails/InverseOf:
+  Exclude:
+    - 'app/models/spree/review.rb'
+
+# Offense count: 2
+Rails/ReflectionClassName:
+  Exclude:
+    - 'app/models/spree/feedback_review.rb'
+    - 'app/models/spree/review.rb'
+
+# Offense count: 5
+# Configuration parameters: ForbiddenMethods, AllowedMethods.
+# ForbiddenMethods: decrement!, decrement_counter, increment!, increment_counter, insert, insert!, insert_all, insert_all!, toggle!, touch, touch_all, update_all, update_attribute, update_column, update_columns, update_counters, upsert, upsert_all
+Rails/SkipsModelValidations:
+  Exclude:
+    - 'app/controllers/spree/admin/reviews_controller.rb'
+    - 'db/migrate/20120123141326_recalculate_ratings.rb'
+    - 'spec/controllers/spree/admin/reviews_controller_spec.rb'
+    - 'spec/models/product_spec.rb'
+
+# Offense count: 10
+# Cop supports --auto-correct.
+# Configuration parameters: AutoCorrect, EnforcedStyle.
+# SupportedStyles: nested, compact
+Style/ClassAndModuleChildren:
+  Exclude:
+    - 'app/controllers/spree/admin/feedback_reviews_controller.rb'
+    - 'app/controllers/spree/admin/review_settings_controller.rb'
+    - 'app/controllers/spree/admin/reviews_controller.rb'
+    - 'app/controllers/spree/feedback_reviews_controller.rb'
+    - 'app/controllers/spree/reviews_controller.rb'
+    - 'app/helpers/spree/reviews_helper.rb'
+    - 'app/models/spree/feedback_review.rb'
+    - 'app/models/spree/review.rb'
+    - 'app/models/spree/reviews_ability.rb'
+    - 'app/models/spree/reviews_configuration.rb'
+
+# Offense count: 2
+Style/ClassVars:
+  Exclude:
+    - 'app/decorators/helpers/solidus_reviews/spree/api/api_helpers_decorator.rb'
+
+# Offense count: 1
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: annotated, template, unannotated
+Style/FormatStringToken:
+  Exclude:
+    - 'app/decorators/models/solidus_reviews/spree/product_decorator.rb'
+
+# Offense count: 5
+# Configuration parameters: MinBodyLength.
+Style/GuardClause:
+  Exclude:
+    - 'app/controllers/spree/admin/reviews_controller.rb'
+    - 'lib/controllers/spree/api/feedback_reviews_controller.rb'
+    - 'lib/controllers/spree/api/reviews_controller.rb'
+
+# Offense count: 1
+Style/OptionalBooleanParameter:
+  Exclude:
+    - 'app/helpers/spree/reviews_helper.rb'
+
+# Offense count: 10
+# Cop supports --auto-correct.
+# Configuration parameters: AutoCorrect, AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
+# URISchemes: http, https
+Layout/LineLength:
+  Max: 150