abstract_feature_branch 1.3.0 → 1.3.1

data/spec/ext/feature_branch__feature_enabled_spec.rb

@@ -1,274 +0,0 @@
-require 'spec_helper'
-
-describe 'feature_branch object extensions' do
-  before do
-    @app_env_backup = AbstractFeatureBranch.application_environment
-    @app_root_backup = AbstractFeatureBranch.application_root
-    AbstractFeatureBranch.logger.warn 'Environment variable ABSTRACT_FEATURE_BRANCH_FEATURE1 already set, potentially conflicting with another test' if ENV.keys.include?('ABSTRACT_FEATURE_BRANCH_FEATURE1')
-    AbstractFeatureBranch.logger.warn 'Environment variable Abstract_Feature_Branch_Feature2 already set, potentially conflicting with another test' if ENV.keys.include?('Abstract_Feature_Branch_Feature2')
-    AbstractFeatureBranch.logger.warn 'Environment variable abstract_feature_branch_feature3 already set, potentially conflicting with another test' if ENV.keys.include?('abstract_feature_branch_feature3')
-  end
-  after do
-    ENV.delete('ABSTRACT_FEATURE_BRANCH_FEATURE1')
-    ENV.delete('Abstract_Feature_Branch_Feature2')
-    ENV.delete('abstract_feature_branch_feature3')
-    AbstractFeatureBranch.application_root = @app_root_backup
-    AbstractFeatureBranch.application_environment = @app_env_backup
-    AbstractFeatureBranch.unload_application_features
-    AbstractFeatureBranch.user_features_storage.keys.each do |key|
-      AbstractFeatureBranch.user_features_storage.del(key)
-    end
-  end
-  describe '#feature_enabled?' do
-    it 'determines whether a feature is enabled or not in features configuration (case-insensitive string or symbol feature names)' do
-      feature_enabled?('Feature1').should == true
-      feature_enabled?(:FEATURE2).should == true
-      feature_enabled?(:feature3).should == false
-    end
-    it 'returns nil for an invalid feature name' do
-      feature_enabled?(:invalid_feature_that_does_not_exist).should be_nil
-    end
-    it 'allows environment variables (case-insensitive booleans) to override configuration file' do
-      ENV['ABSTRACT_FEATURE_BRANCH_FEATURE1'] = 'FALSE'
-      ENV['Abstract_Feature_Branch_Feature2'] = 'False'
-      ENV['abstract_feature_branch_feature3'] = 'true'
-      AbstractFeatureBranch.load_application_features
-      feature_enabled?(:feature1).should == false
-      feature_enabled?(:feature2).should == false
-      feature_enabled?(:feature3).should == true
-      feature_enabled?(:feature4a).should == true #not overridden
-    end
-    it 'allows redis variables (case-insensitive booleans) to override configuration file' do
-      AbstractFeatureBranch.unload_application_features
-      AbstractFeatureBranch.user_features_storage.hset('abstract_feature_branch', 'FEATURE1', 'FALSE')
-      AbstractFeatureBranch.user_features_storage.hset('abstract_feature_branch', 'Feature2', 'False')
-      AbstractFeatureBranch.user_features_storage.hset('abstract_feature_branch', 'feature3', 'true')
-      AbstractFeatureBranch.load_application_features
-      feature_enabled?(:feature1).should == false
-      feature_enabled?(:feature2).should == false
-      feature_enabled?(:feature3).should == true
-      feature_enabled?(:feature4a).should == true #not overridden
-    end
-    it 'allows local configuration file to override main configuration file in test environment' do
-      feature_enabled?(:feature4).should == false
-      feature_enabled?(:feature5).should == true
-    end
-    it 'has no local configuration for production environment' do
-      AbstractFeatureBranch.application_environment = 'production'
-      feature_enabled?(:feature4).should == true
-      feature_enabled?(:feature5).should be_nil
-    end
-    it 'works with Rails environment' do
-      Object.class_eval do
-        module Rails
-          def self.root
-            File.expand_path(File.join(__FILE__, '..', '..', 'fixtures', 'application_rails_config'))
-          end
-          def self.env
-            'staging'
-          end
-        end
-      end
-      AbstractFeatureBranch.initialize_application_root
-      AbstractFeatureBranch.initialize_application_environment
-      AbstractFeatureBranch.load_application_features
-      feature_enabled?(:feature1).should == true
-      feature_enabled?(:feature2).should == false
-      feature_enabled?(:feature3).should == false
-      feature_enabled?(:feature4).should be_nil
-      feature_enabled?(:feature4a).should be_nil
-      feature_enabled?(:feature5).should be_nil
-    end
-    context 'per user' do
-      let(:user_id) { 'email1@example.com' }
-      let(:another_user_id) { 'another_email@example.com' }
-      before do
-        AbstractFeatureBranch.user_features_storage.del("#{AbstractFeatureBranch::ENV_FEATURE_PREFIX}feature6")
-      end
-      after do
-        AbstractFeatureBranch.user_features_storage.del("#{AbstractFeatureBranch::ENV_FEATURE_PREFIX}feature6")
-      end
-      it 'behaves as expected if member list is empty, regardless of the user provided' do
-        feature_enabled?('feature6').should == false
-        feature_enabled?('feature6', nil).should == false
-        feature_enabled?('feature6', user_id).should == false
-      end
-      it 'behaves as expected if member list is not empty, and user provided is in member list' do
-        AbstractFeatureBranch.toggle_features_for_user(user_id, :feature6 => true)
-        feature_enabled?('feature6').should == false
-        feature_enabled?('feature6', user_id).should == true
-      end
-      it 'behaves as expected if member list is not empty, and user provided is not in member list' do
-        AbstractFeatureBranch.toggle_features_for_user(another_user_id, :feature6 => true)
-        feature_enabled?('feature6', user_id).should == false
-      end
-    end
-    context 'cacheability' do
-      before do
-        @development_application_root = File.expand_path(File.join(__FILE__, '..', '..', 'fixtures', 'application_development_config'))
-        @feature_file_reference_path = File.join(@development_application_root, 'config', 'features.reference.yml')
-        @feature_file_path = File.join(@development_application_root, 'config', 'features.yml')
-        FileUtils.cp(@feature_file_reference_path, @feature_file_path)
-        AbstractFeatureBranch.application_root = @development_application_root
-      end
-      after do
-        FileUtils.rm(@feature_file_path)
-      end
-      it 'refreshes features at runtime in development (without forcing load of application features again)' do
-        AbstractFeatureBranch.application_environment = 'development'
-        feature_enabled?(:feature1).should == false
-        feature_enabled?(:feature2).should == true
-        File.open(@feature_file_path, 'w+') do |file|
-          file << <<-CONTENT
-defaults: &defaults
-
-development:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-          CONTENT
-        end
-        feature_enabled?(:feature1).should == true
-        feature_enabled?(:feature2).should == false
-      end
-
-      %w(test staging production).each do |environment|
-        it "does not refresh features at runtime in #{environment} (without forcing load of application features again)" do
-          AbstractFeatureBranch.application_environment = environment
-          AbstractFeatureBranch.load_application_features
-          feature_enabled?(:feature1).should == false
-          feature_enabled?(:feature2).should == true
-          File.open(@feature_file_path, 'w+') do |file|
-            file << <<-CONTENT
-defaults: &defaults
-
-development:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-            CONTENT
-          end
-          feature_enabled?(:feature1).should == false
-          feature_enabled?(:feature2).should == true
-        end
-      end
-
-      it 'does not refresh features at runtime in development when overridden' do
-        AbstractFeatureBranch.application_environment = 'development'
-        AbstractFeatureBranch.cacheable = {:development => true}
-        AbstractFeatureBranch.load_application_features
-        feature_enabled?(:feature1).should == false
-        feature_enabled?(:feature2).should == true
-        File.open(@feature_file_path, 'w+') do |file|
-          file << <<-CONTENT
-defaults: &defaults
-
-development:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-            CONTENT
-          end
-        feature_enabled?(:feature1).should == false
-        feature_enabled?(:feature2).should == true
-      end
-
-      %w(test staging production).each do |environment|
-        it "does refresh features at runtime in #{environment} (without forcing load of application features again)" do
-          AbstractFeatureBranch.application_environment = environment
-          AbstractFeatureBranch.cacheable = {:test => false, :staging => false, :production => false}
-          feature_enabled?(:feature1).should == false
-          feature_enabled?(:feature2).should == true
-          File.open(@feature_file_path, 'w+') do |file|
-            file << <<-CONTENT
-defaults: &defaults
-
-development:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-            CONTENT
-          end
-          feature_enabled?(:feature1).should == true
-          feature_enabled?(:feature2).should == false
-        end
-      end
-    end
-  end
-  describe 'self#feature_enabled?' do
-    after do
-      Object.send(:remove_const, :TestObject)
-    end
-    # No need to retest all instance test cases, just a spot check due to implementation reuse
-    it 'determines whether a feature is enabled or not in features configuration' do
-      class TestObject
-        def self.hit_me
-          feature_enabled?(:feature1).should == true
-          feature_enabled?(:feature2).should == true
-          feature_enabled?(:feature3).should == false
-        end
-      end
-      TestObject.hit_me
-    end
-  end
-end

