govuk_content_models 42.0.0 → 42.0.1

data/test/models/simple_smart_answer_node_test.rb

@@ -1,134 +0,0 @@
-require "test_helper"
-
-class SimpleSmartAnswerNodeTest < ActiveSupport::TestCase
-
-  context "given a smart answer exists" do
-    setup do
-      @edition = FactoryGirl.create(:simple_smart_answer_edition)
-
-      @atts = {
-        title: "How much wood could a woodchuck chuck if a woodchuck could chuck wood?",
-        slug: "how-much-wood-could-a-woodchuck-chuck-if-a-woodchuck-could-chuck-wood",
-        body: "This is a serious question.",
-        kind: "question"
-      }
-    end
-
-    should "be able to create a valid node" do
-      @node = @edition.nodes.build(@atts)
-
-      assert @node.save!
-
-      @edition.reload
-
-      assert_equal "how-much-wood-could-a-woodchuck-chuck-if-a-woodchuck-could-chuck-wood", @edition.nodes.first.slug
-      assert_equal "How much wood could a woodchuck chuck if a woodchuck could chuck wood?", @edition.nodes.first.title
-      assert_equal "This is a serious question.", @edition.nodes.first.body
-    end
-
-    should "not be valid without a slug" do
-      @node = @edition.nodes.build( @atts.merge(slug: "") )
-
-      assert ! @node.valid?
-      assert_equal [:slug], @node.errors.keys
-    end
-
-    should "not be valid with an invalid slug" do
-      @node = @edition.nodes.build(@atts)
-
-      [
-        'under_score',
-        'space space',
-        'punct.u&ation',
-      ].each do |slug|
-        @node.slug = slug
-        refute @node.valid?
-      end
-    end
-
-    should "not be valid without a title" do
-      @node = @edition.nodes.build( @atts.merge(title: "") )
-
-      assert ! @node.valid?
-      assert_equal [:title], @node.errors.keys
-    end
-
-    should "not be valid without a kind" do
-      @node = @edition.nodes.build(@atts.merge(:kind => nil))
-      assert ! @node.valid?
-
-      assert_equal [:kind], @node.errors.keys
-    end
-
-    should "not be valid with a kind other than 'question' or 'outcome'" do
-      @node = @edition.nodes.build(@atts.merge(:kind => 'blah'))
-      assert ! @node.valid?
-
-      assert_equal [:kind], @node.errors.keys
-    end
-
-    should "create options using nested attributes" do
-      @node = @edition.nodes.create!(@atts.merge(:options_attributes => [
-        { :label => "Yes", :next_node => "yes" },
-        { :label => "No", :next_node => "no" }
-      ]))
-
-      @node.reload
-      assert_equal 2, @node.options.count
-      assert_equal ["Yes", "No"], @node.options.all.map(&:label)
-      assert_equal ["yes", "no"], @node.options.all.map(&:next_node)
-    end
-
-    should "destroy options using nested attributes" do
-      @node = @edition.nodes.create!(@atts.merge(:options_attributes => [
-        { :label => "Yes", :next_node => "yes" },
-        { :label => "No", :next_node => "no" }
-      ]))
-      assert_equal 2, @node.options.count
-
-      @node.update_attributes!(:options_attributes => {
-        "1" => { "id" => @node.options.first.id, "_destroy" => "1" }
-      })
-      @node.reload
-
-      assert_equal 1, @node.options.count
-      assert_equal ["No"], @node.options.all.map(&:label)
-      assert_equal ["no"], @node.options.all.map(&:next_node)
-    end
-
-    should "not be valid if an outcome has options" do
-      @node = @edition.nodes.build(@atts.merge(:kind => 'outcome', options_attributes: [
-        { :label => "Yes", :next_node => "yes" },
-        { :label => "No", :next_node => "no" }
-      ]))
-      assert ! @node.valid?
-
-      assert_equal [:options], @node.errors.keys
-    end
-
-    should "be able to create an outcome without options" do
-      @node = @edition.nodes.build(@atts.merge(:kind => 'outcome', :options_attributes => [] ))
-
-      assert @node.valid?
-      assert @node.save!
-    end
-
-    should "be returned in order" do
-      @nodes = [
-        @edition.nodes.create(@atts.merge(:title => "Third", :order => 3)),
-        @edition.nodes.create(@atts.merge(:title => "First", :order => 1)),
-        @edition.nodes.create(@atts.merge(:title => "Second", :order => 2)),
-      ]
-
-      assert_equal ["First","Second","Third"], @edition.nodes.all.map(&:title)
-    end
-
-    should "expose the simple smart answer edition" do
-      @node = @edition.nodes.build(@atts)
-
-      assert_equal @node.edition, @edition
-    end
-
-  end
-
-end

