flipper-ui 0.16.0 → 0.18.0

data/lib/flipper/ui/views/features.erb

@@ -1,43 +1,57 @@
 <% if @show_blank_slate %>
   <div class="jumbotron text-center">
-    <span class="mega-octicon octicon-plus"></span>
-    <span class="mega-octicon octicon-list-unordered"></span>
-    <span class="mega-octicon octicon-zap"></span>
-    <h4>But I've got a blank space baby...</h4>
-    <p>And I'll flip your features.</p>
-    <div class="embed-responsive embed-responsive-16by9">
-      <iframe class="embed-responsive-item" width="560" height="315" src="https://www.youtube.com/embed/e-ORhEE9VVg" frameborder="0" allowfullscreen></iframe>
-    </div>
+    <% if Flipper::UI.configuration.fun %>
+      <h4>But I've got a blank space baby...</h4>
+      <p>And I'll flip your features.</p>
+      <%- if Flipper::UI.configuration.feature_creation_enabled -%>
+        <p>
+          <a class="btn btn-primary btn-sm" href="<%= script_name %>/features/new">Add Feature</a>
+        </p>
+      <%- end -%>
+      <div class="embed-responsive embed-responsive-16by9">
+        <iframe class="embed-responsive-item" width="560" height="315" src="https://www.youtube.com/embed/e-ORhEE9VVg" frameborder="0" allowfullscreen></iframe>
+      </div>
+    <% else %>
+      <h4>Getting Started</h4>
+      <p class="mb-1">You have not added any features to configure yet.</p>
+      <%- if Flipper::UI.configuration.feature_creation_enabled -%>
+        <p class="mt-2">
+          <a class="btn btn-primary btn-sm" href="<%= script_name %>/features/new">Add Feature</a>
+        </p>
+      <% else %>
+        <p>
+          Check the <a href="https://github.com/jnunemaker/flipper#examples">examples</a> to
+          learn how to add one.
+        </p>
+      <%- end -%>
+    <% end %>
   </div>
 <% else %>
   <div class="card">
-    <h4 class="card-header">Features</h4>
-    <table class="table">
-      <thead>
-        <tr class="d-flex">
-          <th class="col-1">
-            <span class="octicon octicon-squirrel"></span>
-          </th>
-          <th class="col">Feature</th>
-          <th class="col">Enabled Gates</th>
-        </tr>
-      </thead>
-      <tbody>
-        <% @features.each do |feature| %>
-          <tr class="d-flex">
-            <td class="col-1">
-              <span class="octicon octicon-squirrel <%= feature.color_class %>"></span>
-            </td>
-            <td class="col">
-              <a href="<%= "#{script_name}/features/#{feature.key}" %>">
-                <%= feature.key %></a>
-            </td>
-            <td class="col">
-              <%= feature.pretty_enabled_gate_names %>
-            </td>
-          </tr>
-        <% end %>
-      </tbody>
-    </table>
+    <div class="card-header">
+      <%- if Flipper::UI.configuration.feature_creation_enabled -%>
+        <div class="float-right">
+          <a class="btn btn-primary btn-sm" href="<%= script_name %>/features/new">Add Feature</a>
+        </div>
+      <%- end -%>
+      <h4 class="m-0">Features</h4>
+    </div>
+    <div class="card-body py-0">
+      <% @features.each do |feature| %>
+        <div class="feature row align-items-center mt-0 px-3 border-bottom">
+          <div class="col-1 col-md-auto">
+            <span class="octicon octicon-squirrel <%= feature.color_class %>" data-toggle="tooltip" title=<%= feature.state.to_s.capitalize %>></span>
+          </div>
+          <div class="col-10">
+            <a href="<%= "#{script_name}/features/#{feature.key}" %>" class="d-block px-0 py-3 btn text-left text-dark">
+              <div class="text-truncate"><%= feature.key %></div>
+              <div class="text-muted text-truncate">
+                <%== feature.gates_in_words %>
+              </div>
+            </a>
+          </div>
+        </div>
+      <% end %>
+    </div>
   </div>
 <% end %>

data/lib/flipper/ui/views/layout.erb

@@ -6,17 +6,12 @@
     <meta http-equiv="X-UA-Compatible" content="IE=edge">
     <meta name="viewport" content="width=device-width, initial-scale=1">
 
+    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm" crossorigin="anonymous">
     <link rel="stylesheet" href="<%= script_name %>/octicons/octicons.css">
     <link rel="stylesheet" href="<%= script_name %>/css/application.css">
