propel_api 0.2.1 → 0.3.0

data/lib/generators/propel_api/templates/tests/integration_test_template.rb.tt

@@ -8,6 +8,16 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
     @organization = organizations(:acme_org)
     @user = users(:john_user)
     @agency = agencies(:marketing_agency)
+<% polymorphic_associations.each do |assoc| -%>
+<% assoc[:parent_types].each_with_index do |parent_type, index| -%>
+<% if index == 0 -%>
+    @<%= parent_type.underscore %> = <%= parent_type.underscore.pluralize %>(<%= parent_type.underscore == 'agency' ? ':marketing_agency' : ':one' %>)
+    @<%= assoc[:field_name] %> = @<%= parent_type.underscore %>  # Default polymorphic parent for tests
+<% else -%>
+    @<%= parent_type.underscore %> = <%= parent_type.underscore.pluralize %>(<%= index == 1 ? ':marketing_agency' : ':one' %>)
+<% end -%>
+<% end -%>
+<% end -%>
 <% if singular_table_name == 'organization' -%>
     @<%= singular_table_name %> = <%= table_name %>(:acme_org)
 <% elsif singular_table_name == 'user' -%>
@@ -21,7 +31,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
     @auth_headers = { 'Authorization' => "Bearer #{@token}" }
     
     # Ensure test <%= singular_table_name %> belongs to test user's organization
-<% unless singular_table_name == 'organization' -%>
+<% if has_organization_reference? -%>
     @<%= singular_table_name %>.update!(organization: @organization)
 <% end -%>
   end
@@ -45,7 +55,12 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
 
 <% attributes.each_with_index do |attribute, index| -%>
 <% if attribute.type == :references -%>
+<% if attribute.respond_to?(:polymorphic?) && attribute.polymorphic? -%>
+      <%= attribute.name %>_id: @<%= attribute.name %>.id,
+      <%= attribute.name %>_type: @<%= attribute.name %>.class.name<%= ',' if index < attributes.length - 1 %>
+<% else -%>
       <%= attribute.name %>_id: @<%= attribute.name %>.id<%= ',' if index < attributes.length - 1 %>
+<% end -%>
 <% elsif attribute.type == :string && !reference_names.include?(attribute.name) -%>
 <% if attribute.name.to_s.match?(/email/) -%>
       <%= attribute.name %>: "workflow@example.com"<%= ',' if index < attributes.length - 1 %>
@@ -64,11 +79,18 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
       <%= attribute.name %>: 100.50<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.type == :datetime -%>
       <%= attribute.name %>: Time.current<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(meta|settings)$/) -%>
-<% if attribute.name == 'meta' -%>
-      <%= attribute.name %>: { workflow_test_meta: "test_value", created_by: "integration_test", test_run_id: <%= rand(1000..9999) %> }<%= ',' if index < attributes.length - 1 %>
+<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% if attribute.name == 'metadata' -%>
+      <%= attribute.name %>: { resource_type: "<%= singular_table_name %>", category: "test", tags: ["integration", "auto_generated"], attributes: { priority: "medium", status_info: { verified: true, active: true } } }<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name == 'settings' -%>
-      <%= attribute.name %>: { dark_mode: true, money_format: "usd", language: "en", timezone: "UTC", notifications: { email: true, sms: false } }<%= ',' if index < attributes.length - 1 %>
+      <%= attribute.name %>: { 
+        ui_preferences: { 
+          theme: "dark", 
+          language: "en", 
+          notifications: { email: true, sms: false, push: true } 
+        }, 
+        feature_flags: { beta_features: false, analytics: true } 
+      }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
       <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
 <% end -%>
@@ -131,17 +153,28 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
     if created_<%= singular_table_name %>['<%= attribute.name %>'].present?
       assert_match(/\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}/, created_<%= singular_table_name %>['<%= attribute.name %>'], "<%= attribute.name %> should be ISO timestamp format")
     end
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(meta|settings)$/) -%>
-<% if attribute.name == 'meta' -%>
-    # Meta field should be a hash with test metadata
-    assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>'], "Meta should be a JSON object"
-    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "workflow_test_meta", "Meta should contain test metadata"
+<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% if attribute.name == 'metadata' -%>
+    # Metadata field should be a hash with generic structure
+    assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>'], "Metadata should be a JSON object"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "resource_type", "Metadata should contain resource_type"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "category", "Metadata should contain category"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "tags", "Metadata should contain tags array"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "attributes", "Metadata should contain attributes object"
+    assert_kind_of Array, created_<%= singular_table_name %>['<%= attribute.name %>']['tags'], "Tags should be an array"
+    assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>']['attributes'], "Attributes should be a hash"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>']['attributes'].keys, "priority", "Attributes should contain priority"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>']['attributes'].keys, "status_info", "Attributes should contain status_info"
 <% elsif attribute.name == 'settings' -%>