data/test/models/simple_smart_answer_option_test.rb

@@ -1,97 +0,0 @@
-require "test_helper"
-
-class SimpleSmartAnswerOptionTest < ActiveSupport::TestCase
-
-  context "given a smart answer exists with a node" do
-    setup do
-      @node = SimpleSmartAnswerEdition::Node.new(:slug => "question1", :title => "Question One?", :kind => "question")
-      @edition = FactoryGirl.create(:simple_smart_answer_edition, :nodes => [
-        @node,
-        SimpleSmartAnswerEdition::Node.new(:slug => "outcome1", :title => "Outcome One", :kind => "outcome")
-      ])
-
-      @atts = {
-        label: "Yes",
-        next_node: "yes"
-      }
-    end
-
-    should "be able to create a valid option" do
-      @option = @node.options.build(@atts)
-
-      assert @option.save!
-      @node.reload
-
-      assert_equal "Yes", @node.options.first.label
-      assert_equal "yes", @node.options.first.next_node
-    end
-
-    should "not be valid without a label" do
-      @option = @node.options.build(@atts.merge(label: nil))
-
-      assert !@option.valid?
-      assert @option.errors.keys.include?(:label)
-    end
-
-    should "not be valid without the next node" do
-      @option = @node.options.build(@atts.merge(next_node: nil))
-
-      assert !@option.valid?
-      assert @option.errors.keys.include?(:next_node)
-    end
-
-    should "expose the node" do
-      @option = @node.options.create(@atts)
-      @option.reload
-
-      assert_equal @node, @option.node
-    end
-
-    should "return in order" do
-      @options = [
-        @node.options.create(@atts.merge(:label => "Third", :next_node => "baz", :order => 3)),
-        @node.options.create(@atts.merge(:label => "First", :next_node => "foo", :order => 1)),
-        @node.options.create(@atts.merge(:label => "Second", :next_node => "bar", :order => 2)),
-      ]
-
-      assert_equal ["First","Second","Third"], @node.options.all.map(&:label)
-      assert_equal ["foo","bar","baz"], @node.options.all.map(&:next_node)
-    end
-
-    context "slug" do
-      should "generate a slug from the label if blank" do
-        @option = @node.options.build(@atts)
-
-        assert @option.valid?
-        assert_equal "yes", @option.slug
-      end
-
-      should "keep the slug up to date if the label changes" do
-        @option = @node.options.create(@atts.merge(slug: "most-likely"))
-        @option.label = "Most of the times"
-        assert @option.valid?
-        assert_equal "most-of-the-times", @option.slug
-      end
-
-      should "not overwrite a given slug" do
-        @option = @node.options.build(@atts.merge(:slug => "fooey"))
-
-        assert @option.valid?
-        assert_equal "fooey", @option.slug
-      end
-
-      should "not be valid with an invalid slug" do
-        @option = @node.options.build(@atts)
-
-        [
-          'under_score',
-          'space space',
-          'punct.u&ation',
-        ].each do |slug|
-          @option.slug = slug
-          refute @option.valid?
-        end
-      end
-    end
-  end
-end

data/test/models/time_zone_test.rb

