From 364140cd086c2da86c479260c6587052745c5172 Mon Sep 17 00:00:00 2001 From: Tom Levy Date: Fri, 22 Dec 2023 19:12:46 +1300 Subject: [PATCH] Remove unused and broken pages for test sets/cases --- app/controllers/test_cases_controller.rb | 82 --------------------- app/controllers/test_sets_controller.rb | 93 ------------------------ app/views/test_cases/index.html.erb | 39 ---------- app/views/test_sets/index.html.erb | 52 ------------- config/routes.rb | 3 - 5 files changed, 269 deletions(-) delete mode 100644 app/controllers/test_cases_controller.rb delete mode 100644 app/controllers/test_sets_controller.rb delete mode 100644 app/views/test_cases/index.html.erb delete mode 100644 app/views/test_sets/index.html.erb diff --git a/app/controllers/test_cases_controller.rb b/app/controllers/test_cases_controller.rb deleted file mode 100644 index f73468be..00000000 --- a/app/controllers/test_cases_controller.rb +++ /dev/null @@ -1,82 +0,0 @@ -class TestCasesController < ApplicationController - #filter_resource_access :collection => [] - - def permitted_params - @_permitted_params ||= begin - permitted_attributes = [:input, :output, :name] - params.require(:test_case).permit(*permitted_attributes) - end - end - - # GET /test_cases - # GET /test_cases.xml - def index - @problem = Problem.find(params[:problem_id]) - if params[:problem_id] - permitted_to! :inspect, @problem - logger.debug "problem is " + params[:problem_id] - @test_cases = @problem.test_cases - else - raise Pundit::NotAuthorizedError - @test_cases = @test_cases.distinct - end - - respond_to do |format| - format.html # index.html.erb - format.xml { render :xml => @test_cases } - end - end - - # GET /test_cases/1 - # GET /test_cases/1.xml - def show - @test_case = TestCase.find(params[:id]) - permitted_to! :inspect, @test_case.problems.first - respond_to do |format| - format.html # show.html.erb - format.xml { render :xml => @test_case } - end - end - - # GET /test_cases/1/edit - def edit - @test_case = TestCase.find(params[:id]) - @test_case.problems.each do |problem| - permitted_to! :update, problem - end - end - - # PUT /test_cases/1 - # PUT /test_cases/1.xml - def update - @test_case = TestCase.find(params[:id]) - @test_case.problems.each do |problem| - permitted_to! :update, problem - end - - respond_to do |format| - if @test_case.update_attributes(permitted_params) - format.html { redirect_to(@test_case, :notice => 'Test case was successfully updated.') } - format.xml { head :ok } - else - format.html { render :action => "edit" } - format.xml { render :xml => @test_case.errors, :status => :unprocessable_entity } - end - end - end - - # DELETE /test_cases/1 - # DELETE /test_cases/1.xml - def destroy - @test_case = TestCase.find(params[:id]) - @test_case.problems.each do |problem| - permitted_to! :update, problem - end - @test_case.destroy - - respond_to do |format| - format.html { redirect_to(test_cases_url) } - format.xml { head :ok } - end - end -end diff --git a/app/controllers/test_sets_controller.rb b/app/controllers/test_sets_controller.rb deleted file mode 100644 index 0a821ddb..00000000 --- a/app/controllers/test_sets_controller.rb +++ /dev/null @@ -1,93 +0,0 @@ -class TestSetsController < ApplicationController - - def permitted_params - @_permitted_params ||= begin - permitted_attributes = [:name, :problem_id, :points] - params.require(:test_set).permit(*permitted_attributes) - end - end - - # GET /test_sets - # GET /test_sets.json - def index - permitted_to! :inspect, Problem.find(params[:problem_id]) - @test_sets = TestSet.where(:problem_id => params[:problem_id]) - - respond_to do |format| - format.html # index.html.erb - format.json { render :json => @test_sets } - end - end - - # GET /test_sets/1 - # GET /test_sets/1.json - def show - @test_set = TestSet.find(params[:id]) - permitted_to! :inspect, @test_set.problem - respond_to do |format| - format.html # show.html.erb - format.json { render :json => @test_set } - end - end - - # GET /test_sets/new - # GET /test_sets/new.json - def new - raise Pundit::NotAuthorizedError - respond_to do |format| - format.html # new.html.erb - format.json { render :json => @test_set } - end - end - - # GET /test_sets/1/edit - def edit - end - - # POST /test_sets - # POST /test_sets.json - def create - raise Pundit::NotAuthorizedError - @test_set = TestSet.new(permitted_params) - permitted_to! :create, @test_set - - respond_to do |format| - if @test_set.save - format.html { redirect_to @test_set, :notice => 'Test set was successfully created.' } - format.json { render :json => @test_set, :status => :created, :location => @test_set } - else - format.html { render :action => "new" } - format.json { render :json => @test_set.errors, :status => :unprocessable_entity } - end - end - end - - # PUT /test_sets/1 - # PUT /test_sets/1.json - def update - @test_set = TestSet.find(params[:id]) - permitted_to! :update, @test_set.problem - respond_to do |format| - if @test_set.update_attributes(permitted_params) - format.html { redirect_to @test_set, :notice => 'Test set was successfully updated.' } - format.json { head :ok } - else - format.html { render :action => "edit" } - format.json { render :json => @test_set.errors, :status => :unprocessable_entity } - end - end - end - - # DELETE /test_sets/1 - # DELETE /test_sets/1.json - def destroy - @test_set = TestSet.find(params[:id]) - permitted_to! :destroy, @test_set - @test_set.destroy - - respond_to do |format| - format.html { redirect_to test_sets_url } - format.json { head :ok } - end - end -end diff --git a/app/views/test_cases/index.html.erb b/app/views/test_cases/index.html.erb deleted file mode 100644 index 39b1c61f..00000000 --- a/app/views/test_cases/index.html.erb +++ /dev/null @@ -1,39 +0,0 @@ -<% content_for :title, "Test Cases" %> - - - - - - - - - - <% if permitted_to? :update, TestSet %> - - <% end %> - <% if permitted_to? :destroy, TestSet %> - - <% end %> - - -<% @test_cases.each do |test_case| %> - - - - - - - - <% if permitted_to? :update, test_case %> - - <% end %> - <% if permitted_to? :destroy, test_case %> - - <% end %> - -<% end %> -
InputOutputTest SetsNameProblems
<%= simple_format(test_case.input.truncate(100)) %><%= simple_format(test_case.output.truncate(100)) %><%= test_case.test_set_ids.to_s %><%= test_case.name %><%= test_case.problem_ids %><%= link_to 'Show', test_case %><%= link_to 'Edit', edit_test_case_path(test_case) %><%= link_to 'Destroy', test_case, :data => { :confirm => 'Are you sure?' }, :method => :delete %>
- -
- -<%= link_to 'New Test case', new_test_case_path %> diff --git a/app/views/test_sets/index.html.erb b/app/views/test_sets/index.html.erb deleted file mode 100644 index 9e4e645c..00000000 --- a/app/views/test_sets/index.html.erb +++ /dev/null @@ -1,52 +0,0 @@ -<% content_for :title, "Test Sets" %> - -<% @colspan = 4 %> - - - - - - - <% if permitted_to? :update, TestSet %> - <% @colspan += 1 %> - - <% end %> - <% if permitted_to? :destroy, TestSet %> - <% @colspan += 1 %> - - <% end %> - - -<% @test_sets.each do |test_set| %> - - - - - - <% if permitted_to? :update, test_set %> - - <% end %> - <% if permitted_to? :destroy, test_set %> - - <% end %> - - - - -<% end %> -
ProblemNamePoints
<%= test_set.problem_id %><%= test_set.name %><%= test_set.points %><%= link_to 'Show', test_set %><%= link_to 'Edit', edit_test_set_path(test_set) %><%= link_to 'Destroy', test_set, :data => { :confirm => 'Are you sure?' }, :method => :delete %>
- - <% test_set.test_cases.each do |test_case| %> - - - - - - - <% end %> -
<%= test_case.name %>
Input: <%= test_case.input[0..200].gsub("\n","\\n") %>
Output: <%= test_case.output[0..200].gsub("\n","\\n") %>
<%= link_to 'Show', test_case %>
-
- -
- -<%= link_to 'New Test set', new_test_set_path if permitted_to? :create, TestSet %> diff --git a/config/routes.rb b/config/routes.rb index b78cbc62..ae01fac4 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -18,9 +18,6 @@ #end end - #resources :test_sets - #resources :test_cases - resources :evaluators resources :settings