apipie-rails 0.9.1 → 0.9.2

data/spec/lib/generator/swagger/param_description/type_spec.rb

@@ -0,0 +1,178 @@
+require 'spec_helper'
+
+describe Apipie::Generator::Swagger::ParamDescription::Type do
+  let(:validator_options) { {} }
+  let(:param_description_options) { {}.merge(validator_options) }
+  let(:with_null) { false }
+  let(:http_method) { :GET }
+  let(:path) { '/api' }
+  let(:validator) { String }
+
+  let(:base_dsl_data) do
+    {
+      :api => false,
+      :api_args => [],
+      :api_from_routes => nil,
+      :errors => [],
+      :tag_list => [],
+      :returns => {},
+      :params => [],
+      :headers => [],
+      :resource_id => nil,
+      :short_description => nil,
+      :description => nil,
+      :examples => [],
+      :see => [],
+      :formats => nil,
+      :api_versions => [],
+      :meta => nil,
+      :show => true,
+      :deprecated => false
+    }
+  end
+
+  let(:dsl_data) do
+    base_dsl_data.merge({
+      api_args: [
+        [
+          http_method,
+          path,
+          'description',
+          { deprecated: true }
+        ]
+      ]
+    })
+  end
+
+  let(:resource_desc) do
+    Apipie::ResourceDescription.new(UsersController, 'dummy')
+  end
+
+  let(:method_desc) do
+    Apipie::MethodDescription.new(:show, resource_desc, dsl_data)
+  end
+
+  let(:param_description_name) { 'some_param' }
+
+  let(:param_description) do
+    Apipie::ParamDescription.new(
+      method_desc,
+      param_description_name,
+      validator,
+      param_description_options
+    )
+  end
+
+  let(:type_definition) do
+    described_class.
+      new(param_description, with_null: with_null, controller_method: 'index').
+      to_hash
+  end
+
+  describe 'type' do
+    subject { type_definition[:type] }
+
+    it { is_expected.to eq(validator.to_s.downcase) }
+
+    context 'when validator is enum' do
+      let(:validator) { ['name', 'enum'] }
+
+      it { is_expected.to eq('string') }
+    end
+
+    context 'when validator is a Hash' do
+      let(:validator) { Hash }
+
+      it { is_expected.to eq('object') }
+    end
+  end
+
+  describe 'items' do
+    let(:items) { type_definition[:items] }
+
+    subject { items }
+
+    context 'when has validator is Array' do
+      let(:validator) { Array }
+
+      it { is_expected.to eq({ type: 'string' }) }
+
+      context 'of Hash' do
+        let(:validator_options) { { of: Hash } }
+
+        let(:reference) do
+          Apipie::Generator::Swagger::OperationId.
+            new(http_method: http_method, path: path, param: param_description_name).
+            to_s
+        end
+
+        it { is_expected.to eq({ type: 'string' }) }
+
+        context 'and swagger_json_input_uses_refs is set to true' do
+          before { Apipie.configuration.swagger_json_input_uses_refs = true }
+          after { Apipie.configuration.swagger_json_input_uses_refs = false }
+
+          it 'returns the reference' do
+            expect(subject).to eq({ '$ref' => reference })
+          end
+
+        end
+      end
+
+      describe 'enum' do
+        subject { items[:enum] }
+
+        it { is_expected.to be_blank }
+
+        context 'when validator is Array' do
+          let(:validator) { Array }
+
+          it { is_expected.to be_blank }
+
+          context 'and an array of in values is given' do
+            let(:enum_values) { ['enum-value-1', 'enum-value-2'] }
+            let(:validator_options) { { in: ['enum-value-1', 'enum-value-2'] } }
+
+            it { is_expected.to eq(enum_values) }
+          end
+        end
+      end
+    end
+  end
+
+  describe 'enum' do
+    subject { type_definition[:enum] }
+
+    context 'and an array of in values is given' do
+      let(:validator) { ['enum-value-1', 'enum-value-2'] }
+
+      it { is_expected.to eq(validator) }
+    end
+  end
+
+  describe 'additionalProperties' do
+    subject { type_definition[:additionalProperties] }
+
+    it  { is_expected.to be_blank }
+
+    context 'when validator is a Hash' do
+      let(:validator) { Hash }
+
+      it { is_expected.to be_truthy }
+    end
+  end
+
+  describe 'warnings' do
+    before { Singleton.__init__(Apipie::Generator::Swagger::WarningWriter) }
+
+    subject { type_definition }
+
+    context 'when validator is a Hash' do
+      let(:validator) { Hash }
+
+      it 'outputs a hash without internal typespec warning' do
+        expect { subject }.to output(/is a generic Hash without an internal type specification/).to_stderr
+      end
+    end
+  end
+end

