Add migration tests for user mail notification settings (#24277)
This commit is contained in:
		
							parent
							
								
									e084b5b82d
								
							
						
					
					
						commit
						0d70deee53
					
				@ -16,7 +16,7 @@ jobs:
 | 
				
			|||||||
      - id: skip_check
 | 
					      - id: skip_check
 | 
				
			||||||
        uses: fkirc/skip-duplicate-actions@v5
 | 
					        uses: fkirc/skip-duplicate-actions@v5
 | 
				
			||||||
        with:
 | 
					        with:
 | 
				
			||||||
          paths: '["Gemfile*", ".ruby-version", "**/*.rb", ".github/workflows/test-migrations-one-step.yml"]'
 | 
					          paths: '["Gemfile*", ".ruby-version", "**/*.rb", ".github/workflows/test-migrations-one-step.yml", "lib/tasks/tests.rake"]'
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  test:
 | 
					  test:
 | 
				
			||||||
    runs-on: ubuntu-latest
 | 
					    runs-on: ubuntu-latest
 | 
				
			||||||
 | 
				
			|||||||
@ -16,7 +16,7 @@ jobs:
 | 
				
			|||||||
      - id: skip_check
 | 
					      - id: skip_check
 | 
				
			||||||
        uses: fkirc/skip-duplicate-actions@v5
 | 
					        uses: fkirc/skip-duplicate-actions@v5
 | 
				
			||||||
        with:
 | 
					        with:
 | 
				
			||||||
          paths: '["Gemfile*", ".ruby-version", "**/*.rb", ".github/workflows/test-migrations-two-step.yml"]'
 | 
					          paths: '["Gemfile*", ".ruby-version", "**/*.rb", ".github/workflows/test-migrations-two-step.yml", "lib/tasks/tests.rake"]'
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  test:
 | 
					  test:
 | 
				
			||||||
    runs-on: ubuntu-latest
 | 
					    runs-on: ubuntu-latest
 | 
				
			||||||
 | 
				
			|||||||
@ -53,6 +53,11 @@ namespace :tests do
 | 
				
			|||||||
        puts 'Admin::ActionLog email domain block records not updated as expected'
 | 
					        puts 'Admin::ActionLog email domain block records not updated as expected'
 | 
				
			||||||
        exit(1)
 | 
					        exit(1)
 | 
				
			||||||
      end
 | 
					      end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					      unless User.find(1).settings.notification_emails['favourite'] == true && User.find(1).settings.notification_emails['mention'] == false
 | 
				
			||||||
 | 
					        puts 'User settings not kept as expected'
 | 
				
			||||||
 | 
					        exit(1)
 | 
				
			||||||
 | 
					      end
 | 
				
			||||||
    end
 | 
					    end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    desc 'Populate the database with test data for 2.4.3'
 | 
					    desc 'Populate the database with test data for 2.4.3'
 | 
				
			||||||
@ -98,6 +103,11 @@ namespace :tests do
 | 
				
			|||||||
          (1, 'destroy', 'EmailDomainBlock', 1, now(), now()),
 | 
					          (1, 'destroy', 'EmailDomainBlock', 1, now(), now()),
 | 
				
			||||||
          (1, 'destroy', 'Status', 1, now(), now()),
 | 
					          (1, 'destroy', 'Status', 1, now(), now()),
 | 
				
			||||||
          (1, 'destroy', 'CustomEmoji', 3, now(), now());
 | 
					          (1, 'destroy', 'CustomEmoji', 3, now(), now());
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        INSERT INTO "settings"
 | 
				
			||||||
 | 
					          (id, thing_type, thing_id, var, value, created_at, updated_at)
 | 
				
			||||||
 | 
					        VALUES
 | 
				
			||||||
 | 
					          (3, 'User', 1, 'notification_emails', E'--- !ruby/hash:ActiveSupport::HashWithIndifferentAccess\nfollow: false\nreblog: true\nfavourite: true\nmention: false\nfollow_request: true\ndigest: true\nreport: true\npending_account: false\ntrending_tag: true\nappeal: true\n', now(), now());
 | 
				
			||||||
      SQL
 | 
					      SQL
 | 
				
			||||||
    end
 | 
					    end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user