Merge pull request #320 from roomorama/feature/fee-percentages-for-hosts

Show hosts fee percentage on suppliers#show

CHANGELOG.md

@@ -4,6 +4,10 @@ This file summarises the most important changes that went live on each release
 of Concierge. Please check the Wiki entry on the release process to understand
 how this file is formatted and how the process works.
 
+## Unreleased
+### Added
+- fee percentage for hosts on Web app on suppliers#show.
+
 ## [0.8.0] - 2016-09-08
 ### Added
 - Kigo and KigoLegacy integration: metadata sync, calendar sync, quoting, booking, cancelling support.

apps/web/templates/suppliers/show.html.erb

@@ -13,6 +13,7 @@
         <th>#</th>
         <th>Identifier</th>
         <th>Roomorama Username</th>
+        <th>Fees Applied</th>
         <th>Access Token</th>
         <th># Properties</th>
       </tr>
@@ -24,6 +25,7 @@
           <td><%= host.id %></td>
           <td><code><%= host.identifier %></code></td>
           <td><%= host.username %></td>
+          <td><%= format_fee_percentage(host.fee_percentage) %></td>
           <td><code><%= format_access_token(host) %></code></td>
           <td><%= host_properties(host) %></td>
         </tr>

apps/web/views/suppliers/show.rb

@@ -22,6 +22,17 @@ def host_properties(host)
       number_formatter.present(total)
     end
 
+    # formats the +fee+, given as a float, to a percentage notation, with a single
+    # digit precision.
+    #
+    # Example
+    #
+    #   host.fee_percentage                        # => 8.0
+    #   format_fee_percentage(host.fee_percentage) # => "8.0%"
+    def format_fee_percentage(fee)
+      sprintf("%.1f%", fee)
+    end
+
     # in order not to show the access token used by a host in its entirety,
     # this extracts the first 5 digits of a +Host+ access token, and appends
     # +...+ to indicate the content is just a substring.

spec/web/views/suppliers/show_spec.rb

@@ -23,7 +23,7 @@
 
   before do
     2.times { |n|
-      create_host(supplier_id: supplier.id, identifier: "host#{n}", access_token: "token#{n}")
+      create_host(supplier_id: supplier.id, fee_percentage: (n + 2).to_f, identifier: "host#{n}", access_token: "token#{n}")
     }
   end
 
@@ -39,6 +39,8 @@
   it "renders a list of hosts for the supplier" do
     expect(sanitized).to include %(<td><code>host0</code></td>)
     expect(sanitized).to include %(<td><code>host1</code></td>)
+    expect(sanitized).to include %(<td>2.0%</td>)
+    expect(sanitized).to include %(<td>3.0%</td>)
 
     expect(sanitized).to include %(<td><code>token...</code></td>)
   end