data/spec/fixtures/application_development_config/config/features.reference.yml

@@ -1,21 +0,0 @@
-defaults: &defaults
-
-development:
-  <<: *defaults
-  FEATURE1: false
-  Feature2: true
-
-test:
-  <<: *defaults
-  FEATURE1: false
-  Feature2: true
-
-staging:
-  <<: *defaults
-  FEATURE1: false
-  Feature2: true
-
-production:
-  <<: *defaults
-  FEATURE1: false
-  Feature2: true

data/spec/fixtures/application_no_config/no_config

@@ -1 +0,0 @@
-# just a stub to allow directory to be committed to git

data/spec/fixtures/application_rails_config/config/features.local.yml

@@ -1,16 +0,0 @@
-defaults: &defaults
-
-
-development:
-  <<: *defaults
-
-test:
-  <<: *defaults
-
-staging:
-  <<: *defaults
-  feature3: false
-
-production:
-  <<: *defaults
-

data/spec/fixtures/application_rails_config/config/features.yml

@@ -1,20 +0,0 @@
-defaults: &defaults
-  FEATURE1: true
-  Feature2: true
-  feature3: true
-
-development:
-  <<: *defaults
-
-test:
-  <<: *defaults
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  Feature2: false
-  feature3: true
-
-production:
-  <<: *defaults
-