-
-    <!-- HTML5 shim and Respond.js for IE8 support of HTML5 elements and media queries -->
-    <!--[if lt IE 9]>
-      <script src="https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js"></script>
-      <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>
-    <![endif]-->
   </head>
-  <body>
-    <div class="container">
+  <body class="py-4">
+    <div class="container mw-600">
       <%- unless Flipper::UI.configuration.banner_text.nil? -%>
         <div class="alert alert-<%= Flipper::UI.configuration.banner_class %> text-center font-weight-bold">
           <%= Flipper::UI.configuration.banner_text %>
@@ -24,7 +19,7 @@
       <%- end -%>
 
       <nav aria-label="breadcrumb">
-        <ol class="breadcrumb bg-white border border-light align-items-center">
+        <ol class="breadcrumb bg-white border align-items-center mb-4">
           <% @breadcrumbs.each do |breadcrumb| %>
             <li class="breadcrumb-item <% if breadcrumb.active? %>active<% end %>">
               <% if breadcrumb.active? %>
@@ -34,11 +29,6 @@
               <% end %>
             </li>
           <% end %>
-          <li class="ml-auto">
-            <%- if Flipper::UI.configuration.feature_creation_enabled -%>
-              <a class="btn btn-sm btn-light" href="<%= script_name %>/features/new">Add Feature</a>
-            <%- end -%>
-          </li>
         </ol>
       </nav>
 

data/lib/flipper/version.rb

@@ -1,3 +1,3 @@
 module Flipper
-  VERSION = '0.16.0'.freeze
+  VERSION = '0.18.0'.freeze
 end

data/spec/flipper/ui/action_spec.rb

@@ -1,59 +1,78 @@
 require 'helper'
 
 RSpec.describe Flipper::UI::Action do
-  let(:action_subclass) do
-    Class.new(described_class) do
-      def noooope
-        raise 'should never run this'
-      end
+  describe 'request methods' do
+    let(:action_subclass) do
+      Class.new(described_class) do
+        def noooope
+          raise 'should never run this'
+        end
 
-      def get
-        [200, {}, 'get']
-      end
+        def get
+          [200, {}, 'get']
+        end
 
-      def post
-        [200, {}, 'post']
-      end
+        def post
+          [200, {}, 'post']
+        end
 
-      def put
-        [200, {}, 'put']
-      end
+        def put
+          [200, {}, 'put']
+        end
 
-      def delete
-        [200, {}, 'delete']
+        def delete
+          [200, {}, 'delete']
+        end
       end
     end
-  end
 
-  it "won't run method that isn't whitelisted" do
-    fake_request = Struct.new(:request_method, :env, :session).new('NOOOOPE', {}, {})
-    action = action_subclass.new(flipper, fake_request)
-    expect do
-      action.run
-    end.to raise_error(Flipper::UI::RequestMethodNotSupported)
-  end
+    it "won't run method that isn't whitelisted" do
+      fake_request = Struct.new(:request_method, :env, :session).new('NOOOOPE', {}, {})
+      action = action_subclass.new(flipper, fake_request)
+      expect do
+        action.run
+      end.to raise_error(Flipper::UI::RequestMethodNotSupported)
+    end
 
-  it 'will run get' do
-    fake_request = Struct.new(:request_method, :env, :session).new('GET', {}, {})
-    action = action_subclass.new(flipper, fake_request)
-    expect(action.run).to eq([200, {}, 'get'])
-  end
+    it 'will run get' do
+      fake_request = Struct.new(:request_method, :env, :session).new('GET', {}, {})
+      action = action_subclass.new(flipper, fake_request)
+      expect(action.run).to eq([200, {}, 'get'])
+    end
 
-  it 'will run post' do
-    fake_request = Struct.new(:request_method, :env, :session).new('POST', {}, {})
-    action = action_subclass.new(flipper, fake_request)
-    expect(action.run).to eq([200, {}, 'post'])
-  end
+    it 'will run post' do
+      fake_request = Struct.new(:request_method, :env, :session).new('POST', {}, {})
+      action = action_subclass.new(flipper, fake_request)
+      expect(action.run).to eq([200, {}, 'post'])
+    end
 
-  it 'will run put' do
-    fake_request = Struct.new(:request_method, :env, :session).new('PUT', {}, {})
-    action = action_subclass.new(flipper, fake_request)
-    expect(action.run).to eq([200, {}, 'put'])
+    it 'will run put' do
+      fake_request = Struct.new(:request_method, :env, :session).new('PUT', {}, {})
+      action = action_subclass.new(flipper, fake_request)
+      expect(action.run).to eq([200, {}, 'put'])
+    end
   end
 
