propel_api 0.3.1.2 → 0.3.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9e5a2cb9edf143dc6d0f7582e01e51abe3c1ea0cf7e37c0ba9077fe38be5c9fb
-  data.tar.gz: c13fe800d5804bb1fc9846c35baff955e38680ea099e8167b5c00fa66309c6ce
+  metadata.gz: 0bce938f1d94ec21c630795fd3732dc295d3555cb9b3907d9db74cfeac8ee59b
+  data.tar.gz: 15e9bec7ffc083869da29a883af7e5e9bdccbcf1e17c96a8ab8288f4dc7d37e6
 SHA512:
-  metadata.gz: 53fea1d7c31195a2411974a1a3149e2757b77574824639616a8af0b1f80b8e5234b6dead7663f46c84575b07a8ad7560d53ea06fca1dd328f31e0a15214620e9
-  data.tar.gz: e635decc36265a721ca638ff706383cce7e55b87a0bcef1f64c7bf3c6a549facc7845d5aaf29579914264a0ec2ba08493f159b65972ccb20b28fe4952b3fe200
+  metadata.gz: f76dd64b8ab5f5e6ec831f0e920e6b3d19502694a3c51ef8d01ef82524692c869c37bce68df5b813d5ebb64d24c5eb37cddca62608ae7edcd8ecbac06c247b21
+  data.tar.gz: c6a464ff6524bdcd2943dfbde5ee6f4624f9acac724ef9f508a22e66f4225d1b216259953a76e12f5a326b57f996873f8ba98225bfd85f182b23e36c108b9349

data/CHANGELOG.md

@@ -10,6 +10,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Planned Features
 - GraphQL adapter support
 
+## [0.3.1.3] - 2025-09-11
+
+### 🐛 Bug Fixes
+- **Polymorphic Association Facet Includes**: Fixed facet includes for polymorphic associations in model templates
+  - Polymorphic associations are now properly included in `details` facet for eager loading
+  - Enhanced model template to handle polymorphic associations in facet includes correctly
+  - Fixed Schedule model and similar polymorphic association issues
+- **Data Integrity Test Fixes**: Improved model test template for more reliable data integrity tests
+  - Fixed hardcoded test expectations to match actual test data creation
+  - Better handling of polymorphic associations in integrity tests
+  - Enhanced test data generation for name/title fields to use realistic values (e.g., "integrity_test")
+  - Added proper JSON/JSONB field testing support
+
 ## [0.3.1.2] - 2025-09-11
 
 ### 🎉 New Features
@@ -27,20 +40,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Command-line flag support for per-generator control
   - Backward compatible with existing configurations
 
-### 🐛 Bug Fixes
-- **Polymorphic Association Facet Includes**: Fixed facet includes for polymorphic associations in model templates
-  - Polymorphic associations are now properly included in `details` facet for eager loading
-  - Enhanced model template to handle polymorphic associations in facet includes correctly
-  - Fixed Schedule model and similar polymorphic association issues
-- **Data Integrity Test Fixes**: Improved model test template for more reliable data integrity tests
-  - Fixed hardcoded test expectations to match actual test data creation
-  - Better handling of polymorphic associations in integrity tests
-  - Enhanced test data generation for name/title fields to use realistic values
-  - Added proper JSON/JSONB field testing support
-- **HTTP Status Code Consistency**: Fixed tokens controller to use consistent HTTP status codes
-  - Changed validation errors from `:unprocessable_content` (422) to `:bad_request` (400)
-  - Better alignment with standard HTTP parameter validation practices
-  - Improved error handling consistency across authentication endpoints
 
 ## [0.3.1.1] - 2025-09-11
 

data/lib/generators/propel_api/core/named_base.rb

@@ -544,7 +544,14 @@ module PropelApi
         polymorphic_attrs = @attributes.select { |attr| attr.type == :references && attr.respond_to?(:polymorphic?) && attr.polymorphic? }
         
         polymorphic_attrs.map do |attr|
