From f9655d3c62fc6c59e8646306ca8d22b4706d144b Mon Sep 17 00:00:00 2001
From: Claire <claire.github-309c@sitedethib.com>
Date: Fri, 28 Aug 2026 11:49:30 +0000
Subject: [PATCH] Add some spec coverage (#40306)

---
 spec/lib/private_address_check_spec.rb        |  2 +-
 .../process_activity_service_spec.rb          | 40 +++++++++++++++++++
 .../email_address_validator_spec.rb           | 25 ++++++++++++
 3 files changed, 66 insertions(+), 1 deletion(-)
 create mode 100644 spec/validators/email_address_validator_spec.rb

diff --git a/spec/lib/private_address_check_spec.rb b/spec/lib/private_address_check_spec.rb
index 20a94983d9e..0c02055187e 100644
--- a/spec/lib/private_address_check_spec.rb
+++ b/spec/lib/private_address_check_spec.rb
@@ -4,7 +4,7 @@ require 'rails_helper'
 
 RSpec.describe PrivateAddressCheck do
   describe 'private_address?' do
-    let(:private_ips) { %w(192.168.1.7 0.0.0.0 127.0.0.1 ::ffff:0.0.0.1) }
+    let(:private_ips) { %w(192.168.1.7 0.0.0.0 127.0.0.1 ::ffff:0.0.0.1 ::127.0.0.1 ::ffff:127.0.0.1 ::ffff:10.0.0.1 ::ffff:169.254.169.254 ::) }
 
     it 'returns true for private addresses' do
       expect(private_ips)
diff --git a/spec/services/activitypub/process_activity_service_spec.rb b/spec/services/activitypub/process_activity_service_spec.rb
index 290058efd7a..e716571ecad 100644
--- a/spec/services/activitypub/process_activity_service_spec.rb
+++ b/spec/services/activitypub/process_activity_service_spec.rb
@@ -162,6 +162,46 @@ RSpec.describe ActivityPub::ProcessActivityService do
         expect(ActivityPub::Activity).to_not have_received(:factory)
       end
 
+      context 'when receiving a status with an unsupported JSON-LD keyword' do
+        let(:payload) do
+          {
+            '@context': 'https://www.w3.org/ns/activitystreams',
+            id: 'foo',
+            type: 'Announce',
+            actor: ActivityPub::TagManager.instance.uri_for(actor),
+            '@reverse': {
+              object: {
+                type: 'Undo',
+                id: 'bar',
+                actor: ActivityPub::TagManager.instance.uri_for(actor),
+              },
+            },
+            object: {
+              id: 'bar',
+              type: 'Note',
+              content: 'Lorem ipsum',
+            },
+            signature: {
+              type: 'RsaSignature2017',
+              creator: "#{ActivityPub::TagManager.instance.uri_for(actor)}#foo",
+              created: '2022-03-09T21:57:25Z',
+              signatureValue: 'foo',
+            },
+          }
+        end
+
+        it 'does not process payload' do
+          signature_double = instance_double(ActivityPub::LinkedDataSignature, verify_actor!: nil)
+          allow(ActivityPub::LinkedDataSignature).to receive(:new).and_return(signature_double)
+          allow(ActivityPub::Activity).to receive(:factory)
+
+          subject.call(json, forwarder)
+
+          expect(signature_double).to_not have_received(:verify_actor!)
+          expect(ActivityPub::Activity).to_not have_received(:factory)
+        end
+      end
+
       context 'when receiving a fabricated status' do
         let!(:actor) do
           Fabricate(:account,
diff --git a/spec/validators/email_address_validator_spec.rb b/spec/validators/email_address_validator_spec.rb
new file mode 100644
index 00000000000..17398269099
--- /dev/null
+++ b/spec/validators/email_address_validator_spec.rb
@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe EmailAddressValidator do
+  subject { record_class.new }
+
+  context 'with no options' do
+    let(:record_class) do
+      Class.new do
+        include ActiveModel::Validations
+
+        def self.name = 'Record'
+
+        attr_accessor :email
+
+        validates :email, email_address: true
+      end
+    end
+
+    it { is_expected.to allow_value('foo@example.com').for(:email) }
+    it { is_expected.to_not allow_value('foo @ example.com').for(:email) }
+    it { is_expected.to_not allow_value('foo@example.com%foo.example.com').for(:email) }
+  end
+end