alexandria-book-collection-manager 0.7.2 → 0.7.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 68616d81acb2df9a9457285c4393b569a4d217c9e5b25371ad0e2addf3e24d93
-  data.tar.gz: 2f3f920293bfc91c7d23cb380c2b0a9806ca0eecf3112639ff859db8fee10faf
+  metadata.gz: 88a834cc5346bc0704749533797d91e6ff2a04c372e066fe9f91ca239baa63f1
+  data.tar.gz: 4a68efa9e12431f6ea356325b4bcee4a702a7f1f49710860a399657866c158e6
 SHA512:
-  metadata.gz: e80f5e730a5d0e81e776968853974043cd672abf8e9aaffd568a9ae82c80c70c5a6ff86a6306072809690743e96fb473b32c292a09f2fc349b6a5b60abdbf07c
-  data.tar.gz: a7a037e7ac5cb5c91cf2f707fa77a34996fd1935cce22113fd0555868d146e47da892d64206bc61f065a3c853e4603d6e80c1e117be347d378e0ee9c989a4f32
+  metadata.gz: 213ed219cd387370ff664b32b108aff3e6655e07579962f436f013616e02e423ef17afb6ed6bc6f036d2c5de863a9b51f8c5d5de5e8f33b9a51a0dff6522da61
+  data.tar.gz: c6440f573914defb43a7a9f451a4cf5e9091275039137c4dabd02d2290c72e87a17d35f62db08f7669d1a66f51ea53e5ef1b1e170cdd3beea237df1d3a1f5dd4

data/.github/dependabot.yml

@@ -0,0 +1,9 @@
+# Documentation for all configuration options:
+# https://help.github.com/github/administering-a-repository/configuration-options-for-dependency-updates
+
+version: 2
+updates:
+  - package-ecosystem: "bundler"
+    directory: "/"
+    schedule:
+      interval: "daily"

data/.github/workflows/ruby.yml

@@ -0,0 +1,77 @@
+# This workflow will download a prebuilt Ruby version, install dependencies and
+# run tests with Rake
+# For more information see: https://github.com/marketplace/actions/setup-ruby-jruby-and-truffleruby
+
+name: Ruby
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  test:
+
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        ruby: [2.5, 2.6, 2.7]
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Install non-ruby dependencies
+      run: |
+        # Needed for gtk3 gem
+        sudo apt-get install libgtk-3-dev
+        # Needed for gstreamer gem
+        sudo apt-get install libgstreamer1.0-dev
+        # Needed for GooCanvas::Canvas widget
+        sudo apt-get install libgoocanvas-2.0-dev
+        # Needed for zoom gem
+        sudo apt-get install libyaz-dev
+        # Needed for intltool-merge executable
+        sudo apt-get install intltool
+        # Needed for gconftool-2 executable
+        sudo apt-get install gconf2
+        # Needed to provide A11y dbus service to silence warnings
+        sudo apt-get install at-spi2-core
+        # Needed to set up sound player pipeline
+        sudo apt-get install gstreamer1.0-plugins-good
+        # Needed to play sound
+        sudo apt-get install pulseaudio
+        # Provides xvfb-run
+        sudo apt-get install xvfb
+
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+        bundler-cache: true
+    - name: Run unit tests
+      run: xvfb-run bundle exec rake spec:unit
+    - name: Run end-to-end tests in a dbus session
+      run: xvfb-run dbus-run-session bundle exec rake spec:end_to_end
+
+  rubocop:
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Install non-ruby dependencies
+      run: |
+        # Needed for gtk3 gem
+        sudo apt-get install libgtk-3-dev
+        # Needed for gstreamer gem
+        sudo apt-get install libgstreamer1.0-dev
+        # Needed for zoom gem
+        sudo apt-get install libyaz-dev
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 2.7
+        bundler-cache: true
+    - name: Run RuboCop
+      run: bundle exec rubocop -P

data/.gitignore