-          parent_types = polymorphic_parent_types[attr.name.to_s] || discover_available_models.first(1)
+          parent_types = polymorphic_parent_types[attr.name.to_s] || discover_available_models.first(3)
+          
+          # Fallback to User if no parent types specified and no models auto-discovered
+          if parent_types.empty?
+            say "Warning: No parent types specified for polymorphic association '#{attr.name}' and no models found to auto-discover.", :yellow
+            say "Falling back to 'User' as parent type. Use --parents ModelA,ModelB,ModelC to specify different parent types.", :yellow
+            parent_types = ['User']
+          end
           
           # Create one association entry per polymorphic field with all parent types
           {
@@ -554,7 +561,7 @@ module PropelApi
             parent_types: parent_types,
             # For fixture generation, we'll use the first parent type for fixture "one", second for "two", etc.
             primary_parent_type: parent_types.first,
-            primary_parent_table: parent_types.first.underscore.pluralize
+            primary_parent_table: parent_types.first&.underscore&.pluralize
           }
         end
       end
@@ -572,6 +579,10 @@ module PropelApi
       fixture_name = case parent_type.underscore
       when 'agency'
         ['marketing_agency', 'tech_agency', 'sales_agency'][fixture_index] || 'marketing_agency'
+      when 'user'
+        ['john_user', 'jane_user', 'confirmed_user'][fixture_index] || 'john_user'
+      when 'organization'
+        ['acme_org', 'tech_startup', 'acme_org'][fixture_index] || 'acme_org'
       else
         ['one', 'two', 'three'][fixture_index] || 'one'
       end
@@ -588,27 +599,46 @@ module PropelApi
     private
 
     def parse_polymorphic_parent_types
-      return {} unless defined?(options) && options[:parents]
+      return {} unless defined?(options) && options[:parents] && !options[:parents].empty?
+      
+      # With the simplified syntax, we apply the same parent types to all polymorphic associations
+      parent_types = options[:parents].map(&:strip).map(&:camelize)
       
+      # Debug: Show what we parsed
+      say "Debug: Parsed parent types from --parents: #{parent_types.inspect}", :cyan if parent_types.any?
+      
+      # Find all polymorphic associations and apply the same parent types to each
       parsed_types = {}
-      options[:parents].each do |field_name, parent_list|
-        parsed_types[field_name] = parent_list.split(',').map(&:strip).map(&:camelize)
+      attrs = respond_to?(:attributes) ? attributes : (@attributes || [])
+      
+      if attrs && !attrs.empty?
+        polymorphic_attrs = attrs.select { |attr| attr.type == :references && attr.respond_to?(:polymorphic?) && attr.polymorphic? }
+        polymorphic_attrs.each do |attr|
+          parsed_types[attr.name.to_s] = parent_types
+          say "Debug: Applied parent types #{parent_types.inspect} to polymorphic field '#{attr.name}'", :cyan
+        end
+      else
+        say "Debug: No attributes found when parsing polymorphic parent types", :yellow
       end
+      
       parsed_types
     end
 
     def discover_available_models
       # Auto-discover available models from app/models directory
       models_dir = File.join(destination_root, 'app/models')
-      return ['Post'] unless File.directory?(models_dir) # Fallback
+      return [] unless File.directory?(models_dir) # No fallback - return empty array
       
       model_files = Dir[File.join(models_dir, '*.rb')]
-      model_files.map do |file|
+      models = model_files.map do |file|
         File.basename(file, '.rb').camelize
       end.reject do |model|
         # Skip common non-domain models
         %w[ApplicationRecord Concerns].include?(model)
       end.sort
+      
+      # Ensure we always return at least one model as fallback
+      models.empty? ? [] : models
     end
   end
 end 

data/lib/generators/propel_api/templates/tests/fixtures_template.yml.tt

@@ -97,13 +97,13 @@ one:
   <%= attribute.name %>: "2024-06-15 10:30:00"
 <% elsif attribute.type == :time -%>
   <%= attribute.name %>: "10:30:00"
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% if attribute.name == 'metadata' -%>
   <%= attribute.name %>: { resource_type: "<%= singular_table_name %>", category: "test", tags: ["fixture", "one"], attributes: { priority: "high", status_info: { verified: true, active: true } } }
 <% elsif attribute.name == 'settings' -%>
   <%= attribute.name %>: { ui_preferences: { theme: "light", language: "en", notifications: { email: true, sms: false, push: true } }, feature_flags: { beta_features: false, analytics: true } }
 <% else -%>
