govuk_ab_testing 2.4.2 → 2.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cf9c70a0d36f4d97dec173bfc21738a3266700997a1af70cf3e5fac2b654c313
-  data.tar.gz: 7d63ce7fe82da0f11076b29498a6188691ee765bad005805bfecdea1fc022894
+  metadata.gz: 6a292ccc01842df0545f9ddfeec92414895271e6127c1cd9905ea30a152f3f58
+  data.tar.gz: 75d28b74e22e0efa9137e7e033daebafd697012fd9f2f17a876fc40c4ef3e66c
 SHA512:
-  metadata.gz: 59ca9f4101ddea9d142c492052ceedc08af2a29e35601a25afd6e057e976ab4e7745ca9dac4b27156210122314df78da84ee4d42c84fd09fef32ec56a9017ae7
-  data.tar.gz: 9543b01d05c4e12ed4b8c0c029b3c5ff2e313b283541b47cdef0d2135c5273938d5cdcb5304981320e58a99a2399d92749729f2fc76680bbecfca60c6e938fab
+  metadata.gz: 0d88b6a20a1c3ffae1bc55abb2deba0cfeec443a12c8d0adad615475de3b889c3ec502bb8bdef5d49fff548cef4bc7bf56eaf826d11322d7a4a554dd8b92c9c2
+  data.tar.gz: e7d98c27fa20cde80ea0ddfbe28be2f9274a6a668257b2cc266387e0db12b1a374b1dbecff1826cceb1bfd9e8aa38386a243c7437bae3eb1d4b31a32b9112525

data/.github/dependabot.yml

@@ -0,0 +1,10 @@
+version: 2
+updates:
+  - package-ecosystem: bundler
+    directory: /
+    schedule:
+      interval: daily
+  - package-ecosystem: "github-actions"
+    directory: /
+    schedule:
+      interval: daily

data/.github/workflows/ci.yml

@@ -0,0 +1,36 @@
+on: [push, pull_request]
+
+jobs:
+  # This matrix job runs the test suite against multiple Ruby versions
+  test_matrix:
+    strategy:
+      fail-fast: false
+      matrix:
+        # Due to https://github.com/actions/runner/issues/849, we have to use quotes for '3.0'
+        ruby: ['3.0', 3.1, 3.2]
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - run: bundle exec rake
+
+  # Branch protection rules cannot directly depend on status checks from matrix jobs.
+  # So instead we define `test` as a dummy job which only runs after the preceding `test_matrix` checks have passed.
+  # Solution inspired by: https://github.community/t/status-check-for-a-matrix-jobs/127354/3
+  test:
+    needs: test_matrix
+    runs-on: ubuntu-latest
+    steps:
+      - run: echo "All matrix tests have passed 🚀"
+
+  publish:
+    needs: test
+    if: ${{ github.ref == 'refs/heads/main' }}
+    permissions:
+      contents: write
+    uses: alphagov/govuk-infrastructure/.github/workflows/publish-rubygem.yml@main
+    secrets:
+      GEM_HOST_API_KEY: ${{ secrets.ALPHAGOV_RUBYGEMS_API_KEY }}

data/.ruby-version

@@ -1 +1 @@
-2.6.6
+3.0.5

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 2.4.3
+
+* Updated gem dependencies and development ruby version to 3.0.5
+* Fixed new rubocop violations
+
 ## 2.4.2
 
 * Updated gem dependencies and development ruby version to 2.6.6

data/README.md

@@ -2,9 +2,7 @@
 
 Gem to help with A/B testing on the GOV.UK platform.
 
-## Technical documentation
-
-### Installation
+## Installation
 
 Add this line to your application's Gemfile:
 
@@ -16,35 +14,27 @@ And then execute:
 
     $ bundle
 
-### Usage
+## Pre-requisites
 
 Before starting this, you'll need to:
 