data/spec/lib/generator/swagger/param_description_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe Apipie::Generator::Swagger::ParamDescription do
+  describe '.create_for_missing_param' do
+    let(:name) { 'ok' }
+
+    let(:method_description) do
+      Apipie.get_method_description(UsersController, :update)
+    end
+
+    subject do
+      described_class.create_for_missing_param(method_description, name)
+    end
+
+    it 'creates a required param description' do
+      expect(subject.required).to eq(true)
+    end
+
+    it 'has the correct name' do
+      expect(subject.name).to eq(name)
+    end
+
+    it 'has been created from path' do
+      expect(subject.options[:added_from_path]).to eq(true)
+    end
+  end
+end
+

data/spec/lib/generator/swagger/type_extractor_spec.rb

@@ -1,15 +1,11 @@
 require 'spec_helper'
 
 describe Apipie::Generator::Swagger::TypeExtractor do
-  let(:validator) {}
   let(:extractor) { described_class.new(validator) }
 
-  describe '#extarct_with_warnings' do
+  shared_examples 'extractable method' do
     let(:warnings) { {} }
-
-    before { Apipie.configuration.swagger_suppress_warnings = false }
-
-    subject { extractor.extract_with_warnings(warnings) }
+    let(:validator) {}
 
     it { is_expected.to eq(Apipie::Generator::Swagger::TypeExtractor::TYPES[:string]) }
 
@@ -38,21 +34,45 @@ describe Apipie::Generator::Swagger::TypeExtractor do
           it 'returns a boolean type' do
             expect(subject).to eq(Apipie::Generator::Swagger::TypeExtractor::TYPES[:boolean])
           end
+        end
+      end
+    end
+  end
+
+  describe '#extract' do
+    subject { extractor.extract }
 
-          context 'and a boolean warning is passed' do
-            let(:boolean_warning) do
-              Apipie::Generator::Swagger::Warning.for_code(
-                Apipie::Generator::Swagger::Warning::INFERRING_BOOLEAN_CODE,
-                'SampleController#action',
-                { parameter: 'some-param' }
-              )
-            end
+    it_behaves_like 'extractable method'
+  end
+
+  describe '#extarct_with_warnings' do
+    before { Apipie.configuration.swagger_suppress_warnings = false }
+
+    subject { extractor.extract_with_warnings(warnings) }
+
+    it_behaves_like 'extractable method'
+
+    context "when enum validator is used" do
+      context "and has `true`, `false` as values" do
+        let(:enum_values) { [true, false] }
+
+        let(:validator) do
+          Apipie::ResponseDescriptionAdapter::PropDesc::Validator.new('some-type', enum_values)
+        end
+
+        context 'and a boolean warning is passed' do
+          let(:boolean_warning) do
+            Apipie::Generator::Swagger::Warning.for_code(
+              Apipie::Generator::Swagger::Warning::INFERRING_BOOLEAN_CODE,
+              'SampleController#action',
+              { parameter: 'some-param' }
+            )
+          end
 
