Fixing class/method casing and how api key is passed

Author: winrid

README.md

@@ -29,12 +29,12 @@ This library contains the generated API client and the SSO utilities to make wor
 
 ### Public vs Secured APIs
 
-For the API client, there are two classes, `DefaultAPI` and `PublicAPI`. The `DefaultAPI` contains methods that require your API key, and `PublicAPI` contains api calls
+For the API client, there are two classes, `DefaultApi` and `PublicApi`. The `DefaultApi` contains methods that require your API key, and `PublicApi` contains api calls
 that can be made directly from a browser/mobile device/etc without authentication.
 
 ## Quick Start
 
-### Using Authenticated APIs (DefaultAPI)
+### Using Authenticated APIs (DefaultApi)
 
 **Important:** You must set your API key on the ApiClient before making authenticated requests. If you don't, requests will fail with a 401 error.
 
@@ -46,10 +46,10 @@ config = FastCommentsClient::Configuration.new
 api_client = FastCommentsClient::ApiClient.new(config)
 
 # REQUIRED: Set your API key (get this from your FastComments dashboard)
-config.api_key['ApiKeyAuth'] = 'YOUR_API_KEY_HERE'
+config.api_key['x-api-key'] = 'YOUR_API_KEY_HERE'
 
 # Create the API instance with the configured client
-api = FastCommentsClient::DefaultAPI.new(api_client)
+api = FastCommentsClient::DefaultApi.new(api_client)
 
 # Now you can make authenticated API calls
 begin
@@ -71,14 +71,14 @@ rescue FastCommentsClient::ApiError => e
 end
 ```
 
-### Using Public APIs (PublicAPI)
+### Using Public APIs (PublicApi)
 
 Public endpoints don't require authentication:
 
 ```ruby
 require 'fastcomments-client'
 