@@ -3,12 +3,16 @@ config.save
 alexandria.desktop
 alexandria.desktop.in.h
 .svn-authors
-lib/alexandria/default_preferences.rb
 Gemfile.lock
 .yardoc/
 share/locale/
 share/omf/alexandria/*.omf
+spec/examples.txt
 pkg/
 coverage/
 tmp/
+po/*.po~
+po/*.mo
+po/*.pot
 *.gem
+*.glade~

data/.hound.yml

@@ -0,0 +1,2 @@
+ruby:
+  config_file: .rubocop.yml

data/.rubocop.yml

@@ -1,18 +1,47 @@
+inherit_from: .rubocop_todo.yml
+
+require:
+  - rubocop-performance
+  - rubocop-rspec
+  - rubocop-i18n
+
 AllCops:
   Exclude:
     - 'lib/alexandria/default_preferences.rb'
     - 'pkg/**/*'
-  TargetRubyVersion: 2.3
+    - 'vendor/bundle/**/*'
+  NewCops: enable
+  TargetRubyVersion: 2.5
+
+# Choose GetText-style I18n cops
+GetText:
+  Enabled: true
+  # Exclude non-user-facing code
+  Exclude:
+    - Rakefile
+    - 'util/**/*'
+    - 'spec/**/*'
+RailsI18n:
+  Enabled: false
 
-# Dot at end of line makes it clearer that the line is not done
-Layout/DotPosition:
-  EnforcedStyle: trailing
+# This code has a lot of pre-existing translations with percent-style
+# formatting. Maybe one day someone will re-do all of them ...
+GetText/DecorateStringFormattingUsingPercent:
+  Enabled: false
 
-# Multi-line assignment should be simply indented. Aligning them makes it even
-# harder to keep a sane line length.
-Layout/MultilineOperationIndentation:
-  EnforcedStyle: indented
+# Make BeginEndAlignment behavior match EndAlignment
+Layout/BeginEndAlignment:
+  EnforcedStyleAlignWith: begin
+
+Layout/HashAlignment:
+  EnforcedColonStyle: key
+  EnforcedHashRocketStyle: table
 
+# Be lenient with line length
+Layout/LineLength:
+  Max: 92
+
+# Don't force lonely closing parentheses
 Layout/MultilineMethodCallBraceLayout:
   EnforcedStyle: same_line
 
@@ -21,71 +50,98 @@ Layout/MultilineMethodCallBraceLayout:
 Layout/MultilineMethodCallIndentation:
   EnforcedStyle: indented
 
-# Make syntax for empty and filled blocks consistent
+# Multi-line assignment should be simply indented. Aligning them makes it even
+# harder to keep a sane line length.
+Layout/MultilineOperationIndentation:
+  EnforcedStyle: indented
+
+# Force consistent spacing independent of block contents
 Layout/SpaceBeforeBlockBraces:
   EnforcedStyleForEmptyBraces: space
 
+# Assume the programmer knows how bracketed block syntax works
+Lint/AmbiguousBlockAssociation:
+  Enabled: false
+
 # Allow if (foo = get_foo) style
 Lint/AssignmentInCondition:
   AllowSafeAssignment: true
 
+# This cop's reach is too broad
+# See https://github.com/rubocop-hq/rubocop/pull/8480
+Lint/MissingSuper:
+  Enabled: false
+
 # Blocks in specs can be any size
 Metrics/BlockLength:
   Exclude:
     - 'spec/**/*'
 
-# Be lenient with line length
-Metrics/LineLength:
-  Max: 110
-
 Naming/AccessorMethodName:
   Enabled: false
 
