feat(ui): display repository URL on the pacticipant relationships page

https://pact.canny.io/feature-requests/p/display-repo-url-for-provider-and-consumer-in-broker-ui
pact-foundation · Feb 27, 2020 · 5d285e9 · 5d285e9
1 parent 58dea9a
commit 5d285e9
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 4 deletions.
diff --git a/lib/pact_broker/test/test_data_builder.rb b/lib/pact_broker/test/test_data_builder.rb
@@ -111,7 +111,7 @@ def create_tag_with_hierarchy pacticipant_name, pacticipant_version, tag_name
 
       def create_pacticipant pacticipant_name, params = {}
         params.delete(:comment)
-        @pacticipant = PactBroker::Domain::Pacticipant.create({ :name => pacticipant_name }.merge(params))
+        @pacticipant = PactBroker::Domain::Pacticipant.create({ :name => pacticipant_name, repository_url: "https://github.com/example-organization/#{pacticipant_name}" }.merge(params))
         self
       end
 

diff --git a/lib/pact_broker/ui/controllers/groups.rb b/lib/pact_broker/ui/controllers/groups.rb
@@ -10,10 +10,13 @@ class Groups < Base
         include PactBroker::Services
 
         get ":name" do
+          pacticipant = pacticipant_service.find_pacticipant_by_name(params[:name])
           erb :'groups/show.html', {
-            locals: {
-              csv_path: "/groups/#{params[:name]}.csv",
-              pacticipant_name: params[:name]}
+              locals: {
+                csv_path: "/groups/#{params[:name]}.csv",
+                pacticipant_name: params[:name],
+                repository_url: pacticipant&.repository_url
+              }
             }, {
               layout: 'layouts/main'
             }

diff --git a/lib/pact_broker/ui/views/groups/show.html.erb b/lib/pact_broker/ui/views/groups/show.html.erb
@@ -1,3 +1,5 @@
+<% require 'sanitize' %>
+
 <!DOCTYPE html>
 <html lang="en">
 <head>
@@ -30,6 +32,19 @@ body{
 </head>
 <body>
 <h1>Network graph of <%= pacticipant_name %> relationships</h1>
+
+<% if repository_url %>
+<p>Repository URL:
+
+<%
+repository_link = "<a href='#{repository_url}'>#{escape_html(repository_url)}</a>"
+%>
+
+<%= Sanitize.fragment(repository_link, Sanitize::Config::BASIC) %>
+
+<% end %>
+
+
 <script type="text/javascript">
 var windowWidth, windowHeight, svg;
 

diff --git a/pact_broker.gemspec b/pact_broker.gemspec
@@ -63,6 +63,7 @@ Gem::Specification.new do |gem|
   gem.add_runtime_dependency 'dry-logic', '0.4.2' # Later version cases ArgumentError: wrong number of arguments
   gem.add_runtime_dependency 'table_print', '~> 1.5'
   gem.add_runtime_dependency 'semantic_logger', '~> 4.3'
+  gem.add_runtime_dependency 'sanitize', '~> 5.1'
 
   gem.add_development_dependency 'pact', '~>1.14'
   gem.add_development_dependency 'rspec-pact-matchers', '~>0.1'