shopify-routes 0.0.3 → 0.0.4

data/.rspec

@@ -0,0 +1 @@
+--colour

data/.travis.yml

@@ -0,0 +1 @@
+rvm: 1.9.3

data/Rakefile

@@ -1,2 +1,8 @@
 #!/usr/bin/env rake
 require "bundler/gem_tasks"
+require "rspec/core/rake_task"
+
+task :default => :spec
+
+desc "Run specs"
+RSpec::Core::RakeTask.new

data/lib/shopify-routes.rb

@@ -58,8 +58,9 @@ module ShopifyAPI
     # @param [String] path The path for the route
     # @param [Array] actions List of actions to draw for this route
     # @param [String] nom Alias for the route if the path name does not suffice
+    # @param [String] suffix An optional suffix to include at the end of the redirect URI
     # @api private
-    def match(path, actions, nom = path)
+    def match(path, actions, nom = path, suffix = "")
 
       # only draw this route if it's included
       return unless include?(path)
@@ -67,8 +68,8 @@ module ShopifyAPI
       actions.each do |action|
 
         # create the route, and it's name
-        route = "*shop/admin/#{path.to_s}"
-        name  = nom.to_s
+        route   = "*shop/admin/#{path.to_s}"
+        name    = nom.to_s
 
         case action
 
@@ -81,6 +82,7 @@ module ShopifyAPI
         when :new
           route << "/new"
           name = "new_#{name.singularize}"
+          suffix = "/new"
 
         # custom routes
         else
@@ -91,7 +93,12 @@ module ShopifyAPI
         end
 
         # draw the route
-        match_admin_path route => path, as: name
+        options = {
+          route => path,
+          :as   => name
+        }
+
+        match_admin_path options, suffix
       end
 
     end
@@ -102,7 +109,7 @@ module ShopifyAPI
     # @param [String] resource The name of the resource - plural or singular
     # @param [ActionDispatch::Routing::RouteSet] The router to add the matches
     # @api private
-    def match_admin_path(options = {})
+    def match_admin_path(options = {}, suffix = "")
 
       # use the router from the options, or the cached one
       router = options[:router] || @router
@@ -126,7 +133,7 @@ module ShopifyAPI
         uri << "/#{params[:id]}" if params.include?(:id)
 
         # return the redirect URI
-        uri
+        uri + suffix
       },
 
       :as => options[:as]
@@ -182,7 +189,7 @@ module ShopifyAPI
       match :links,                       [:index],               "navigation"
       match :marketing,                   [:index],               "promotions"
       match :themes,                      [:index, :show]
-      match :themes,                      [":id/settings"],       "theme_settings"
+      match :themes,                      [":id/settings"],       "theme_settings",       "/settings"
       match :general_preferences,         [:index],               "general_settings"
       match :countries,                   [:index, :show, :new],  "regions"
       match :payments,                    [:index],               "checkout_and_payment"

data/shopify-routes.gemspec

@@ -12,7 +12,9 @@ Gem::Specification.new do |gem|
   gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   gem.name          = "shopify-routes"
   gem.require_paths = ["lib"]
-  gem.version       = "0.0.3"
+  gem.version       = "0.0.4"
 
-  gem.add_runtime_dependency "shopify_api", ">= 3.0.0"
+  gem.add_runtime_dependency      "rails",                            ">= 3.0.0"
+  gem.add_runtime_dependency      "shopify_api",                      ">= 3.0.0"
+  gem.add_development_dependency  "rspec-rails",                      "~> 2.9.0"
 end

data/spec/lib/shopify-routes_spec.rb

