Permalink
Browse files

GOV.UK URLs are always HTTPS

  • Loading branch information...
1 parent 4aaa277 commit 1bd840cc3d908b374dd0b1564615790966856cfa @jamiecobbett jamiecobbett committed Nov 26, 2012
@@ -1,6 +1,6 @@
<%= f.inputs :name => "URL(s) affected" do %>
- <%= f.input :url1, :label => "URL 1", :input_html => {:class => "span6", :placeholder => "http://www.gov.uk/"} %>
- <%= f.input :url2, :label => "URL 2", :input_html => {:class => "span6", :placeholder => "http://www.gov.uk/"} %>
- <%= f.input :url3, :label => "URL 3", :input_html => {:class => "span6", :placeholder => "http://www.gov.uk/"} %>
+ <%= f.input :url1, :label => "URL 1", :input_html => {:class => "span6", :placeholder => "https://www.gov.uk/"} %>
+ <%= f.input :url2, :label => "URL 2", :input_html => {:class => "span6", :placeholder => "https://www.gov.uk/"} %>
+ <%= f.input :url3, :label => "URL 3", :input_html => {:class => "span6", :placeholder => "https://www.gov.uk/"} %>
<%= f.input :details_of_change, :as => :text, :label => "Details of the requested change", :required => :true, :input_html => {:class => "span6", :rows => 6, :cols => 50, :"aria-required" => true } %>
<% end %>
@@ -11,15 +11,15 @@ Feature: General requests
Scenario: successful request
When the user submits the following general request:
| Details | URL |
- | The site is down | http://www.gov.uk |
+ | The site is down | https://www.gov.uk |
Then the following ticket is raised in ZenDesk:
| Subject | Requester email | Requester name | Phone | Job title | Organisation |
| Govt Agency General Issue | john.smith@email.com | John Smith | 12345 | Developer | cabinet_office |
And the ticket is tagged with "govt_agency_general"
And the comment on the ticket is:
"""
[Url]
- http://www.gov.uk
+ https://www.gov.uk
[Additional]
The site is down
View
@@ -3,7 +3,7 @@ def valid_content_change_request_params
{"content_change_request" =>
{ "requester_attributes" => valid_requester_params,
"time_constraint_attributes" => valid_time_constraint_params,
- "url1" => "http://www.gov.uk",
+ "url1" => "https://www.gov.uk",
"details_of_change" => "Content is wrong"}
}
end
@@ -6,9 +6,9 @@ class ContentChangeRequestTest < Test::Unit::TestCase
should allow_value("yes").for(:inside_government)
- should allow_value("http://www.gov.uk").for(:url1)
- should allow_value("http://www.gov.uk").for(:url2)
- should allow_value("http://www.gov.uk").for(:url3)
+ should allow_value("https://www.gov.uk").for(:url1)
+ should allow_value("https://www.gov.uk").for(:url2)
+ should allow_value("https://www.gov.uk").for(:url3)
should "allow time constraints" do
request = ContentChangeRequest.new(:time_constraint => stub("time constraint", :valid? => true))
@@ -3,7 +3,7 @@
class GeneralRequestTest < Test::Unit::TestCase
should validate_presence_of(:requester)
- should allow_value("http://www.gov.uk").for(:url)
+ should allow_value("https://www.gov.uk").for(:url)
should allow_value("a comment").for(:additional)
end

0 comments on commit 1bd840c

Please sign in to comment.