-  <%= attribute.name %>: { test_data: "fixture_one" }
+  <%= attribute.name %>: { field_type: "jsonb_field", test_data: { string_value: "sample_text_one", numeric_value: 42, boolean_value: true, array_value: ["item1", "item2", "item3"], nested_object: { property: "value", settings: ["option1", "option2"] } }, metadata: { generated_by: "test_fixture", field_name: "<%= attribute.name %>", fixture_id: "one" } }
 <% end -%>
 <% else -%>
   <%= attribute.name %>: "test_value_one"
@@ -201,13 +201,13 @@ two:
   <%= attribute.name %>: "2023-12-25 15:45:00"
 <% elsif attribute.type == :time -%>
   <%= attribute.name %>: "15:45:00"
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% if attribute.name == 'metadata' -%>
   <%= attribute.name %>: { resource_type: "<%= singular_table_name %>", category: "premium", tags: ["fixture", "two"], attributes: { priority: "medium", status_info: { verified: false, active: true } } }
 <% elsif attribute.name == 'settings' -%>
   <%= attribute.name %>: { ui_preferences: { theme: "dark", language: "fr", notifications: { email: true, sms: true, push: false } }, feature_flags: { beta_features: true, analytics: false } }
 <% else -%>
-  <%= attribute.name %>: { test_data: "fixture_two" }
+  <%= attribute.name %>: { field_type: "jsonb_field", test_data: { string_value: "sample_text_two", numeric_value: 84, boolean_value: false, array_value: ["item4", "item5", "item6"], nested_object: { property: "different_value", settings: ["option3", "option4"] } }, metadata: { generated_by: "test_fixture", field_name: "<%= attribute.name %>", fixture_id: "two" } }
 <% end -%>
 <% else -%>
   <%= attribute.name %>: "test_value_two"
@@ -305,13 +305,13 @@ three:
   <%= attribute.name %>: "2025-03-10 08:15:00"
 <% elsif attribute.type == :time -%>
   <%= attribute.name %>: "08:15:00"
-<% elsif attribute.type == :json || attribute.name.to_s.match?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% if attribute.name == 'metadata' -%>
   <%= attribute.name %>: { resource_type: "<%= singular_table_name %>", category: "basic", tags: ["fixture", "three"], attributes: { priority: "low", status_info: { verified: true, active: false } } }
 <% elsif attribute.name == 'settings' -%>
   <%= attribute.name %>: { ui_preferences: { theme: "auto", language: "es", notifications: { email: false, sms: true, push: true } }, feature_flags: { beta_features: false, analytics: true } }
 <% else -%>
-  <%= attribute.name %>: { test_data: "fixture_three" }
+  <%= attribute.name %>: { field_type: "jsonb_field", test_data: { string_value: "sample_text_three", numeric_value: 126, boolean_value: true, array_value: ["item7", "item8", "item9"], nested_object: { property: "final_value", settings: ["option5", "option6"] } }, metadata: { generated_by: "test_fixture", field_name: "<%= attribute.name %>", fixture_id: "three" } }
 <% end -%>
 <% else -%>
   <%= attribute.name %>: "test_value_three"

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

@@ -89,7 +89,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
       <%= attribute.name %>: "password123"<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name.to_s.match?(/\A(url|website|web_address|domain|domain_name)\z/i) -%>
       <%= attribute.name %>: "https://workflow.example.com"<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.name.to_s.end_with?('_type') -%>
+<% elsif attribute.name.to_s.end_with?('_type') && attribute.name.to_s.match?(/_parent_type$/) -%>
       <%= attribute.name %>: @<%= attribute.name.gsub('_type', '') %>.class.name<%= ',' if index < attributes.length - 1 %>
 <% else -%>
       <%= attribute.name %>: "Workflow Test <%= attribute.name.humanize %>"<%= ',' if index < attributes.length - 1 %>
@@ -104,7 +104,7 @@ 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?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% 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' -%>
@@ -117,7 +117,24 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
         feature_flags: { beta_features: false, analytics: true } 
       }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