data/spec/fixtures/application_ugly_config_reference/config/another_application_configuration.yml

@@ -1,31 +0,0 @@
-common: &default_settings
-  license_key: <%= ENV["LICENSE_KEY"] %>
-  app_name: <%= ENV["APP_NAME"] %>
-  monitor_mode: true
-  developer_mode: false
-  log_level: info
-
-  browser_monitoring:
-      auto_instrument: true
-
-  audit_log:
-    enabled: false
-
-
-development:
-  <<: *default_settings
-  monitor_mode: false
-  developer_mode: true
-
-test:
-  <<: *default_settings
-  monitor_mode: false
-
-production:
-  <<: *default_settings
-  monitor_mode: true
-
-staging:
-  <<: *default_settings
-  monitor_mode: true
-  app_name: <%= ENV["APP_NAME"] %> (Staging)

data/spec/fixtures/application_ugly_config_reference/config/database.yml

@@ -1,17 +0,0 @@
-development:
-  adapter: sqlite3
-  database: db/development.sqlite3
-  pool: 5
-  timeout: 5000
-
-test:
-  adapter: sqlite3
-  database: db/test.sqlite3
-  pool: 5
-  timeout: 5000
-
-production:
-  adapter: sqlite3
-  database: db/production.sqlite3
-  pool: 5
-  timeout: 5000

