pbbuilder 0.13.3 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 10736ed13e1665f1bf45095973eb464ab2bcef87f3d69fe4bb8f89e3f35e0474
-  data.tar.gz: a2a8522c756cdf99638d7aef7f3ad7ed9d009ab24b46e499f2f7e0d4a3427b5e
+  metadata.gz: e89f023f27d95d8d11cdf454148ef955b0234b6d7e59f40e30859e09c02761cb
+  data.tar.gz: 108693d16d963bc46ad96d3f79ed4b8078f8efc9c3a7f798c32b7bdd5cc98e73
 SHA512:
-  metadata.gz: 366ce4bf8ef0195f67f9961cd15b5ebc8311b3c15c192d54a5a064a33a34e9e7696a2114afab7f4734bb1e89d61769ae823a91567562741bfc4bb0e9a41a4d3d
-  data.tar.gz: 88d5bc81285ac6e48531328b03a2194b5939dd40ca8bfe003502486f7009c22bb55f1a18c2767f28153942801c7f913526d5a5c18294373a00fcffbfd6daeef4
+  metadata.gz: aa478fc7fad5954801385b731b7fc77296d732e8f13a6bc5688417dfdeb247a7135f3ecbce14f88abc441a87cbc6ad669d3f5547b917f80bfddfbac7e0b2c9a8
+  data.tar.gz: 2fe79ab3cfde6b908907d55eaf37054cb0b530f4e98dbf44d732bef82c46054f0efd4c6ba3a05827915cc855f9cc70a671adc2316c691edd5147ababec573ec6

data/CHANGELOG.md

@@ -3,6 +3,14 @@ All notable changes to this project will be documented in this file.
 
 This format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 0.15.0
+### Changed
+- #merge! method was refactored to accomodate caching for all data types (especially those that are :repeated)
+
+## 0.14.0
+### Added
+- Adding `frozen_string_literal: true` to all files.
+
 ## 0.13.2 2023.02.3
 ### Fixed
 - In case ActiveSupport::Cache::FileStore in Rails is used as a cache, File.atomic_write can have a race condition and fail to rename temporary file. We're attempting to recover from that, by catching this specific error and returning a value.

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 # Specify your gem's dependencies in pbbuilder.gemspec.

data/README.md