-public_api = FastCommentsClient::PublicAPI.new
+public_api = FastCommentsClient::PublicApi.new
 
 begin
   response = public_api.get_comments_public(
@@ -93,8 +93,8 @@ end
 
 ### Common Issues
 
-1. **401 "missing-api-key" error**: Make sure you set `config.api_key['ApiKeyAuth'] = 'YOUR_KEY'` before creating the DefaultAPI instance.
-2. **Wrong API class**: Use `DefaultAPI` for server-side authenticated requests, `PublicAPI` for client-side/public requests.
+1. **401 "missing-api-key" error**: Make sure you set `config.api_key['x-api-key'] = 'YOUR_KEY'` before creating the DefaultApi instance.
+2. **Wrong API class**: Use `DefaultApi` for server-side authenticated requests, `PublicApi` for client-side/public requests.
 3. **Null API key**: The SDK will silently skip authentication if the API key is null, leading to 401 errors.
 
 ## Notes
@@ -131,7 +131,7 @@ puts "SSO Token: #{token}"
 # Use the SSO token to make an authenticated API call
 config = FastCommentsClient::Configuration.new
 api_client = FastCommentsClient::ApiClient.new(config)
-public_api = FastCommentsClient::PublicAPI.new(api_client)
+public_api = FastCommentsClient::PublicApi.new(api_client)
 
 response = public_api.get_comments_public(
   tenant_id: 'your-tenant-id',
@@ -165,7 +165,7 @@ puts "Secure SSO Token: #{token}"
 # Use the SSO token to make an authenticated API call
 config = FastCommentsClient::Configuration.new
 api_client = FastCommentsClient::ApiClient.new(config)
-public_api = FastCommentsClient::PublicAPI.new(api_client)
+public_api = FastCommentsClient::PublicApi.new(api_client)
 
 response = public_api.get_comments_public(
   tenant_id: 'your-tenant-id',

spec/sso_integration_spec.rb

@@ -26,14 +26,14 @@ def get_tenant_id
   end
 
   let(:public_api) do
-    FastCommentsClient::PublicAPI.new(api_client)
+    FastCommentsClient::PublicApi.new(api_client)
   end
 
   let(:default_api) do
     config = FastCommentsClient::Configuration.new
-    config.api_key['ApiKeyAuth'] = API_KEY
+    config.api_key['x-api-key'] = API_KEY
     client = FastCommentsClient::ApiClient.new(config)
-    FastCommentsClient::DefaultAPI.new(client)
+    FastCommentsClient::DefaultApi.new(client)
   end
 
   let(:mock_secure_user) do
@@ -61,9 +61,9 @@ def get_tenant_id
         sso_token = sso.create_token
 
         response = public_api.get_comments_public(
-          tenant_id: TENANT_ID,
-          url_id: 'sdk-test-page-secure',
-          sso: sso_token
+          TENANT_ID,
+          'sdk-test-page-secure',
+          { sso: sso_token }
         )
 
         expect(response).not_to be_nil
@@ -83,21 +83,34 @@ def get_tenant_id
         }
 
         response = public_api.create_comment_public(
-          tenant_id: TENANT_ID,
-          url_id: 'sdk-test-page-secure-comment',
-          broadcast_id: "test-#{timestamp}",
-          comment_data: comment_data,
-          sso: sso_token
+          TENANT_ID,
+          'sdk-test-page-secure-comment',
+          "test-#{timestamp}",
+          comment_data,
+          { sso: sso_token }
         )
 
         expect(response).not_to be_nil
       end
 
     it 'gets comments with DefaultApi' do
+        # First create the SSO user
+        user_data = FastCommentsClient::CreateAPISSOUserData.new({
+          id: mock_secure_user.user_id,
+          email: mock_secure_user.email,
+          username: mock_secure_user.username,
+          avatar_src: mock_secure_user.avatar
+        })
+
+        default_api.add_sso_user(TENANT_ID, user_data)
+
+        # Then get comments with that user's context
         response = default_api.get_comments(
-          tenant_id: TENANT_ID,
-          url_id: 'sdk-test-page-secure-admin',
-          context_user_id: mock_secure_user.user_id
+          TENANT_ID,
+          {
+            url_id: 'sdk-test-page-secure-admin',
+            context_user_id: mock_secure_user.user_id
+          }
         )
 
         expect(response).not_to be_nil
@@ -110,9 +123,9 @@ def get_tenant_id
         sso_token = sso.create_token
 
         response = public_api.get_comments_public(
-          tenant_id: TENANT_ID,
-          url_id: 'sdk-test-page-simple',
-          sso: sso_token
+          TENANT_ID,
+          'sdk-test-page-simple',
+          { sso: sso_token }
         )
 
         expect(response).not_to be_nil
@@ -132,11 +145,11 @@ def get_tenant_id
         }
 
         response = public_api.create_comment_public(
-          tenant_id: TENANT_ID,
-          url_id: 'sdk-test-page-simple-comment',
-          broadcast_id: "test-#{timestamp}",
-          comment_data: comment_data,
-          sso: sso_token
+          TENANT_ID,
+          'sdk-test-page-simple-comment',
+          "test-#{timestamp}",
+          comment_data,
+          { sso: sso_token }
         )
 
         expect(response).not_to be_nil
@@ -150,9 +163,9 @@ def get_tenant_id
 
         expect {
           public_api.get_comments_public(
-            tenant_id: 'invalid-tenant-123',
-            url_id: 'sdk-test-page-secure',
-            sso: sso_token
+            'invalid-tenant-123',
+            'sdk-test-page-secure',
+            { sso: sso_token }
           )
         }.to raise_error(FastCommentsClient::ApiError) do |error|
           expect(error.code).to be >= 400
@@ -164,9 +177,9 @@ def get_tenant_id
 
         expect {
           public_api.get_comments_public(
-            tenant_id: TENANT_ID,
-            url_id: 'sdk-test-page-secure',
-            sso: malformed_sso
+            TENANT_ID,
+            'sdk-test-page-secure',
+            { sso: malformed_sso }
           )
         }.to raise_error(FastCommentsClient::ApiError) do |error|
           expect(error.code).to be >= 400