praxis 2.0.pre.38 → 2.0.pre.40

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b2f139dbbe5f48ceddf013eb18a84dc43fcdf41af033b19a40f565f8606b961f
-  data.tar.gz: 69b0200fd2ead2d713af52b7a0c56ca60a19bc4cb118ff0284017e7f1d19cda2
+  metadata.gz: 19745c4d9f6ebe88d95a2118df3b7a41343a2076b2a17d051793fed535575b9d
+  data.tar.gz: af9eb1d60cf4367f0ceb0e7aadc589b7f7d862fa161815ccc05c46eaa08f8881
 SHA512:
-  metadata.gz: cdcaed25ce9a8735f76a77e8f805432579eb4afa8893d0abf39b8054cd09053dc9a0103c1ea64d2330baa4280dc9d8b9a522022afce009c90814f367e3295920
-  data.tar.gz: 6a146584703308be70f70a54e3ebb53b10ff038578a5851e333cad95392729576e133c5079a85052487f3af023c831888e7cc2a8dd075b93b7aa8c7c30b5dc24
+  metadata.gz: 4a0a13bfeb6d1fd4512483fd4d341e1f606f1b391d0eda220019b5032f2593be4e79bd7061d375de2b97630eecd19feffa89db594d7c553c11163084b6bb00e1
+  data.tar.gz: e0ef5e1cc96d638496ef60f0ae4aaf82ccc2cecbf49cae5794642525283e201bfb600b4f02f4a85d1cd7b16988e6dc05a5a08162c7d9f894906017beba271de3

data/CHANGELOG.md

@@ -1,8 +1,16 @@
 # Praxis Changelog
 