-  it 'will run delete' do
-    fake_request = Struct.new(:request_method, :env, :session).new('DELETE', {}, {})
-    action = action_subclass.new(flipper, fake_request)
-    expect(action.run).to eq([200, {}, 'delete'])
+  describe 'FeatureNameFromRoute' do
+    let(:action_subclass) do
+      Class.new(described_class) do |parent|
+        include parent::FeatureNameFromRoute
+
+        route %r{\A/features/(?<feature_name>.*)\Z}
+
+        def get
+          [200, { feature_name: feature_name }, 'get']
+        end
+      end
+    end
+
+    it 'decodes feature_name' do
+      requested_feature_name = Rack::Utils.escape("team:side_pane")
+      fake_request = Struct
+                     .new(:request_method, :env, :session, :path_info)
+                     .new('GET', {}, {}, "/features/#{requested_feature_name}")
+      action = action_subclass.new(flipper, fake_request)
+      expect(action.run).to eq([200, { feature_name: "team:side_pane" }, 'get'])
+    end
   end
 end

data/spec/flipper/ui/actions/actors_gate_spec.rb

@@ -44,7 +44,7 @@ RSpec.describe Flipper::UI::Actions::ActorsGate do
 
   describe 'POST /features/:feature/actors' do
     context 'enabling an actor' do
-      let(:value) { 'User:6' }
+      let(:value) { 'User;6' }
 
       before do
         post 'features/search/actors',
@@ -53,7 +53,7 @@ RSpec.describe Flipper::UI::Actions::ActorsGate do
       end
 
       it 'adds item to members' do
-        expect(flipper[:search].actors_value).to include('User:6')
+        expect(flipper[:search].actors_value).to include('User;6')
       end
 
       it 'redirects back to feature' do
@@ -62,10 +62,10 @@ RSpec.describe Flipper::UI::Actions::ActorsGate do
       end
 
       context 'value contains whitespace' do
-        let(:value) { '  User:6  ' }
+        let(:value) { '  User;6  ' }
 
         it 'adds item without whitespace' do
-          expect(flipper[:search].actors_value).to include('User:6')
+          expect(flipper[:search].actors_value).to include('User;6')
         end
       end
 
@@ -73,39 +73,35 @@ RSpec.describe Flipper::UI::Actions::ActorsGate do
         context 'empty value' do
           let(:value) { '' }
 
-          # rubocop:disable Metrics/LineLength
           it 'redirects back to feature' do
             expect(last_response.status).to be(302)
             expect(last_response.headers['Location']).to eq('/features/search/actors?error=%22%22+is+not+a+valid+actor+value.')
           end
-          # rubocop:enable Metrics/LineLength
         end
 
         context 'nil value' do
           let(:value) { nil }
 
-          # rubocop:disable Metrics/LineLength
           it 'redirects back to feature' do
             expect(last_response.status).to be(302)
             expect(last_response.headers['Location']).to eq('/features/search/actors?error=%22%22+is+not+a+valid+actor+value.')
           end
-          # rubocop:enable Metrics/LineLength
         end
       end
     end
 
     context 'disabling an actor' do
-      let(:value) { 'User:6' }
+      let(:value) { 'User;6' }
 
       before do
-        flipper[:search].enable_actor Flipper::Actor.new('User:6')
+        flipper[:search].enable_actor Flipper::Actor.new('User;6')
         post 'features/search/actors',
              { 'value' => value, 'operation' => 'disable', 'authenticity_token' => token },
              'rack.session' => session
       end
 
       it 'removes item from members' do
-        expect(flipper[:search].actors_value).not_to include('User:6')
+        expect(flipper[:search].actors_value).not_to include('User;6')
       end
 
       it 'redirects back to feature' do
@@ -114,10 +110,10 @@ RSpec.describe Flipper::UI::Actions::ActorsGate do
       end
 
       context 'value contains whitespace' do
-        let(:value) { '  User:6  ' }
+        let(:value) { '  User;6  ' }
 
         it 'removes item without whitespace' do
-          expect(flipper[:search].actors_value).not_to include('User:6')
+          expect(flipper[:search].actors_value).not_to include('User;6')
         end
       end
     end

data/spec/flipper/ui/actions/feature_spec.rb

@@ -70,6 +70,15 @@ RSpec.describe Flipper::UI::Actions::Feature do
 
   describe 'GET /features/:feature' do
     before do
+      Flipper::UI.configure do |config|
+        config.descriptions_source = lambda { |_keys|
+          {
+            "stats" => "Most awesome stats",
+            "search" => "Most in-depth search",
+          }
+        }
+      end
+
       get '/features/search'
     end
 
@@ -81,10 +90,11 @@ RSpec.describe Flipper::UI::Actions::Feature do
       expect(last_response.body).to include('search')
       expect(last_response.body).to include('Enable')
       expect(last_response.body).to include('Disable')
-      expect(last_response.body).to include('Actors')
-      expect(last_response.body).to include('Groups')
-      expect(last_response.body).to include('Percentage of Time')
-      expect(last_response.body).to include('Percentage of Actors')
+      expect(last_response.body).to include('No actors enabled')
+      expect(last_response.body).to include('No groups enabled')
+      expect(last_response.body).to include('Enabled for 0% of time')
+      expect(last_response.body).to include('Enabled for 0% of actors')
+      expect(last_response.body).to include('Most in-depth search')
     end
   end
 
@@ -99,12 +109,6 @@ RSpec.describe Flipper::UI::Actions::Feature do
 
     it 'renders template' do
       expect(last_response.body).to include('search_features')
-      expect(last_response.body).to include('Enable')
-      expect(last_response.body).to include('Disable')
-      expect(last_response.body).to include('Actors')
-      expect(last_response.body).to include('Groups')
-      expect(last_response.body).to include('Percentage of Time')
-      expect(last_response.body).to include('Percentage of Actors')
     end
   end
 
@@ -119,12 +123,6 @@ RSpec.describe Flipper::UI::Actions::Feature do
 
     it 'renders template' do
       expect(last_response.body).to include('a/b')
-      expect(last_response.body).to include('Enable')
-      expect(last_response.body).to include('Disable')
-      expect(last_response.body).to include('Actors')
-      expect(last_response.body).to include('Groups')
-      expect(last_response.body).to include('Percentage of Time')
-      expect(last_response.body).to include('Percentage of Actors')
     end
   end
 end

data/spec/flipper/ui/actions/features_spec.rb

@@ -13,19 +13,58 @@ RSpec.describe Flipper::UI::Actions::Features do
   end
 
   describe 'GET /features' do
-    before do
-      flipper[:stats].enable
-      flipper[:search].enable
-      get '/features'
-    end
+    context "when there are some features" do
+      before do
+        flipper[:stats].enable
+        flipper[:search].enable
+        get '/features'
+      end
 
-    it 'responds with success' do
-      expect(last_response.status).to be(200)
+      it 'responds with success' do
+        expect(last_response.status).to be(200)
+      end
+
+      it 'renders template' do
+        expect(last_response.body).to include('stats')
+        expect(last_response.body).to include('search')
+      end
     end
 
-    it 'renders template' do
-      expect(last_response.body).to include('stats')
-      expect(last_response.body).to include('search')
+    context "when there are no features to list" do
+      before do
+        @original_fun_enabled = Flipper::UI.configuration.fun
+        Flipper::UI.configuration.fun = fun_mode
+      end
+
+      after do
+        Flipper::UI.configuration.fun = @original_fun_enabled
+      end
+
+      context "when fun mode is enabled" do
+        let(:fun_mode) { true }
+        before { get '/features' }
+
+        it 'responds with success' do
+          expect(last_response.status).to be(200)
+        end
+
+        it 'renders template' do
+          expect(last_response.body).to include('And I\'ll flip your features.')
+        end
+      end
+
+      context "when fun mode is disabled" do
+        let(:fun_mode) { false }
+        before { get '/features' }
+
+        it 'responds with success' do
+          expect(last_response.status).to be(200)
+        end
+
+        it 'renders template' do
+          expect(last_response.body).to include('You have not added any features to configure yet.')
+        end
+      end
     end
   end
 
@@ -72,12 +111,10 @@ RSpec.describe Flipper::UI::Actions::Features do
             expect(flipper.features.map(&:key)).to eq([])
           end
 
-          # rubocop:disable Metrics/LineLength
           it 'redirects back to feature' do
             expect(last_response.status).to be(302)
             expect(last_response.headers['Location']).to eq('/features/new?error=%22%22+is+not+a+valid+feature+name.')
           end
-          # rubocop:enable Metrics/LineLength
         end
 
         context 'nil feature name' do
@@ -87,12 +124,10 @@ RSpec.describe Flipper::UI::Actions::Features do
             expect(flipper.features.map(&:key)).to eq([])
           end
 
-          # rubocop:disable Metrics/LineLength
           it 'redirects back to feature' do
             expect(last_response.status).to be(302)
             expect(last_response.headers['Location']).to eq('/features/new?error=%22%22+is+not+a+valid+feature+name.')
           end
-          # rubocop:enable Metrics/LineLength
         end
       end
     end