active_recall 1.3.0 → 1.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8177fef2ccf96ffc5a4ec4c6c4a35ad553af43bb620132c734ff29eaf9947990
-  data.tar.gz: 7e055537fc620a7227502ff0e0c2b99db1753cc1be5d4a384bb5e8d179b06a8a
+  metadata.gz: 45416341e36349534f31c031163739ad2c68da6141239c1429a3218900cc8882
+  data.tar.gz: 20e7473dab6cec259e94e9aa146e8429c2f8f6e4dcb7ba79da81e283f18bd172
 SHA512:
-  metadata.gz: 59999a3e84db489a6d4052c1df9bb99a255c954116e837982df837e8c1d4198e0753738028a52db20a11468c82fd436a322ebe9eeac8885f3a10afefdb33ca7d
-  data.tar.gz: b249d23bbd2f286602c07f41ba1dde323c493310a644edf5b210f24df68d0beb56abebcadfe0b785d78b6df3a8834e8e35cb281f7a47bc696fed6cd7dd9a8e8e
+  metadata.gz: b17e1c048223c597767b5e411e70ce308126fff101e2879da81481b94207236cf2410b26f8cd4b3efacdbd500f3d5088d0ead858a62d2924e728c445ef4b15b8
+  data.tar.gz: aea8f7d8a27454d2f120f30d10c6fca12ba951c2a56ed5a23f26bd99b39b60f9250edcacd72939c3b59b9bc43f24e224e32f10364f9973a18ad2471760779f8c

data/.github/workflows/tests.yml

@@ -0,0 +1,46 @@
+name: Tests 
+
+on:
+  pull_request:
+    branches:
+      - master
+  push:
+    branches:
+      - master
+
+jobs:
+  unit_tests:
+    name: Unit Tests
+    if: "contains(github.event.commits[0].message, '[ci skip]') == false"
+    strategy:
+      fail-fast: false
+      matrix:
+        os:
+          - macos 
+          - ubuntu
+        ruby:
+          - 2.5
+          - 2.6
+          - 2.7
+          - 3.0
+        allow_failures:
+          - false
+        include:
+          - os: ubuntu
+            ruby: ruby-head
+            allow_failures: false
+    env:
+      BUNDLE_GEMFILE: "${{ matrix.gemfile }}"
+      ALLOW_FAILURES: "${{ matrix.allow_failures }}"
+    runs-on: ${{ matrix.os }}-latest
+    continue-on-error: ${{ endsWith(matrix.ruby, 'head') || matrix.ruby == 'debug' }}
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+      - name: Setup Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - name: Test
+        run: bundle exec rake spec || $ALLOW_FAILURES

data/.ruby-version

@@ -1 +1 @@
-2.5.3
+3.0.0

data/Gemfile

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 source 'https://rubygems.org'
-ruby '2.5.3'
 
 git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 

data/Gemfile.lock

@@ -1,30 +1,30 @@
 PATH
   remote: .
   specs:
-    active_recall (1.3.0)
-      activerecord (>= 5.2.3)
-      activesupport (>= 5.2.3)
+    active_recall (1.6.2)
+      activerecord (>= 5.2.3, <= 6.2)
+      activesupport (>= 5.2.3, <= 6.2)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activemodel (6.0.2.1)
-      activesupport (= 6.0.2.1)
-    activerecord (6.0.2.1)
-      activemodel (= 6.0.2.1)
-      activesupport (= 6.0.2.1)
-    activesupport (6.0.2.1)
+    activemodel (6.1.3)
+      activesupport (= 6.1.3)
+    activerecord (6.1.3)
+      activemodel (= 6.1.3)
+      activesupport (= 6.1.3)
+    activesupport (6.1.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-      zeitwerk (~> 2.2)
-    concurrent-ruby (1.1.5)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
+    concurrent-ruby (1.1.8)
     diff-lcs (1.3)
-    i18n (1.7.0)
+    i18n (1.8.9)
       concurrent-ruby (~> 1.0)
-    minitest (5.13.0)
-    rake (10.5.0)
+    minitest (5.14.4)
+    rake (12.3.3)
     rdoc (6.2.0)
     rspec (3.8.0)
       rspec-core (~> 3.8.0)
@@ -39,25 +39,20 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.8.0)
     rspec-support (3.8.2)
-    sqlite3 (1.4.1)
-    thread_safe (0.3.6)
-    tzinfo (1.2.6)
-      thread_safe (~> 0.1)
-    zeitwerk (2.2.2)
+    sqlite3 (1.4.2)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    zeitwerk (2.4.2)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
   active_recall!
