From 8b2c152803b9118a9ae2ae3085d3bb88fee7fb53 Mon Sep 17 00:00:00 2001 From: Gregory Ballantine Date: Tue, 19 Aug 2025 19:15:37 -0400 Subject: [PATCH] Added some Content-Type unit tests --- spec/controllers/benchmark_controller_spec.rb | 12 ++++++++++++ spec/controllers/hardware_controller_spec.rb | 12 ++++++++++++ spec/controllers/index_controller_spec.rb | 4 ++++ spec/controllers/reports_controller_spec.rb | 6 +++++- spec/controllers/test_controller_spec.rb | 12 ++++++++++++ 5 files changed, 45 insertions(+), 1 deletion(-) diff --git a/spec/controllers/benchmark_controller_spec.rb b/spec/controllers/benchmark_controller_spec.rb index 5981cbd..2e47d5f 100644 --- a/spec/controllers/benchmark_controller_spec.rb +++ b/spec/controllers/benchmark_controller_spec.rb @@ -11,6 +11,10 @@ RSpec.describe(BenchmarkController) do expect(last_response).to(be_redirect) end + it 'Benchmark base route is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it 'Benchmark base route Location header points to /benchmark/list' do expect(last_response['Location']).to(eq("#{BASE_URL}/benchmark/list")) end @@ -24,6 +28,10 @@ RSpec.describe(BenchmarkController) do expect(last_response).to(be_ok) end + it 'Benchmark list page is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it "Benchmark list page contains 'List of benchmarks' on page." do expect(last_response.body).to(include('List of benchmarks')) end @@ -37,6 +45,10 @@ RSpec.describe(BenchmarkController) do expect(last_response).to(be_ok) end + it 'Benchmark add page is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it "Benchmark add page contains 'Add new benchmark' on page." do expect(last_response.body).to(include('Add new benchmark')) end diff --git a/spec/controllers/hardware_controller_spec.rb b/spec/controllers/hardware_controller_spec.rb index 6e0235c..f36140c 100644 --- a/spec/controllers/hardware_controller_spec.rb +++ b/spec/controllers/hardware_controller_spec.rb @@ -11,6 +11,10 @@ RSpec.describe(HardwareController) do expect(last_response).to(be_redirect) end + it 'Hardware base route is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it 'Hardware base route Location header points to /hardware/list' do expect(last_response['Location']).to(eq("#{BASE_URL}/hardware/list")) end @@ -24,6 +28,10 @@ RSpec.describe(HardwareController) do expect(last_response).to(be_ok) end + it 'Hardware list page is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it "Hardware list page contains 'List of hardware' on page." do expect(last_response.body).to(include('List of hardware')) end @@ -37,6 +45,10 @@ RSpec.describe(HardwareController) do expect(last_response).to(be_ok) end + it 'Hardware add page is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it "Hardware add page contains 'Add new hardware' on page." do expect(last_response.body).to(include('Add new hardware')) end diff --git a/spec/controllers/index_controller_spec.rb b/spec/controllers/index_controller_spec.rb index a5442b5..d3eb6e6 100644 --- a/spec/controllers/index_controller_spec.rb +++ b/spec/controllers/index_controller_spec.rb @@ -10,6 +10,10 @@ RSpec.describe(IndexController) do expect(last_response).to(be_ok) end + it 'Dashboard is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it "Dashboard contains 'Game Data' on page (nav bar should be loaded)." do expect(last_response.body).to(include('Game Data')) end diff --git a/spec/controllers/reports_controller_spec.rb b/spec/controllers/reports_controller_spec.rb index 585d404..edb5647 100644 --- a/spec/controllers/reports_controller_spec.rb +++ b/spec/controllers/reports_controller_spec.rb @@ -10,7 +10,11 @@ RSpec.describe(ReportsController) do expect(last_response).to(be_ok) end - it "Reports page contains 'Generate report' on page." do + it 'Reports page is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + + it 'Reports page contains "Generate report" on page.' do expect(last_response.body).to(include('Generate report')) end end diff --git a/spec/controllers/test_controller_spec.rb b/spec/controllers/test_controller_spec.rb index 8b74707..8fd8be0 100644 --- a/spec/controllers/test_controller_spec.rb +++ b/spec/controllers/test_controller_spec.rb @@ -11,6 +11,10 @@ RSpec.describe(TestController) do expect(last_response).to(be_redirect) end + it 'Test base route is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it 'Test base route Location header points to /test/list' do expect(last_response['Location']).to(eq("#{BASE_URL}/test/list")) end @@ -24,6 +28,10 @@ RSpec.describe(TestController) do expect(last_response).to(be_ok) end + it 'Test list page is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it "Test list page contains 'List of tests' on page." do expect(last_response.body).to(include('List of tests')) end @@ -37,6 +45,10 @@ RSpec.describe(TestController) do expect(last_response).to(be_ok) end + it 'Test add page is an HTML response' do + expect(last_response['Content-Type']).to(include('text/html')) + end + it "Test add page contains 'Add new test' on page." do expect(last_response.body).to(include('Add new test')) end