cinch-storage 1.0.0 → 1.2.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: 112eeeb9907a13161c643e3242a57153f138a941abfe612cc7fc96731969df2a
+  data.tar.gz: 6af933b544576f5f2d34f379475bee24d44dd8cd73be9dcc2708f3c51762fdce
+SHA512:
+  metadata.gz: c02e483ce7080587718908926c3048c7d63f2aa7d8b5820656597b0a2945a1d048c08915ed84f597867cef92de5e8a6df428e655bac37e1a42724c3b4f7b0b6f
+  data.tar.gz: fa4d71472116de7c499d13a74dcf523bca6ed27838d7798dabcf8e7ca2cc7b750d14138ab07767a3756bec0c2b62545739bec9c94738fcf436c2821f1057b9ff

data/.travis.yml

@@ -1,4 +1,23 @@
 language: ruby
+env:
+  global:
+    - "JRUBY_OPTS=-Xcext.enabled=true"
 rvm:
-  - 1.9.2
+  - 2.1.0
+  - 2.0.0
   - 1.9.3
+  - 1.9.2
+  - 1.8.7
+  - jruby-18mode
+  - jruby-19mode
+  - ruby-head
+  - jruby-head
+  - ree
+matrix:
+  allow_failures:
+    - rvm: 1.8.7
+    - rvm: ree
+    - rvm: jruby-18mode
+    - rvm: jruby-19mode
+    - rvm: jruby-head
+  fast_finish: true

data/README.md

@@ -26,6 +26,10 @@ Or install it yourself as:
 
 ## Usage
 
+First, require it.
+
+    require 'cinch/storage'
+
 In your plugin you will need to spin up a Storage object as part of your init.
 
     def initialize(*args)
@@ -42,14 +46,10 @@ data collision.
       @storage.save
     end
 
-## This is Horrible Code!
-
-The module is very fragile at the moment as a result of being written in like 20 minutes
-or something as a stopgap.
+## Think this is terrible?
 
-However, since it doesn't look like Cinch::Storage will be happening very soon I will
-be going back through and adding tests and error handling and proper debugging and a
-little tighter integration with Cinch.
+I wouldn't disagree; this is my first time writing anyting remotely
+ORM-like. Please feel free to fork and submit pulls to increase sanity.
 
 ## Contributing
 

data/cinch-storage.gemspec

@@ -4,23 +4,24 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'cinch/storage/version'
 
 Gem::Specification.new do |gem|
-  gem.name          = "cinch-storage"
+  gem.name          = 'cinch-storage'
   gem.version       = Cinch::Storage::VERSION
-  gem.authors       = ["Brian Haberer"]
-  gem.email         = ["bhaberer@gmail.com"]
-  gem.description   = %q{Simle YAML backed Storage solution for Cinch plugins.}
-  gem.summary       = %q{Simle YAML backed Storage solution for Cinch plugins.}
-  gem.homepage      = "https://github.com/bhaberer/cinch-storage"
-
+  gem.authors       = ['Brian Haberer']
+  gem.email         = ['bhaberer@gmail.com']
+  gem.description   = %q(Simple YAML backed Storage for Cinch plugins.)
+  gem.summary       = %q(Data Storage for Cinch plugins.)
+  gem.homepage      = 'https://github.com/bhaberer/cinch-storage'
+  gem.license       = 'MIT'
   gem.files         = `git ls-files`.split($/)
-  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
-  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
-  gem.require_paths = ["lib"]
+  gem.executables   = gem.files.grep(/^bin\//).map { |f| File.basename(f) }
+  gem.test_files    = gem.files.grep(/^(test|spec|features)\//)
+  gem.require_paths = ['lib']
 
-  gem.add_development_dependency 'rake'
-  gem.add_development_dependency 'rspec'
-  gem.add_development_dependency 'coveralls'
-  gem.add_development_dependency 'cinch-test'
+  gem.add_development_dependency 'rake', '~> 12.3.3'
+  gem.add_development_dependency 'rspec', '~> 3'
+  gem.add_development_dependency 'coveralls', '~> 0.7'
+  gem.add_development_dependency 'cinch-test', '~> 0.1', '>= 0.1.2'
 
-  gem.add_dependency    'psych',        '~> 1.3.4'
+  gem.add_dependency 'cinch', '~> 2', '>= 2.0.8'
+  gem.add_dependency 'psych', '~> 2.0', '>= 2.0.4'
 end

data/lib/cinch/storage.rb

@@ -0,0 +1,28 @@
+# -*- encoding: utf-8 -*-
+require 'cinch/storage/version'
+require 'psych'
+
+module Cinch
+  # Addon for Cinch to allow for simple YAML storage of data
+  class Storage
+    attr_accessor :filename, :data
+
+    def initialize(file, init = {})
+      @filename = file
+      @data = Psych.load(File.open(@filename)) if File.exist?(@filename)
+      @data ||= init
+    end
+
+    def save
+      File.open(@filename, 'w') do |file|
+        Psych.dump(@data, file)
+      end
+    end
+
+    def synced_save(bot = @bot)
+      bot.synchronize(@filename) do
+        save
+      end
+    end
+  end
+end

data/lib/cinch/storage/version.rb

@@ -1,5 +1,7 @@
+# -*- encoding: utf-8 -*-
 module Cinch
-  module Storage
-    VERSION = "1.0.0"
+  # versioning info
+  class Storage
+    VERSION = '1.2.1'
   end
 end

data/spec/cinch-storage_spec.rb

@@ -1,43 +1,56 @@
 require 'spec_helper'
 
+Thing = Struct.new(:foo)
+
 class MyPlugin
   include Cinch::Plugin
 
   def initialize(*args)
     super
-    @storage = CinchStorage.new(config[:filename] || '/dev/null', [])
+    @storage = Cinch::Storage.new(config[:filename] || '/dev/null', [])
   end
 
-  match /store (.*)/
 
-  def execute(m, thing)
-    @storage.data << thing
+  match /store (.*)/,       method: :store_text
+  match /storething (.*)/,  method: :store_object
+
+  def store_text(m, text)
+    @storage.data << text
+    @storage.synced_save(@bot)
+  end
+
+  def store_object(m, text)
+    @storage.data << Thing.new(text)
     @storage.synced_save(@bot)
   end
 end
 
 
-describe CinchStorage do
+describe Cinch::Storage do
   include Cinch::Test
 
   before(:each) do
     @filename = '/tmp/storage_test.tmp'
     File.delete(@filename) if File.exist?(@filename)
-    @bot = make_bot(MyPlugin, { :filename => @filename })
+    @bot = make_bot(MyPlugin, { filename: @filename })
   end
 
   it 'should allow users to store information between bot runs' do
-    storage = CinchStorage.new(@filename, [])
+    storage = Cinch::Storage.new(@filename, [])
     storage.data << 'foo'
     storage.save
 
-    CinchStorage.new(@filename).data.
-      should include 'foo'
+    expect(Cinch::Storage.new(@filename).data)
+      .to include('foo')
   end
 
   it 'should handle multiple bot threads attempting to write to the same file' do
-    10.times { |x| get_replies(make_message(@bot, "!store #{x}"))}
-    CinchStorage.new(@filename).data.length.
-      should == 10
+    10.times { |x| get_replies(make_message(@bot, "!store #{x}")) }
+    expect(Cinch::Storage.new(@filename).data.length).to eq(10)
+  end
+
+  it 'should write object data to the disk' do
+    get_replies(make_message(@bot, "!storething foobar"))
+    expect(Cinch::Storage.new(@filename).data.first).to eq(Thing.new('foobar'))
   end
 end

data/spec/spec_helper.rb

@@ -1,4 +1,4 @@
 require 'coveralls'
 Coveralls.wear!
-require 'cinch-storage'
+require 'cinch/storage'
 require 'cinch/test'

metadata

@@ -1,139 +1,158 @@
 --- !ruby/object:Gem::Specification
 name: cinch-storage
 version: !ruby/object:Gem::Version
-  version: 1.0.0
-  prerelease: 
+  version: 1.2.1
 platform: ruby
 authors:
 - Brian Haberer
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2013-07-04 00:00:00.000000000 Z
+date: 2021-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 12.3.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 12.3.3
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: coveralls
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: cinch-test
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.1.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.1.2
+- !ruby/object:Gem::Dependency
+  name: cinch
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.8
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.8
 - !ruby/object:Gem::Dependency
   name: psych
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.3.4
+        version: 2.0.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.3.4
-description: Simle YAML backed Storage solution for Cinch plugins.
+        version: 2.0.4
+description: Simple YAML backed Storage for Cinch plugins.
 email:
 - bhaberer@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .travis.yml
+- ".gitignore"
+- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
 - cinch-storage.gemspec
-- lib/cinch-storage.rb
+- lib/cinch/storage.rb
 - lib/cinch/storage/version.rb
 - spec/cinch-storage_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/bhaberer/cinch-storage
-licenses: []
-post_install_message: 
+licenses:
+- MIT
+metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 1.8.24
-signing_key: 
-specification_version: 3
-summary: Simle YAML backed Storage solution for Cinch plugins.
+rubygems_version: 3.2.6
+signing_key:
+specification_version: 4
+summary: Data Storage for Cinch plugins.
 test_files:
 - spec/cinch-storage_spec.rb
 - spec/spec_helper.rb
-has_rdoc: 

data/lib/cinch-storage.rb

@@ -1,25 +0,0 @@
-require 'cinch/storage/version'
-require 'psych'
-
-class CinchStorage
-  attr_accessor :filename, :data
-
-  def initialize(file, init = Hash.new)
-    @filename = file
-    @data = YAML::load(File.open(@filename)) if File::exist?(@filename)
-    @data ||= init
-  end
-
-  def save
-    File.open(@filename, 'w') do |file|
-      YAML::dump(@data, file)
-    end
-  end
-
-  def synced_save(bot = @bot)
-    bot.synchronize(@filename) do
-      save
-    end
-  end
-end
-