-    # Settings field should be a hash with user preferences 
+    # Settings field should be a hash with generic structure
     assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>'], "Settings should be a JSON object"
-    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "dark_mode", "Settings should contain UI preferences"
-    # Rails conventionally serializes booleans in JSON as strings  
-    assert_equal "true", created_<%= singular_table_name %>['<%= attribute.name %>']['dark_mode'], "Dark mode should be enabled in test settings"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "ui_preferences", "Settings should contain ui_preferences"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "feature_flags", "Settings should contain feature_flags"
+    assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>']['ui_preferences'], "UI preferences should be a hash"
+    assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>']['feature_flags'], "Feature flags should be a hash"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>']['ui_preferences'].keys, "theme", "UI preferences should contain theme"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>']['ui_preferences'].keys, "notifications", "UI preferences should contain notifications"
+    assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>']['ui_preferences']['notifications'], "Notifications should be a hash"
 <% else -%>
     assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>'], "<%= attribute.name %> should be a JSON object"
 <% end -%>
@@ -368,7 +401,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
     unique_id = "#{Time.current.to_i}_#{SecureRandom.hex(6)}"
     test_params = { email_address: "tenancy_test_#{unique_id}@example.com", username: "tenancy_test_#{unique_id}", password: "password123", agency_id: @agency.id }
 <% else -%>
-    test_params = { title: "Test <%= class_name %>", agency_id: @agency.id }
+    test_params = { title: "Test <%= class_name %>", agency_id: @agency.id<% polymorphic_associations.each do |assoc| -%>, <%= assoc[:field_name] %>_id: @<%= assoc[:field_name] %>.id, <%= assoc[:field_name] %>_type: @<%= assoc[:field_name] %>.class.name<% end -%> }
 <% end -%>
 <% else -%>
     # Model without agency - test missing organization_id only  
@@ -376,7 +409,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
     unique_id = "#{Time.current.to_i}_#{SecureRandom.hex(6)}"
     test_params = { email_address: "tenancy_test_#{unique_id}@example.com", username: "tenancy_test_#{unique_id}", password: "password123" }
 <% else -%>
-    test_params = { title: "Test <%= class_name %>" }
+    test_params = { title: "Test <%= class_name %>"<% polymorphic_associations.each do |assoc| -%>, <%= assoc[:field_name] %>_id: @<%= assoc[:field_name] %>.id, <%= assoc[:field_name] %>_type: @<%= assoc[:field_name] %>.class.name<% end -%> }
 <% end -%>
 <% end -%>
     
@@ -386,7 +419,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
     
     if require_org_id<% if has_agency_id %> || false<% end -%> # Agency models test organization_id behavior
       # Strict mode: Should return 422 when tenancy context is required but missing
-      assert_response :unprocessable_entity
+      assert_response :unprocessable_content
       
       error_response = JSON.parse(response.body)
       assert_includes error_response.keys, 'errors'
@@ -446,7 +479,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
          headers: @auth_headers
 <% else -%>
     post <%= api_route_helper %>_url,
-         params: { data: { organization_id: @organization.id, agency_id: 99999, title: "Test <%= class_name %>" } },
+         params: { data: { organization_id: @organization.id, agency_id: 99999, title: "Test <%= class_name %>"<% polymorphic_associations.each do |assoc| -%>, <%= assoc[:field_name] %>_id: @<%= assoc[:field_name] %>.id, <%= assoc[:field_name] %>_type: @<%= assoc[:field_name] %>.class.name<% end -%> } },
          headers: @auth_headers
 <% end -%>
     
@@ -465,7 +498,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
          headers: @auth_headers
 <% else -%>
     post <%= api_route_helper %>_url,
