Add submission_url to config and refactor logic into services

Gemfile

@@ -1,19 +1,17 @@
 source "https://rubygems.org"
-# Add dependencies required to use your gem here.
-# Example:
+
 gem "activesupport", ">= 2.3.5"
-gem "oj", "~> 3.13.12"
+gem 'oj', '~> 3.16', '>= 3.16.3'
 gem "octokit", "~> 5.0"
 gem "zip"
 gem "faraday-retry", "~> 1.0.3"
 gem "rake", "~> 13"
-# Add dependencies to develop your gem here.
-# Include everything needed to run rake, tests, features, etc.
+
 group :development do
   gem "rspec", "~> 3.5.0"
   gem "rdoc", "~> 6.1"
-  gem "bundler", "~> 2.1.4"
-  gem "juwelier", "~> 2.1.0"
+  gem "bundler", "~> 2.6.7"
+  gem "juwelier", "~> 2.4.9"
   gem "simplecov", ">= 0"
   gem "pry", "~> 0"
   gem "pry-byebug", "~> 3"

Gemfile.lock

@@ -8,6 +8,7 @@ GEM
       tzinfo (~> 2.0)
     addressable (2.8.4)
       public_suffix (>= 2.0.2, < 6.0)
+    bigdecimal (3.2.2)
     binding_of_caller (1.0.0)
       debug_inspector (>= 0.0.1)
     builder (3.2.4)
@@ -56,17 +57,21 @@ GEM
     i18n (1.14.1)
       concurrent-ruby (~> 1.0)
     interception (0.5)
-    juwelier (2.1.3)
+    juwelier (2.4.9)
       builder
-      bundler (>= 1.13)
-      git (>= 1.2.5)
+      bundler
+      git
       github_api
-      highline (>= 1.6.15)
-      nokogiri (>= 1.5.10)
+      highline
+      kamelcase (~> 0)
+      nokogiri
+      psych
       rake
       rdoc
-      semver
+      semver2
     jwt (2.7.0)
+    kamelcase (0.0.2)
+      semver2 (~> 3)
     method_source (1.0.0)
     mini_portile2 (2.8.1)
     minitest (5.19.0)
@@ -85,7 +90,10 @@ GEM
     octokit (5.6.1)
       faraday (>= 1, < 3)
       sawyer (~> 0.9)
-    oj (3.13.23)
+    oj (3.16.11)
+      bigdecimal (>= 3.0)
+      ostruct (>= 0.2)
+    ostruct (0.6.3)
     pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
@@ -130,7 +138,7 @@ GEM
     sawyer (0.9.2)
       addressable (>= 2.3.5)
       faraday (>= 0.17.3, < 3)
-    semver (1.0.1)
+    semver2 (3.4.2)
     simplecov (0.21.2)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
@@ -152,11 +160,11 @@ PLATFORMS
 
 DEPENDENCIES
   activesupport (>= 2.3.5)
-  bundler (~> 2.1.4)
+  bundler (~> 2.6.7)
   faraday-retry (~> 1.0.3)
-  juwelier (~> 2.1.0)
+  juwelier (~> 2.4.9)
   octokit (~> 5.0)
-  oj (~> 3.13.12)
+  oj (~> 3.16, >= 3.16.3)
   pry (~> 0)
   pry-byebug (~> 3)
   pry-doc (~> 0)
@@ -170,4 +178,4 @@ DEPENDENCIES
   zip
 
 BUNDLED WITH
-   2.1.4
+   2.6.7

README.markdown

@@ -1,7 +1,6 @@
 # grade_runner
 
-A Ruby client for [firstdraft Grades](https://grades.firstdraft.com)
-
+A Ruby client for [Grades](https://grades.dpi.dev)
 
 ## Installation
 
@@ -48,6 +47,7 @@ if Rails.env.development? || Rails.env.test?
   GradeRunner.config do |config|
     config.default_points = 1           # default 1
     config.override_local_specs = false # default true
+    config.submission_url = "https://your-grading-server-url.com" # Set custom submission URL
   end
 end
 ```

Rakefile

@@ -21,6 +21,9 @@ Juwelier::Tasks.new do |gem|
   gem.email = ["raghu@firstdraft.com", "jelani@firstdraft.com"]
   gem.authors = ["Raghu Betina", "Jelani Woods"]
 
+  # Note: rspec tests do not yet support 3.4 (BigDecimal)
+  gem.required_ruby_version = Gem::Requirement.new(">= 2")
+
   # dependencies defined in Gemfile
 end
 Juwelier::RubygemsDotOrgTasks.new

VERSION

@@ -1 +1 @@
-0.0.13
+0.0.14

grade_runner.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: grade_runner 0.0.13 ruby lib
+# stub: grade_runner 0.0.14 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "grade_runner".freeze
-  s.version = "0.0.13"
+  s.version = "0.0.14"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["Raghu Betina".freeze, "Jelani Woods".freeze]
-  s.date = "2025-03-17"
+  s.date = "2025-08-18"
   s.description = "This gem runs your RSpec test suite and posts the JSON output to grades.firstdraft.com.".freeze
   s.email = ["raghu@firstdraft.com".freeze, "jelani@firstdraft.com".freeze]
   s.extra_rdoc_files = [
@@ -29,57 +29,45 @@ Gem::Specification.new do |s|
     "VERSION",
     "grade_runner.gemspec",
     "lib/grade_runner.rb",
+    "lib/grade_runner/formatters/hint_formatter.rb",
+    "lib/grade_runner/formatters/json_output_formatter.rb",
     "lib/grade_runner/railtie.rb",
     "lib/grade_runner/runner.rb",
-    "lib/tasks/grade.rake",
-    "lib/tasks/grade_runner.rake",
+    "lib/grade_runner/services/config_service.rb",
+    "lib/grade_runner/services/github_service.rb",
+    "lib/grade_runner/services/grade_service.rb",
+    "lib/grade_runner/services/spec_service.rb",
+    "lib/grade_runner/services/token_service.rb",
+    "lib/grade_runner/tasks.rb",
+    "lib/grade_runner/tasks/grade.rake",
+    "lib/grade_runner/tasks/grade_runner.rake",
+    "lib/grade_runner/utils/path_utils.rb",
     "spec/spec_helper.rb"
   ]
   s.homepage = "http://github.com/firstdraft/grade_runner".freeze
   s.licenses = ["MIT".freeze]
-  s.rubygems_version = "3.1.6".freeze
+  s.required_ruby_version = Gem::Requirement.new(">= 2".freeze)
+  s.rubygems_version = "3.4.6".freeze
   s.summary = "A Ruby client for [firstdraft Grades](https://grades.firstdraft.com)".freeze
 
-  if s.respond_to? :specification_version then
-    s.specification_version = 4
-  end
+  s.specification_version = 4
 
-  if s.respond_to? :add_runtime_dependency then
-    s.add_runtime_dependency(%q<activesupport>.freeze, [">= 2.3.5"])
-    s.add_runtime_dependency(%q<oj>.freeze, ["~> 3.13.12"])
-    s.add_runtime_dependency(%q<octokit>.freeze, ["~> 5.0"])
-    s.add_runtime_dependency(%q<zip>.freeze, [">= 0"])
-    s.add_runtime_dependency(%q<faraday-retry>.freeze, ["~> 1.0.3"])
-    s.add_runtime_dependency(%q<rake>.freeze, ["~> 13"])
-    s.add_development_dependency(%q<rspec>.freeze, ["~> 3.5.0"])
-    s.add_development_dependency(%q<rdoc>.freeze, ["~> 6.1"])
-    s.add_development_dependency(%q<bundler>.freeze, ["~> 2.1.4"])
-    s.add_development_dependency(%q<juwelier>.freeze, ["~> 2.1.0"])
-    s.add_development_dependency(%q<simplecov>.freeze, [">= 0"])
-    s.add_development_dependency(%q<pry>.freeze, ["~> 0"])
-    s.add_development_dependency(%q<pry-byebug>.freeze, ["~> 3"])
-    s.add_development_dependency(%q<pry-doc>.freeze, ["~> 0"])
-    s.add_development_dependency(%q<pry-remote>.freeze, ["~> 0"])
-    s.add_development_dependency(%q<pry-rescue>.freeze, ["~> 1"])
-    s.add_development_dependency(%q<pry-stack_explorer>.freeze, ["~> 0"])
-  else
-    s.add_dependency(%q<activesupport>.freeze, [">= 2.3.5"])
-    s.add_dependency(%q<oj>.freeze, ["~> 3.13.12"])
-    s.add_dependency(%q<octokit>.freeze, ["~> 5.0"])
-    s.add_dependency(%q<zip>.freeze, [">= 0"])
-    s.add_dependency(%q<faraday-retry>.freeze, ["~> 1.0.3"])
-    s.add_dependency(%q<rake>.freeze, ["~> 13"])
-    s.add_dependency(%q<rspec>.freeze, ["~> 3.5.0"])
-    s.add_dependency(%q<rdoc>.freeze, ["~> 6.1"])
-    s.add_dependency(%q<bundler>.freeze, ["~> 2.1.4"])
-    s.add_dependency(%q<juwelier>.freeze, ["~> 2.1.0"])
-    s.add_dependency(%q<simplecov>.freeze, [">= 0"])
-    s.add_dependency(%q<pry>.freeze, ["~> 0"])
-    s.add_dependency(%q<pry-byebug>.freeze, ["~> 3"])
-    s.add_dependency(%q<pry-doc>.freeze, ["~> 0"])
-    s.add_dependency(%q<pry-remote>.freeze, ["~> 0"])
-    s.add_dependency(%q<pry-rescue>.freeze, ["~> 1"])
-    s.add_dependency(%q<pry-stack_explorer>.freeze, ["~> 0"])
-  end
+  s.add_runtime_dependency(%q<activesupport>.freeze, [">= 2.3.5"])
+  s.add_runtime_dependency(%q<oj>.freeze, ["~> 3.16", ">= 3.16.3"])
+  s.add_runtime_dependency(%q<octokit>.freeze, ["~> 5.0"])
+  s.add_runtime_dependency(%q<zip>.freeze, [">= 0"])
+  s.add_runtime_dependency(%q<faraday-retry>.freeze, ["~> 1.0.3"])
+  s.add_runtime_dependency(%q<rake>.freeze, ["~> 13"])
+  s.add_development_dependency(%q<rspec>.freeze, ["~> 3.5.0"])
+  s.add_development_dependency(%q<rdoc>.freeze, ["~> 6.1"])
+  s.add_development_dependency(%q<bundler>.freeze, ["~> 2.6.7"])
+  s.add_development_dependency(%q<juwelier>.freeze, ["~> 2.4.9"])
+  s.add_development_dependency(%q<simplecov>.freeze, [">= 0"])
+  s.add_development_dependency(%q<pry>.freeze, ["~> 0"])
+  s.add_development_dependency(%q<pry-byebug>.freeze, ["~> 3"])
+  s.add_development_dependency(%q<pry-doc>.freeze, ["~> 0"])
+  s.add_development_dependency(%q<pry-remote>.freeze, ["~> 0"])
+  s.add_development_dependency(%q<pry-rescue>.freeze, ["~> 1"])
+  s.add_development_dependency(%q<pry-stack_explorer>.freeze, ["~> 0"])
 end
 

lib/grade_runner.rb

@@ -1,11 +1,19 @@
 require "grade_runner/runner"
+require "grade_runner/utils/path_utils"
+require "grade_runner/services/token_service"
+require "grade_runner/services/config_service"
+require "grade_runner/services/github_service"
+require "grade_runner/services/spec_service"
+require "grade_runner/services/grade_service"
 require "grade_runner/railtie" if defined?(Rails)
 
 module GradeRunner
   class Error < StandardError; end
 
+  DEFAULT_SUBMISSION_URL = "https://grades.firstdraft.com"
+
   class << self
-    attr_writer :default_points, :override_local_specs
+    attr_writer :default_points, :override_local_specs, :submission_url
 
     def default_points
       @default_points || 1
@@ -19,8 +27,16 @@ def override_local_specs
       end
     end
 
+    def submission_url
+      @submission_url || DEFAULT_SUBMISSION_URL
+    end
+
     def config
       yield self
     end
+
+    def project_root
+      Utils::PathUtils.project_root
+    end
   end
 end

lib/grade_runner/formatters/json_output_formatter.rb

@@ -1,5 +1,6 @@
-RSpec::Support.require_rspec_core "formatters/json_formatter"
 require "oj"
+require "rspec/core"
+
 class JsonOutputFormatter < RSpec::Core::Formatters::JsonFormatter
   RSpec::Core::Formatters.register self, :dump_summary
 
@@ -8,7 +9,7 @@ def dump_summary(summary)
       examples.
       map { |example| example.metadata.fetch(:points, GradeRunner.default_points).to_i }.
       sum
-      
+
     earned_points = summary.
       examples.
       select { |example| example.execution_result.status == :passed }.
@@ -17,7 +18,7 @@ def dump_summary(summary)
 
     score = (earned_points.to_f / total_points).round(4)
     score = 0 if score.nan?
-    
+
     @output_hash[:summary] = {
       duration: summary.duration,
       example_count: summary.example_count,
@@ -29,14 +30,14 @@ def dump_summary(summary)
       score: score
     }
     result = (@output_hash[:summary][:score] * 100).round(2)
-    
+
     if summary.errors_outside_of_examples_count.positive?
       result = "An error occurred while running tests"
     else
       result = result.to_s + "%"
     end
-    
-    
+
+
     @output_hash[:summary_line] = [
       "#{summary.example_count} #{summary.example_count == 1 ? "test" : "tests"}",
       "#{summary.failure_count} failures",
@@ -48,7 +49,7 @@ def dump_summary(summary)
   def close(_notification)
     output.write  Oj.dump @output_hash
   end
-  
+
   private
 
   def format_example(example)

lib/grade_runner/railtie.rb

@@ -2,10 +2,6 @@
 
 module GradeRunner
   class Railtie < ::Rails::Railtie
-
-     rake_tasks do
-      load "tasks/grade_runner.rake"
-      load "tasks/grade.rake"
-     end
+    rake_tasks { load "grade_runner/tasks.rb" }
   end
 end

lib/grade_runner/runner.rb

@@ -3,7 +3,6 @@
 
 module GradeRunner
   class Runner
-
     def initialize(submission_root_url, grades_access_token, rspec_output_json, username, reponame, sha, source)
       @submission_url = submission_root_url + submission_path
       @grades_access_token = grades_access_token
@@ -47,6 +46,5 @@ def data
         source: @source
       }
     end
-
   end
 end