Change default column value to 0 on user stats

Gemfile.lock

@@ -585,4 +585,4 @@ RUBY VERSION
    ruby 3.3.5p100
 
 BUNDLED WITH
-   2.5.18
+   2.5.23

db/migrate/20241128052830_fix_users_table_defaults.rb

@@ -0,0 +1,10 @@
+class FixUsersTableDefaults < ActiveRecord::Migration[7.2]
+  def change
+    change_column_default :users, :num_machines_added, default: 0
+    change_column_default :users, :num_machines_removed, default: 0
+    change_column_default :users, :num_locations_suggested, default: 0
+    change_column_default :users, :num_lmx_comments_left, default: 0
+    change_column_default :users, :num_msx_scores_added, default: 0
+    change_column_default :users, :user_submissions_count, default: 0
+  end
+end

db/migrate/20241128060236_fix_users_table_defaults_again.rb

@@ -0,0 +1,16 @@
+class FixUsersTableDefaultsAgain < ActiveRecord::Migration[7.2]
+  def change
+    change_column_null :users, :num_machines_added, false
+    change_column_null :users, :num_machines_removed, false
+    change_column_null :users, :num_locations_suggested, false
+    change_column_null :users, :num_lmx_comments_left, false
+    change_column_null :users, :num_msx_scores_added, false
+    change_column_null :users, :user_submissions_count, false
+    change_column_default :users, :num_machines_added, from: nil, to: 0
+    change_column_default :users, :num_machines_removed, from: nil, to: 0
+    change_column_default :users, :num_locations_suggested, from: nil, to: 0
+    change_column_default :users, :num_lmx_comments_left, from: nil, to: 0
+    change_column_default :users, :num_msx_scores_added, from: nil, to: 0
+    change_column_default :users, :user_submissions_count, from: nil, to: 0
+  end
+end

db/structure.sql

@@ -926,13 +926,13 @@ CREATE TABLE public.users (
     is_disabled boolean,
     authentication_token character varying(30),
     security_test character varying,
-    user_submissions_count integer,
+    user_submissions_count integer DEFAULT 0 NOT NULL,
     flag character varying,
-    num_machines_added integer,
-    num_machines_removed integer,
-    num_locations_suggested integer,
-    num_lmx_comments_left integer,
-    num_msx_scores_added integer,
+    num_machines_added integer DEFAULT 0 NOT NULL,
+    num_machines_removed integer DEFAULT 0 NOT NULL,
+    num_locations_suggested integer DEFAULT 0 NOT NULL,
+    num_lmx_comments_left integer DEFAULT 0 NOT NULL,
+    num_msx_scores_added integer DEFAULT 0 NOT NULL,
     contributor_rank character varying,
     admin_title character varying
 );
@@ -1795,6 +1795,8 @@ ALTER TABLE ONLY public.active_storage_attachments
 SET search_path TO "$user", public;
 
 INSERT INTO "schema_migrations" (version) VALUES
+('20241128060236'),
+('20241128052830'),
 ('20241127215612'),
 ('20241127054132'),
 ('20241127051246'),

spec/features/users_spec.rb

@@ -94,6 +94,16 @@
       expect(page).to_not have_content('Saved Locations:')
     end
 
+    it 'returns zeros when appropriate' do
+      visit "/users/#{@user.id}/profile"
+
+      expect(page).to have_content("0\nTOTAL CONTRIBUTIONS")
+      expect(page).to have_content("0\nMACHINES ADDED")
+      expect(page).to have_content("0\nMACHINES REMOVED")
+      expect(page).to have_content("0\nMACHINE COMMENTS")
+      expect(page).to have_content("0\nLOCATIONS SUBMITTED")
+    end
+
     it 'adds commas to high scores' do
       FactoryBot.create(:user_submission, user: @user, location: FactoryBot.create(:location, id: 500, name: 'Location One'), machine: FactoryBot.create(:machine, name: 'Machine One'), submission_type: UserSubmission::NEW_SCORE_TYPE, submission: 'ssw added a high score of 1000000 on Machine One at Location One', created_at: '2016-01-02')