artemis 0.7.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b64a8fd49d4d32ab4fd0dbda0f20005b68129fc1eee01de0f525424a19d1d28
-  data.tar.gz: eae9f56e2f7bbd832278cf1546c931824ea5f7bfa969e28cdf9dbe2e8a6e4ab6
+  metadata.gz: fcf01b2987aaf247cee4afa02982fd582208ea0b48f3042ed207e7bb5ed6fdd8
+  data.tar.gz: 4eea51e39bbe709d579d08a409d09c2a95495590ab99609834d0af735cc0c46a
 SHA512:
-  metadata.gz: 061e522011314b8f3a712a1a13c504653e7b7979e17ff239298d98742f08ae3fea14cd9772970dd72dd1c9e361572d6c70b076f3b0e4ee552f9d1fc1e2a7e69d
-  data.tar.gz: 2d1fec853399e9ec52b74e3c5c7eae407a9affe39222f7719d993f098ef839cd2da72dfbb353f967c025c2d72b464dbdc3f56dadfa4e596b2d9ea32c43d6119f
+  metadata.gz: a0f862a4301a613dfe0fadefe9bdbad4170f2b6af1ef6f5db859fb13d40dc0065a1d93d37cab4f44790c90c1b624bf62d4b7777ebaa67e38028be0816d14d19d
+  data.tar.gz: 0b197e80bf6ffe84c489423804d8b67fa18f37c57679015308598284090d5ab9adcdea587a27c6e49312c741642e5fe03675a594e6b66695af08dc344fb76db3

data/.github/workflows/ruby.yml

@@ -5,24 +5,34 @@ on:
   - pull_request
 
 jobs:
-  build:
+  mri:
     strategy:
       matrix:
         ruby_version:
+          - '3.2'
           - '3.1'
           - '3.0'
           - '2.7'
           - '2.6'
-          # - 'jruby-9.2.17.0'
         gemfile:
+          - gemfiles/rails_71.gemfile
           - gemfiles/rails_70.gemfile
           - gemfiles/rails_61.gemfile
           - gemfiles/rails_60.gemfile
           - gemfiles/rails_52.gemfile
           - gemfiles/rails_51.gemfile
           - gemfiles/rails_50.gemfile
-          # - gemfiles/rails_edge.gemfile
         exclude:
+          - ruby_version: '3.2'
+            gemfile: gemfiles/rails_61.gemfile
+          - ruby_version: '3.2'
+            gemfile: gemfiles/rails_60.gemfile
+          - ruby_version: '3.2'
+            gemfile: gemfiles/rails_52.gemfile
+          - ruby_version: '3.2'
+            gemfile: gemfiles/rails_51.gemfile
+          - ruby_version: '3.2'
+            gemfile: gemfiles/rails_50.gemfile
           - ruby_version: '3.1'
             gemfile: gemfiles/rails_61.gemfile
           - ruby_version: '3.1'
@@ -39,21 +49,23 @@ jobs:
             gemfile: gemfiles/rails_51.gemfile
           - ruby_version: '3.0'
             gemfile: gemfiles/rails_50.gemfile
+          - ruby_version: '2.7'
+            gemfile: gemfiles/rails_71.gemfile
           - ruby_version: '2.7'
             gemfile: gemfiles/rails_52.gemfile
           - ruby_version: '2.7'
             gemfile: gemfiles/rails_51.gemfile
           - ruby_version: '2.7'
             gemfile: gemfiles/rails_50.gemfile
+          - ruby_version: '2.6'
+            gemfile: gemfiles/rails_71.gemfile
           - ruby_version: '2.6'
             gemfile: gemfiles/rails_70.gemfile
-          # - ruby_version: '2.6'
-          #   gemfile: gemfiles/rails_edge.gemfile
-    runs-on: ubuntu-18.04
+    runs-on: ubuntu-22.04
     env:
       BUNDLE_GEMFILE: ${{ matrix.gemfile }}
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
       - name: Install curl
         run: sudo apt-get install curl libcurl4-openssl-dev
       - name: Set up Ruby
@@ -62,3 +74,67 @@ jobs:
           ruby-version: ${{ matrix.ruby_version }}
           bundler-cache: true
       - run: bundle exec rake
+
+  rails_edge:
+    needs:
+      - mri
+    runs-on: ubuntu-22.04
+    env:
+      BUNDLE_GEMFILE: gemfiles/rails_edge.gemfile
+    steps:
+      - uses: actions/checkout@v3
+      - name: Install curl
+        run: sudo apt-get install curl libcurl4-openssl-dev
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 3.2
+          bundler-cache: true
+      - run: bundle exec rake || echo "Rails edge test is done."
+
+  ruby_edge:
+    needs:
+      - mri
+    strategy:
+      matrix:
+        ruby_version:
+          - 'ruby-head'
+        gemfile:
+          - gemfiles/rails_edge.gemfile
+          - gemfiles/rails_71.gemfile
+    runs-on: ubuntu-22.04
+    env:
+      BUNDLE_GEMFILE: ${{ matrix.gemfile }}
+    steps:
+      - uses: actions/checkout@v3
+      - name: Install curl
+        run: sudo apt-get install curl libcurl4-openssl-dev
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby_version }}
+          bundler-cache: true
+      - run: bundle exec rake || echo "Ruby edge test is done."
+
+#  The curb gem does not work well with JRuby, so skipping for now...
+#  jruby:
+#    needs:
+#      - mri
+#    strategy:
+#      matrix:
+#        ruby_version:
+#          - 'jruby-9.4'
+#          - 'jruby-head'
+#        gemfile:
+#          - gemfiles/rails_70.gemfile
+#    runs-on: ubuntu-22.04
+#    env:
+#      BUNDLE_GEMFILE: ${{ matrix.gemfile }}
+#    steps:
+#      - uses: actions/checkout@v3
+#      - name: Set up Ruby
+#        uses: ruby/setup-ruby@v1
+#        with:
+#          ruby-version: ${{ matrix.ruby_version }}
+#          bundler-cache: true
+#      - run: bundle exec rake || echo "JRuby test is done."

data/Appraisals

@@ -4,6 +4,15 @@ appraise "rails_edge" do
     gem "railties"
     gem "activesupport"
   end
+
+  gem "rackup"
+end
+
+appraise "rails_71" do
+  gem "rails", '~> 7.1.0.beta1'
+  gem "railties", '~> 7.1.0.beta1'
+  gem "activesupport", '~> 7.1.0.beta1'
+  gem "rackup"
 end
 
 appraise "rails_70" do

data/CHANGELOG.md

@@ -1,7 +1,24 @@
+## v0.9.0
 
-## v0.7.0
+#### New Features
 
-_<sup>unreleased</sup>_
+- Rails 7.1.0.beta1 is now officially supported
+
+#### Fixes
+
+- Fixes an issue where `graphql` gem `2.1.0` may not work with `graphql-client` (`b144ee2f`)
+
+## [v0.8.0](https://github.com/yuki24/artemis/tree/v0.8.0)
+
+_<sup>released at 2023-01-05 05:29:37 UTC</sup>_
+
+#### New Features
+
+- Ruby 3.2 is now officially supported
+
+## [v0.7.0](https://github.com/yuki24/artemis/tree/v0.7.0)
+
+_<sup>released at 2022-03-05 08:24:45 UTC</sup>_
 
 #### Features
 

data/artemis.gemspec

@@ -17,7 +17,7 @@ Gem::Specification.new do |spec|
 
   spec.add_dependency "activesupport", ">= 4.0.0"
   spec.add_dependency "railties", ">= 4.0.0"
-  spec.add_dependency "graphql", ">= 1.8"
+  spec.add_dependency "graphql", "< 2.1"
   spec.add_dependency "graphql-client", ">= 0.13.0"
 
   spec.add_development_dependency "appraisal", ">= 2.2"

data/gemfiles/rails_71.gemfile

@@ -0,0 +1,14 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "pry"
+gem "pry-byebug", platforms: :mri
+gem "curb", ">= 0.9.6"
+gem "webrick"
+gem "rails", "~> 7.1.0.beta1"
+gem "railties", "~> 7.1.0.beta1"
+gem "activesupport", "~> 7.1.0.beta1"
+gem "rackup"
+
+gemspec path: "../"

data/gemfiles/rails_edge.gemfile

@@ -12,5 +12,6 @@ gem "pry"
 gem "pry-byebug", platforms: :mri
 gem "curb", ">= 0.9.6"
 gem "webrick"
+gem "rackup"
 
 gemspec path: "../"

data/lib/artemis/adapters/net_http_persistent_adapter.rb

@@ -8,6 +8,12 @@ rescue LoadError
   # no-op... Rails 7.0 requires this.
 end
 
+begin
+  require 'active_support/deprecation'
+  require 'active_support/deprecator'
+rescue LoadError
+end
+
 require 'active_support/core_ext/numeric/time'
 require 'net/http/persistent'
 

data/lib/artemis/railtie.rb

@@ -61,7 +61,7 @@ module Artemis
       if app.config.eager_load && app.config.cache_classes
         Artemis::GraphQLEndpoint.registered_services.each do |endpoint_name|
           begin
-            require endpoint_name # Rails 7.0 requires this.
+            require_dependency endpoint_name # Rails 7.0+ requires this.
           rescue LoadError
             # no-op...
           end

data/lib/artemis/version.rb

@@ -1,3 +1,3 @@
 module Artemis
-  VERSION = "0.7.0"
+  VERSION = "0.9.0"
 end

data/spec/adapters_spec.rb

@@ -2,6 +2,13 @@ require 'json'
 require 'rack'
 require 'webrick'
 
+RACK_SERVER = begin
+                require 'rackup/handler/webrick'
+                Rackup::Handler::WEBrick
+              rescue LoadError
+                Rack::Handler::WEBrick
+              end
+
 describe 'Adapters' do
   FakeServer = ->(env) {
     case env['PATH_INFO']
@@ -61,7 +68,7 @@ describe 'Adapters' do
     Artemis::Adapters::AbstractAdapter.send(:attr_writer, :uri, :timeout)
 
     @server_thread = Thread.new do
-      Rack::Handler::WEBrick.run(FakeServer, Port: 8000, Logger: WEBrick::Log.new('/dev/null'), AccessLog: [])
+      RACK_SERVER.run(FakeServer, Port: 8000, Logger: WEBrick::Log.new('/dev/null'), AccessLog: [])
     end
 
     loop do
@@ -75,7 +82,7 @@ describe 'Adapters' do
   end
 
   after :all do
-    Rack::Handler::WEBrick.shutdown
+    RACK_SERVER.shutdown
     @server_thread.terminate
   end
 

data/spec/autoloading_spec.rb

@@ -1,56 +1,63 @@
 describe "#{GraphQL::Client} Autoloading" do
   describe ".load_constant" do
     it "loads the specified constant if there is a matching graphql file" do
-      Metaphysics.send(:remove_const, :Artist) if Metaphysics.constants.include?(:Artist)
+      Github.send(:remove_const, :User) if Github.constants.include?(:User)
 
-      Metaphysics.load_constant(:Artist)
+      Github.load_constant(:User)
 
-      expect(defined?(Metaphysics::Artist)).to eq('constant')
+      expect(defined?(Github::User)).to eq('constant')
     end
 
     it "does nothing and returns nil if there is no matching file" do
-      expect(Metaphysics.load_constant(:DoesNotExist)).to be_nil
+      expect(Github.load_constant(:DoesNotExist)).to be_nil
     end
   end
 
   describe ".preload!" do
     it "preloads all the graphQL files in the query paths" do
-      %i(Artist Artists Artwork ArtistFields)
-        .select {|const_name| Metaphysics.constants.include?(const_name) }
-        .each {|const_name| Metaphysics.send(:remove_const, const_name) }
+      %i(User UserRepositories Repository RepositoryFields)
+        .select {|const_name| Github.constants.include?(const_name) }
+        .each {|const_name| Github.send(:remove_const, const_name) }
 
-      Metaphysics.preload!
+      Github.preload!
 
-      expect(defined?(Metaphysics::Artist)).to eq('constant')
-      expect(defined?(Metaphysics::Artwork)).to eq('constant')
+      expect(defined?(Github::User)).to eq('constant')
+      expect(defined?(Github::Repository)).to eq('constant')
     end
   end
 
   it "dynamically loads the matching GraphQL query and sets it to a constant" do
-    Metaphysics.send(:remove_const, :Artist) if Metaphysics.constants.include?(:Artist)
+    Github.send(:remove_const, :User) if Github.constants.include?(:User)
 
-    query = Metaphysics::Artist
+    query = Github::User
 
     expect(query.document.to_query_string).to eq(<<~GRAPHQL.strip)
-      query Metaphysics__Artist($id: String!) {
-        artist(id: $id) {
+      query Github__User {
+        user(login: "yuki24") {
+          id
           name
-          bio
-          birthday
         }
       }
     GRAPHQL
   end
 
   it "dynamically loads the matching GraphQL fragment and sets it to a constant" do
-    Metaphysics.send(:remove_const, :ArtistFields) if Metaphysics.constants.include?(:ArtistFields)
+    Github.send(:remove_const, :RepositoryFields) if Github.constants.include?(:RepositoryFields)
 
-    query = Metaphysics::ArtistFields
+    query = Github::RepositoryFields
 
     expect(query.document.to_query_string).to eq(<<~GRAPHQL.strip)
-      fragment Metaphysics__ArtistFields on Artist {
-        hometown
-        deathday
+      fragment Github__RepositoryFields on Repository {
+        name
+        nameWithOwner
+        url
+        updatedAt
+        languages(first: 1) {
+          nodes {
+            name
+            color
+          }
+        }
       }
     GRAPHQL
   end
@@ -58,25 +65,24 @@ describe "#{GraphQL::Client} Autoloading" do
   it "correctly loads the matching GraphQL query even when the top-level constant with the same name exists" do
     # In Ruby <= 2.4 top-level constants can be looked up through a namespace, which turned out to be a bad practice.
     # This has been removed in 2.5, but in earlier versions still suffer from this behaviour.
-    Metaphysics.send(:remove_const, :Artist) if Metaphysics.constants.include?(:Artist)
-    Object.send(:remove_const, :Artist) if Object.constants.include?(:Artist)
+    Github.send(:remove_const, :User) if Github.constants.include?(:User)
+    Object.send(:remove_const, :User) if Object.constants.include?(:User)
 
     begin
-      Object.send(:const_set, :Artist, 1)
+      Object.send(:const_set, :User, 1)
 
-      Metaphysics.artist
+      Github.user
     ensure
-      Object.send(:remove_const, :Artist)
+      Object.send(:remove_const, :User)
     end
 
-    query = Metaphysics::Artist
+    query = Github::User
 
     expect(query.document.to_query_string).to eq(<<~GRAPHQL.strip)
-      query Metaphysics__Artist($id: String!) {
-        artist(id: $id) {
+      query Github__User {
+        user(login: "yuki24") {
+          id
           name
-          bio
-          birthday
         }
       }
     GRAPHQL
@@ -84,63 +90,63 @@ describe "#{GraphQL::Client} Autoloading" do
 
   it "raises an exception when the path was resolved but the file does not exist" do
     begin
-      Metaphysics.graphql_file_paths << "metaphysics/removed.graphql"
+      Github.graphql_file_paths << "github/removed.graphql"
 
-      expect { Metaphysics::Removed }.to raise_error(Errno::ENOENT)
+      expect { Github::Removed }.to raise_error(Errno::ENOENT)
     ensure
-      Metaphysics.graphql_file_paths.delete("metaphysics/removed.graphql")
+      Github.graphql_file_paths.delete("github/removed.graphql")
     end
   end
 
   it "raises an NameError when there is no graphql file that matches the const name" do
-    expect { Metaphysics::DoesNotExist }.to raise_error(NameError)
+    expect { Github::DoesNotExist }.to raise_error(NameError)
   end
 
   xit "defines the query method when the matching class method gets called for the first time" do
-    Metaphysics.undef_method(:artwork) if Metaphysics.public_instance_methods.include?(:artwork)
+    Github.undef_method(:user) if Github.public_instance_methods.include?(:user)
 
-    Metaphysics.artwork
+    Github.user
 
-    expect(Metaphysics.public_instance_methods).to include(:artwork)
+    expect(Github.public_instance_methods).to include(:user)
   end
 
   it "raises an NameError when there is no graphql file that matches the class method name" do
-    expect { Metaphysics.does_not_exist }.to raise_error(NameError)
+    expect { Github.does_not_exist }.to raise_error(NameError)
   end
 
   it "raises an NameError when the class method name matches a fragment name" do
-    expect { Metaphysics.artist_fragment }.to raise_error(NameError)
+    expect { Github.repository_fields_fragment }.to raise_error(NameError)
   end
 
   it "responds to a class method that has a matching graphQL file" do
-    expect(Metaphysics).to respond_to(:artwork)
+    expect(Github).to respond_to(:user)
   end
 
   it "does not respond to class methods that do not have a matching graphQL file" do
-    expect(Metaphysics).not_to respond_to(:does_not_exist)
+    expect(Github).not_to respond_to(:does_not_exist)
   end
 
   xit "defines the query method when the matching instance method gets called for the first time" do
-    Metaphysics.undef_method(:artwork) if Metaphysics.public_instance_methods.include?(:artwork)
+    Github.undef_method(:user) if Github.public_instance_methods.include?(:user)
 
-    Metaphysics.new.artwork
+    Github.new.user
 
-    expect(Metaphysics.public_instance_methods).to include(:artwork)
+    expect(Github.public_instance_methods).to include(:user)
   end
 
   it "raises an NameError when there is no graphql file that matches the instance method name" do
-    expect { Metaphysics.new.does_not_exist }.to raise_error(NameError)
+    expect { Github.new.does_not_exist }.to raise_error(NameError)
   end
 
   it "raises an NameError when the instance method name matches a fragment name" do
-    expect { Metaphysics.new.artist_fragment }.to raise_error(NameError)
+    expect { Github.new.repository_fields_fragment }.to raise_error(NameError)
   end
 
   it "responds to the method that has a matching graphQL file" do
-    expect(Metaphysics.new).to respond_to(:artwork)
+    expect(Github.new).to respond_to(:user)
   end
 
   it "does not respond to methods that do not have a matching graphQL file" do
-    expect(Metaphysics.new).not_to respond_to(:does_not_exist)
+    expect(Github.new).not_to respond_to(:does_not_exist)
   end
 end

data/spec/callbacks_spec.rb

@@ -3,7 +3,7 @@ require 'active_support/core_ext/module/attribute_accessors'
 describe "#{GraphQL::Client} Callbacks" do
   Client = Class.new(Artemis::Client) do
     def self.name
-      'Metaphysics'
+      'Github'
     end
 
     mattr_accessor :before_callback, :after_callback
@@ -35,20 +35,20 @@ describe "#{GraphQL::Client} Callbacks" do
 
   describe ".before_execute" do
     it "gets invoked before executing" do
-      Client.artist(id: 'yayoi-kusama', context: { user_id: 'yuki24' })
+      Client.repository(owner: "yuki24", name: "artemis", context: { user_id: 'yuki24' })
 
       document, operation_name, variables, context = Client.before_callback
 
-      expect(document).to eq(Client::Artist.document)
-      expect(operation_name).to eq('Client__Artist')
-      expect(variables).to eq('id' => 'yayoi-kusama')
+      expect(document).to eq(Client::Repository.document)
+      expect(operation_name).to eq('Client__Repository')
+      expect(variables).to eq("name" => "artemis", "owner" => "yuki24")
       expect(context).to eq(user_id: 'yuki24')
     end
   end
 
   describe ".after_execute" do
     it "gets invoked after executing" do
-      Client.artwork
+      Client.user
 
       data, errors, extensions = Client.after_callback