-      <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
+      <%= attribute.name %>: {
+        field_type: "jsonb_field",
+        test_data: {
+          string_value: "api_test_data",
+          numeric_value: 123,
+          boolean_value: true,
+          array_value: ["test1", "test2", "test3"],
+          nested_object: {
+            property: "api_value",
+            settings: ["api_option1", "api_option2"]
+          }
+        },
+        metadata: {
+          generated_by: "integration_test",
+          field_name: "<%= attribute.name %>",
+          test_id: "workflow_test"
+        }
+      }<%= ',' if index < attributes.length - 1 %>
 <% end -%>
 
 <% elsif !reference_names.include?(attribute.name) -%>
@@ -161,6 +178,8 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
 <% elsif attribute.name.to_s.match?(/password/) -%>
     # Password fields should NOT be returned in API responses for security
     assert_not_includes created_<%= singular_table_name %>.keys, '<%= attribute.name %>', "Password fields should be filtered from API responses"
+<% elsif attribute.name.to_s.match?(/\A(url|website|web_address|domain|domain_name)\z/i) -%>
+    assert_equal "https://workflow.example.com", created_<%= singular_table_name %>['<%= attribute.name %>']
 <% else -%>
     assert_equal "Workflow Test <%= attribute.name.humanize %>", created_<%= singular_table_name %>['<%= attribute.name %>']
 <% end -%>
@@ -173,12 +192,12 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
 <% elsif attribute.type == :decimal || attribute.type == :float -%>
     # Rails conventionally serializes decimal/float values as strings in JSON
     assert_equal "100.5", created_<%= singular_table_name %>['<%= attribute.name %>']
-<% elsif attribute.type == :datetime || attribute.type == :timestamp || attribute.name.to_s.match?(/_at$/) -%>
+<% elsif attribute.type == :datetime || attribute.type == :timestamp || (attribute.name.to_s.match?(/_at$/) && attribute.type != :json && attribute.type != :jsonb) -%>
     # Datetime fields should be properly formatted ISO timestamps, not exact values
     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?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% 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"
@@ -201,7 +220,35 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
     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"
+    # Generic JSON field should have standard test structure
+    assert_kind_of Hash, created_<%= singular_table_name %>['<%= attribute.name %>'], "<%= attribute.name.humanize %> should be a JSON object"
+    
+    # Verify main structure
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "field_type", "<%= attribute.name.humanize %> should contain field_type"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "test_data", "<%= attribute.name.humanize %> should contain test_data"
+    assert_includes created_<%= singular_table_name %>['<%= attribute.name %>'].keys, "metadata", "<%= attribute.name.humanize %> should contain metadata"
+    
+    # Verify test_data structure (tests all JSON data types)
+    test_data = created_<%= singular_table_name %>['<%= attribute.name %>']['test_data']
+    assert_kind_of Hash, test_data, "test_data should be a hash"
+    assert_includes test_data.keys, "string_value", "test_data should contain string_value"
+    assert_includes test_data.keys, "numeric_value", "test_data should contain numeric_value"
+    assert_includes test_data.keys, "boolean_value", "test_data should contain boolean_value"
+    assert_includes test_data.keys, "array_value", "test_data should contain array_value"
+    assert_includes test_data.keys, "nested_object", "test_data should contain nested_object"
+    
+    # Verify data types
+    assert_kind_of String, test_data['string_value'], "string_value should be a string"
+    assert_kind_of Integer, test_data['numeric_value'], "numeric_value should be a number"
+    assert_kind_of Array, test_data['array_value'], "array_value should be an array"
+    assert_kind_of Hash, test_data['nested_object'], "nested_object should be a hash"
+    
+    # Verify metadata structure
+    metadata = created_<%= singular_table_name %>['<%= attribute.name %>']['metadata']
+    assert_kind_of Hash, metadata, "metadata should be a hash"
+    assert_includes metadata.keys, "generated_by", "metadata should contain generated_by"
+    assert_includes metadata.keys, "field_name", "metadata should contain field_name"
+    assert_equal "<%= attribute.name %>", metadata['field_name'], "metadata field_name should match attribute name"
 <% end -%>
 <% else -%>
     assert_equal "workflow_test", created_<%= singular_table_name %>['<%= attribute.name %>']