-- [read the documentation](https://docs.publishing.service.gov.uk/manual/ab-testing.html) on how to set up an a/b test. The cookie and header name in [fastly-configure](https://github.com/alphagov/fastly-configure) and  [CDN config](https://github.digital.cabinet-office.gov.uk/gds/cdn-configs) must match the test name parameter that you pass to the Gem. The cookie name is case-sensitive.
-- configure Google Analytics (guidelines to follow)
+- [Read the documentation](https://docs.publishing.service.gov.uk/manual/ab-testing.html) for an overview on how a/b testing works on GOV.UK. 
+- The cookie and header name in [govuk-cdn-config](https://github.com/alphagov/govuk-cdn-config/blob/master/ab_tests/ab_tests.yaml) must match the test name parameter that you pass to the Gem. The cookie name is case-sensitive.
 
-To enable testing in the app, your Rails app needs:
+## Usage
 
-1. Some piece of logic to be A/B tested
-2. A HTML meta tag that will be used to measure the results, and which specifies
-the dimension to use in Google Analytics
-3. A response HTTP header that tells Fastly you're doing an A/B test
+### Outline 
 
-Start by defining which acceptance testing framework you will use. This gem
-supports both Capybara and ActiveSupport. In order to configure it, add this to
-your test helper file:
+To enable testing in the app, your Rails app needs:
 
-```
-GovukAbTesting.configure do |config|
-  config.acceptance_test_framework = :capybara # or :active_support
-end
-```
+1. [Some piece of logic to be A/B tested](#1-example-ab-test-logic)
+2. [A response HTTP header that tells Fastly you're doing an A/B test](#2-http-response-header-to-fastly)
+3. [A HTML meta tag that will be used to measure the results, and which specifies
+   the dimension to use in Google Analytics](#3-add-html-metatag-tags-to-your-layouts)
 
-If we use capybara, the gem expects `page` to be defined in the scope of the
-test cases. If we use ActiveSupport, the gem expects `@request` to be defined in
-the scope of the test cases.
+### 1. Example A/B test logic
 
-Now, let's say you have this controller:
+Let's say you have this controller:
 
 ```ruby
 # app/controllers/party_controller.rb
@@ -73,10 +63,15 @@ end
 
 In this example, we are running a multivariate test with 3 options being
 tested: the existing version (control), and two title changes. The minimum
-number of variants in any test should be two.
+number of variants in any test should be two. 
+
+### 2. HTTP response header to Fastly
+
+The `configure_response` method used in the example in `step 1` sends the response header. The header helps Fastly to understand which variant was returned to the user and cache appropriately.
+
+### 3. Add HTML metatag tags to your layouts
 
-Then, add this to your layouts, so that we have a meta tag that can be picked up
-by the extension and analytics.
+This is for the extension and analytics.
 
 ```html
 <!-- application.html.erb -->
@@ -86,11 +81,76 @@ by the extension and analytics.
 ```
 
 The analytics meta tag will include the allowed variants so the extension knows
-which variants to suggest the user.
+which variants to suggest to the user.
+
+## Running the test suite for the gem
+
+`bundle exec rake`
+
+## Acceptance testing
+
+Start by defining which acceptance testing framework you will use. This gem
+supports both Capybara and ActiveSupport. In order to configure it, add this to
+your test helper file:
+
+```
+GovukAbTesting.configure do |config|
+  config.acceptance_test_framework = :capybara # or :active_support
+end
+```
+
+If we use capybara, the gem expects `page` to be defined in the scope of the
+test cases. If we use ActiveSupport, the gem expects `@request` to be defined in
+the scope of the test cases.
+
+### Test helpers
+
+#### RSpec
+
+It is also possible to use `with_variant` and all the individual setup and
+assertions steps in RSpec tests. Here is an example of a Capybara feature file:
+
+```ruby
+# spec/features/ab_testing_spec.rb
+feature "Viewing a page with an A/B test" do
+  include GovukAbTesting::RspecHelpers
+
+  scenario "viewing the B version of the page" do
+    with_variant your_ab_test_name: 'B' do
+      visit root_path
+
+      expect(page).to have_breadcrumbs
+      expect(page).to have_beta_label
+    end
+  end
+end
+```
+
+And here is an RSpec controller test:
+
+```ruby
+# spec/controllers/some_controller_spec.rb
+describe SomeController, type :controller do
+  include GovukAbTesting::RspecHelpers
+
+  # RSpec doesn't render views for controller specs by default
+  render_views
 
-#### Test helpers
+  it "should render the B version of the page" do
+    with_variant your_ab_test_name: 'B' do
+      get :index
+    end
+  end
+end
+```
 
-##### Minitest
+As with the `minitest` version, you can also pass in the following options to
+`with_variant`:
+
+- `assert_meta_tag: false`
+- `dimension: <number>`
+
+#### Minitest
 
 The most common usage of an A/B test is to serve two different variants of the
 same page. In this situation, you can test the controller using `with_variant`.
@@ -169,71 +229,22 @@ class PartyControllerTest < ActionController::TestCase
 end
 ```
 
-##### RSpec
+## API documentation
 
-It is also possible to use `with_variant` and all the individual setup and
-assertions steps in RSpec tests. Here is an example of a Capybara feature file:
+See [RubyDoc](http://www.rubydoc.info/gems/govuk_ab_testing) for documentation including all of the assertions for tests.
 
-```ruby
-# spec/features/ab_testing_spec.rb
-feature "Viewing a page with an A/B test" do
-  include GovukAbTesting::RspecHelpers
-
-  scenario "viewing the B version of the page" do
-    with_variant your_ab_test_name: 'B' do
-      visit root_path
-
-      expect(page).to have_breadcrumbs
-      expect(page).to have_beta_label
-    end
-  end
-end
-```
-
-And here is an RSpec controller test:
-
-```ruby
-# spec/controllers/some_controller_spec.rb
-describe SomeController, type :controller do
-  include GovukAbTesting::RspecHelpers
-
-  # RSpec doesn't render views for controller specs by default
-  render_views
-
-  it "should render the B version of the page" do
-    with_variant your_ab_test_name: 'B' do
-      get :index
-    end
-  end
-end
-```
-
-As with the `minitest` version, you can also pass in the following options to
-`with_variant`:
-
-- `assert_meta_tag: false`
-- `dimension: <number>`
-
-### Running the test suite
+To run a Yard server locally to preview documentation, run:
 
-`bundle exec rake`
+    $ bundle exec yard server --reload
 
-### Testing in a browser
+## Checking your A/B test in a browser
 
 If you want to test this behaviour in a browser then you should use the
-[GOV.UK Toolkit for Chrome](https://github.com/alphagov/govuk-toolkit-chrome).
+[GOV.UK Toolkit browser extension](https://github.com/alphagov/govuk-browser-extension).
 
 This detects when you have a test running on a page and enables you to choose
 between variants.
 
-### Documentation
-
-See [RubyDoc](http://www.rubydoc.info/gems/govuk_ab_testing) for some limited documentation.
-
-To run a Yard server locally to preview documentation, run:
-
-    $ bundle exec yard server --reload
-
 ## Licence
 
-[MIT License](LICENCE.txt)
+[MIT License](LICENCE)

data/govuk_ab_testing.gemspec

@@ -12,8 +12,9 @@ Gem::Specification.new do |spec|
   spec.description   = "Gem to help with A/B testing on the GOV.UK platform"
   spec.homepage      = "https://github.com/alphagov/govuk_ab_testing"
   spec.license       = "MIT"
+  spec.required_ruby_version = ">= 3.0"
 
-  spec.files         = `git ls-files -z`.split("\x0").reject do |f|
+  spec.files = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test|spec|features)/})
   end
   spec.bindir        = "exe"
@@ -22,6 +23,6 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec"
-  spec.add_development_dependency "rubocop-govuk"
+  spec.add_development_dependency "rubocop-govuk", "4.10.0"
   spec.add_development_dependency "yard"
 end

data/lib/govuk_ab_testing/ab_test.rb

@@ -1,9 +1,6 @@
 module GovukAbTesting
   class AbTest
-    attr_reader :ab_test_name
-    attr_reader :dimension
-    attr_reader :allowed_variants
-    attr_reader :control_variant
+    attr_reader :ab_test_name, :dimension, :allowed_variants, :control_variant
 
     alias_method :name, :ab_test_name
 

data/lib/govuk_ab_testing/configuration.rb

@@ -21,16 +21,14 @@ module GovukAbTesting
     end
 
     def framework_class
-      @framework_class ||= begin
-        case config[:acceptance_test_framework]
-        when :capybara
-          GovukAbTesting::AcceptanceTests::Capybara
-        when :active_support
-          GovukAbTesting::AcceptanceTests::ActiveSupport
-        else
-          raise "Invalid framework #{acceptance_test_framework}"
-        end
-      end
+      @framework_class ||= case config[:acceptance_test_framework]
+                           when :capybara
+                             GovukAbTesting::AcceptanceTests::Capybara
+                           when :active_support
+                             GovukAbTesting::AcceptanceTests::ActiveSupport
+                           else
+                             raise "Invalid framework #{acceptance_test_framework}"
+                           end
     end
   end
 end

data/lib/govuk_ab_testing/requested_variant.rb

@@ -63,10 +63,14 @@ module GovukAbTesting
     #
     # @return [String]
     def analytics_meta_tag
-      '<meta name="govuk:ab-test" ' \
-        'content="' + ab_test.meta_tag_name + ":" + variant_name + '" ' \
-        'data-analytics-dimension="' + @dimension.to_s + '" ' \
-        'data-allowed-variants="' + ab_test.allowed_variants.join(",") + '">'
+      tag = <<~HTML
+        <meta name="govuk:ab-test"
+          content="#{ab_test.meta_tag_name}:#{variant_name}"
+          data-analytics-dimension="#{@dimension}"
+          data-allowed-variants="#{ab_test.allowed_variants.join(',')}">
+      HTML
+
+      tag.gsub(/\n/, "")
     end
   end
 end

data/lib/govuk_ab_testing/version.rb

@@ -1,3 +1,3 @@
 module GovukAbTesting
-  VERSION = "2.4.2".freeze
+  VERSION = "2.4.3".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: govuk_ab_testing
 version: !ruby/object:Gem::Version
-  version: 2.4.2
+  version: 2.4.3
 platform: ruby
 authors:
 - GOV.UK Dev
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-08-04 00:00:00.000000000 Z
+date: 2023-03-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -42,16 +42,16 @@ dependencies:
   name: rubocop-govuk
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.10.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.10.0
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -73,14 +73,15 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/dependabot.yml"
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
 - ".ruby-version"
 - CHANGELOG.md
 - Gemfile
-- Jenkinsfile
-- LICENSE.txt
+- LICENCE
 - README.md
 - Rakefile
 - bin/console
@@ -111,14 +112,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.4.9
 signing_key: 
 specification_version: 4
 summary: Gem to help with A/B testing on the GOV.UK platform

data/Jenkinsfile

@@ -1,7 +0,0 @@
-#!/usr/bin/env groovy
-
-library("govuk")
-
-node {
-  govuk.buildProject()
-}