-# Allow multi-line blocks of all sorts
-Style/BlockDelimiters:
-  Enabled: false
+# Avoid one-letter exception variable name
+Naming/RescuedExceptionsVariableName:
+  PreferredName: ex
+
+Performance/EndWith:
+  AutoCorrect: true
+
+Performance/StartWith:
+  AutoCorrect: true
+
+# End-to-end specs do not describe classes
+RSpec/DescribeClass:
+  Exclude:
+    - 'spec/end_to_end/**/*_spec.rb'
+
+RSpec/ExampleLength:
+  Max: 8
+  Exclude:
+    - 'spec/end_to_end/**/*_spec.rb'
 
 # Allow and/or for control flow only
 Style/AndOr:
-  Enabled: conditionals
-
-# Prefer old school format strings
-Style/FormatStringToken:
-  EnforcedStyle: unannotated
+  EnforcedStyle: conditionals
 
 # Require at least two dependent lines before suggesting a guard clause
 Style/GuardClause:
   MinBodyLength: 2
 
-# Not all objects that handle #== 0 also handle #zero?
-Style/NumericPredicate:
-  Enabled: false
-
 # Sometimes an if statement just looks better than next with a guard clause
 Style/Next:
   Enabled: false
 
+# Not all objects that handle #== 0 also handle #zero?
+Style/NumericPredicate:
+  Enabled: false
+
 # Use older RuboCop default
 Style/PercentLiteralDelimiters:
   PreferredDelimiters:
+    '%W': ()
     '%w': ()
 
 # Allow explicit return with multiple return values
 Style/RedundantReturn:
   AllowMultipleReturnValues: true
 
-# Always use raise to raise exceptions
-Style/SignalException:
-  EnforcedStyle: only_raise
-
 Style/SingleLineBlockParams:
   Enabled: false
 
-# I like to have symbols always look like symbols
+# Allow all ways to construct strings for now
+Style/StringConcatenation:
+  Enabled: false
+
+# Do not commit to use of interpolation
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+
+# Prefer symbols to look like symbols
 Style/SymbolArray:
   EnforcedStyle: brackets
 
-# Accessors are only trivial if they match the ivar name.
+# Accessors are only trivial if they match the ivar name
 Style/TrivialAccessors:
   ExactNameMatch: true
 
@@ -96,9 +152,3 @@ Style/TrailingCommaInArrayLiteral:
 # Alexandria uses regular arrays
 Style/WordArray:
   Enabled: false
-
-# Not all objects that handle #length also handle #empty?
-Style/ZeroLengthPredicate:
-  Enabled: false
-
-inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -1,170 +1,100 @@
 # This configuration was generated by
-# `rubocop --auto-gen-config`
-# on 2018-03-18 12:56:55 +0100 using RuboCop version 0.53.0.
+# `rubocop --auto-gen-config --no-offense-counts --no-auto-gen-timestamp`
+# using RuboCop version 0.93.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: 12
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: auto_detection, squiggly, active_support, powerpack, unindent
-Layout/IndentHeredoc:
-  Exclude:
-    - 'Rakefile'
-    - 'bin/alexandria'
-    - 'lib/alexandria/export_library.rb'
-    - 'lib/alexandria/ui/dialogs/about_dialog.rb'
-
-# Offense count: 6
-Lint/HandleExceptions:
-  Exclude:
-    - 'lib/alexandria/import_library_csv.rb'
-    - 'lib/alexandria/models/library.rb'
-    - 'lib/alexandria/ui/dialogs/book_properties_dialog.rb'
-    - 'lib/alexandria/ui/dialogs/import_dialog.rb'
-
-# Offense count: 2
-Lint/IneffectiveAccessModifier:
-  Exclude:
-    - 'lib/alexandria/logging.rb'
-    - 'lib/alexandria/web_themes.rb'
-
-# Offense count: 1
-Lint/UnreachableCode:
-  Exclude:
-    - 'lib/alexandria/ui/dialogs/export_dialog.rb'
-
-# Offense count: 1
-Lint/UriEscapeUnescape:
-  Exclude:
-    - 'lib/alexandria/book_providers/amazon_ecs_util.rb'
-
-# Offense count: 2
-# Configuration parameters: ContextCreatingMethods, MethodCreatingMethods.
-Lint/UselessAccessModifier:
-  Exclude:
-    - 'lib/alexandria/logging.rb'
-
-# Offense count: 2
-Lint/UselessAssignment:
-  Exclude:
-    - 'lib/alexandria/models/library.rb'
-    - 'lib/alexandria/smart_library.rb'
-
-# Offense count: 165
+# Configuration parameters: IgnoredMethods.
 Metrics/AbcSize:
-  Max: 161
+  Max: 111
 
-# Offense count: 22
-# Configuration parameters: CountComments, ExcludedMethods.
+# Configuration parameters: CountComments, CountAsOne, ExcludedMethods.
+# ExcludedMethods: refine
 Metrics/BlockLength:
-  Max: 67
+  Max: 64
 
-# Offense count: 13
 # Configuration parameters: CountBlocks.
 Metrics/BlockNesting:
   Max: 5
 
-# Offense count: 32
-# Configuration parameters: CountComments.
+# Configuration parameters: CountComments, CountAsOne.
 Metrics/ClassLength:
-  Max: 994
+  Max: 990
 
-# Offense count: 68
+# Configuration parameters: IgnoredMethods.
 Metrics/CyclomaticComplexity:
-  Max: 28
+  Max: 27
 
-# Offense count: 219
-# Configuration parameters: CountComments.
+# Configuration parameters: CountComments, CountAsOne, ExcludedMethods.
 Metrics/MethodLength:
-  Max: 109
+  Max: 98
 
-# Offense count: 2
-# Configuration parameters: CountComments.
+# Configuration parameters: CountComments, CountAsOne.
 Metrics/ModuleLength:
-  Max: 348
+  Max: 361
 
-# Offense count: 10
 # Configuration parameters: CountKeywordArgs.
 Metrics/ParameterLists:
   Max: 7
 
-# Offense count: 69
+# Configuration parameters: IgnoredMethods.
 Metrics/PerceivedComplexity:
-  Max: 29
+  Max: 27
 
-# Offense count: 8
 Naming/BinaryOperatorParameterName:
   Exclude:
-    - 'lib/alexandria/book_providers.rb'
     - 'lib/alexandria/book_providers/amazon_ecs_util.rb'
-    - 'lib/alexandria/models/book.rb'
-    - 'lib/alexandria/models/library.rb'
-    - 'lib/alexandria/smart_library.rb'
-    - 'lib/alexandria/ui/multi_drag_treeview.rb'
 
-# Offense count: 15
-# Configuration parameters: Blacklist.
-# Blacklist: END, (?-mix:EO[A-Z]{1})
+# Configuration parameters: ForbiddenDelimiters.
+# ForbiddenDelimiters: (?-mix:(^|\s)(EO[A-Z]{1}|END)(\s|$))
 Naming/HeredocDelimiterNaming:
   Exclude:
     - 'Rakefile'
     - 'bin/alexandria'
     - 'lib/alexandria/export_library.rb'
-    - 'lib/alexandria/ui/dialogs/about_dialog.rb'
-
-# Offense count: 2
-Naming/MemoizedInstanceVariableName:
-  Exclude:
-    - 'lib/alexandria/logging.rb'
+    - 'lib/alexandria/ui/about_dialog.rb'
 
-# Offense count: 1
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: snake_case, camelCase
-Naming/MethodName:
-  Exclude:
-    - 'lib/alexandria/book_providers/mcu.rb'
-
-# Offense count: 4
-# Configuration parameters: NamePrefix, NamePrefixBlacklist, NameWhitelist, MethodDefinitionMacros.
+# Configuration parameters: NamePrefix, ForbiddenPrefixes, AllowedMethods, MethodDefinitionMacros.
 # NamePrefix: is_, has_, have_