@@ -1,48 +0,0 @@
-require 'test_helper'
-
-class TimeZoneTest < ActiveSupport::TestCase
-  def first_day_of_summer_time
-    Time.zone.parse("2013-04-01")
-  end
-
-  def wintertime
-    Time.zone.parse("2013-01-01")
-  end
-
-  context "use_activesupport_time_zone is set to true, Time.zone is set to 'London'" do
-    setup do
-      # This context has already been set in the local mongoid.yml, and in test_helper.rb
-    end
-
-    should "still store the date in UTC" do
-      Timecop.freeze(wintertime) do
-        FactoryGirl.create(:artefact)
-        assert_equal 'UTC', Artefact.last[:created_at].zone
-        assert_equal 'GMT', Artefact.last.created_at.zone
-      end
-    end
-
-    should "use the Time.zone time zone for dot-methods" do
-      Timecop.freeze(wintertime) do
-        FactoryGirl.create(:artefact)
-        assert_equal 'GMT', Artefact.last.created_at.zone
-      end
-    end
-
-    context "it is currently British Summer Time" do
-      should "still store the date in UTC" do
-        Timecop.freeze(first_day_of_summer_time) do
-          FactoryGirl.create(:artefact)
-          assert_equal 'UTC', Artefact.last[:created_at].zone
-        end
-      end
-
-      should "use the time zone with offset for dot-methods" do
-        Timecop.freeze(first_day_of_summer_time) do
-          FactoryGirl.create(:artefact)
-          assert_equal 'BST', Artefact.last.created_at.zone
-        end
-      end
-    end
-  end
-end

data/test/models/transaction_edition_test.rb

@@ -1,36 +0,0 @@
-require 'test_helper'
-
-class TransactionEditionTest < ActiveSupport::TestCase
-
-  setup do
-    @artefact = FactoryGirl.create(:artefact)
-  end
-
-  context 'Department analytics profiles' do
-    should "only allow valid Google Analytics profiles" do
-      transaction = FactoryGirl.create(:transaction_edition, panopticon_id: @artefact.id)
-
-      ['invalid', 'ua-12345', 'UA-1234A-1'].each do |id|
-        transaction.department_analytics_profile = id
-        refute transaction.valid?
-      end
-
-      ['ua-123456-1', 'UA-00-10'].each do |id|
-        transaction.department_analytics_profile = id
-        assert transaction.valid?
-      end
-    end
-  end
-
-  context "indexable_content" do
-    should "include the introduction without markup" do
-      transaction = FactoryGirl.create(:transaction_edition, introduction: "## introduction", more_information: "", panopticon_id: @artefact.id)
-      assert_equal "introduction", transaction.indexable_content
-    end
-
-    should "include the more_information without markup" do
-      transaction = FactoryGirl.create(:transaction_edition, more_information: "## more info", introduction: "", panopticon_id: @artefact.id)
-      assert_equal "more info", transaction.indexable_content
-    end
-  end
-end

data/test/models/travel_advice_edition_test.rb

