BRD-20938 fix rubocop errors

Author: jamesh-gomo

lib/scorm_engine/api/endpoints/courses.rb

@@ -35,24 +35,24 @@ def get_courses(options = {})
           result = Enumerator.new do |enum|
             if single_course_id
               # Single course endpoint returns course data directly
-              if response.success? && response.raw_response.body.is_a?(Hash)
+              if response.success? && response.raw_response.body.is_a?(Hash) # rubocop:disable Style/IfUnlessModifier
                 enum << ScormEngine::Models::Course.new_from_api(response.raw_response.body)
               end
             else
               # Multiple courses endpoint returns array in "courses" key
               loop do
                 break unless response.success? && response.raw_response.body.is_a?(Hash)
-                
+
                 courses = response.raw_response.body["courses"]
                 break unless courses.is_a?(Array)
-                
+
                 courses.each do |course|
                   enum << ScormEngine::Models::Course.new_from_api(course)
                 end
-                
+
                 more_url = response.raw_response.body["more"]
                 break if more_url.nil?
-                
+
                 response = get(more_url)
               end
             end

lib/scorm_engine/models/course_import.rb

@@ -34,9 +34,7 @@ def self.new_from_api(options = {})
           this.status          = (options["status"] || import_result["status"])&.upcase
           this.parser_warnings = import_result["parserWarnings"]
 
-          if import_result.key?("course")
-            this.course = Course.new_from_api(import_result["course"])
-          end
+          this.course = Course.new_from_api(import_result["course"]) if import_result.key?("course")
 
         elsif options.keys == ["result"]
           # Initial import response (legacy format: {"result" => "job-id"})
@@ -49,9 +47,7 @@ def self.new_from_api(options = {})
           this.status          = options["status"]&.upcase
           this.parser_warnings = options["parserWarnings"]
 
-          if options.key?("course")
-            this.course = Course.new_from_api(options["course"])
-          end
+          this.course = Course.new_from_api(options["course"]) if options.key?("course")
         end
 
         this