-# NamePrefixBlacklist: is_, has_, have_
-# NameWhitelist: is_a?
+# ForbiddenPrefixes: is_, has_, have_
+# AllowedMethods: is_a?
 # MethodDefinitionMacros: define_method, define_singleton_method
 Naming/PredicateName:
   Exclude:
     - 'spec/**/*'
     - 'lib/alexandria/book_providers/amazon_ecs_util.rb'
-    - 'lib/alexandria/ui/dialogs/smart_library_properties_dialog_base.rb'
+    - 'lib/alexandria/ui/smart_library_properties_dialog_base.rb'
     - 'lib/alexandria/web_themes.rb'
 
-# Offense count: 10
-# Configuration parameters: MinNameLength, AllowNamesEndingInNumbers, AllowedNames, ForbiddenNames.
-# AllowedNames: io, id
-Naming/UncommunicativeMethodParamName:
+# Configuration parameters: CustomTransform, IgnoreMethods, SpecSuffixOnly.
+RSpec/FilePath:
   Exclude:
-    - 'lib/alexandria/book_providers/amazon_ecs_util.rb'
-    - 'lib/alexandria/book_providers/proxis.rb'
-    - 'lib/alexandria/book_providers/pseudomarc.rb'
-    - 'lib/alexandria/smart_library.rb'
-    - 'lib/alexandria/ui/callbacks.rb'
-    - 'lib/alexandria/ui/init.rb'
-    - 'util/rake/fileinstall.rb'
+    - 'spec/alexandria/ui/iconview_spec.rb'
+    - 'spec/alexandria/ui/sound_spec.rb'
+
+RSpec/MultipleExpectations:
+  Max: 7
 
-# Offense count: 8
 # Cop supports --auto-correct.
-# Configuration parameters: AutoCorrect, EnforcedStyle.
+# Configuration parameters: Strict, EnforcedStyle, AllowedExplicitMatchers.
+# SupportedStyles: inflected, explicit
+RSpec/PredicateMatcher:
+  Exclude:
+    - 'spec/alexandria/export_library_spec.rb'
+    - 'spec/alexandria/library_spec.rb'
+    - 'spec/alexandria/library_store_spec.rb'
+    - 'spec/alexandria/scanners/cue_cat_spec.rb'
+
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle.
 # SupportedStyles: nested, compact
 Style/ClassAndModuleChildren:
   Exclude:
-    - 'lib/alexandria/ui/dialogs/import_dialog.rb'
-    - 'lib/alexandria/ui/dialogs/preferences_dialog.rb'
-    - 'lib/alexandria/ui/icons.rb'
     - 'lib/alexandria/ui/init.rb'
+    - 'lib/alexandria/ui/provider_preferences_base_dialog.rb'
 
-# Offense count: 17
 Style/ClassVars:
   Exclude:
     - 'lib/alexandria/book_providers/amazon_ecs_util.rb'
@@ -172,94 +102,35 @@ Style/ClassVars:
     - 'lib/alexandria/logging.rb'
     - 'lib/alexandria/models/library.rb'
     - 'lib/alexandria/smart_library.rb'
-    - 'lib/alexandria/ui/dialogs/acquire_dialog.rb'
-    - 'lib/alexandria/ui/dialogs/book_properties_dialog_base.rb'
-    - 'lib/alexandria/ui/dialogs/new_book_dialog.rb'
-
-# Offense count: 12
-Style/CommentedKeyword:
-  Exclude:
-    - 'lib/alexandria/book_providers.rb'
-    - 'lib/alexandria/book_providers/barnes_and_noble.rb'
-    - 'lib/alexandria/book_providers/worldcat.rb'
-    - 'lib/alexandria/ui/dialogs/book_properties_dialog_base.rb'
-    - 'lib/alexandria/ui/dialogs/new_book_dialog.rb'
-    - 'util/rake/fileinstall.rb'
+    - 'lib/alexandria/ui/acquire_dialog.rb'
+    - 'lib/alexandria/ui/book_properties_dialog_base.rb'
+    - 'lib/alexandria/ui/new_book_dialog.rb'
 
