Merge pull request #65 from sandbergja/distinct_employers

Don't duplicate employers in the employer listings, helps with #63

app/models/employer.rb

@@ -9,7 +9,7 @@ class Employer < ApplicationRecord
 
   default_scope { order(name: :asc) }
 
-  scope :with_published_jobs, -> { joins(:jobs).where(jobs: { published: true }) }
+  scope :with_published_jobs, -> { joins(:jobs).where(jobs: { published: true }).distinct }
 
   validates :name, presence: true
 

spec/models/employer_spec.rb

@@ -1,5 +1,9 @@
 require 'rails_helper'
 
 RSpec.describe Employer, type: :model do
-  pending "add some examples to (or delete) #{__FILE__}"
+  it 'uses distinct when listing employers' do
+    employer = described_class.create name: 'The Library'
+    (1..5).each { |job| Job.create employer: employer, title: job, description: job, job_type: job, published: true }
+    expect(described_class.with_published_jobs.count).to eq(1)
+  end
 end