-         params: { data: { organization_id: 99999, title: "Test <%= class_name %>" } },
+         params: { data: { organization_id: 99999, title: "Test <%= class_name %>"<% polymorphic_associations.each do |assoc| -%>, <%= assoc[:field_name] %>_id: @<%= assoc[:field_name] %>.id, <%= assoc[:field_name] %>_type: @<%= assoc[:field_name] %>.class.name<% end -%> } },
          headers: @auth_headers
 <% end -%>
     
@@ -551,11 +584,19 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
       <%= attribute.name %>: true<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.type == :decimal || attribute.type == :float -%>
       <%= attribute.name %>: 11.1<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(meta|settings)$/) -%>
-<% if attribute.name == 'meta' -%>
-      <%= attribute.name %>: { org1_test_meta: "org1_value", tenant: "org1", test_scenario: "multi_tenancy" }<%= ',' if index < attributes.length - 1 %>
+<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% if attribute.name == 'metadata' -%>
+      <%= attribute.name %>: { 
+        resource_type: "<%= singular_table_name %>", 
+        category: "employee", 
+        tags: ["org1", "department_test"], 
+        attributes: { 
+          priority: "medium", 
+          status_info: { verified: true, active: true } 
+        } 
+      }<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name == 'settings' -%>
-      <%= attribute.name %>: { dark_mode: false, money_format: "usd", language: "en", org1_preference: true }<%= ',' if index < attributes.length - 1 %>
+      <%= attribute.name %>: { dark_mode: false, notifications: { email: true, sms: true } }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
       <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
 <% end -%>
@@ -589,11 +630,19 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
       <%= attribute.name %>: false<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.type == :decimal || attribute.type == :float -%>
       <%= attribute.name %>: 22.2<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(meta|settings)$/) -%>
-<% if attribute.name == 'meta' -%>
-      <%= attribute.name %>: { org2_test_meta: "org2_value", tenant: "org2", test_scenario: "multi_tenancy" }<%= ',' if index < attributes.length - 1 %>
+<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% if attribute.name == 'metadata' -%>
+      <%= attribute.name %>: { 
+        resource_type: "<%= singular_table_name %>", 
+        category: "manager", 
+        tags: ["org2", "department_test"], 
+        attributes: { 
+          priority: "high", 
+          status_info: { verified: true, active: true } 
+        } 
+      }<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name == 'settings' -%>
-      <%= attribute.name %>: { dark_mode: true, money_format: "eur", language: "fr", org2_preference: true }<%= ',' if index < attributes.length - 1 %>
+      <%= attribute.name %>: { dark_mode: true, notifications: { email: false, sms: true } }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
       <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
 <% end -%>
@@ -667,7 +716,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
     
     # Verify core fields are present
     assert_includes <%= singular_table_name %>_data.keys, 'id'
-<% unless options[:skip_tenancy] || class_name == 'Organization' -%>
+<% if has_organization_reference? -%>
     assert_includes <%= singular_table_name %>_data.keys, 'organization'
     
     # Verify associations are properly included
@@ -737,11 +786,26 @@ if attributes.any? { |attr| attr.type == :references && attr.name == 'user' } &&
       <%= attribute.name %>: true<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.type == :decimal || attribute.type == :float -%>
       <%= attribute.name %>: 50.5<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(meta|settings)$/) -%>
-<% if attribute.name == 'meta' -%>
-      <%= attribute.name %>: { concurrent_test_meta: "concurrent_value", test_type: "concurrent", thread_id: 1 }<%= ',' if index < attributes.length - 1 %>
+<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% if attribute.name == 'metadata' -%>
+      <%= attribute.name %>: { 
+        resource_type: "<%= singular_table_name %>", 
+        category: "admin", 
+        tags: ["concurrent", "test"], 
+        attributes: { 
+          priority: "high", 
+          status_info: { verified: true, active: true } 
+        } 
+      }<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name == 'settings' -%>
-      <%= attribute.name %>: { dark_mode: true, money_format: "gbp", language: "en", concurrent_mode: true }<%= ',' if index < attributes.length - 1 %>
+      <%= attribute.name %>: { 
+        ui_preferences: { 
+          theme: "dark", 
+          language: "en", 
+          notifications: { email: true, sms: false, push: true } 
+        }, 
+        feature_flags: { beta_features: false, analytics: true } 
+      }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
       <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
 <% end -%>
