FIX: Improve avatar loading, and add tests

Follow-up from 4e2cc9c

spec/components/auth/managed_authenticator_spec.rb

@@ -114,6 +114,40 @@ describe Auth::ManagedAuthenticator do
         expect(result.user.id).to eq(user.id)
       end
     end
+
+    describe "avatar on update" do
+      let(:user) { Fabricate(:user) }
+      let!(:associated) { UserAssociatedAccount.create!(user: user, provider_name: 'myauth', provider_uid: "1234") }
+
+      it "schedules the job upon update correctly" do
+        # No image supplied, do not schedule
+        expect { result = authenticator.after_authenticate(hash) }
+          .to change { Jobs::DownloadAvatarFromUrl.jobs.count }.by(0)
+
+        # Image supplied, schedule
+        expect { result = authenticator.after_authenticate(hash.deep_merge(info: { image: "https://some.domain/image.jpg" })) }
+          .to change { Jobs::DownloadAvatarFromUrl.jobs.count }.by(1)
+
+        # User already has profile picture, don't schedule
+        user.user_avatar = Fabricate(:user_avatar, custom_upload: Fabricate(:upload))
+        user.save!
+        expect { result = authenticator.after_authenticate(hash.deep_merge(info: { image: "https://some.domain/image.jpg" })) }
+          .to change { Jobs::DownloadAvatarFromUrl.jobs.count }.by(0)
+      end
+    end
+
+    describe "avatar on create" do
+      let(:user) { Fabricate(:user) }
+      it "doesn't schedule with no image" do
+        expect { result = authenticator.after_create_account(user, extra_data: hash) }
+          .to change { Jobs::DownloadAvatarFromUrl.jobs.count }.by(0)
+      end
+
+      it "schedules with image" do
+        expect { result = authenticator.after_create_account(user, extra_data: hash.deep_merge(info: { image: "https://some.domain/image.jpg" })) }
+          .to change { Jobs::DownloadAvatarFromUrl.jobs.count }.by(1)
+      end
+    end
   end
 
   describe 'description_for_user' do