govuk_content_models 30.0.0 → 31.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ace1431a4755075f7fc7d6a9ec4ec9edf0db97cb
-  data.tar.gz: b4744764d54f9a8eff22384617d8c700b4e8caea
+  metadata.gz: f180214b4c41e4eeeca68831ad6ba2ca55f1ef7f
+  data.tar.gz: c3620788cedec2c668987ae8d8b563e527e080e3
 SHA512:
-  metadata.gz: 1be2224113f915f7df753dd478728911cddd4439a008457b2d99d63da82d28999acae2e4420c6ceee4df3321cef3eb2dcafc29da8fd6887b45ab4c464c814b4a
-  data.tar.gz: a55c774b064aca486549a103d45111ac4a6d581148ec2c6ab30dc77f26129413c4d7b8b4fe7b0768eeac0557b765a2f8bce0a787afba817399edf363e36c2c88
+  metadata.gz: acf06907c1430d183ac52ae67cc858f67d68cab01f6963a406683fbcd8738c15595016aed443495e3310b0068f4eaea003214a4a2bfc48fc94950c7f0398eb54
+  data.tar.gz: 4b276c0907a6b77a0f0201c7e2e27d4e4c852a3e611f09f46d07e284a8f294270caab42e98745a54dd06159cc2ffc2d96b653f914cd926fc4136a9c37aa64349

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # CHANGELOG
 
+## 31.0.0
+
+- `Edition.find_or_create_from_panopticon_data` no longer takes a third
+   parameter.
+
 ## 30.0.0
 
 - Add:

data/app/models/edition.rb

@@ -224,8 +224,7 @@ class Edition
     new_edition
   end
 
-  def self.find_or_create_from_panopticon_data(panopticon_id,
-                                               importing_user, api_credentials)
+  def self.find_or_create_from_panopticon_data(panopticon_id, importing_user)
     existing_publication = Edition.where(panopticon_id: panopticon_id)
                                   .order_by([:version_number, :desc]).first
     return existing_publication if existing_publication

data/lib/govuk_content_models/version.rb

@@ -1,4 +1,4 @@
 module GovukContentModels
   # Changing this causes Jenkins to tag and release the gem into the wild
-  VERSION = "30.0.0"
+  VERSION = "31.0.0"
 end

data/test/models/artefact_test.rb

@@ -393,7 +393,7 @@ class ArtefactTest < ActiveSupport::TestCase
     )
 
     user1 = FactoryGirl.create(:user)
-    edition = AnswerEdition.find_or_create_from_panopticon_data(artefact.id, user1, {})
+    edition = AnswerEdition.find_or_create_from_panopticon_data(artefact.id, user1)
     edition.state = "published"
     edition.save!
 
@@ -414,7 +414,7 @@ class ArtefactTest < ActiveSupport::TestCase
         owning_app: "publisher",
     )
     user1 = FactoryGirl.create(:user)
-    edition = AnswerEdition.find_or_create_from_panopticon_data(artefact.id, user1, {})
+    edition = AnswerEdition.find_or_create_from_panopticon_data(artefact.id, user1)
 
     refute artefact.any_editions_published?
 

data/test/models/edition_test.rb

@@ -451,7 +451,7 @@ class EditionTest < ActiveSupport::TestCase
     a = Artefact.find(artefact.id)
     user = User.create
 
-    publication = Edition.find_or_create_from_panopticon_data(artefact.id, user, {})
+    publication = Edition.find_or_create_from_panopticon_data(artefact.id, user)
 
     assert_kind_of AnswerEdition, publication
     assert_equal artefact.name, publication.title
@@ -530,7 +530,7 @@ class EditionTest < ActiveSupport::TestCase
 
   test "deleting a newer draft of a published edition removes sibling information" do
     user1 = FactoryGirl.create(:user)
-    edition = AnswerEdition.find_or_create_from_panopticon_data(@artefact.id, user1, {})
+    edition = AnswerEdition.find_or_create_from_panopticon_data(@artefact.id, user1)
     edition.update_attribute(:state, "published")
     second_edition = edition.build_clone
     second_edition.save!
@@ -546,7 +546,7 @@ class EditionTest < ActiveSupport::TestCase
 
   test "the latest edition should remove sibling_in_progress details if it is present" do
     user1 = FactoryGirl.create(:user)
-    edition = AnswerEdition.find_or_create_from_panopticon_data(@artefact.id, user1, {})
+    edition = AnswerEdition.find_or_create_from_panopticon_data(@artefact.id, user1)
     edition.update_attribute(:state, "published")
 
     # simulate a document having a newer edition destroyed (previous behaviour).
@@ -561,7 +561,7 @@ class EditionTest < ActiveSupport::TestCase
     FactoryGirl.create(:live_tag, tag_id: "test-section", title: "Test section", tag_type: "section")
 
     user1 = FactoryGirl.create(:user)
-    edition = AnswerEdition.find_or_create_from_panopticon_data(@artefact.id, user1, {})
+    edition = AnswerEdition.find_or_create_from_panopticon_data(@artefact.id, user1)
 
     assert_difference "Artefact.count", -1 do
       edition.destroy
@@ -572,7 +572,7 @@ class EditionTest < ActiveSupport::TestCase
 
     FactoryGirl.create(:live_tag, tag_id: "test-section", title: "Test section", tag_type: "section")
     user1 = FactoryGirl.create(:user)
-    edition = AnswerEdition.find_or_create_from_panopticon_data(@artefact.id, user1, {})
+    edition = AnswerEdition.find_or_create_from_panopticon_data(@artefact.id, user1)
     edition.update_attribute(:state, "published")
 
     edition.reload
@@ -905,7 +905,7 @@ class EditionTest < ActiveSupport::TestCase
         owning_app: "publisher",
     )
 
-    edition = AnswerEdition.find_or_create_from_panopticon_data(artefact.id, user, {})
+    edition = AnswerEdition.find_or_create_from_panopticon_data(artefact.id, user)
 
     assert_equal user.name, edition.creator
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: govuk_content_models
 version: !ruby/object:Gem::Version
-  version: 30.0.0
+  version: 31.0.0
 platform: ruby
 authors:
 - Paul Battley