@@ -837,11 +901,19 @@ if attributes.any? { |attr| attr.type == :references && attr.name == 'user' } &&
         <%= attribute.name %>: i.even?<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.type == :decimal || attribute.type == :float -%>
         <%= attribute.name %>: i * 10.5<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(meta|settings)$/) -%>
-<% if attribute.name == 'meta' -%>
-        <%= attribute.name %>: { bulk_test_meta: "bulk_#{i}", batch_number: i, test_type: "bulk_operations" }<%= ',' if index < attributes.length - 1 %>
+<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% if attribute.name == 'metadata' -%>
+        <%= attribute.name %>: { 
+          resource_type: "<%= singular_table_name %>", 
+          category: "employee", 
+          tags: ["bulk", "test_#{i}"], 
+          attributes: { 
+            priority: "medium", 
+            status_info: { verified: true, active: true } 
+          } 
+        }<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name == 'settings' -%>
-        <%= attribute.name %>: { dark_mode: i.even?, money_format: (i.even? ? "usd" : "eur"), language: "en", "bulk_preference_#{i}": true }<%= ',' if index < attributes.length - 1 %>
+        <%= attribute.name %>: { dark_mode: i.even?, notifications: { email: true, sms: i.even? } }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
         <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
 <% end -%>

data/lib/generators/propel_api/templates/tests/model_test_template.rb.tt

@@ -8,6 +8,16 @@ class <%= class_name %>Test < ActiveSupport::TestCase
     @organization = organizations(:acme_org)
     @user = users(:john_user)
     @agency = agencies(:marketing_agency)
+<% polymorphic_associations.each do |assoc| -%>
+<% assoc[:parent_types].each_with_index do |parent_type, index| -%>
+<% if index == 0 -%>
+    @<%= parent_type.underscore %> = <%= parent_type.underscore.pluralize %>(<%= parent_type.underscore == 'agency' ? ':marketing_agency' : ':one' %>)
+    @<%= assoc[:field_name] %> = @<%= parent_type.underscore %>  # Default polymorphic parent for tests
+<% else -%>
+    @<%= parent_type.underscore %> = <%= parent_type.underscore.pluralize %>(<%= index == 1 ? ':marketing_agency' : ':one' %>)
+<% end -%>
+<% end -%>
+<% end -%>
 <% if singular_table_name == 'organization' -%>
     @<%= singular_table_name %> = <%= table_name %>(:acme_org)
 <% elsif singular_table_name == 'user' -%>
@@ -24,7 +34,17 @@ class <%= class_name %>Test < ActiveSupport::TestCase
   
   test "should belong to <%= attribute.name %>" do
     assert_respond_to @<%= singular_table_name %>, :<%= attribute.name %>
+<% if attribute.respond_to?(:polymorphic?) && attribute.polymorphic? -%>
+    assert @<%= singular_table_name %>.<%= attribute.name %>.present?
+    # Polymorphic association - check against valid parent types
+<% polymorphic_associations.each do |assoc| -%>
+<% if assoc[:field_name] == attribute.name -%>
+    assert(<%= assoc[:parent_types].map { |type| "@#{singular_table_name}.#{attribute.name}.is_a?(#{type})" }.join(' || ') %>)
+<% end -%>
+<% end -%>
+<% else -%>
     assert_kind_of <%= attribute.name.camelize %>, @<%= singular_table_name %>.<%= attribute.name %>
+<% end -%>
   end
   
   test "should be invalid without <%= attribute.name %>" do

data/lib/propel_api.rb

@@ -1,3 +1,3 @@
 module PropelApi
-  VERSION = "0.2.1"
+  VERSION = "0.3.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: propel_api
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Ryan Martin, Rafael Pivato, Chi Putera
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-09-02 00:00:00.000000000 Z
+date: 2025-09-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -30,6 +30,26 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
+- !ruby/object:Gem::Dependency
+  name: pagy
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '8.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '8.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '10.0'
 description: A comprehensive Rails generator that creates complete API resources with
   models, controllers, tests, and realistic seed data. Supports both JSON Facet and
   Graphiti serialization engines.