data/spec/fixtures/application_ugly_config_reference/config/features/admin.local.yml

@@ -1,44 +0,0 @@
-defaults: &defaults
-
-  feature4a: true
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-development:
-  <<: *defaults
-  FEATURE1: true
-
-  feature4a: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-
-  feature4a: true
-  feature3: false
-  Feature2: true
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-
-  feature4a: true
-  Feature2: true
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-  feature4a: true
-

data/spec/fixtures/application_ugly_config_reference/config/features/admin.yml

@@ -1,44 +0,0 @@
-defaults: &defaults
-
-  feature4a: true
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-development:
-  <<: *defaults
-  FEATURE1: true
-
-  feature4a: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-
-  feature4a: true
-  feature3: false
-  Feature2: true
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-
-  feature4a: true
-  Feature2: true
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-  feature4a: true
-

data/spec/fixtures/application_ugly_config_reference/config/features/feature_empty_config.local.yml

@@ -1,13 +0,0 @@
-defaults: &defaults
-
-development:
-  <<: *defaults
-
-test:
-  <<: *defaults
-
-staging:
-  <<: *defaults
-
-production:
-  <<: *defaults

data/spec/fixtures/application_ugly_config_reference/config/features/including_comments.local.yml

@@ -1,52 +0,0 @@
-# This file allows you to override any feature configuration locally without it being committed to git
-# It is recommended to use this file only for temporary overrides. Once done, make final change in main .yml
-
-defaults: &defaults
-
-  feature4a: true
-  FEATURE1: true
-  feature4: true
-# Some comment in between features (will get deleted by beautifier)
-  feature3: false
-
-  # another comment in between features (will get deleted by beautifier)
-  Feature2: true
-
-development:
-    # yet another comment in between features (will get deleted by beautifier)
-  <<: *defaults
-  FEATURE1: true
-
-  feature4a: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-
-  feature4a: true
-  feature3: false
-  Feature2: true
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-
-  feature4a: true
-  Feature2: true
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-  feature4a: true
-
-    # absolutely deleted by beautifier

data/spec/fixtures/application_ugly_config_reference/config/features/internal/wiki.local.yml

@@ -1,44 +0,0 @@
-defaults: &defaults
-
-  feature4a: true
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-development:
-  <<: *defaults
-  FEATURE1: true
-
-  feature4a: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-
-  feature4a: true
-  feature3: false
-  Feature2: true
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-
-  feature4a: true
-  Feature2: true
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-  feature4a: true
-

data/spec/fixtures/application_ugly_config_reference/config/features/internal/wiki.yml

@@ -1,44 +0,0 @@
-defaults: &defaults
-
-  feature4a: true
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-development:
-  <<: *defaults
-  FEATURE1: true
-
-  feature4a: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-
-  feature4a: true
-  feature3: false
-  Feature2: true
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-
-  feature4a: true
-  Feature2: true
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-  feature4a: true
-

data/spec/fixtures/application_ugly_config_reference/config/features/public.local.yml

@@ -1,44 +0,0 @@
-defaults: &defaults
-
-  feature4a: true
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-development:
-  <<: *defaults
-  FEATURE1: true
-
-  feature4a: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-test:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-
-  feature4a: true
-  feature3: false
-  Feature2: true
-
-staging:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-
-  feature4a: true
-  Feature2: true
-
-production:
-  <<: *defaults
-  FEATURE1: true
-  feature4: true
-  feature3: false
-  Feature2: true
-
-  feature4a: true
-