@@ -599,7 +646,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
       <%= attribute.name %>: "org1_secure_password"<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name.to_s.match?(/\A(url|website|web_address|domain|domain_name)\z/i) -%>
       <%= attribute.name %>: "https://org1.example.com"<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.name.to_s.end_with?('_type') -%>
+<% elsif attribute.name.to_s.end_with?('_type') && attribute.name.to_s.match?(/_parent_type$/) -%>
       <%= attribute.name %>: @<%= attribute.name.gsub('_type', '') %>.class.name<%= ',' if index < attributes.length - 1 %>
 <% else -%>
       <%= attribute.name %>: "Org 1 <%= attribute.name.humanize %>"<%= ',' if index < attributes.length - 1 %>
@@ -612,7 +659,7 @@ 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?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% if attribute.name == 'metadata' -%>
       <%= attribute.name %>: { 
         resource_type: "<%= singular_table_name %>", 
@@ -626,7 +673,24 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
 <% elsif attribute.name == 'settings' -%>
       <%= attribute.name %>: { dark_mode: false, notifications: { email: true, sms: true } }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
-      <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
+      <%= attribute.name %>: {
+        field_type: "jsonb_field",
+        test_data: {
+          string_value: "api_test_data",
+          numeric_value: 123,
+          boolean_value: true,
+          array_value: ["test1", "test2", "test3"],
+          nested_object: {
+            property: "api_value",
+            settings: ["api_option1", "api_option2"]
+          }
+        },
+        metadata: {
+          generated_by: "integration_test",
+          field_name: "<%= attribute.name %>",
+          test_id: "workflow_test"
+        }
+      }<%= ',' if index < attributes.length - 1 %>
 <% end -%>
 <% else -%>
       <%= attribute.name %>: "org1_value"<%= ',' if index < attributes.length - 1 %>
@@ -649,7 +713,7 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
       <%= attribute.name %>: "org2_secure_password"<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name.to_s.match?(/\A(url|website|web_address|domain|domain_name)\z/i) -%>
       <%= attribute.name %>: "https://org2.example.com"<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.name.to_s.end_with?('_type') -%>
+<% elsif attribute.name.to_s.end_with?('_type') && attribute.name.to_s.match?(/_parent_type$/) -%>
       <%= attribute.name %>: @<%= attribute.name.gsub('_type', '') %>.class.name<%= ',' if index < attributes.length - 1 %>
 <% else -%>
       <%= attribute.name %>: "Org 2 <%= attribute.name.humanize %>"<%= ',' if index < attributes.length - 1 %>
@@ -662,7 +726,7 @@ 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?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% if attribute.name == 'metadata' -%>
       <%= attribute.name %>: { 
         resource_type: "<%= singular_table_name %>", 
@@ -676,7 +740,24 @@ class <%= class_name %>ApiTest < ActionDispatch::IntegrationTest
 <% elsif attribute.name == 'settings' -%>
       <%= attribute.name %>: { dark_mode: true, notifications: { email: false, sms: true } }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
-      <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
+      <%= attribute.name %>: {
+        field_type: "jsonb_field",
+        test_data: {
+          string_value: "api_test_data",
+          numeric_value: 123,
+          boolean_value: true,
+          array_value: ["test1", "test2", "test3"],
+          nested_object: {
+            property: "api_value",
+            settings: ["api_option1", "api_option2"]
+          }
+        },
+        metadata: {
+          generated_by: "integration_test",
+          field_name: "<%= attribute.name %>",
+          test_id: "workflow_test"
+        }
+      }<%= ',' if index < attributes.length - 1 %>
 <% end -%>
 <% else -%>
       <%= attribute.name %>: "org2_value"<%= ',' if index < attributes.length - 1 %>
@@ -809,7 +890,7 @@ if attributes.any? { |attr| attr.type == :references && attr.name == 'user' } &&
       <%= attribute.name %>: "concurrent@example.com"<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name.to_s.match?(/\A(url|website|web_address|domain|domain_name)\z/i) -%>
       <%= attribute.name %>: "https://concurrent.example.com"<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.name.to_s.end_with?('_type') -%>
+<% elsif attribute.name.to_s.end_with?('_type') && attribute.name.to_s.match?(/_parent_type$/) -%>
       <%= attribute.name %>: @<%= attribute.name.gsub('_type', '') %>.class.name<%= ',' if index < attributes.length - 1 %>
 <% else -%>
       <%= attribute.name %>: "Concurrent Test <%= attribute.name.humanize %>"<%= ',' if index < attributes.length - 1 %>
@@ -822,7 +903,7 @@ 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?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% if attribute.name == 'metadata' -%>
       <%= attribute.name %>: { 
         resource_type: "<%= singular_table_name %>", 
@@ -843,7 +924,24 @@ if attributes.any? { |attr| attr.type == :references && attr.name == 'user' } &&
         feature_flags: { beta_features: false, analytics: true } 
       }<%= ',' if index < attributes.length - 1 %>
 <% else -%>
-      <%= attribute.name %>: {}<%= ',' if index < attributes.length - 1 %>
+      <%= attribute.name %>: {
+        field_type: "jsonb_field",
+        test_data: {
+          string_value: "api_test_data",
+          numeric_value: 123,
+          boolean_value: true,
+          array_value: ["test1", "test2", "test3"],
+          nested_object: {
+            property: "api_value",
+            settings: ["api_option1", "api_option2"]
+          }
+        },
+        metadata: {
+          generated_by: "integration_test",
+          field_name: "<%= attribute.name %>",
+          test_id: "workflow_test"
+        }
+      }<%= ',' if index < attributes.length - 1 %>
 <% end -%>
 <% else -%>
       <%= attribute.name %>: "concurrent_test"<%= ',' if index < attributes.length - 1 %>
@@ -928,7 +1026,7 @@ if attributes.any? { |attr| attr.type == :references && attr.name == 'user' } &&
         <%= attribute.name %>: "bulk#{i}@example.com"<%= ',' if index < attributes.length - 1 %>
 <% elsif attribute.name.to_s.match?(/\A(url|website|web_address|domain|domain_name)\z/i) -%>
         <%= attribute.name %>: "https://bulk#{i}.example.com"<%= ',' if index < attributes.length - 1 %>
-<% elsif attribute.name.to_s.end_with?('_type') -%>
+<% elsif attribute.name.to_s.end_with?('_type') && attribute.name.to_s.match?(/_parent_type$/) -%>
         <%= attribute.name %>: @<%= attribute.name.gsub('_type', '') %>.class.name<%= ',' if index < attributes.length - 1 %>
 <% else -%>
         <%= attribute.name %>: "Bulk Test <%= attribute.name.humanize %> #{i}"<%= ',' if index < attributes.length - 1 %>
@@ -941,7 +1039,7 @@ 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?(/^(metadata|settings)$/) -%>
+<% elsif attribute.type == :json || attribute.type == :jsonb -%>
 <% if attribute.name == 'metadata' -%>
         <%= attribute.name %>: { 
           resource_type: "<%= singular_table_name %>", 
@@ -955,7 +1053,24 @@ if attributes.any? { |attr| attr.type == :references && attr.name == 'user' } &&
 <% elsif attribute.name == 'settings' -%>
         <%= 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 %>
+        <%= attribute.name %>: {
+        field_type: "jsonb_field",
+        test_data: {
+          string_value: "api_test_data",
+          numeric_value: 123,
+          boolean_value: true,
+          array_value: ["test1", "test2", "test3"],
+          nested_object: {
+            property: "api_value",
+            settings: ["api_option1", "api_option2"]
+          }
+        },
+        metadata: {
+          generated_by: "integration_test",
+          field_name: "<%= attribute.name %>",
+          test_id: "workflow_test"
+        }
+      }<%= ',' if index < attributes.length - 1 %>
 <% end -%>
 <% else -%>
         <%= attribute.name %>: "bulk_#{i}"<%= ',' if index < attributes.length - 1 %>

data/lib/propel_api.rb

@@ -1,3 +1,3 @@
 module PropelApi
-  VERSION = "0.3.1.2"
+  VERSION = "0.3.1.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: propel_api
 version: !ruby/object:Gem::Version
-  version: 0.3.1.2
+  version: 0.3.1.3
 platform: ruby
 authors:
 - Ryan Martin, Rafael Pivato, Chi Putera