Merge pull request #17 from alphagov/dont-crash-for-host

Don't crash when `host` isn't set
alphagov · Apr 18, 2018 · 492e621 · 492e621
2 parents d02b4df + 8e78293
commit 492e621
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/govuk_tech_docs/meta_tags.rb b/lib/govuk_tech_docs/meta_tags.rb
@@ -53,7 +53,7 @@ def page_title
     end
 
     def host
-      config[:tech_docs][:host]
+      config[:tech_docs][:host].to_s
     end
 
     def locals

diff --git a/spec/govuk_tech_docs/meta_tags_spec.rb b/spec/govuk_tech_docs/meta_tags_spec.rb
@@ -80,6 +80,19 @@ def generate_title(site_name:, page_title:)
 
       expect(tags["og:title"]).to eql("The local variable title.")
     end
+
+    it 'works even when no config is set' do
+      current_page = double("current_page",
+        data: double("page_frontmatter", description: "The description.", title: "The Title"),
+        url: "/foo.html",
+        metadata: { locals: { title: "The local variable title." } })
+
+      config = { tech_docs: {} }
+
+      tags = GovukTechDocs::MetaTags.new(config, current_page).tags
+
+      expect(tags).to be_a(Hash)
+    end
   end
 
   def generate_config(config = {})