+## 2.0.pre40
+
+- Prevent IRB from running console in a new Thread (#405)
+
+## 2.0.pre.39
+
+- Revert JSON handler back to `json` gem, as `oj` was too unstable when used with ActiveSupport.
+
 ## 2.0.pre.38
 
-- Stoped calling ::Oj.mimic_JSON in Praxis::Handlers::JSON. It breaks ActiveSupport::JSON's html escaping when called.
+- Stopped calling ::Oj.mimic_JSON in Praxis::Handlers::JSON. It breaks ActiveSupport::JSON's html escaping when called.
 
 ## 2.0.pre.37
 

data/gemfiles/.gitignore

@@ -0,0 +1,3 @@
+/.bundle
+/vendor
+w

data/gemfiles/active_6.gemfile

@@ -11,7 +11,6 @@ gem 'sequel', '~> 5'
 group :test do
   gem 'builder'
   gem 'link_header'
-  gem 'oj'
   gem 'parslet'
 end
 

data/gemfiles/active_7.gemfile

@@ -4,14 +4,16 @@
 
 source 'https://rubygems.org'
 
-gem 'activerecord', '>=7'
+# TODO: figure out incompatibility between ActiveRecord 7.1 and SQLite
+# @see https://github.com/praxis/praxis/pull/405#issuecomment-1780570599
+# Until then, we're pinned to 7.0.x for purposes of testing.
+gem 'activerecord', '~> 7.0.8'
 gem 'rubocop'
 gem 'sequel', '~> 5'
 
 group :test do
   gem 'builder'
   gem 'link_header'
-  gem 'oj'
   gem 'parslet'
 end
 

data/lib/praxis/handlers/json.rb

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 module Praxis
   module Handlers
     class JSON
@@ -7,11 +5,11 @@ module Praxis
       #
       # @raise [Praxis::Exceptions::InvalidConfiguration] if the handler is unsupported
       def initialize
-        require 'oj'
+        require 'json'
       rescue LoadError
         # Should never happen since JSON is a default gem; might as well be cautious!
         raise Praxis::Exceptions::InvalidConfiguration,
-              'JSON handler depends on oj ~> 3; please add it to your Gemfile'
+              "JSON handler depends on json ~> 1.0; please add it to your Gemfile"
       end
 
       # Parse a JSON document into structured data.
@@ -20,9 +18,8 @@ module Praxis
       # @return [Hash,Array] the structured-data representation of the document
       def parse(document)
         # Try to be nice and accept an empty string as an empty payload (seems nice to do for dumb http clients)
-        return nil if document.nil? || document == ''
-
-        ::Oj.load(document)
+        return nil if (document.nil? || document == '')
+        ::JSON.parse(document)
       end
 
       # Generate a pretty-printed JSON document from structured data.
@@ -30,7 +27,7 @@ module Praxis
       # @param [Hash,Array] structured_data
       # @return [String]
       def generate(structured_data)
-        ::Oj.dump(structured_data, indent: 2)
+        ::JSON.pretty_generate(structured_data)
       end
     end
   end

data/lib/praxis/tasks/console.rb

@@ -5,38 +5,35 @@ namespace :praxis do
   task :console do
     # Use irb if available (which it almost always is).
     require 'irb'
+    # Ensure that we save history just like normal IRB
+    require 'irb/ext/save-history'
 
     Rake::Task['praxis:environment'].invoke
 
+    basedir = ::Praxis::Application.instance.root
+    nickname = File.basename(basedir)
+
     # Keep IRB.setup from complaining about bad ARGV options
     old_argv = ARGV.dup
     ARGV.clear
-    IRB.setup nil
+    IRB.setup(basedir)
     ARGV.concat(old_argv)
 
-    # Ensure that multi-irb has a context to work with (and, indirectly an instance of IRB::Irb).
-    IRB.conf[:MAIN_CONTEXT] = IRB::Irb.new.context
-
-    # Allow reentrant IRB
-    require 'irb/ext/multi-irb'
-
-    # Ensure that we save history just like normal IRB
-    require 'irb/ext/save-history'
-
     # Remove main object from prompt (its stringify is not useful)
-    nickname = File.basename(::Praxis::Application.instance.root)
     IRB.conf[:PROMPT][:DEFAULT] = {
       PROMPT_I: "%N(#{nickname}):%03n:%i> ",
       PROMPT_N: "%N(#{nickname}):%03n:%i> ",
       PROMPT_S: "%N(#{nickname}):%03n:%i%l ",
       PROMPT_C: "%N(#{nickname}):%03n:%i* ",
     }
-
     # Disable inefficient, distracting autocomplete
     IRB.conf[:USE_AUTOCOMPLETE] = false
 
-    # Invoke the REPL, then cleanly shut down
-    IRB.irb(nil, Praxis::Application.instance)
+    # Invoke the REPL, setting the workspace binding to the application object.
+    IRB::Irb.new(IRB::WorkSpace.new(::Praxis::Application.instance)).run(
+      IRB.conf,
+    )
+    # Cleanly shut down to ensure we save history
     IRB.irb_at_exit
   end
 end

data/lib/praxis/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Praxis
-  VERSION = '2.0.pre.38'
+  VERSION = '2.0.pre.40'
 end

data/spec/functional_cloud_spec.rb

@@ -12,30 +12,39 @@ describe 'Functional specs' do
   context 'index' do
     context 'with a valid request' do
       it 'is successful' do
-        get '/api/clouds/1/instances?api_version=1.0', nil, 'global_session' => session
+        get '/api/clouds/1/instances?api_version=1.0',
+            nil,
+            'global_session' => session
         expect(last_response.headers['Content-Type']).to(
-          eq('application/vnd.acme.instance;type=collection')
+          eq('application/vnd.acme.instance;type=collection'),
         )
       end
     end
 
     context 'with a path param that can not load' do
       it 'returns a useful error' do
-        get '/api/clouds/invalid/instances?api_version=1.0', nil, 'global_session' => session
+        get '/api/clouds/invalid/instances?api_version=1.0',
+            nil,
+            'global_session' => session
 
         expect(last_response.status).to eq 400
 
         response = JSON.parse(last_response.body)
         expect(response['name']).to eq 'ValidationError'
         expect(response['summary']).to eq 'Error loading params.'
-        expect(response['errors']).to match_array([/Error loading attribute \$\.params\.cloud_id/])
+        expect(response['errors']).to match_array(
+          [/Error loading attribute \$\.params\.cloud_id/],
+        )
         expect(response['cause']['name']).to eq 'ArgumentError'
       end
     end
 
     context 'with a header that can not load' do
       it 'returns a useful error' do
-        get '/api/clouds/1/instances?api_version=1.0', nil, 'global_session' => session, 'HTTP_ACCOUNT_ID' => 'invalid'
+        get '/api/clouds/1/instances?api_version=1.0',
+            nil,
+            'global_session' => session,
+            'HTTP_ACCOUNT_ID' => 'invalid'
 
         expect(last_response.status).to eq 400
 
@@ -43,14 +52,18 @@ describe 'Functional specs' do
 
         expect(response['name']).to eq 'ValidationError'
         expect(response['summary']).to eq 'Error loading headers.'
-        expect(response['errors']).to match_array([/Error loading attribute .*Account-Id"/])
+        expect(response['errors']).to match_array(
+          [/Error loading attribute .*Account-Id"/],
+        )
         expect(response['cause']['name']).to eq 'ArgumentError'
       end
     end
 
     context 'with a param that is invalid' do
       it 'returns a useful error' do
-        get '/api/clouds/-1/instances?api_version=1.0', nil, 'global_session' => session
+        get '/api/clouds/-1/instances?api_version=1.0',
+            nil,
+            'global_session' => session
 
         expect(last_response.status).to eq 400
 
@@ -58,13 +71,18 @@ describe 'Functional specs' do
 
         expect(response['name']).to eq 'ValidationError'
         expect(response['summary']).to eq 'Error validating request data.'
-        expect(response['errors']).to match_array([/.*cloud_id.*is smaller than the allowed min/])
+        expect(response['errors']).to match_array(
+          [/.*cloud_id.*is smaller than the allowed min/],
+        )
       end
     end
 
     context 'with a header that is invalid' do
       it 'returns a useful error' do
-        get '/api/clouds/1/instances?api_version=1.0', nil, 'global_session' => session, 'HTTP_ACCOUNT_ID' => '-1'
+        get '/api/clouds/1/instances?api_version=1.0',
+            nil,
+            'global_session' => session,
+            'HTTP_ACCOUNT_ID' => '-1'
 
         expect(last_response.status).to eq 400
 
@@ -72,7 +90,9 @@ describe 'Functional specs' do
 
         expect(response['name']).to eq 'ValidationError'
         expect(response['summary']).to eq 'Error validating request data.'
-        expect(response['errors']).to match_array([/.*headers.*Account-Id.*is smaller than the allowed min/])
+        expect(response['errors']).to match_array(
+          [/.*headers.*Account-Id.*is smaller than the allowed min/],
+        )
       end
     end
 
@@ -87,7 +107,10 @@ describe 'Functional specs' do
       end
 
       it 'fails to validate the response' do
-        get '/api/clouds/1/instances?response_content_type=somejunk&api_version=1.0', nil, 'HTTP_FOO' => 'bar', 'global_session' => session
+        get '/api/clouds/1/instances?response_content_type=somejunk&api_version=1.0',
+            nil,
+            'HTTP_FOO' => 'bar',
+            'global_session' => session
         expect(last_response.status).to eq(500)
         response = JSON.parse(last_response.body)
 
@@ -97,16 +120,22 @@ describe 'Functional specs' do
       end
 
       context 'with response validation disabled' do
-        let(:praxis_config) { double('praxis_config', validate_responses: false) }
+        let(:praxis_config) do
+          double('praxis_config', validate_responses: false)
+        end
         let(:config) { double('config', praxis: praxis_config) }
 
         before do
-          expect(Praxis::Application.instance.config).to receive(:praxis).and_return(praxis_config)
+          expect(Praxis::Application.instance.config).to receive(
+            :praxis,
+          ).and_return(praxis_config)
         end
 
         it 'does not validate the response and succeeds' do
           expect do
-            get '/api/clouds/1/instances?response_content_type=somejunk&api_version=1.0', nil, 'global_session' => session
+            get '/api/clouds/1/instances?response_content_type=somejunk&api_version=1.0',
+                nil,
+                'global_session' => session
           end.to_not raise_error
         end
       end
@@ -114,10 +143,14 @@ describe 'Functional specs' do
     context 'with a valid request but misusing request content-type' do
       it 'is still successful and does not get confused about the sister post action' do
         the_body = StringIO.new('') # This is a GET request passing a body
-        get '/api/clouds/1/instances?api_version=1.0', nil, 'rack.input' => the_body, 'CONTENT_TYPE' => 'application/json', 'global_session' => session
+        get '/api/clouds/1/instances?api_version=1.0',
+            nil,
+            'rack.input' => the_body,
+            'CONTENT_TYPE' => 'application/json',
+            'global_session' => session
         expect(last_response.status).to eq(200)
         expect(last_response.headers['Content-Type']).to(
-          eq('application/vnd.acme.instance;type=collection')
+          eq('application/vnd.acme.instance;type=collection'),
         )
       end
     end
@@ -126,30 +159,40 @@ describe 'Functional specs' do
   context 'index using POST sister action' do
     context 'with a valid request' do
       it 'is successful and round trips the content type we pass in the body' do
-        payload = { response_content_type: 'application/vnd.acme.instance; type=collection; other=thing' }
-        post '/api/clouds/1/instances/actions/index_using_post?api_version=1.0', JSON.dump(payload), 'CONTENT_TYPE' => 'application/json', 'global_session' => session
+        payload = {
+          response_content_type:
+            'application/vnd.acme.instance; type=collection; other=thing',
+        }
+        post '/api/clouds/1/instances/actions/index_using_post?api_version=1.0',
+             JSON.dump(payload),
+             'CONTENT_TYPE' => 'application/json',
+             'global_session' => session
         expect(last_response.status).to eq(200)
         expect(last_response.headers['Content-Type']).to(
-          eq(payload[:response_content_type])
+          eq(payload[:response_content_type]),
         )
       end
     end
   end
   it 'works' do
     the_body = StringIO.new('{}') # This is a funny, GET request expecting a body
-    get '/api/clouds/1/instances/2?junk=foo&api_version=1.0', nil, 'rack.input' => the_body, 'CONTENT_TYPE' => 'application/json', 'global_session' => session
+    get '/api/clouds/1/instances/2?junk=foo&api_version=1.0',
+        nil,
+        'rack.input' => the_body,
+        'CONTENT_TYPE' => 'application/json',
+        'global_session' => session
     expect(last_response.status).to eq(200)
     expected = {
       'cloud_id' => 1,
       'id' => 2,
       'junk' => 'foo',
       'other_params' => {
-        'some_date' => '2012-12-21T00:00:00.000+00:00',
-        'fail_filter' => false
+        'some_date' => '2012-12-21T00:00:00+00:00',
+        'fail_filter' => false,
       },
       'payload' => {
-        'optional' => 'not given'
-      }
+        'optional' => 'not given',
+      },
     }
 
     expect(JSON.parse(last_response.body)).to eq(expected)
@@ -160,13 +203,17 @@ describe 'Functional specs' do
   end
 
   it 'returns early when making the before filter break' do
-    get '/api/clouds/1/instances/2?junk=foo&api_version=1.0&fail_filter=true', nil, 'global_session' => session
+    get '/api/clouds/1/instances/2?junk=foo&api_version=1.0&fail_filter=true',
+        nil,
+        'global_session' => session
     expect(last_response.status).to eq(401)
   end
 
   context 'bulk_create multipart' do
     let(:instance) { Instance.example }
-    let(:instance_json) { JSON.pretty_generate(instance.render(fields: { id: true, name: true })) }
+    let(:instance_json) do
+      JSON.pretty_generate(instance.render(fields: { id: true, name: true }))
+    end
 
     let(:form) do
       form_data = MIME::Multipart::FormData.new
@@ -179,9 +226,13 @@ describe 'Functional specs' do
     let(:body) { form.body.to_s }
 
     it 'works' do
-      post '/api/clouds/1/instances?api_version=1.0', body, 'CONTENT_TYPE' => content_type, 'global_session' => session
+      post '/api/clouds/1/instances?api_version=1.0',
+           body,
+           'CONTENT_TYPE' => content_type,
+           'global_session' => session
 
-      _reponse_preamble, response = Praxis::MultipartParser.parse(last_response.headers, last_response.body)
+      _reponse_preamble, response =
+        Praxis::MultipartParser.parse(last_response.headers, last_response.body)
       expect(response).to have(1).item
 
       instance_part = response.first
@@ -194,7 +245,9 @@ describe 'Functional specs' do
 
       response_instance = JSON.parse(instance_part.body)
       expect(response_instance['key']).to eq(instance.id)
-      expect(response_instance['value'].values).to eq(instance.render(fields: { id: true, name: true }).values)
+      expect(response_instance['value'].values).to eq(
+        instance.render(fields: { id: true, name: true }).values,
+      )
     end
   end
 
@@ -216,7 +269,10 @@ describe 'Functional specs' do
 
     context 'with a valid payload' do
       before do
-        post '/api/clouds/1/instances/2/files?api_version=1.0', body, 'CONTENT_TYPE' => content_type, 'global_session' => session
+        post '/api/clouds/1/instances/2/files?api_version=1.0',
+             body,
+             'CONTENT_TYPE' => content_type,
+             'global_session' => session
       end
 
       subject(:response) { JSON.parse(last_response.body) }
@@ -244,11 +300,16 @@ describe 'Functional specs' do
       let(:body) { form.body.to_s }
 
       it 'returns an error' do
-        post '/api/clouds/1/instances/2/files?api_version=1.0', body, 'CONTENT_TYPE' => content_type, 'global_session' => session
+        post '/api/clouds/1/instances/2/files?api_version=1.0',
+             body,
+             'CONTENT_TYPE' => content_type,
+             'global_session' => session
         response = JSON.parse(last_response.body)
 
         expect(response['name']).to eq('ValidationError')
-        expect(response['errors']).to eq(['Attribute $.payload.destination_path is required'])
+        expect(response['errors']).to eq(
+          ['Attribute $.payload.destination_path is required'],
+        )
       end
     end
 
@@ -273,9 +334,14 @@ describe 'Functional specs' do
       subject(:response) { JSON.parse(last_response.body) }
 
       before do
-        post '/api/clouds/1/instances/2/files?api_version=1.0', body, 'CONTENT_TYPE' => content_type, 'global_session' => session
+        post '/api/clouds/1/instances/2/files?api_version=1.0',
+             body,
+             'CONTENT_TYPE' => content_type,
+             'global_session' => session
+      end
+      its(:keys) do
+        should eq(%w[destination_path name filename type contents options])
       end
-      its(:keys) { should eq(%w[destination_path name filename type contents options]) }
       its(['options']) { should eq({ 'extra_thing' => 'I am extra' }) }
     end
   end
@@ -283,11 +349,15 @@ describe 'Functional specs' do
   context 'not found and API versions' do
     context 'when no version is specified' do
       it 'it tells you which available api versions would match' do
-        get '/api/clouds/1/instances/2?junk=foo', nil, 'global_session' => session
+        get '/api/clouds/1/instances/2?junk=foo',
+            nil,
+            'global_session' => session
 
         expect(last_response.status).to eq(404)
         expect(last_response.headers['Content-Type']).to eq('text/plain')
-        expect(last_response.body).to eq('NotFound. Your request did not specify an API version. Available versions = "1.0".')
+        expect(last_response.body).to eq(
+          'NotFound. Your request did not specify an API version. Available versions = "1.0".',
+        )
       end
       it 'it just gives you a simple not found when nothing would have matched' do
         get '/foobar?junk=foo', nil, 'global_session' => session
@@ -300,43 +370,58 @@ describe 'Functional specs' do
 
     context 'when some version is specified, but wrong' do
       it 'it tells you which possible correcte api versions exist' do
-        get '/api/clouds/1/instances/2?junk=foo&api_version=50.0', nil, 'global_session' => session
+        get '/api/clouds/1/instances/2?junk=foo&api_version=50.0',
+            nil,
+            'global_session' => session
 
         expect(last_response.status).to eq(404)
         expect(last_response.headers['Content-Type']).to eq('text/plain')
-        expect(last_response.body).to eq('NotFound. Your request specified API version = "50.0". Available versions = "1.0".')
+        expect(last_response.body).to eq(
+          'NotFound. Your request specified API version = "50.0". Available versions = "1.0".',
+        )
       end
     end
   end
 
   context 'volumes' do
-    before do
-      header 'X-Api-Version', '1.0'
-    end
+    before { header 'X-Api-Version', '1.0' }
 
     context 'when no authorization header is passed' do
       it 'works as expected' do
-        get '/api/clouds/1/volumes/123?junk=stuff', nil, 'global_session' => session
+        get '/api/clouds/1/volumes/123?junk=stuff',
+            nil,
+            'global_session' => session
         expect(last_response.status).to eq(200)
         expect(Volume.load(last_response.body).validate).to be_empty
-        expect(last_response.headers['Content-Type']).to eq('application/vnd.acme.volume')
+        expect(last_response.headers['Content-Type']).to eq(
+          'application/vnd.acme.volume',
+        )
       end
     end
     context 'when an authorization header is passed' do
       it 'returns 401 when it does not match "secret" ' do
-        get '/api/clouds/1/volumes/123?junk=stuff', nil, 'HTTP_AUTHORIZATION' => 'foobar', 'global_session' => session
+        get '/api/clouds/1/volumes/123?junk=stuff',
+            nil,
+            'HTTP_AUTHORIZATION' => 'foobar',
+            'global_session' => session
         expect(last_response.status).to eq(401)
         expect(last_response.body).to match(/Authentication info is invalid/)
       end
       it 'succeeds as expected when it matches "secret" ' do
-        get '/api/clouds/1/volumes/123?junk=stuff', nil, 'HTTP_AUTHORIZATION' => 'the secret', 'global_session' => session
+        get '/api/clouds/1/volumes/123?junk=stuff',
+            nil,
+            'HTTP_AUTHORIZATION' => 'the secret',
+            'global_session' => session
         expect(last_response.status).to eq(200)
       end
     end
 
     context 'index action with no args defined' do
       it 'dispatches successfully' do
-        get '/api/clouds/1/volumes', nil, 'HTTP_AUTHORIZATION' => 'the secret', 'global_session' => session
+        get '/api/clouds/1/volumes',
+            nil,
+            'HTTP_AUTHORIZATION' => 'the secret',
+            'global_session' => session
         expect(last_response.status).to eq(200)
       end
     end
@@ -345,22 +430,32 @@ describe 'Functional specs' do
   context 'wildcard verb routing' do
     let(:content_type) { 'application/json' }
     it 'can terminate instances with POST' do
-      post '/api/clouds/23/instances/1/terminate?api_version=1.0', nil, 'CONTENT_TYPE' => content_type, 'global_session' => session
+      post '/api/clouds/23/instances/1/terminate?api_version=1.0',
+           nil,
+           'CONTENT_TYPE' => content_type,
+           'global_session' => session
       expect(last_response.status).to eq(200)
     end
     it 'can terminate instances with DELETE' do
-      post '/api/clouds/23/instances/1/terminate?api_version=1.0', nil, 'CONTENT_TYPE' => content_type, 'global_session' => session
+      post '/api/clouds/23/instances/1/terminate?api_version=1.0',
+           nil,
+           'CONTENT_TYPE' => content_type,
+           'global_session' => session
       expect(last_response.status).to eq(200)
     end
   end
 
   context 'route options' do
     it 'reach the endpoint that does not match the except clause' do
-      get '/api/clouds/23/otherinstances/_action/test?api_version=1.0', nil, 'global_session' => session
+      get '/api/clouds/23/otherinstances/_action/test?api_version=1.0',
+          nil,
+          'global_session' => session
       expect(last_response.status).to eq(200)
     end
     it 'does NOT reach the endpoint that matches the except clause' do
-      get '/api/clouds/23/otherinstances/_action/exceptional?api_version=1.0', nil, 'global_session' => session
+      get '/api/clouds/23/otherinstances/_action/exceptional?api_version=1.0',
+          nil,
+          'global_session' => session
       expect(last_response.status).to eq(404)
     end
   end
@@ -369,12 +464,17 @@ describe 'Functional specs' do
     let(:content_type) { 'application/json' }
 
     it 'can terminate' do
-      post '/api/clouds/23/instances/1/terminate?api_version=1.0', nil, 'global_session' => session, 'CONTENT_TYPE' => content_type
+      post '/api/clouds/23/instances/1/terminate?api_version=1.0',
+           nil,
+           'global_session' => session,
+           'CONTENT_TYPE' => content_type
       expect(last_response.status).to eq(200)
     end
 
     it 'can not stop' do
-      post '/api/clouds/23/instances/1/stop?api_version=1.0', '', 'global_session' => session
+      post '/api/clouds/23/instances/1/stop?api_version=1.0',
+           '',
+           'global_session' => session
       expect(last_response.status).to eq(403)
     end
   end
@@ -384,13 +484,18 @@ describe 'Functional specs' do
     let(:content_type) { 'application/json' }
 
     before do
-      post '/api/clouds/1/instances/2/terminate?api_version=1.0', body, 'CONTENT_TYPE' => content_type, 'global_session' => session
+      post '/api/clouds/1/instances/2/terminate?api_version=1.0',
+           body,
+           'CONTENT_TYPE' => content_type,
+           'global_session' => session
     end
 
     it 'returns a useful error message' do
       body = JSON.parse(last_response.body)
       expect(body['name']).to eq('ValidationError')
-      expect(body['summary']).to match("Error loading payload. Used Content-Type: 'application/json'")
+      expect(body['summary']).to match(
+        "Error loading payload. Used Content-Type: 'application/json'",
+      )
       expect(body['errors']).to_not be_empty
     end
   end
@@ -400,7 +505,10 @@ describe 'Functional specs' do
     let(:content_type) { 'application/json' }
 
     before do
-      patch '/api/clouds/1/instances/3?api_version=1.0', body, 'CONTENT_TYPE' => content_type, 'global_session' => session
+      patch '/api/clouds/1/instances/3?api_version=1.0',
+            body,
+            'CONTENT_TYPE' => content_type,
+            'global_session' => session
     end
 
     subject(:response_body) { JSON.parse(last_response.body) }
@@ -429,7 +537,11 @@ describe 'Functional specs' do
 
       its(['name']) { should eq 'ValidationError' }
       its(['summary']) { should eq 'Error validating response' }
-      its(['errors']) { should match_array [/\$\.name value \(Invalid Name\) does not match regexp/] }
+      its(['errors']) do
+        should match_array [
+                             /\$\.name value \(Invalid Name\) does not match regexp/,
+                           ]
+      end
 
       it 'returns a validation error' do
         expect(last_response.status).to eq(500)

data/spec/spec_helper.rb

@@ -25,17 +25,13 @@ require 'rack/test'
 require 'rspec/its'
 require 'rspec/collection_matchers'
 
-require 'oj'
 require 'json'
-Oj.mimic_JSON
 
 Dir["#{File.dirname(__FILE__)}/../lib/praxis/plugins/*.rb"].sort.each do |file|
   require file
 end
 
-Dir["#{File.dirname(__FILE__)}/support/*.rb"].sort.each do |file|
-  require file
-end
+Dir["#{File.dirname(__FILE__)}/support/*.rb"].sort.each { |file| require file }
 
 def suppress_output
   original_stdout = $stdout.clone
@@ -57,9 +53,7 @@ RSpec.configure do |config|
   end
 
   config.before(:each) do
-    Praxis::Blueprint.cache = Hash.new do |hash, key|
-      hash[key] = {}
-    end
+    Praxis::Blueprint.cache = Hash.new { |hash, key| hash[key] = {} }
   end
 
   config.before(:all) do

data/tasks/thor/templates/generator/example_app/Gemfile

@@ -4,7 +4,6 @@ source 'https://rubygems.org'
 
 gem 'activerecord'
 gem 'link_header' # For pagination extensions
-gem 'oj' # For fast JSON de/serialization handlers
 gem 'parslet' # For field selection extension
 gem 'praxis'
 gem 'puma' # A much better web server than the default webrick
@@ -20,4 +19,4 @@ group :development, :test do
 
   gem 'pry'
   gem 'pry-byebug'
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: praxis
 version: !ruby/object:Gem::Version
-  version: 2.0.pre.38
+  version: 2.0.pre.40
 platform: ruby
 authors:
 - Josep M. Blanquer
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-05 00:00:00.000000000 Z
+date: 2023-11-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -387,6 +387,7 @@ files:
 - Rakefile
 - SELECTOR_NOTES.txt
 - bin/praxis
+- gemfiles/.gitignore
 - gemfiles/active_6.gemfile
 - gemfiles/active_7.gemfile
 - lib/praxis.rb