-# Offense count: 118
 Style/Documentation:
   Enabled: false
 
-# Offense count: 6
 # Configuration parameters: EnforcedStyle.
-# SupportedStyles: each, for
-Style/For:
-  Exclude:
-    - 'lib/alexandria/book_providers/amazon_ecs_util.rb'
-    - 'lib/alexandria/book_providers/douban.rb'
-    - 'lib/alexandria/book_providers/renaud.rb'
-    - 'spec/alexandria/library_spec.rb'
-
-# Offense count: 34
-# Configuration parameters: MinBodyLength.
-Style/GuardClause:
-  Enabled: false
-
-# Offense count: 4
-Style/IdenticalConditionalBranches:
-  Exclude:
-    - 'lib/alexandria/book_providers/amazon_aws.rb'
-    - 'lib/alexandria/ui/ui_manager.rb'
-
-# Offense count: 4
-Style/IfInsideElse:
-  Exclude:
-    - 'lib/alexandria/book_providers/proxis.rb'
-    - 'lib/alexandria/book_providers/web.rb'
-    - 'lib/alexandria/preferences.rb'
-    - 'lib/alexandria/ui/ui_manager.rb'
-
-# Offense count: 2
-Style/MethodMissing:
-  Exclude:
-    - 'lib/alexandria/book_providers.rb'
-    - 'lib/alexandria/preferences.rb'
-
-# Offense count: 1
-Style/MultilineTernaryOperator:
+# SupportedStyles: annotated, template, unannotated
+Style/FormatStringToken:
   Exclude:
+    - 'lib/alexandria/ui/acquire_dialog.rb'
+    - 'lib/alexandria/ui/dndable.rb'
     - 'lib/alexandria/ui/listview.rb'
+    - 'lib/alexandria/ui/new_book_dialog.rb'
 
-# Offense count: 23
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, AllowInnerSlashes.
-# SupportedStyles: slashes, percent_r, mixed
-Style/RegexpLiteral:
+# Configuration parameters: AllowedMethods.
+# AllowedMethods: respond_to_missing?
+Style/OptionalBooleanParameter:
   Exclude:
-    - 'lib/alexandria/book_providers/adlibris.rb'
-    - 'lib/alexandria/book_providers/amazon_ecs_util.rb'
-    - 'lib/alexandria/book_providers/douban.rb'
-    - 'lib/alexandria/book_providers/mcu.rb'
-    - 'lib/alexandria/book_providers/proxis.rb'
-    - 'lib/alexandria/book_providers/pseudomarc.rb'
-    - 'lib/alexandria/book_providers/renaud.rb'
+    - 'lib/alexandria/book_providers.rb'
+    - 'lib/alexandria/book_providers/barnes_and_noble.rb'
     - 'lib/alexandria/book_providers/siciliano.rb'
     - 'lib/alexandria/book_providers/thalia.rb'
+    - 'lib/alexandria/book_providers/worldcat.rb'
+    - 'lib/alexandria/console.rb'
+    - 'lib/alexandria/execution_queue.rb'
+    - 'lib/alexandria/export_format.rb'
+    - 'lib/alexandria/library_sort_order.rb'
     - 'lib/alexandria/models/library.rb'
-    - 'lib/alexandria/ui/dialogs/book_properties_dialog_base.rb'
-    - 'lib/alexandria/ui/sidepane.rb'
+    - 'lib/alexandria/ui/ui_manager.rb'
     - 'util/rake/fileinstall.rb'
-    - 'util/rake/gettextgenerate.rb'
-
-# Offense count: 78
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: implicit, explicit
-Style/RescueStandardError:
-  Enabled: false
-
-# Offense count: 3
-Style/StructInheritance:
-  Exclude:
-    - 'lib/alexandria/smart_library.rb'
-    - 'lib/alexandria/ui/multi_drag_treeview.rb'