-  bundler (>= 1.16)
-  rake (~> 10.0)
+  rake (>= 12.0)
   rdoc
-  rspec (~> 3.0)
+  rspec (>= 3.0)
   sqlite3
 
-RUBY VERSION
-   ruby 2.5.3p105
-
 BUNDLED WITH
-   1.17.3
+   2.2.3

data/README.md

@@ -1,8 +1,8 @@
-# ActiveRecall [![Build Status](https://travis-ci.org/jaysonvirissimo/active_recall.svg?branch=master)](https://travis-ci.org/jaysonvirissimo/active_recall)
+# ActiveRecall
 
 **ActiveRecall** is a spaced-repetition system that allows you to treat arbitrary [ActiveRecord](https://github.com/rails/rails/tree/master/activerecord) models as if they were flashcards to be learned and reviewed.
 It it based on, and is intended to be backwards compatible with, the [okubo](https://github.com/rgravina/okubo) gem.
-The primary differentiating features are that it lets the user specify the scheduling algorithm and is fully compatible with Rails 6.
+The primary differentiating features are that it lets the user specify the scheduling algorithm and is fully compatible with Rails 6.0 (and 6.1) and Ruby 3.0.
 
 ## Installation
 
@@ -116,9 +116,12 @@ user.words.next #=> nil
 
 ## Development
 
-After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
+After checking out the repo, run `bin/setup` to install dependencies.
+Then, run `rake spec` to run the tests.
+You can also run `bin/console` for an interactive prompt that will allow you to experiment.
 
-To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and tags, and push the `.gem` file to [rubygems.org](https://rubygems.org).
+To install this gem onto your local machine, run `bundle exec rake install`.
+To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and tags, and push the `.gem` file to [rubygems.org](https://rubygems.org).
 
 ## Contributing
 

data/active_recall.gemspec

@@ -32,11 +32,10 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'bundler', '>= 1.16'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rake', '>= 12.0'
   spec.add_development_dependency 'rdoc'
-  spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'rspec', '>= 3.0'
   spec.add_development_dependency 'sqlite3'
-  spec.add_runtime_dependency 'activerecord', '>= 5.2.3'
-  spec.add_runtime_dependency 'activesupport', '>= 5.2.3'
+  spec.add_runtime_dependency 'activerecord', '>= 5.2.3', '<= 6.2'
+  spec.add_runtime_dependency 'activesupport', '>= 5.2.3', '<= 6.2'
 end

data/lib/active_recall.rb

@@ -10,7 +10,7 @@ require 'active_recall/models/deck'
 require 'active_recall/models/item'
 require 'active_recall/version'
 
-ActiveRecord::Base.send(:include, ActiveRecall::Base)
+ActiveRecord::Base.include ActiveRecall::Base
 
 module ActiveRecall
   class << self

data/lib/active_recall/algorithms/fibonacci_sequence.rb

@@ -2,7 +2,7 @@
 
 module ActiveRecall
   class FibonacciSequence
-    def self.right(box:, current_time: Time.current, times_right:, times_wrong:)
+    def self.right(box:, times_right:, times_wrong:, current_time: Time.current)
       new(
         box: box,
         current_time: current_time,
@@ -11,7 +11,7 @@ module ActiveRecall
       ).right
     end
 
-    def self.wrong(box:, current_time: Time.current, times_right:, times_wrong:)
+    def self.wrong(box:, times_right:, times_wrong:, current_time: Time.current)
       new(
         box: box,
         current_time: current_time,
@@ -20,7 +20,7 @@ module ActiveRecall
       ).wrong
     end
 
-    def initialize(box:, current_time: Time.current, times_right:, times_wrong:)
+    def initialize(box:, times_right:, times_wrong:, current_time: Time.current)
       @box = box
       @current_time = current_time
       @times_right = times_right

data/lib/active_recall/algorithms/leitner_system.rb

@@ -4,7 +4,7 @@ module ActiveRecall
   class LeitnerSystem
     DELAYS = [3, 7, 14, 30, 60, 120, 240].freeze
 
-    def self.right(box:, current_time: Time.current, times_right:, times_wrong:)
+    def self.right(box:, times_right:, times_wrong:, current_time: Time.current)
       new(
         box: box,
         current_time: current_time,
@@ -13,7 +13,7 @@ module ActiveRecall
       ).right
     end
 
-    def self.wrong(box:, current_time: Time.current, times_right:, times_wrong:)
+    def self.wrong(box:, times_right:, times_wrong:, current_time: Time.current)
       new(
         box: box,
         current_time: current_time,
@@ -22,7 +22,7 @@ module ActiveRecall
       ).wrong
     end
 
-    def initialize(box:, current_time: Time.current, times_right:, times_wrong:)
+    def initialize(box:, times_right:, times_wrong:, current_time: Time.current)
       @box = box
       @current_time = current_time
       @times_right = times_right

data/lib/active_recall/deck_methods.rb

@@ -3,10 +3,10 @@
 module ActiveRecall
   module DeckMethods
     def deck
-      d = ActiveRecall::Deck.where(user_id: id, user_type: self.class.name).first_or_create
+      d = ActiveRecall::Deck.find_or_create_by(user_id: id, user_type: self.class.name)
       d.source_class.module_eval do
         def stats
-          ActiveRecall::Item.where(source_id: id, source_type: self.class.name).first
+          ActiveRecall::Item.find_by(source_id: id, source_type: self.class.name)
         end
       end
       d

data/lib/active_recall/item_methods.rb

@@ -3,15 +3,11 @@
 module ActiveRecall
   module ItemMethods
     def right_answer_for!(item)
-      i = deck.items.where(source_id: item.id).first
-      i.right!
-      i.save!
+      deck.items.find_by(source_id: item.id).right!
     end
 
     def wrong_answer_for!(item)
-      i = deck.items.where(source_id: item.id).first
-      i.wrong!
-      i.save!
+      deck.items.find_by(source_id: item.id).wrong!
     end
   end
 end

data/lib/active_recall/models/deck.rb

@@ -37,16 +37,19 @@ module ActiveRecall
     end
 
     def delete(source)
-      item = ActiveRecall::Item.new(deck: self, source_id: source.id, source_type: source.class.name)
-      item.destroy
+      ActiveRecall::Item
+        .find_by(deck: self, source_id: source.id, source_type: source.class.name)
+        .destroy
     end
 
+    # OPTIMIZE: Attempt in active record, rather than array of Ruby records
     def review
       %i[untested failed expired].inject([]) do |words, s|
         words += items.send(s).order(random_order_function).map(&:source)
       end
     end
 
+    # OPTIMIZE: Use optimized #review and build only the record to be returned
     def next
       word = nil
       %i[untested failed expired].each do |category|

data/lib/active_recall/models/item.rb

@@ -22,11 +22,11 @@ module ActiveRecall
     end
 
     def right!
-      update!(algorithm_class.right(scoring_attributes))
+      update!(algorithm_class.right(**scoring_attributes))
     end
 
     def wrong!
-      update!(algorithm_class.wrong(scoring_attributes))
+      update!(algorithm_class.wrong(**scoring_attributes))
     end
 
     private

data/lib/active_recall/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveRecall
-  VERSION = '1.3.0'
+  VERSION = '1.6.2'
 end

data/lib/generators/active_recall/active_recall_generator.rb

@@ -20,9 +20,7 @@ class ActiveRecallGenerator < Rails::Generators::Base
   def create_migration_files
     create_migration_file_if_not_exist 'create_active_recall_tables'
     create_migration_file_if_not_exist 'add_active_recall_item_answer_counts'
-    if options['migrate_data']
-      create_migration_file_if_not_exist 'migrate_okubo_to_active_recall'
-    end
+    create_migration_file_if_not_exist 'migrate_okubo_to_active_recall' if options['migrate_data']
   end
 
   private

metadata

@@ -1,44 +1,30 @@
 --- !ruby/object:Gem::Specification
 name: active_recall
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.6.2
 platform: ruby
 authors:
 - Robert Gravina
 - Jayson Virissimo
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-01-02 00:00:00.000000000 Z
+date: 2021-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.16'
+        version: '12.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.16'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.0'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -57,14 +43,14 @@ dependencies:
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
@@ -88,6 +74,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.2.3
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: '6.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -95,6 +84,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.2.3
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: '6.2'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
@@ -102,6 +94,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.2.3
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: '6.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -109,6 +104,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.2.3
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: '6.2'
 description: A spaced-repetition system to be used with ActiveRecord models
 email:
 - robert.gravina@gmail.com
@@ -117,10 +115,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/tests.yml"
 - ".gitignore"
 - ".rspec"
 - ".ruby-version"
-- ".travis.yml"
 - Gemfile
 - Gemfile.lock
 - LICENSE
@@ -148,7 +146,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org/
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -163,9 +161,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: A spaced-repetition system
 test_files: []

data/.travis.yml

@@ -1,5 +0,0 @@
-language: ruby
-cache: bundler
-rvm:
-  - 2.5.3
-before_install: gem install bundler -v 1.17.3