propel_api 0.3.1.2 → 0.3.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9e5a2cb9edf143dc6d0f7582e01e51abe3c1ea0cf7e37c0ba9077fe38be5c9fb
-  data.tar.gz: c13fe800d5804bb1fc9846c35baff955e38680ea099e8167b5c00fa66309c6ce
+  metadata.gz: 57d463a766fd115825dccf217caabd982bf849c9aac5a9dfafc7ef11b2a75c04
+  data.tar.gz: 171026f0732372a319fb79946d3c7991293542a7b080bda4559b28427017b741
 SHA512:
-  metadata.gz: 53fea1d7c31195a2411974a1a3149e2757b77574824639616a8af0b1f80b8e5234b6dead7663f46c84575b07a8ad7560d53ea06fca1dd328f31e0a15214620e9
-  data.tar.gz: e635decc36265a721ca638ff706383cce7e55b87a0bcef1f64c7bf3c6a549facc7845d5aaf29579914264a0ec2ba08493f159b65972ccb20b28fe4952b3fe200
+  metadata.gz: 0576b9c86f00a1dde3691dfafc7049d6ffcc58c0717c98cab5739b3bee7c354046f711f3eaf061caa11f6c43c29f43008e988c55d706e84dfaa57e8ca0266de4
+  data.tar.gz: fd24e624bd103eb1bdd96c4e85701198b77964716ebf35d267b1d08f75ee7fc8f6016b85c5374eedc111bb7371af8210779874050db707c906290129fdc1d828

data/CHANGELOG.md

@@ -10,6 +10,35 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Planned Features
 - GraphQL adapter support
 
+## [0.3.1.4] - 2025-09-11
+
+### 🐛 Critical Bug Fixes
+- **Polymorphic Parents Parsing Regression**: Fixed `NoMethodError: undefined method 'strip' for Array` 
+  - Reverted to original, working `parse_polymorphic_parent_types` implementation
+  - Fixed overcomplicated parsing logic that incorrectly assumed Thor gives Arrays
+  - Thor consistently provides comma-separated String values that need simple `.split(',')` handling
+  - Resolves crash when using `--parents` option with polymorphic generators
+
+### 📖 Documentation Improvements
+- **--parents Flag Usage Guide**: Added comprehensive documentation for correct --parents syntax
+  - Fixed misleading error messages that suggested unsupported syntax
+  - Added clear examples of correct vs incorrect --parents usage
+  - Enhanced README with detailed usage guide and common pitfalls
+  - Updated all generator descriptions to show required field_name:Parent1,Parent2 syntax
+
+## [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 +56,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
 
@@ -92,7 +107,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Integration Test Templates**: Enhanced error handling and parameter validation in generated integration tests
 
 ### ⚠️ Breaking Changes
-- **REQUIRED --parents for Polymorphic**: Polymorphic associations now require explicit `--parents Parent1,Parent2` declaration. Auto-discovery fallback has been removed. Generator will block with confirmation if --parents is missing.
+- **REQUIRED --parents for Polymorphic**: Polymorphic associations now require explicit `--parents field_name:Parent1,Parent2` declaration. Auto-discovery fallback has been removed. Generator will block with confirmation if --parents is missing.
 
 ## [0.3.0] - 2025-01-15
 

data/README.md

@@ -55,10 +55,10 @@ rails generate propel_api:resource User name:string email:string role:string
 rails generate propel_api:resource Article title:string content:text user:references category:references
 
 # 🎉 NEW: Polymorphic associations (v0.3.0) - REQUIRES --parents flag
-rails generate propel_api:resource Comment content:text commentable:references{polymorphic} --parents Post,Video,Product
+rails generate propel_api:resource Comment content:text commentable:references{polymorphic} --parents commentable:Post,Video,Product
 
 # Polymorphic with tenancy
-rails generate propel_api:resource Review rating:integer comment:text review_parent:polymorphic --parents Product,Agency
+rails generate propel_api:resource Review rating:integer comment:text review_parent:polymorphic --parents review_parent:Product,Agency
 
 # With Graphiti adapter
 rails generate propel_api Product name:string price:decimal --adapter=graphiti
@@ -81,10 +81,10 @@ PropelApi now provides comprehensive support for polymorphic associations with a
 
 ```bash
 # Generate a Comment that can belong to Posts, Videos, or Products
-rails generate propel_api:resource Comment body:text commentable:references{polymorphic} --parents Post,Video,Product
+rails generate propel_api:resource Comment body:text commentable:references{polymorphic} --parents commentable:Post,Video,Product
 
-# Generate a Review with full tenancy support
-rails generate propel_api:resource Review rating:integer comment:text review_parent:references{polymorphic} --parents Product,Agency
+# Generate a Review with full tenancy support  
+rails generate propel_api:resource Review rating:integer comment:text review_parent:references{polymorphic} --parents review_parent:Product,Agency
 ```
 
 ### Key Features
@@ -94,7 +94,8 @@ rails generate propel_api:resource Review rating:integer comment:text review_par
 - No need for complex quote escaping or nested syntax
 
 #### 🎯 Required Parent Specification
-- `--parents Parent1,Parent2,Parent3` defines valid parent models (REQUIRED for polymorphic associations)
+- `--parents field_name:Parent1,Parent2,Parent3` defines valid parent models (REQUIRED for polymorphic associations)
+- Field name must match the polymorphic association name (e.g., `commentable`, `schedule_parent`)
 - Automatically generates varied test data using different parent types
 - Generator will block with confirmation if --parents is not provided
 
@@ -139,6 +140,58 @@ class Api::V1::CommentsController < Api::V1::ApiController
 end
 ```
 
+### --parents Flag Usage Guide
+
+⚠️ **IMPORTANT**: The `--parents` flag requires specific syntax to work correctly.
+
+#### ✅ **Correct Syntax:**
+```bash
+# Single polymorphic field
+--parents field_name:Parent1,Parent2,Parent3
+
+# Real examples:
+--parents commentable:Post,Photo,Video
+--parents schedule_parent:User,Room,MeetingLink
+--parents review_parent:Product,Agency
+
+# Multiple polymorphic fields  
+--parents commentable:Post,Photo schedule_parent:User,Room
+```
+
+#### ❌ **Incorrect Syntax (Will Fail):**
+```bash
+# Missing field name (Thor can't parse this)
+--parents Post,Photo,Video
+
+# Array brackets (not supported)
+--parents commentable:[Post,Photo,Video]
+
+# JSON-like syntax (not supported)  
+--parents "{commentable: [Post,Photo,Video]}"
+```
+
+#### 🔑 **Key Rules:**
+1. **Always include field name**: `field_name:Parent1,Parent2`
+2. **Field name must match polymorphic association**: If your field is `commentable:polymorphic`, use `--parents commentable:...`
+3. **Use commas to separate parent types**: `Post,Photo,Video` not `Post Photo Video`
+4. **Whitespace is automatically cleaned**: `Post, Photo, Video` works fine
+
+### Complete Usage Examples
+
+```bash
+# Standard resource with polymorphic association
+rails generate propel_api:resource Comment content:text commentable:polymorphic --parents commentable:Post,Photo,Video
+
+# Multi-tenant polymorphic resource
+rails generate propel_api:resource Review rating:integer organization:references agency:references review_parent:polymorphic --parents review_parent:Product,Agency,User
+
+# Schedule system with polymorphic parent
+rails generate propel_api:resource Schedule name:string start_time:datetime schedule_parent:polymorphic --parents schedule_parent:User,Room,MeetingLink
+
+# Multiple polymorphic associations in one resource
+rails generate propel_api:resource Attachment file_name:string attachable:polymorphic owner:polymorphic --parents attachable:Post,Comment,Email owner:User,Agency
+```
+
 ### Breaking Changes from v0.2.x
 
 ⚠️ **Migration Required**: The polymorphic syntax and test generation has been completely rewritten for better reliability and Rails compatibility.
@@ -148,7 +201,7 @@ end
 
 The new approach provides:
 - More reliable test generation
-- Better fixture management
+- Better fixture management  
 - Cleaner, more maintainable code
 - Full Rails compatibility
 

data/lib/generators/propel_api/controller/controller_generator.rb

@@ -52,7 +52,7 @@ module PropelApi
     class_option :parents,
                  type: :hash,
                  default: {},
-                 desc: "Specify parent types for polymorphic associations (e.g., --parents commentable:Post,Photo,Video)"
+                 desc: "Specify parent types for polymorphic associations. REQUIRED syntax: --parents model_polymorphic_name:Parent1,Parent2,Parent3 (e.g., --parents commentable:Post,Photo,Video)"
 
     def validate_model_exists
       # Ensure attributes are parsed before validation

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 #{attr.name}:Parent1,Parent2,Parent3 to specify 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
@@ -600,15 +611,18 @@ module PropelApi
     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/resource/resource_generator.rb

@@ -74,7 +74,7 @@ module PropelApi
   class_option :parents,
                type: :hash,
                default: {},
-               desc: "Specify parent types for polymorphic associations (e.g., --parents commentable:Post,Photo,Video)"
+               desc: "Specify parent types for polymorphic associations. REQUIRED syntax: --parents field_name:Parent1,Parent2,Parent3 (e.g., --parents commentable:Post,Photo,Video)"
 
   def create_migration
     initialize_propel_api_settings

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.4"
 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.4
 platform: ruby
 authors:
 - Ryan Martin, Rafael Pivato, Chi Putera