-            let(:warnings) { { boolean: boolean_warning } }
+          let(:warnings) { { boolean: boolean_warning } }
 
-            it 'outputs the warning' do
-              expect { subject }.to output(boolean_warning.warning_message).to_stderr
-            end
+          it 'outputs the warning' do
+            expect { subject }.to output(boolean_warning.warning_message).to_stderr
           end
         end
       end

data/spec/lib/param_group_spec.rb

@@ -43,17 +43,17 @@ describe "param groups" do
   end
 
   it "should replace parameter name in a group when it comes from concern" do
-    expect(Apipie["overridden_concern_resources#update"].params.has_key?(:user)).to eq(true)
+    expect(Apipie["overridden_concern_resources#update"].params.key?(:user)).to eq(true)
   end
 
   it "shouldn't replace parameter name in a group redefined in the controller" do
-    expect(Apipie["overridden_concern_resources#create"].params.has_key?(:concern)).to eq(true)
-    expect(Apipie["overridden_concern_resources#create"].params.has_key?(:user)).to eq(false)
+    expect(Apipie["overridden_concern_resources#create"].params.key?(:concern)).to eq(true)
+    expect(Apipie["overridden_concern_resources#create"].params.key?(:user)).to eq(false)
   end
 
 it "shouldn't replace name of a parameter defined in the controller" do
-    expect(Apipie["overridden_concern_resources#custom"].params.has_key?(:concern)).to eq(true)
-    expect(Apipie["overridden_concern_resources#custom"].params.has_key?(:user)).to eq(false)
+    expect(Apipie["overridden_concern_resources#custom"].params.key?(:concern)).to eq(true)
+    expect(Apipie["overridden_concern_resources#custom"].params.key?(:user)).to eq(false)
   end
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: apipie-rails
 version: !ruby/object:Gem::Version
-  version: 0.9.1
+  version: 0.9.2
 platform: ruby
 authors:
 - Pavel Pokorny
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-01-16 00:00:00.000000000 Z
+date: 2023-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -235,6 +235,15 @@ files:
 - lib/apipie/extractor/recorder.rb
 - lib/apipie/extractor/writer.rb
 - lib/apipie/generator/generator.rb
+- lib/apipie/generator/swagger/context.rb
+- lib/apipie/generator/swagger/operation_id.rb
+- lib/apipie/generator/swagger/param_description.rb
+- lib/apipie/generator/swagger/param_description/builder.rb
+- lib/apipie/generator/swagger/param_description/composite.rb
+- lib/apipie/generator/swagger/param_description/description.rb
+- lib/apipie/generator/swagger/param_description/in.rb
+- lib/apipie/generator/swagger/param_description/name.rb
+- lib/apipie/generator/swagger/param_description/type.rb
 - lib/apipie/generator/swagger/swagger.rb
 - lib/apipie/generator/swagger/type.rb
 - lib/apipie/generator/swagger/type_extractor.rb
@@ -340,6 +349,15 @@ files:
 - spec/lib/extractor/middleware_spec.rb
 - spec/lib/extractor/writer_spec.rb
 - spec/lib/file_handler_spec.rb
+- spec/lib/generator/swagger/context_spec.rb
+- spec/lib/generator/swagger/operation_id_spec.rb
+- spec/lib/generator/swagger/param_description/builder_spec.rb
+- spec/lib/generator/swagger/param_description/composite_spec.rb
+- spec/lib/generator/swagger/param_description/description_spec.rb
+- spec/lib/generator/swagger/param_description/in_spec.rb
+- spec/lib/generator/swagger/param_description/name_spec.rb
+- spec/lib/generator/swagger/param_description/type_spec.rb
+- spec/lib/generator/swagger/param_description_spec.rb
 - spec/lib/generator/swagger/type_extractor_spec.rb
 - spec/lib/generator/swagger/warning_spec.rb
 - spec/lib/generator/swagger/warning_writer_spec.rb