diff --git a/spec/controllers/test_controller_spec.rb b/spec/controllers/test_controller_spec.rb index cea8d8c..e8e00e2 100644 --- a/spec/controllers/test_controller_spec.rb +++ b/spec/controllers/test_controller_spec.rb @@ -59,10 +59,11 @@ RSpec.describe(TestController) do before do @hardware = Hardware.create(name: 'Test Hardware', type: 'gpu') @benchmark = Benchmark.create(name: 'Test Benchmark', scoring: 'fps') + @benchmark_profile = BenchmarkProfile.create(label: 'Test Benchmark Profile', settings: '{}', benchmark_id: @benchmark.id) request_data = { test_name: 'Test Test', test_hardware: @hardware.id, - 'test_benchmarks[]': [@benchmark.id], + 'test_benchmarks[]': [@benchmark_profile.id], test_description: 'Test for testing' } post '/test/add', request_data @@ -93,11 +94,11 @@ RSpec.describe(TestController) do end it 'Test add POST route created test has benchmarks.' do - expect(Test.first.benchmarks.length).to(eq(1)) + expect(Test.first.benchmark_profiles.length).to(eq(1)) end it 'Test add POST route created test\'s benchmark can be read.' do - expect(Test.first.benchmarks[0].id).to(eq(@benchmark.id)) + expect(Test.first.benchmark_profiles[0].id).to(eq(@benchmark.id)) end it 'Test add POST route created test has description.' do @@ -110,12 +111,13 @@ RSpec.describe(TestController) do before do @hardware = Hardware.create(name: 'Test Hardware', type: 'gpu') @benchmark = Benchmark.create(name: 'Test Benchmark', scoring: 'fps') + @benchmark_profile = BenchmarkProfile.create(label: 'Test Benchmark Profile', settings: '{}', benchmark_id: @benchmark.id) @test = Test.create( name: 'Test Test', hardware_id: @hardware.id, description: 'Test for testing' ) - @test.add_benchmark(@benchmark) + @test.add_benchmark_profile(@benchmark_profile) get "/test/#{@test.id}" end @@ -141,12 +143,13 @@ RSpec.describe(TestController) do before do @hardware = Hardware.create(name: 'Test Hardware', type: 'gpu') @benchmark = Benchmark.create(name: 'Test Benchmark', scoring: 'fps') + @benchmark_profile = BenchmarkProfile.create(label: 'Test Benchmark Profile', settings: '{}', benchmark_id: @benchmark.id) @test = Test.create( name: 'Test Test', hardware_id: @hardware.id, description: 'Test for testing' ) - @test.add_benchmark(@benchmark) + @test.add_benchmark_profile(@benchmark_profile) get "/test/#{@test.id}/edit" end diff --git a/src/controllers/test.rb b/src/controllers/test.rb index 6e0de42..2c61180 100644 --- a/src/controllers/test.rb +++ b/src/controllers/test.rb @@ -42,7 +42,7 @@ class TestController < BaseController benchmarks = Array(params[:test_benchmarks]) # associate the benchmarks to the test benchmarks.each do |b| - tst.add_benchmark(b) + tst.add_benchmark_profile(b) end redirect "/test/#{tst.id}" @@ -81,13 +81,13 @@ class TestController < BaseController selected_benchmarks = Array(params[:test_benchmarks]) # remove benchmarks no longer associated with the test - tst.benchmarks.dup.each do |b| - tst.remove_benchmark(b.id) unless selected_benchmarks.include?(b.id) + tst.benchmark_profiles.dup.each do |b| + tst.remove_benchmark_profile(b.id) unless selected_benchmarks.include?(b.id) end # associate the benchmarks to the test selected_benchmarks.each do |b| - tst.add_benchmark(b) unless tst.benchmark?(b) + tst.add_benchmark_profile(b) unless tst.benchmark?(b) end redirect "/test/#{tst.id}" diff --git a/src/models/test.rb b/src/models/test.rb index 6a6ac85..9fd4fa0 100644 --- a/src/models/test.rb +++ b/src/models/test.rb @@ -7,8 +7,8 @@ class Test < Sequel::Model many_to_one :hardware many_to_many :benchmark_profiles - def benchmark?(benchmark_id) - return benchmark_profiles_dataset.where(Sequel[:benchmark][:id] => benchmark_id).any? + def benchmark?(benchmark_profile_id) + return benchmark_profiles_dataset.where(Sequel[:benchmark_profiles][:id] => benchmark_profile_id).any? end end diff --git a/views/test/add.erb b/views/test/add.erb index 6c9e2d0..3fcf5a1 100644 --- a/views/test/add.erb +++ b/views/test/add.erb @@ -28,8 +28,10 @@ diff --git a/views/test/edit.erb b/views/test/edit.erb index 154ed86..2b6484d 100644 --- a/views/test/edit.erb +++ b/views/test/edit.erb @@ -28,8 +28,10 @@ diff --git a/views/test/index.erb b/views/test/index.erb index 7717b15..5a6bb15 100644 --- a/views/test/index.erb +++ b/views/test/index.erb @@ -26,7 +26,7 @@ <% tests.each do |t| %> <%= t.name %> - <%= t.benchmarks.length %> + <%= t.benchmark_profiles.length %> <%= date_format(t.created_at) %> <%= date_format(t.updated_at) %> diff --git a/views/test/view.erb b/views/test/view.erb index 001ac24..a58e538 100644 --- a/views/test/view.erb +++ b/views/test/view.erb @@ -28,8 +28,8 @@
@@ -71,7 +71,7 @@ - <% test.benchmarks.each do |benchmark| %> + <% test.benchmark_profiles.each do |benchmark| %> <% end %>