@@ -1,469 +0,0 @@
-# encoding: UTF-8
-
-require "test_helper"
-
-class TravelAdviceEditionTest < ActiveSupport::TestCase
-
-  should "have correct fields" do
-    ed = TravelAdviceEdition.new
-    ed.title = "Travel advice for Aruba"
-    ed.overview = "This gives travel advice for Aruba"
-    ed.country_slug = 'aruba'
-    ed.alert_status = [ 'avoid_all_but_essential_travel_to_parts', 'avoid_all_travel_to_parts' ]
-    ed.summary = "This is the summary of stuff going on in Aruba"
-    ed.version_number = 4
-    ed.image_id = "id_from_the_asset_manager_for_an_image"
-    ed.document_id = "id_from_the_asset_manager_for_a_document"
-    ed.published_at = Time.zone.parse('2013-02-21T14:56:22Z')
-    ed.minor_update = true
-    ed.change_description = "Some things"
-    ed.synonyms = ["Foo", "Bar"]
-    ed.parts.build(:title => "Part One", :slug => "one")
-    ed.save!
-
-    ed = TravelAdviceEdition.first
-    assert_equal "Travel advice for Aruba", ed.title
-    assert_equal "This gives travel advice for Aruba", ed.overview
-    assert_equal 'aruba', ed.country_slug
-    assert_equal [ 'avoid_all_but_essential_travel_to_parts', 'avoid_all_travel_to_parts' ], ed.alert_status
-    assert_equal "This is the summary of stuff going on in Aruba", ed.summary
-    assert_equal 4, ed.version_number
-    assert_equal "id_from_the_asset_manager_for_an_image", ed.image_id
-    assert_equal "id_from_the_asset_manager_for_a_document", ed.document_id
-    assert_equal Time.zone.parse('2013-02-21T14:56:22Z'), ed.published_at
-    assert_equal true, ed.minor_update
-    assert_equal ["Foo", "Bar"], ed.synonyms
-    assert_equal "Some things", ed.change_description
-    assert_equal "Part One", ed.parts.first.title
-  end
-
-  context "validations" do
-    setup do
-      @ta = FactoryGirl.build(:travel_advice_edition)
-    end
-
-    should "require a country slug" do
-      @ta.country_slug = ''
-      assert ! @ta.valid?
-      assert_includes @ta.errors.messages[:country_slug], "can't be blank"
-    end
-
-    should "require a title" do
-      @ta.title = ''
-      assert ! @ta.valid?
-      assert_includes @ta.errors.messages[:title], "can't be blank"
-    end
-
-    context "on state" do
-      should "only allow one edition in draft per slug" do
-        another_edition = FactoryGirl.create(:travel_advice_edition,
-                                             :country_slug => @ta.country_slug)
-        @ta.state = 'draft'
-        assert ! @ta.valid?
-        assert_includes @ta.errors.messages[:state], "is already taken"
-      end
-
-      should "only allow one edition in published per slug" do
-        another_edition = FactoryGirl.create(:published_travel_advice_edition,
-                                             :country_slug => @ta.country_slug)
-        @ta.state = 'published'
-        assert ! @ta.valid?
-        assert_includes @ta.errors.messages[:state], "is already taken"
-      end
-
-      should "allow multiple editions in archived per slug" do
-        another_edition = FactoryGirl.create(:archived_travel_advice_edition,
-                                             :country_slug => @ta.country_slug)
-        @ta.save!
-        @ta.state = 'archived'
-        assert @ta.valid?
-      end
-
-      should "not conflict with itself when validating uniqueness" do
-        @ta.state = 'draft'
-        @ta.save!
-        assert @ta.valid?
-      end
-    end
-
-    context "preventing editing of non-draft" do
-      should "not allow published editions to be edited" do
-        ta = FactoryGirl.create(:published_travel_advice_edition)
-        ta.title = "Fooey"
-        assert ! ta.valid?
-        assert_includes ta.errors.messages[:state], "must be draft to modify"
-      end
-
-      should "not allow archived editions to be edited" do
-        ta = FactoryGirl.create(:archived_travel_advice_edition)
-        ta.title = "Fooey"
-        assert ! ta.valid?
-        assert_includes ta.errors.messages[:state], "must be draft to modify"
-      end
-
-      should "allow publishing draft editions" do
-        ta = FactoryGirl.create(:travel_advice_edition)
-        assert ta.publish
-      end
-
-      should "allow 'save & publish'" do
-        ta = FactoryGirl.create(:travel_advice_edition)
-        ta.title = 'Foo'
-        assert ta.publish
-      end
-
-      should "allow archiving published editions" do
-        ta = FactoryGirl.create(:published_travel_advice_edition)
-        assert ta.archive
-      end
-
-      should "NOT allow 'save & archive'" do
-        ta = FactoryGirl.create(:published_travel_advice_edition)
-        ta.title = 'Foo'
-        assert ! ta.archive
-        assert_includes ta.errors.messages[:state], "must be draft to modify"
-      end
-    end
-
-    context "on alert status" do
-      should "not permit invalid values in the array" do
-        @ta.alert_status = [ 'avoid_all_but_essential_travel_to_parts', 'something_else', 'blah' ]
-        assert ! @ta.valid?
-        assert_includes @ta.errors.messages[:alert_status], "is not in the list"
-      end
-
-      should "permit an empty array" do
-        @ta.alert_status = [ ]
-        assert @ta.valid?
-      end
-
-      # Test that accessing an Array field doesn't mark it as dirty.
-      # mongoid/dirty#changes method is patched in lib/mongoid/monkey_patches.rb
-      # See https://github.com/mongoid/mongoid/issues/2311 for details.
-      should "track changes to alert status accurately" do
-        @ta.alert_status = [ ]
-        @ta.alert_status
-        assert @ta.valid?
-      end
-    end
-
-    context "on version_number" do
-      should "require a version_number" do
-        @ta.save # version_number is automatically populated on create, so save it first.
-        @ta.version_number = ''
-        refute @ta.valid?
-        assert_includes @ta.errors.messages[:version_number], "can't be blank"
-      end
-
-      should "require a unique version_number per slug" do
-        another_edition = FactoryGirl.create(:archived_travel_advice_edition,
-                                             :country_slug => @ta.country_slug,
-                                             :version_number => 3)
-        @ta.version_number = 3
-        refute @ta.valid?
-        assert_includes @ta.errors.messages[:version_number], "is already taken"
-      end
-
-      should "allow matching version_numbers for different slugs" do
-        another_edition = FactoryGirl.create(:archived_travel_advice_edition,
-                                             :country_slug => 'wibble',
-                                             :version_number => 3)
-        @ta.version_number = 3
-        assert @ta.valid?
-      end
-    end
-
-    context "on minor update" do
-      should "not allow first version to be minor update" do
-        @ta.minor_update = true
-        refute @ta.valid?
-        assert_includes @ta.errors.messages[:minor_update], "can't be set for first version"
-      end
-
-      should "allow other versions to be minor updates" do
-        FactoryGirl.create(:published_travel_advice_edition, :country_slug => @ta.country_slug)
-        @ta.minor_update = true
-        assert @ta.valid?
-      end
-    end
-
-    context "on change_description" do
-      should "be required on publish" do
-        @ta.save! # Can't save directly as published, have to save as draft first
-        @ta.change_description = ""
-        @ta.state = "published"
-        refute @ta.valid?
-        assert_includes @ta.errors.messages[:change_description], "can't be blank on publish"
-      end
-
-      should "not be required on publish for a minor update" do
-        FactoryGirl.create(:archived_travel_advice_edition, :country_slug => @ta.country_slug)
-        @ta.version_number = 2 # version one can't be minor update
-        @ta.save! # Can't save directly as published, have to save as draft first
-        @ta.change_description = ""
-        @ta.minor_update = true
-        @ta.state = "published"
-        assert @ta.valid?
-      end
-
-      should "not be required when just saving a draft" do
-        @ta.change_description = ""
-        assert @ta.valid?
-      end
-    end
-  end
-
-  should "have a published scope" do
-    e1 = FactoryGirl.create(:draft_travel_advice_edition)
-    e2 = FactoryGirl.create(:published_travel_advice_edition)
-    e3 = FactoryGirl.create(:archived_travel_advice_edition)
-    e4 = FactoryGirl.create(:published_travel_advice_edition)
-
-    assert_equal [e2, e4].sort, TravelAdviceEdition.published.to_a.sort
-  end
-
-  context "fields on a new edition" do
-    should "be in draft state" do
-      assert TravelAdviceEdition.new.draft?
-    end
-
-    context "populating version_number" do
-      should "set version_number to 1 if there are no existing versions for the country" do
-        ed = TravelAdviceEdition.new(:country_slug => 'foo')
-        ed.valid?
-        assert_equal 1, ed.version_number
-      end
-
-      should "set version_number to the next available version" do
-        FactoryGirl.create(:archived_travel_advice_edition, :country_slug => 'foo', :version_number => 1)
-        FactoryGirl.create(:archived_travel_advice_edition, :country_slug => 'foo', :version_number => 2)
-        FactoryGirl.create(:published_travel_advice_edition, :country_slug => 'foo', :version_number => 4)
-
-        ed = TravelAdviceEdition.new(:country_slug => 'foo')
-        ed.valid?
-        assert_equal 5, ed.version_number
-      end
-
-      should "do nothing if version_number is already set" do
-        ed = TravelAdviceEdition.new(:country_slug => 'foo', :version_number => 42)
-        ed.valid?
-        assert_equal 42, ed.version_number
-      end
-
-      should "do nothing if country_slug is not set" do
-        ed = TravelAdviceEdition.new(:country_slug => '')
-        ed.valid?
-        assert_equal nil, ed.version_number
-      end
-    end
-
-    should "not be minor_update" do
-      assert_equal false, TravelAdviceEdition.new.minor_update
-    end
-  end
-
-  context "building a new version" do
-    setup do
-      @ed = FactoryGirl.create(:travel_advice_edition,
-                               :title => "Aruba",
-                               :overview => "Aruba is not near Wales",
-                               :country_slug => "aruba",
-                               :summary => "## The summary",
-                               :alert_status => ["avoid_all_but_essential_travel_to_whole_country", "avoid_all_travel_to_parts"],
-                               :image_id => "id_from_the_asset_manager_for_an_image",
-                               :document_id => "id_from_the_asset_manager_for_a_document")
-      @ed.parts.build(:title => "Fooey", :slug => 'fooey', :body => "It's all about Fooey")
-      @ed.parts.build(:title => "Gooey", :slug => 'gooey', :body => "It's all about Gooey")
-      @ed.save!
-      @ed.publish!
-    end
-
-    should "build a new instance with the same fields" do
-      new_ed = @ed.build_clone
-      assert new_ed.new_record?
-      assert new_ed.valid?
-      assert_equal @ed.title, new_ed.title
-      assert_equal @ed.country_slug, new_ed.country_slug
-      assert_equal @ed.overview, new_ed.overview
-      assert_equal @ed.summary, new_ed.summary
-      assert_equal @ed.image_id, new_ed.image_id
-      assert_equal @ed.document_id, new_ed.document_id
-      assert_equal @ed.alert_status, new_ed.alert_status
-    end
-
-    should "copy the edition's parts" do
-      new_ed = @ed.build_clone
-      assert_equal ['Fooey', 'Gooey'], new_ed.parts.map(&:title)
-    end
-  end
-
-  context "previous_version" do
-    setup do
-      @ed1 = FactoryGirl.create(:archived_travel_advice_edition, :country_slug => 'foo')
-      @ed2 = FactoryGirl.create(:archived_travel_advice_edition, :country_slug => 'foo')
-      @ed3 = FactoryGirl.create(:archived_travel_advice_edition, :country_slug => 'foo')
-    end
-
-    should "return the previous version" do
-      assert_equal @ed2, @ed3.previous_version
-      assert_equal @ed1, @ed2.previous_version
-    end
-
-    should "return nil if there is no previous version" do
-      assert_equal nil, @ed1.previous_version
-    end
-  end
-
-  context "publishing" do
-    setup do
-      @published = FactoryGirl.create(:published_travel_advice_edition, :country_slug => 'aruba',
-                                      :published_at => 3.days.ago, :change_description => 'Stuff changed')
-      @ed = FactoryGirl.create(:travel_advice_edition, :country_slug => 'aruba')
-      @published.reload
-    end
-
-    should "publish the edition and archive related editions" do
-      @ed.publish!
-      @published.reload
-      assert @ed.published?
-      assert @published.archived?
-    end
-
-    context "setting the published date" do
-      should "set the published_at to now for a normal update" do
-        Timecop.freeze(1.day.from_now) do
-          @ed.publish!
-          # The to_i is necessary to account for the difference in milliseconds
-          # Time from the db only has a resolution in seconds, whereas Time.zone.now is more accurate
-          assert_equal Time.zone.now.utc.to_i, @ed.published_at.to_i
-        end
-      end
-
-      should "set the published_at to the previous version's published_at for a minor update" do
-        @ed.minor_update = true
-        @ed.publish!
-        assert_equal @published.published_at, @ed.published_at
-      end
-    end
-
-    should "set the change_description to the previous version's change_description for a minor update" do
-      @ed.minor_update = true
-      @ed.publish!
-      assert_equal @published.change_description, @ed.change_description
-    end
-  end
-
-  context "setting the reviewed at date" do
-    setup do
-      @published = FactoryGirl.create(:published_travel_advice_edition, :country_slug => 'aruba',
-                                      :published_at => 3.days.ago, :change_description => 'Stuff changed')
-      @published.reviewed_at = 2.days.ago
-      @published.save!
-      @published.reload
-
-      Timecop.freeze(1.days.ago) do
-        # this is done to make sure there's a significant difference in time
-        # between creating the edition and it being published
-        @ed = FactoryGirl.create(:travel_advice_edition, :country_slug => 'aruba')
-      end
-    end
-
-    should "be updated to published time when edition is published" do
-      @ed.change_description = "Did some stuff"
-      @ed.publish!
-      assert_equal @ed.published_at, @ed.reviewed_at
-    end
-
-    should "be set to the previous version's reviewed_at when a minor update is published" do
-      @ed.minor_update = true
-      @ed.publish!
-      assert_equal @published.reviewed_at, @ed.reviewed_at
-    end
-
-    should "be able to be updated without affecting other dates" do
-      published_at = @ed.published_at
-      Timecop.freeze(1.day.from_now) do
-        @ed.reviewed_at = Time.zone.now
-        assert_equal published_at, @ed.published_at
-      end
-    end
-
-    should "be able to update reviewed_at on a published edition" do
-      @ed.minor_update = true
-      @ed.publish!
-      Timecop.freeze(1.day.from_now) do
-        new_time = Time.zone.now
-        @ed.reviewed_at = new_time
-        @ed.save!
-        assert_equal new_time.utc.to_i, @ed.reviewed_at.to_i
-      end
-    end
-  end
-
-  context "indexable content" do
-    setup do
-      @edition = FactoryGirl.build(:travel_advice_edition)
-    end
-
-    should "return summary and all part titles and bodies" do
-      @edition.summary = "The Summary"
-      @edition.parts << Part.new(:title => "Part One", :body => "Some text")
-      @edition.parts << Part.new(:title => "More info", :body => "Some more information")
-      assert_equal "The Summary Part One Some text More info Some more information", @edition.indexable_content
-    end
-
-    should "convert govspeak to plain text" do
-      @edition.summary = "## The Summary"
-      @edition.parts << Part.new(:title => "Part One", :body => "* Some text")
-      assert_equal "The Summary Part One Some text", @edition.indexable_content
-    end
-  end
-
-  context "actions" do
-    setup do
-      @user = FactoryGirl.create(:user)
-      @old = FactoryGirl.create(:archived_travel_advice_edition, :country_slug => 'foo')
-      @edition = FactoryGirl.create(:draft_travel_advice_edition, :country_slug => 'foo')
-    end
-
-    should "not have any actions by default" do
-      assert_equal 0, @edition.actions.size
-    end
-
-    should "add a 'create' action" do
-      @edition.build_action_as(@user, Action::CREATE)
-      assert_equal 1, @edition.actions.size
-      assert_equal Action::CREATE, @edition.actions.first.request_type
-      assert_equal @user, @edition.actions.first.requester
-    end
-
-    should "add a 'new' action with a comment" do
-      @edition.build_action_as(@user, Action::NEW_VERSION, "a comment for the new version")
-      assert_equal 1, @edition.actions.size
-      assert_equal "a comment for the new version", @edition.actions.first.comment
-    end
-
-    context "publish_as" do
-      should "add a 'publish' action with change_description as comment on publish" do
-        @edition.change_description = "## My hovercraft is full of eels!"
-        @edition.publish_as(@user)
-        @edition.reload
-        assert_equal 1, @edition.actions.size
-        action = @edition.actions.last
-        assert_equal Action::PUBLISH, action.request_type
-        assert_equal @user, action.requester
-        assert_equal "My hovercraft is full of eels!", action.comment
-      end
-
-      should "add a 'publish' action with 'Minor update' as comment on publish of a minor_update" do
-        @edition.minor_update = true
-        @edition.publish_as(@user)
-        @edition.reload
-        assert_equal 1, @edition.actions.size
-        action = @edition.actions.last
-        assert_equal Action::PUBLISH, action.request_type
-        assert_equal "Minor update", action.comment
-      end
-    end
-  end
-end