@@ -0,0 +1,208 @@
+require 'spec_helper'
+
+describe "ShopifyAPI.routes" do
+  before do
+    @router = RailsApp::Application.routes
+    @domain = "domain.myshopify.com"
+  end
+
+  def find_route(name)
+    @router.routes.find {|r| r.name == name }
+  end
+
+  def redirect_for(resource, id = nil)
+    route   = find_route(resource)
+    params  = { :shop => @domain }
+    params[:id] = id unless id.nil?
+    route.app.path(params, nil)
+  end
+
+  def path_for(resource, id = nil)
+    path = "https://#{@domain}/admin/#{resource}"
+    path += "/#{id}" unless id.nil?
+    path
+  end
+
+  describe "redirect routes" do
+
+    describe "applications" do
+      subject { redirect_for "applications" }
+      it      { should eq path_for "applications" }
+    end
+
+    describe "orders" do
+      subject { redirect_for "orders" }
+      it      { should eq path_for "orders" }
+    end
+
+    describe "order" do
+      subject { redirect_for "order", "1" }
+      it      { should eq path_for "orders", "1" }
+    end
+
+    describe "customers" do
+      subject { redirect_for "customers" }
+      it      { should eq path_for "customers" }
+    end
+
+    describe "customer" do
+      subject { redirect_for "customer", "1" }
+      it      { should eq path_for "customers", "1" }
+    end
+
+    describe "new_customer" do
+      subject { redirect_for "new_customer" }
+      it      { should eq path_for "customers", "new" }
+    end
+
+    describe "products" do
+      subject { redirect_for "products" }
+      it      { should eq path_for "products" }
+    end
+
+    describe "product" do
+      subject { redirect_for "product", "1" }
+      it      { should eq path_for "products", "1" }
+    end
+
+    describe "new_product" do
+      subject { redirect_for "new_product" }
+      it      { should eq path_for "products", "new" }
+    end
+
+    describe "custom_collections" do
+      subject { redirect_for "custom_collections" }
+      it      { should eq path_for "custom_collections" }
+    end
+
+    describe "custom_collection" do
+      subject { redirect_for "custom_collection", "1" }
+      it      { should eq path_for "custom_collections", "1" }
+    end
+
+    describe "new_custom_collection" do
+      subject { redirect_for "new_custom_collection" }
+      it      { should eq path_for "custom_collections", "new" }
+    end
+
+    describe "smart_collections" do
+      subject { redirect_for "smart_collections" }
+      it      { should eq path_for "smart_collections" }
+    end
+
+    describe "smart_collection" do
+      subject { redirect_for "smart_collection", "1" }
+      it      { should eq path_for "smart_collections", "1" }
+    end
+
+    describe "new_smart_collection" do
+      subject { redirect_for "new_smart_collection" }
+      it      { should eq path_for "smart_collections", "new" }
+    end
+
+    describe "pages" do
+      subject { redirect_for "pages" }
+      it      { should eq path_for "pages" }
+    end
+
+    describe "page" do
+      subject { redirect_for "page", "1" }
+      it      { should eq path_for "pages", "1" }
+    end
+
+    describe "new_page" do
+      subject { redirect_for "new_page" }
+      it      { should eq path_for "pages", "new" }
+    end
+
+    describe "blogs" do
+      subject { redirect_for "blogs" }
+      it      { should eq path_for "blogs" }
+    end
+
+    describe "blog" do
+      subject { redirect_for "blog", "1" }
+      it      { should eq path_for "blogs", "1" }
+    end
+
+    describe "new_blog" do
+      subject { redirect_for "new_blog" }
+      it      { should eq path_for "blogs", "new" }
+    end
+
+    describe "navigation" do
+      subject { redirect_for "navigation" }
+      it      { should eq path_for "links" }
+    end
+
+    describe "promotions" do
+      subject { redirect_for "promotions" }
+      it      { should eq path_for "marketing" }
+    end
+
+    describe "themes" do
+      subject { redirect_for "themes" }
+      it      { should eq path_for "themes" }
+    end
+
+    describe "theme" do
+      subject { redirect_for "theme", "1" }
+      it      { should eq path_for "themes", "1" }
+    end
+
+    describe "theme_settings" do
+      subject { redirect_for "theme_settings", "1" }
+      it      { should eq path_for "themes/1/settings" }
+    end
+
+    describe "general_settings" do
+      subject { redirect_for "general_settings" }
+      it      { should eq path_for "general_preferences" }
+    end
+
+    describe "regions" do
+      subject { redirect_for "regions" }
+      it      { should eq path_for "countries" }
+    end
+
+    describe "checkout_and_payment" do
+      subject { redirect_for "checkout_and_payment" }
+      it      { should eq path_for "payments" }
+    end
+
+    describe "shipping_rates" do
+      subject { redirect_for "shipping_rates" }
+      it      { should eq path_for "shipping" }
+    end
+
+    describe "weight_based_shipping_rate" do
+      subject { redirect_for "weight_based_shipping_rate", "1" }
+      it      { should eq path_for "weight_based_shipping_rates", "1" }
+    end
+
+    describe "price_based_shipping_rate" do
+      subject { redirect_for "price_based_shipping_rate", "1" }
+      it      { should eq path_for "price_based_shipping_rates", "1" }
+    end
+
+    describe "fulfillment_services" do
+      subject { redirect_for "fulfillment_services" }
+      it      { should eq path_for "fulfillment_services" }
+    end
+
+    describe "notifications" do
+      subject { redirect_for "notifications" }
+      it      { should eq path_for "notifications" }
+    end
+
+    describe "notification" do
+      subject { redirect_for "notification", "1" }
+      it      { should eq path_for "notifications", "1" }
+    end
+
+    describe "domains" do
+      subject { redirect_for "domains" }
+      it      { should eq path_for "domains" }
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,11 @@
+require 'rubygems'
+require 'bundler/setup'
+require 'shopify-routes'
+
+# load all the support files
+Dir["spec/support/**/*.rb"].each { |f| require File.expand_path(f) }
+require 'rspec/rails' # this has to be loaded after the support files
+
+RSpec.configure do |config|
+  config.mock_with :rspec
+end

data/spec/support/rails_app.rb

@@ -0,0 +1,21 @@
+require 'rails'
+require "action_controller/railtie"
+
+module RailsApp
+  # Provides a Rails application environment to use when running the specs
+  class Application < Rails::Application
+    config.encoding = 'utf-8'
+  end
+
+  Application.configure do
+    config.active_support.deprecation = :stderr
+    config.logger = Logger.new(STDOUT)
+  end
+end
+
+Rails.env = 'test'
+RailsApp::Application.initialize!
+
+RailsApp::Application.routes.draw do
+  ShopifyAPI.routes(self)
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: shopify-routes
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,22 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-05-24 00:00:00.000000000 Z
+date: 2012-05-25 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: &25042380 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: *25042380
 - !ruby/object:Gem::Dependency
   name: shopify_api
-  requirement: &20192260 !ruby/object:Gem::Requirement
+  requirement: &25058280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,7 +32,18 @@ dependencies:
         version: 3.0.0
   type: :runtime
   prerelease: false
-  version_requirements: *20192260
+  version_requirements: *25058280
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: &25057820 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.9.0
+  type: :development
+  prerelease: false
+  version_requirements: *25057820
 description: Provides routes that redirect back to Shopify pages.
 email:
 - travis.j.haynes@gmail.com
@@ -30,12 +52,17 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .rspec
+- .travis.yml
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
 - lib/shopify-routes.rb
 - shopify-routes.gemspec
+- spec/lib/shopify-routes_spec.rb
+- spec/spec_helper.rb
+- spec/support/rails_app.rb
 homepage: ''
 licenses: []
 post_install_message: