added integration test, little refactoring

Author: igorkasyanchuk

Gemfile

@@ -8,3 +8,12 @@ gem "sqlite3"
 
 # Start debugger with binding.b [https://github.com/ruby/debug]
 # gem "debug", ">= 1.0.0"
+
+gem "sprockets-rails"
+gem "pry"
+gem "wrapped_print"
+
+group :test do
+  gem "capybara"
+  gem "selenium-webdriver"
+end

Gemfile.lock

@@ -1,9 +1,7 @@
 PATH
   remote: .
   specs:
-    lazy_value (0.1.0)
-      ast
-      parser
+    lazy_value (0.0.1)
       rails
 
 GEM
@@ -74,8 +72,18 @@ GEM
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
-    ast (2.4.2)
+    addressable (2.8.4)
+      public_suffix (>= 2.0.2, < 6.0)
     builder (3.2.4)
+    capybara (3.39.0)
+      addressable
+      matrix
+      mini_mime (>= 0.1.3)
+      nokogiri (~> 1.8)
+      rack (>= 1.6.0)
+      rack-test (>= 0.6.3)
+      regexp_parser (>= 1.5, < 3.0)
+      xpath (~> 3.2)
     coderay (1.1.3)
     concurrent-ruby (1.2.2)
     crass (1.0.6)
@@ -94,6 +102,7 @@ GEM
       net-pop
       net-smtp
     marcel (1.0.2)
+    matrix (0.4.2)
     method_source (1.0.0)
     mini_mime (1.1.2)
     minitest (5.18.0)
@@ -109,11 +118,10 @@ GEM
     nio4r (2.5.9)
     nokogiri (1.14.3-x86_64-darwin)
       racc (~> 1.4)
-    parser (3.2.2.0)
-      ast (~> 2.4.1)
     pry (0.14.2)
       coderay (~> 1.1)
       method_source (~> 1.0)
+    public_suffix (5.0.1)
     puma (6.2.1)
       nio4r (~> 2.0)
     racc (1.6.2)
@@ -147,6 +155,13 @@ GEM
       thor (~> 1.0)
       zeitwerk (~> 2.5)
     rake (13.0.6)
+    regexp_parser (2.7.0)
+    rexml (3.2.5)
+    rubyzip (2.3.2)
+    selenium-webdriver (4.8.6)
+      rexml (~> 3.2, >= 3.2.5)
+      rubyzip (>= 1.2.2, < 3.0)
+      websocket (~> 1.0)
     sprockets (4.2.0)
       concurrent-ruby (~> 1.0)
       rack (>= 2.2.4, < 4)
@@ -159,20 +174,25 @@ GEM
     timeout (0.3.2)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
+    websocket (1.2.9)
     websocket-driver (0.7.5)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
     wrapped_print (0.1.1)
       activesupport
+    xpath (3.2.0)
+      nokogiri (~> 1.8)
     zeitwerk (2.6.7)
 
 PLATFORMS
   x86_64-darwin-21
 
 DEPENDENCIES
+  capybara
   lazy_value!
   pry
   puma
+  selenium-webdriver
   sprockets-rails
   sqlite3
   wrapped_print

README.md

@@ -1,10 +1,13 @@
 # LazyValue
+
 Short description and motivation.
 
 ## Usage
+
 How to use my plugin.
 
 ## Installation
+
 Add this line to your application's Gemfile:
 
 ```ruby
@@ -21,8 +24,13 @@ Or install it yourself as:
 $ gem install lazy_value
 ```
 
+## Testing
+
+`bin/rails test:system`.
+
 ## Contributing
-Contribution directions go here.
+
+You are welcome to contribute.
 
 ## License
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

Rakefile

@@ -1,3 +1,7 @@
 require "bundler/setup"
 
 require "bundler/gem_tasks"
+
+require_relative "test/dummy/config/application"
+
+Rails.application.load_tasks

app/controllers/lazy_value/application_controller.rb

@@ -3,7 +3,7 @@ class ApplicationController < ActionController::Base
     REGEX = /<%= lazy_value_tag (?:do|{.+?}) %>(.+?)<% end %>|<%= lazy_value_tag { (.+?) } %>/m
 
     def show
-      data = JSON.parse(LazyValue.message_encryptor.decrypt_and_verify(payload))
+      data = JSON.parse(LazyValue.cryptography.decrypt_and_verify(payload))
       code = File.read(data["path"]).lines[data["lineno"]-1..-1].join
       str = code.match(REGEX)
 

app/helpers/lazy_value/application_helper.rb

@@ -11,7 +11,7 @@ def lazy_value_tag(options: {}, &block)
       options[:class] ||= "lazy-value-tag"
       options[:id] = element_id
 
-      data = LazyValue.message_encryptor.encrypt_and_sign(
+      data = LazyValue.cryptography.encrypt_and_sign(
         {
           path: kaller.path,
           lineno: kaller.lineno

lazy_value.gemspec

@@ -3,20 +3,20 @@ require_relative "lib/lazy_value/version"
 Gem::Specification.new do |spec|
   spec.name        = "lazy_value"
   spec.version     = LazyValue::VERSION
-  spec.authors     = ["Igor Kasyanchuk"]
-  spec.email       = ["[email protected]"]
+  spec.authors     = ["Igor Kasyanchuk", "Liubomyr Manastyretskyi"]
+  spec.email       = ["[email protected]", "[email protected]"]
   spec.homepage    = "https://github.com/railsjazz.com/lazy_value"
-  spec.summary     = "Summary of LazyValue."
-  spec.description = "Description of LazyValue."
+  spec.summary     = "Lazy load values in Rails views."
+  spec.description = "Lazy load values in Rails views."
   spec.license     = "MIT"
 
   spec.metadata["homepage_uri"] = spec.homepage
 
   spec.add_dependency "rails"
-  spec.add_dependency "parser"
-  spec.add_dependency "ast"
   spec.add_development_dependency "puma"
   spec.add_development_dependency "pry"
   spec.add_development_dependency "wrapped_print"
   spec.add_development_dependency "sprockets-rails"
+  spec.add_development_dependency "capybara"
+  spec.add_development_dependency "selenium-webdriver"
 end

lib/lazy_value.rb

@@ -1,13 +1,11 @@
-require 'parser/current'
-require 'ast'
 require "lazy_value/version"
 require "lazy_value/routes"
 require "lazy_value/engine"
 
 module LazyValue
-  mattr_accessor :message_encryptor
+  mattr_accessor :cryptography
 
-  @@message_encryptor = begin
+  @@cryptography = begin
     len   = ActiveSupport::MessageEncryptor.key_len
     salt  = SecureRandom.random_bytes(len)
     key   = ActiveSupport::KeyGenerator.new(SecureRandom.hex(32)).generate_key(salt, len)

lib/lazy_value/version.rb

@@ -1,3 +1,3 @@
 module LazyValue
-  VERSION = "0.1.0"
+  VERSION = "0.0.1"
 end

test/application_system_test_case.rb

@@ -0,0 +1,5 @@
+require "test_helper"
+
+class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
+  driven_by :selenium, using: :chrome, screen_size: [1400, 1400]
+end