diff --git a/config/locales/en.yml b/config/locales/en.yml
index 69e293e..e423344 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -5,7 +5,12 @@ en:
log_out: Log out
sign_up: Sign up
not_authenticated: You need to log in to access this page.
+ edit: Edit
+ delete: Delete
layouts:
+ admin:
+ dashboard: Dashboard
+ game_systems: Game Systems
mailer:
greeting: "Hello, %{name}"
sign_off: |
@@ -18,7 +23,30 @@ en:
error: "Invalid token, could not verify your account."
admin:
index:
+ dashboard: Dashboard
intro: With great power comes great responsibility
+ game_systems:
+ error: Error
+ index:
+ game_systems: Game systems
+ new_game_system: New game system
+ new:
+ new: New game system
+ create: Add game system
+ create:
+ success: “%{name}” has been created.
+ error: “%{name}” could not be created.
+ show:
+ confirm_delete: Are you sure you want to delete “%{name}”?
+ edit:
+ edit: Edit %{name}
+ update: Update %{name}
+ update:
+ success: Successfully updated “%{name}”.
+ error: “%{name}” could not be updated.
+ destroy:
+ success: Successfully deleted “%{name}”.
+ error: “%{name}” could not be deleted.
sessions:
create:
success: "Hello, %{name}!"
diff --git a/config/routes.rb b/config/routes.rb
index dad18fd..e28f5f1 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -13,6 +13,9 @@ Rails.application.routes.draw do
resources :tables, only: [ :index ]
resources :admin, only: [ :index ]
+ namespace :admin do
+ resources :game_systems
+ end
get "up" => "rails/health#show", as: :rails_health_check
end
diff --git a/db/migrate/20240526102908_add_unique_index_to_game_system_name.rb b/db/migrate/20240526102908_add_unique_index_to_game_system_name.rb
new file mode 100644
index 0000000..a86ad43
--- /dev/null
+++ b/db/migrate/20240526102908_add_unique_index_to_game_system_name.rb
@@ -0,0 +1,7 @@
+class AddUniqueIndexToGameSystemName < ActiveRecord::Migration[7.1]
+ def change
+ add_index :game_systems, :name, unique: true
+
+ add_check_constraint :game_systems, "length(name) <= 100", name: "chk_name_max_length"
+ end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 1b3dfdf..7beccff 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -10,7 +10,7 @@
#
# It's strongly recommended that you check this file into your version control system.
-ActiveRecord::Schema[7.1].define(version: 2024_05_26_085315) do
+ActiveRecord::Schema[7.1].define(version: 2024_05_26_102908) do
# These are extensions that must be enabled in order to support this database
enable_extension "plpgsql"
@@ -18,6 +18,8 @@ ActiveRecord::Schema[7.1].define(version: 2024_05_26_085315) do
t.string "name", null: false
t.datetime "created_at", null: false
t.datetime "updated_at", null: false
+ t.index ["name"], name: "index_game_systems_on_name", unique: true
+ t.check_constraint "length(name::text) <= 100", name: "chk_name_max_length"
end
create_table "site_roles", force: :cascade do |t|
diff --git a/test/controllers/admin/game_systems_controller_test.rb b/test/controllers/admin/game_systems_controller_test.rb
new file mode 100644
index 0000000..e9c3ef7
--- /dev/null
+++ b/test/controllers/admin/game_systems_controller_test.rb
@@ -0,0 +1,20 @@
+require "test_helper"
+
+class AdminGameSystemsControllerTest < ActionDispatch::IntegrationTest
+ test "should get game systems index if signed in as admin" do
+ sign_in users(:admin)
+ get admin_game_systems_path
+ assert_response :success
+ end
+
+ test "should not get game systems index if signed in as non-admin user" do
+ sign_in users(:trevor)
+ get admin_game_systems_path
+ assert_response :forbidden
+ end
+
+ test "should not get game systems index if not signed in" do
+ get admin_game_systems_path
+ assert_redirected_to login_path
+ end
+end
diff --git a/test/integration/admin_game_systems_integration_test.rb b/test/integration/admin_game_systems_integration_test.rb
new file mode 100644
index 0000000..a0f7885
--- /dev/null
+++ b/test/integration/admin_game_systems_integration_test.rb
@@ -0,0 +1,64 @@
+require "test_helper"
+
+class AdminGameSystemsIntegrationTest < ActionDispatch::IntegrationTest
+ test "index should show most all game systems" do
+ sign_in users(:admin)
+ get admin_game_systems_path
+ assert_response :success
+
+ assert_select ".game-system", GameSystem.all.size
+ end
+
+ test "show should show game system" do
+ sign_in users(:admin)
+ get admin_game_system_path(game_systems(:troika))
+ assert_response :success
+ assert_select "h2", game_systems(:troika).name
+ end
+
+ test "new should show form to create new game system" do
+ sign_in users(:admin)
+ get new_admin_game_system_path
+ assert_response :success
+
+ assert_select "form[action=?][method=?]", admin_game_systems_path, "post"
+ end
+
+ test "create should create new game system" do
+ sign_in users(:admin)
+ assert_difference "GameSystem.count", 1 do
+ post admin_game_systems_path, params: { game_system: { name: "Test Game System" } }
+ end
+ assert_redirected_to admin_game_system_path(GameSystem.last)
+ end
+
+ test "create should fail if game system already exists" do
+ sign_in users(:admin)
+ assert_no_difference "GameSystem.count" do
+ post admin_game_systems_path, params: { game_system: { name: game_systems(:troika).name } }
+ end
+ assert_response :unprocessable_entity
+ end
+
+ test "edit should show form to edit game system" do
+ sign_in users(:admin)
+ get edit_admin_game_system_path(game_systems(:troika))
+ assert_response :success
+
+ assert_select "form[action=?][method=?]", admin_game_system_path(game_systems(:troika)), "post"
+ end
+
+ test "update should update game system" do
+ sign_in users(:admin)
+ patch admin_game_system_path(game_systems(:troika)), params: { game_system: { name: "Test Game System" } }
+ assert_redirected_to admin_game_system_path(game_systems(:troika))
+ end
+
+ test "destroy should destroy game system" do
+ sign_in users(:admin)
+ assert_difference "GameSystem.count", -1 do
+ delete admin_game_system_path(game_systems(:troika))
+ end
+ assert_redirected_to admin_game_systems_path
+ end
+end