Add an administrator action to refresh autotest schema

Changelog.md

@@ -14,6 +14,7 @@
 - Update autotest settings form UI (#7777)
 - Store start and end date for courses (#7783)
 - Split courses into Current and Past sections for all users (#7801)
+- Add an administrator action on the course settings page to refresh autotest schema (#7828)
 
 ### 🐛 Bug fixes
 - Prevent "No rows found" message from displaying in tables when data is loading (#7790)

app/controllers/admin/courses_controller.rb

@@ -60,6 +60,23 @@ def reset_autotest_connection
       respond_with current_course, location: -> { edit_admin_course_path(current_course) }
     end
 
+    def refresh_autotest_schema
+      settings = current_course.autotest_setting
+      if settings.nil?
+        flash_message(:error, I18n.t('automated_tests.no_autotest_settings'))
+        return respond_with current_course, location: -> { edit_admin_course_path(current_course) }
+      end
+
+      begin
+        schema_json = get_schema(settings)
+        settings.update!(schema: schema_json)
+        flash_message(:success, I18n.t('automated_tests.manage_connection.refresh_schema_success'))
+      rescue StandardError => e
+        flash_message(:error, I18n.t('automated_tests.manage_connection.refresh_schema_failure', error: e.message))
+      end
+      respond_with current_course, location: -> { edit_admin_course_path(current_course) }
+    end
+
     def destroy_lti_deployment
       deployment = LtiDeployment.find(params[:lti_deployment_id])
       deployment.destroy!

app/views/courses/_form.html.erb

@@ -56,6 +56,20 @@
 <% if @current_course.persisted? && allowed_to?(:edit?, with: Admin::CoursePolicy) %>
   <fieldset id="manage-connection" class="<%= @current_course.autotest_setting_id ? '' : 'no-display' %>">
     <legend><span><%= t('automated_tests.manage_connection.title') %></span></legend>
+    <div>
+      <%= link_to t('automated_tests.manage_connection.refresh_schema_bottom'),
+                  refresh_autotest_schema_admin_course_path(@current_course),
+                  method: :post,
+                  class: 'button',
+                  data: { confirm: I18n.t('automated_tests.manage_connection.refresh_schema_confirm') } %>
+      <div class='inline-help'>
+        <p>
+          <%= I18n.t('automated_tests.manage_connection.refresh_schema_tooltip',
+                     url: @current_course.autotest_setting&.url) %>
+        </p>
+      </div>
+    </div>
+    <br>
     <div>
       <%= link_to I18n.t('automated_tests.manage_connection.test'),
                   test_autotest_connection_admin_course_path(@current_course),

config/locales/views/automated_tests/en.yml

@@ -27,6 +27,11 @@ en:
         Please try saving the settings again from the Automated Testing page.
     manage_connection:
       refresh: Refresh Settings
+      refresh_schema_bottom: Refresh autotest schema
+      refresh_schema_confirm: Do you want to refresh autotest schema now?
+      refresh_schema_failure: 'Refresh autotest schema failed with the following error: %{error}.'
+      refresh_schema_success: Refresh autotest schema was successful.
+      refresh_schema_tooltip: Fetch the latest autotest schema from %{url}.
       refresh_tooltip: Resend all test settings to the automated tester at %{url}. This should be used when the automated tester has been reset.
       test: Test Connection
       test_failure: 'Connection to %{url} failed with the following error: %{error}'

config/routes.rb

@@ -104,6 +104,7 @@
       member do
         get 'test_autotest_connection'
         put 'reset_autotest_connection'
+        post 'refresh_autotest_schema'
         delete 'destroy_lti_deployment'
       end
     end

spec/controllers/admin/courses_controller_spec.rb

@@ -69,6 +69,13 @@
           expect(response).to have_http_status(:forbidden)
         end
       end
+
+      describe '#refresh_autotest_schema' do
+        it 'responds with 403' do
+          post_as user, :refresh_autotest_schema, params: { id: course.id }
+          expect(response).to have_http_status(:forbidden)
+        end
+      end
     end
 
     context 'Instructor' do
@@ -354,6 +361,38 @@
       end
     end
 
+    describe '#refresh_autotest_schema' do
+      subject { post_as admin, :refresh_autotest_schema, params: { id: course.id } }
+
+      context 'there is no autotest_setting set' do
+        it 'should flash an error message' do
+          subject
+          expect(response).to have_http_status(:found)
+          expect(flash[:error]).not_to be_empty
+        end
+      end
+
+      context 'there is an autotest_setting' do
+        include_context 'course with an autotest setting'
+        it 'should update the schema' do
+          schema_json = { 'schema' => 'data' }.to_json
+          allow(controller).to receive(:get_schema).and_return(schema_json)
+          subject
+          expect(autotest_setting.reload.schema).to eq(schema_json)
+          expect(flash[:success]).not_to be_empty
+        end
+
+        context 'when the refresh request fails' do
+          before { allow(controller).to receive(:get_schema).and_raise(StandardError) }
+
+          it 'should flash an error message' do
+            subject
+            expect(flash[:error]).not_to be_empty
+          end
+        end
+      end
+    end
+
     describe '#reset_autotest_connection' do
       subject { put_as admin, :reset_autotest_connection, params: { id: course.id } }