Skip to content

Commit

Permalink
Cleanup test
Browse files Browse the repository at this point in the history
secondary_terms test wasn't even testing secondary_terms.
less derp, more good.
  • Loading branch information
atz committed May 27, 2016
1 parent 42d73f5 commit 0fbed3b
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions spec/forms/sufia/batch_upload_form_spec.rb
Expand Up @@ -7,24 +7,20 @@
describe "#primary_terms" do
subject { form.primary_terms }
it { is_expected.to eq [:creator, :keyword, :rights] }
it { is_expected.not_to include(:title) }
end

describe "#secondary_terms" do
subject { form.primary_terms }
it "doesn't have title" do
expect(subject).not_to include(:title)
end
subject { form.secondary_terms }
it { is_expected.not_to include(:title) } # title is per file, not per form
end

describe ".model_name" do
subject { described_class.model_name }
it "has a route_key" do
expect(subject.route_key).to eq 'batch_uploads'
end

it "has a param_key" do
derp = subject
derp.param_key
expect(subject.param_key).to eq 'generic_work'
end
end
Expand Down

0 comments on commit 0fbed3b

Please sign in to comment.