@@ -1,21 +1,26 @@
 # Pbbuilder
 PBBuilder generates [Protobuf](https://developers.google.com/protocol-buffers) Messages with a simple DSL similar to [JBuilder](https://rubygems.org/gems/jbuilder) gem.
 
-## Usage
-It basically works exactly like jbuilder. The main difference is that it can use introspection to figure out what kind of protobuf message it needs to create.
+## Compatibility
+We don't aim to have 100% compatibility with jbuilder gem, but we closely follow jbuilder's API design. 
+
+| | Jbuilder | Pbbuilder |
+|---|---|---|
+|  set! | ✅ | ✅ |
+|  cache! | ✅ | ✅ |
+|  cache_if! | ✅ | ✅ |
+| cache_root! | ✅|  |
+| extract! | ✅ | ✅ |
+| merge! | ✅ | ✅ |
+| deep_format_keys! | ✅ |  |
+| child! | ✅ |  |
+| array! | ✅ |  |
+| ignore_nil! | ✅ |  |
 
+## Usage
+The main difference is that it can use introspection to figure out what kind of protobuf message it needs to create.
 
-Following Pbbuilder code
-```
-person = RPC::Person.new
- Pbbuilder.new(person) do |pb|
-   pb.name "Hello"
-   pb.friends [1, 2, 3] do |number|
-     pb.name "Friend ##{number}"
-   end
- end
-```
-Would produce this message:
+This is an example `.proto` message.
 
 ```
 message Person {
@@ -24,6 +29,52 @@ message Person {
 }
 ```
 
+The following `.pb` file would generate a message of valid Person type.
+```
+person = RPC::Person.new
+
+Pbbuilder.new(person) do |pb|
+  pb.name "Hello"
+  pb.friends [1, 2, 3] do |number|
+    pb.name "Friend ##{number}"
+  end
+end
+```
+
+Under the hood, this DSL is using `method_missing` and `set!` methods. But there are other methods and features to use.
+
+### extract!
+The following `_account.pb.pbbuilder` partial:
+```
+pb.id account.id
+pb.phone_number account.phone_number
+pb.tag account.tag
+```
+
+could be rewritten to a shorter version with a use of `extract!`.
+```
+pb.extract! account, :id, :phone_number, :tag
+```
+
+### Partials
+Given partial `_account.pb.pbuilder`:
+
+```
+pb.name account.name
+pb.registration_date account.created_at
+```
+
+Using partial while passing a variable to it
+
+```
+pb.account partial: "account", account: @account
+```
+
+Here is way to use partials with collection while passing a variable to it
+
+```
+pb.accounts @accounts, partial: "account", as: account
+```
 
 ### Caching
 Fragment caching is supported, it uses Rails.cache and works like caching in HTML templates:
@@ -42,7 +93,6 @@ pb.cache_if! !admin?, "cache-key", expires_in: 10.minutes do
 end
 ```
 
-
 ## Installation
 Add this line to your application's Gemfile:
 
@@ -59,10 +109,20 @@ Or install it yourself as:
 ```bash
 $ gem install pbbuilder
 ```
+## Development
 
-## Contributing
+When debugging, make sure to prepend `::Kernel` to any calls such as `puts` as otherwise the code will think you're trying to add another attribute into protobuf object.
 
-When debugging, make sure you're prepending `::Kernel` to any calls such as `puts` as otherwise the code will think you're trying to add another attribute onto the protobuf.
+In case, you're looking to use breakpoints for debugging purposes - it's better to use `pry`. Just make sure to [change pbbuilder superclass from `ProxyObject/BasicObject` to `Object`](lib/pbbuilder/pbbuilder.rb).
+
+## Testing
+Running `bundle exec appraisal rake test` locally will run entire testsuit with all version of rails. To run tests only for certain rails version do the following `bundle exec appraisal rails-7-0 rake test`
+
+To run only one tests from file - use `m` utility. Like this:
+`bundle exec appraisal rails-7-0 m test/pbbuilder_template_test.rb:182`
+
+## Contributing
+Everyone is welcome to contribute.
 
 ## License
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "bundler/setup"
 require "bundler/gem_tasks"
 require "rake/testtask"

data/lib/pbbuilder/errors.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pbbuilder'
 
 class Pbbuilder

data/lib/pbbuilder/handler.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pbbuilder/template"
 
 # Basically copied and pasted from JbuilderHandler, except it uses Pbbuilder

data/lib/pbbuilder/pbbuilder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Pbbuilder = Class.new(begin
   require 'active_support/proxy_object'
   ActiveSupport::ProxyObject

data/lib/pbbuilder/protobuf_extension.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "google/protobuf/message_exts"
 
 module Google::Protobuf::MessageExts::ClassMethods

data/lib/pbbuilder/railtie.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails'
 require "pbbuilder/handler"
 

data/lib/pbbuilder/template.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # PbbuilderTemplate is an extension of Pbbuilder to be used as a Rails template
 # It adds support for partials.
 class PbbuilderTemplate < Pbbuilder

data/lib/pbbuilder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pbbuilder/pbbuilder"
 require 'pbbuilder/errors'
 require "pbbuilder/protobuf_extension"
@@ -114,33 +116,58 @@ class Pbbuilder
     ::Kernel.raise Pbbuilder::MergeError.build(target!, object) unless object.class == ::Hash
 
     object.each_key do |key|
-      if object[key].respond_to?(:empty?) && object[key].empty?
-        ::Kernel.raise Pbbuilder::MergeError.build(target!, object)
-      end
+      next if object[key].respond_to?(:empty?) && object[key].empty?
 
-      if object[key].class == ::String
-        # pb.fields {"one" => "two"}
-        @message[key.to_s] = object[key]
-      elsif object[key].class == ::Array
-        # pb.tags ['test', 'ok']
-        @message[key.to_s].replace object[key]
-      elsif object[key].class == ::TrueClass || object[key].class == ::FalseClass
-        # pb.boolean true || false
-        @message[key.to_s] = object[key]
-      elsif ( obj = object[key]).class == ::Hash
-        # pb.field_name do
-        #    pb.tags ["ok", "cool"]
-        # end
-        #
+      descriptor = @message.class.descriptor.lookup(key.to_s)
+      ::Kernel.raise ::ArgumentError, "Unknown field #{name}" if descriptor.nil?
 
+      if descriptor.label == :repeated
         # optional empty fields don't show up in @message object,
         # we recreate empty message, so we can fill it with values
         if @message[key.to_s].nil?
-          field_descriptor = @message.class.descriptor.lookup(key.to_s)
-          @message[key.to_s] = _new_message_from_descriptor(field_descriptor)
+          @message[key.to_s] = _new_message_from_descriptor(descriptor)
+        end
+
+        if object[key].respond_to?(:to_hash)
+          object[key].to_hash.each {|k, v| @message[key.to_s][k] = v}
+        elsif object[key].respond_to?(:to_ary)
+          elements = object[key].map do |obj|
+            descriptor.subtype ? descriptor.subtype.msgclass.new(obj) : obj
+          end
+
+          @message[key.to_s].replace(elements)
         end
+      else
+        if object[key].class == ::String
+          # pb.fields {"one" => "two"}
+          @message[key.to_s] = object[key]
+        elsif object[key].class == ::TrueClass || object[key].class == ::FalseClass
+          # pb.boolean true || false
+          @message[key.to_s] = object[key]
+        elsif object[key].class == ::Array
+          # pb.field_name do
+          #    pb.tags ["ok", "cool"]
+          # end
+
+          @message[key.to_s] = object[key]
+        elsif object[key].class == ::Hash
+          if @message[key.to_s].nil?
+            @message[key.to_s] = _new_message_from_descriptor(descriptor)
+          end
 
-        @message[key.to_s] = _scope(@message[key.to_s]) { self.merge!(obj) }
+          object[key].each do |k, v|
+            if object[key][k].respond_to?(:to_hash)
+              _scope(@message[key.to_s][k.to_s]) { self.merge!(object[key][k]) }
+            elsif object[key][k].respond_to?(:to_ary)
+              @message[key.to_s][k.to_s].replace object[key][k]
+            else
+              # Throws an error, if we try to merge nil object into empty value.
+              next if object[key][k].nil? && @message[key.to_s][k.to_s].nil?
+
+              @message[key.to_s][k.to_s] = object[key][k]
+            end
+          end
+        end
       end
     end
   end

data/pbbuilder.gemspec

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 Gem::Specification.new do |spec|
   spec.name = "pbbuilder"
-  spec.version = "0.13.3"
+  spec.version = "0.15.0"
   spec.authors = ["Bouke van der Bijl"]
   spec.email = ["bouke@cheddar.me"]
   spec.homepage = "https://github.com/cheddar-me/pbbuilder"

data/test/pbbuilder_template_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 require "action_view/testing/resolvers"
 
@@ -101,13 +103,13 @@ class PbbuilderTemplateTest < ActiveSupport::TestCase
   end
 
   test "should raise Error in merge! an empty hash" do
-    assert_raise(ActionView::Template::Error) {
+    assert_nothing_raised {
       render(<<-PBBUILDER)
         pb.merge! "name" => {}
       PBBUILDER
     }
 
-    assert_raise(ActionView::Template::Error) {
+    assert_nothing_raised {
       render(<<-PBBUILDER)
         pb.merge! "" => {}
       PBBUILDER
@@ -129,6 +131,39 @@ class PbbuilderTemplateTest < ActiveSupport::TestCase
     assert_equal "suslik", result["name"]
   end
 
+  test "caching repeated partial" do
+    template = <<-PBBUILDER
+      pb.cache! "some-random-key" do
+        pb.friends @friends, partial: "racers/racer", as: :racer
+      end
+    PBBUILDER
+    friends = [Racer.new(1, "Johnny Test", []), Racer.new(2, "Max Verstappen", [])]
+
+    result = assert_nothing_raised { render(template, friends: friends) }
+    assert_equal("Johnny Test", result.friends[0].name)
+    assert_equal("Max Verstappen", result.friends[1].name)
+
+    result = render('pb.cache! "some-random-key" do; end ')
+    assert_equal("Johnny Test", result.friends[0].name)
+    assert_equal("Max Verstappen", result.friends[1].name)
+  end
+
+  test "caching map values" do
+    template = <<-PBBUILDER
+      pb.cache! "some-random-cache-key" do
+        pb.favourite_foods @foods
+      end
+    PBBUILDER
+
+    r = assert_nothing_raised { render( template, foods: {'pizza' => 'yes', 'borsh' => 'false'})}
+    assert_equal('false', r.favourite_foods['borsh'])
+    assert_equal('yes', r.favourite_foods['pizza'])
+
+    result = render('pb.cache! "some-random-cache-key" do; end ')
+    assert_equal('false', result.favourite_foods['borsh'])
+    assert_equal('yes', result.favourite_foods['pizza'])
+  end
+
   test "object fragment caching" do
     render(<<-PBBUILDER)
       pb.cache! "cache-key" do

data/test/pbbuilder_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class PbbuilderTest < ActiveSupport::TestCase

data/test/protobuf_extension_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "test_helper"
 
 class ProtobufExtensionTest < ActiveSupport::TestCase

data/test/test_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "bundler/setup"
 
 require "rails"
@@ -15,6 +17,7 @@ require "google/protobuf"
 require "google/protobuf/field_mask_pb"
 
 require "active_support/testing/autorun"
+require "pry"
 
 ActiveSupport.test_order = :random
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pbbuilder
 version: !ruby/object:Gem::Version
-  version: 0.13.3
+  version: 0.15.0
 platform: ruby
 authors:
 - Bouke van der Bijl
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-13 00:00:00.000000000 Z
+date: 2023-05-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-protobuf