From 93890f5ee0158cbf6de0271ef27962f3dae1fc6b Mon Sep 17 00:00:00 2001
From: Joseph Wynn <joseph@wildlyinaccurate.com>
Date: Sun, 7 Dec 2014 13:14:40 +0000
Subject: [PATCH] Collapse feature steps for brevity

---
 features/responsive-image-block.feature   | 10 ++----
 features/responsive-image.feature         | 44 ++++++-----------------
 features/step_definitions/jekyll_steps.rb |  8 +++++
 3 files changed, 20 insertions(+), 42 deletions(-)

diff --git a/features/responsive-image-block.feature b/features/responsive-image-block.feature
index f8ea4e8..2603e9c 100644
--- a/features/responsive-image-block.feature
+++ b/features/responsive-image-block.feature
@@ -4,10 +4,7 @@ Feature: Jekyll responsive_image_block tag
   In order to dynamically generate my responsive images
 
   Scenario: Simple image tag
-    Given I have a responsive_image configuration with:
-      """
-        template: _includes/responsive-image.html
-      """
+    Given I have a responsive_image configuration with "template" set to "_includes/responsive-image.html"
     And I have a file "index.html" with:
       """
       {% assign path = 'assets/test.png' %}
@@ -23,10 +20,7 @@ Feature: Jekyll responsive_image_block tag
     Then I should see "<img alt=\"Lorem ipsum\" src=\"/assets/test.png\" title=\"Magic rainbow adventure!\"" in "_site/index.html"
 
   Scenario: More complex logic in the block tag
-    Given I have a responsive_image configuration with:
-      """
-        template: _includes/responsive-image.html
-      """
+    Given I have a responsive_image configuration with "template" set to "_includes/responsive-image.html"
     And I have a file "index.html" with:
       """
       {% assign path = 'assets/test.png' %}
diff --git a/features/responsive-image.feature b/features/responsive-image.feature
index 73ff7e0..ea377d7 100644
--- a/features/responsive-image.feature
+++ b/features/responsive-image.feature
@@ -4,38 +4,23 @@ Feature: Jekyll responsive_image tag
   In order to best cater for devices of all sizes
 
   Scenario: Simple image tag
-    Given I have a responsive_image configuration with:
-      """
-        template: _includes/responsive-image.html
-      """
-    And I have a file "index.html" with:
-      """
-      {% responsive_image path: assets/test.png alt: Foobar %}
-      """
+    Given I have a responsive_image configuration with "template" set to "_includes/responsive-image.html"
+    And I have a file "index.html" with "{% responsive_image path: assets/test.png alt: Foobar %}"
     When I run Jekyll
     Then I should see "<img alt=\"Foobar\" src=\"/assets/test.png\"" in "_site/index.html"
 
   Scenario: Adding custom attributes
-    Given I have a responsive_image configuration with:
-      """
-        template: _includes/responsive-image.html
-      """
+    Given I have a responsive_image configuration with "template" set to "_includes/responsive-image.html"
     And I have a file "index.html" with:
       """
-      {% responsive_image path: assets/test.png alt: Foobar title: "Lorem Ipsum" %}
+      {% responsive_image path: assets/test.png alt: 'Foobar bazbar' title: "Lorem Ipsum" %}
       """
     When I run Jekyll
-    Then I should see "<img alt=\"Foobar\" src=\"/assets/test.png\" title=\"Lorem Ipsum\"" in "_site/index.html"
+    Then I should see "<img alt=\"Foobar bazbar\" src=\"/assets/test.png\" title=\"Lorem Ipsum\"" in "_site/index.html"
 
   Scenario: UTF-8 attributes
-    Given I have a responsive_image configuration with:
-      """
-        template: _includes/responsive-image.html
-      """
-    And I have a file "index.html" with:
-      """
-      {% responsive_image path: assets/test.png alt: "かっこいい！ ジェケルが好きです！" %}
-      """
+    Given I have a responsive_image configuration with "template" set to "_includes/responsive-image.html"
+    And I have a file "index.html" with "{% responsive_image path: assets/test.png alt: 'かっこいい！ ジェケルが好きです！' %}"
     When I run Jekyll
     Then I should see "<img alt=\"かっこいい！ ジェケルが好きです！\" src=\"/assets/test.png\"" in "_site/index.html"
 
@@ -47,10 +32,7 @@ Feature: Jekyll responsive_image tag
           - width: 100
           - width: 200
       """
-    And I have a file "index.html" with:
-      """
-      {% responsive_image path: assets/test.png %}
-      """
+    And I have a file "index.html" with "{% responsive_image path: assets/test.png %}"
     When I run Jekyll
     Then I should see "<img alt=\"\" src=\"/assets/test.png\"" in "_site/index.html"
     And I should see "/assets/resized/test-100x50.png 100w" in "_site/index.html"
@@ -68,10 +50,7 @@ Feature: Jekyll responsive_image tag
           - width: 200
           - width: 300
       """
-    And I have a file "index.html" with:
-      """
-      {% responsive_image path: assets/test.png template: _includes/custom-template.html %}
-      """
+    And I have a file "index.html" with "{% responsive_image path: assets/test.png template: _includes/custom-template.html %}"
     When I run Jekyll
     Then I should see "[100, 200, 300]" in "_site/index.html"
 
@@ -83,10 +62,7 @@ Feature: Jekyll responsive_image tag
         sizes:
           - width: 100
       """
-    And I have a file "index.html" with:
-      """
-      {% responsive_image path: assets/test.png %}
-      """
+    And I have a file "index.html" with "{% responsive_image path: assets/test.png %}"
     When I run Jekyll
     Then I should see "/assets/test.png-resized/100/test-50.png 100w" in "_site/index.html"
     And the file "assets/test.png-resized/100/test-50.png" should exist
diff --git a/features/step_definitions/jekyll_steps.rb b/features/step_definitions/jekyll_steps.rb
index 48bfdba..7541bd6 100644
--- a/features/step_definitions/jekyll_steps.rb
+++ b/features/step_definitions/jekyll_steps.rb
@@ -6,10 +6,18 @@ Given /^I have a responsive_image configuration with:$/ do |config|
   write_file('_config.yml', "responsive_image:\n#{config}")
 end
 
+Given /^I have a responsive_image configuration with "(.+)" set to "(.+)"$/ do |config, value|
+  write_file('_config.yml', "responsive_image:\n  #{config}: #{value}")
+end
+
 Given /^I have a file "(.+)" with:$/ do |path, contents|
   write_file(path, "---\n---\n#{contents}")
 end
 
+Given /^I have a file "(.+)" with "(.+)"$/ do |path, contents|
+  write_file(path, "---\n---\n#{contents}")
+end
+
 Then /^I should see "(.+)" in "(.*)"$/ do |text, file|
   assert_match(Regexp.new(text), File.open(file).readlines.join)
 end