disco_app 0.9.4

data/test/integration/synchronises_test.rb

@@ -0,0 +1,55 @@
+require 'test_helper'
+
+class SynchronisesTest < ActionDispatch::IntegrationTest
+  fixtures :all
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+    @product = products(:ipod)
+    @routes = DiscoApp::Engine.routes
+  end
+
+  def teardown
+    @shop = nil
+  end
+
+  test 'new product is created when product created webhook is received' do
+    post_webhook('product_created', :'products/create')
+
+    # Assert the product was created locally, with the correct attributes.
+    product = Product.find(632910392)
+    assert_equal 'IPod Nano - 8GB', product.data['title']
+  end
+
+  test 'existing product is updated when product updated webhook is received' do
+    assert_equal({}, @product.data)
+
+    post_webhook('product_updated', :'products/update')
+
+    # Assert the product was updated locally, with the correct attributes.
+    @product.reload
+    assert_equal 632910393, @product.id
+    assert_equal 'IPod Nano - 8GB', @product.data['title']
+  end
+
+  test 'existing product is deleted when product deleted webhook is receieved' do
+    post_webhook('product_deleted', :'products/delete')
+    assert_equal 0, Product.count
+  end
+
+  private
+
+    def webhooks_url
+      DiscoApp::Engine.routes.url_helpers.webhooks_url
+    end
+
+    def post_webhook(fixture_name, webhook_topic)
+      body = webhook_fixture(fixture_name)
+      post(webhooks_url, body, {
+        HTTP_X_SHOPIFY_TOPIC: webhook_topic,
+        HTTP_X_SHOPIFY_SHOP_DOMAIN: @shop.shopify_domain,
+        HTTP_X_SHOPIFY_HMAC_SHA256: DiscoApp::WebhookService.calculated_hmac(body, ShopifyApp.configuration.secret)
+      })
+    end
+
+end

data/test/jobs/disco_app/app_installed_job_test.rb

@@ -0,0 +1,55 @@
+require 'test_helper'
+
+class DiscoApp::AppInstalledJobTest < ActionController::TestCase
+  include ActiveJob::TestHelper
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+
+    stub_request(:get, "#{@shop.admin_url}/webhooks.json").to_return(status: 200, body: api_fixture('widget_store/webhooks').to_json)
+    stub_request(:post, "#{@shop.admin_url}/webhooks.json").to_return(status: 200)
+    stub_request(:get, "#{@shop.admin_url}/shop.json").to_return(status: 200, body: api_fixture('widget_store/shop').to_json)
+  end
+
+  def teardown
+    @shop = nil
+
+    WebMock.reset!
+  end
+
+  test 'app installed job performs shop update job' do
+    # Assert the main install job can be enqueued and performed.
+    perform_enqueued_jobs do
+      DiscoApp::AppInstalledJob.perform_later(@shop)
+    end
+
+    # Assert the update shop job was performed.
+    @shop.reload
+    assert_equal 'United States', @shop.data['country_name']
+  end
+
+  test 'app installed job automatically subscribes stores to the correct default plan' do
+    @shop.current_subscription.destroy
+
+    perform_enqueued_jobs do
+      DiscoApp::AppInstalledJob.perform_later(@shop)
+    end
+
+    # Assert the shop was subscribed to the development plan.
+    assert_equal disco_app_plans(:development), @shop.current_subscription.plan
+  end
+
+  test 'app installed job automatically subscribes stores to the correct default plan with a plan code and a source' do
+    @shop.current_subscription.destroy
+
+    perform_enqueued_jobs do
+      DiscoApp::AppInstalledJob.perform_later(@shop, 'PODCAST', 'smpodcast')
+    end
+
+    # Assert the shop was subscribed to the development plan.
+    assert_equal disco_app_plans(:development), @shop.current_subscription.plan
+    assert_equal disco_app_plan_codes(:podcast_dev), @shop.current_subscription.plan_code
+    assert_equal 'smpodcast', @shop.current_subscription.source
+  end
+
+end

data/test/jobs/disco_app/app_uninstalled_job_test.rb

@@ -0,0 +1,30 @@
+require 'test_helper'
+
+class DiscoApp::AppUninstalledJobTest < ActionController::TestCase
+  include ActiveJob::TestHelper
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+
+    perform_enqueued_jobs do
+      DiscoApp::AppUninstalledJob.perform_later(@shop, {})
+    end
+  end
+
+  def teardown
+    @shop = nil
+  end
+
+  test 'app uninstalled job changes shop status' do
+    assert_performed_jobs 1
+    @shop.reload
+    assert @shop.uninstalled?
+  end
+
+  test 'app uninstalled job can be extended using concerns' do
+    assert_performed_jobs 1
+    @shop.reload
+    assert_equal 'Nowhere', @shop.data['country_name'] # Assert extended method called.
+  end
+
+end

data/test/models/disco_app/plan_test.rb

@@ -0,0 +1,5 @@
+require 'test_helper'
+
+class DiscoApp::PlanTest < ActiveSupport::TestCase
+
+end

data/test/models/disco_app/renders_assets_test.rb

@@ -0,0 +1,100 @@
+require 'test_helper'
+
+class DiscoApp::RendersAssetsTest < ActiveSupport::TestCase
+  include ActiveJob::TestHelper
+  include DiscoApp::Test::ShopifyAPI
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+    @js_configuration = js_configurations(:js_swedish)
+    @widget_configuration = widget_configurations(:widget_swedish)
+  end
+
+  def teardown
+    @shop = nil
+    @js_configuration = nil
+    @widget_configuration = nil
+  end
+
+  ##
+  # Test queueing behaviour.
+  ##
+
+  test 'rendering of javascript asset group queued when locale changed' do
+    assert_enqueued_with(job: DiscoApp::RenderAssetGroupJob) do
+      @js_configuration.update(locale: :no)
+    end
+  end
+
+  test 'rendering of javascript asset group not queued when label changed' do
+    assert_no_enqueued_jobs do
+      @js_configuration.update(label: 'Sample')
+    end
+  end
+
+  test 'rendering of widget asset group queued when locale changed' do
+    assert_enqueued_with(job: DiscoApp::RenderAssetGroupJob, args: [@widget_configuration.shop, @widget_configuration, 'widget_assets']) do
+      @widget_configuration.update(locale: :no)
+    end
+  end
+
+  test 'rendering of liquid asset group queued when locale changed' do
+    assert_enqueued_with(job: DiscoApp::RenderAssetGroupJob, args: [@widget_configuration.shop, @widget_configuration, 'liquid_assets']) do
+      @widget_configuration.update(locale: :no)
+    end
+  end
+
+  test 'rendering of widget asset group queued when background color changed' do
+    assert_enqueued_with(job: DiscoApp::RenderAssetGroupJob, args: [@widget_configuration.shop, @widget_configuration, 'widget_assets']) do
+      @widget_configuration.update(background_color: '#FF0000')
+    end
+  end
+
+  test 'rendering of liquid asset group not queued when background color changed' do
+    assert_enqueued_jobs 1 do
+      @widget_configuration.update(background_color: '#FF0000')
+    end
+  end
+
+  ##
+  # Test rendering behaviour.
+  ##
+
+  test 'individual js asset renders correctly' do
+    assert_equal asset_fixture('test.js'), @js_configuration.send('render_asset_group_asset', 'test.js', {})
+  end
+
+  test 'js asset group renders and uploads to shopify' do
+    stub_api_request(:put, "#{@shop.admin_url}/assets.json", 'widget_store/assets/create_test_js')
+    @js_configuration.render_asset_group(:js_assets)
+  end
+
+  test 'widget asset group renders and uploads to shopify' do
+    stub_api_request(:put, "#{@shop.admin_url}/assets.json", 'widget_store/assets/create_widget_scss')
+    stub_api_request(:put, "#{@shop.admin_url}/assets.json", 'widget_store/assets/create_widget_js')
+    @widget_configuration.render_asset_group(:widget_assets)
+  end
+
+  test 'script tag asset group renders, uploads to shopify and creates new script tag' do
+    stub_api_request(:put, "#{@shop.admin_url}/assets.json", 'widget_store/assets/create_script_tag_js')
+    stub_api_request(:get, "#{@shop.admin_url}/script_tags.json", 'widget_store/assets/get_script_tags_empty')
+    stub_api_request(:post, "#{@shop.admin_url}/script_tags.json", 'widget_store/assets/create_script_tag')
+    @widget_configuration.render_asset_group(:script_tag_assets)
+  end
+
+  test 'script tag asset group renders, uploads to shopify and updates existing script tag' do
+    stub_api_request(:put, "#{@shop.admin_url}/assets.json", 'widget_store/assets/create_script_tag_js')
+    stub_api_request(:get, "#{@shop.admin_url}/script_tags.json", 'widget_store/assets/get_script_tags_preexisting')
+    stub_api_request(:put, "#{@shop.admin_url}/script_tags/38138886.json", 'widget_store/assets/update_script_tag')
+    @widget_configuration.render_asset_group(:script_tag_assets)
+  end
+
+  private
+
+    # Return an asset fixture as a string.
+    def asset_fixture(path)
+      filename = File.join(File.dirname(File.dirname(File.dirname(__FILE__))), 'fixtures', 'assets', "#{path}")
+      File.read(filename)
+    end
+
+end

data/test/models/disco_app/session_test.rb

@@ -0,0 +1,31 @@
+require 'test_helper'
+
+class DiscoApp::SessionTest < ActiveSupport::TestCase
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+    @session = DiscoApp::Session.create(
+      session_id: 'a91bfc51fa79c9d09d43e2615d9345d4',
+      data: {
+        :shopify => @shop.id,
+        :shopify_domain => @shop.shopify_domain
+      }
+    )
+  end
+
+  test 'logged in sessions are linked to their shop' do
+    assert_equal @shop.id, @session.shop_id
+  end
+
+  test 'can fetch sessions for a particular shop through association' do
+    assert_equal 1, @shop.sessions.size
+  end
+
+  test 'sessions can be deleted by shop' do
+    DiscoApp::Session.create(session_id: 'a91bfc51fa79c9d09d43e2615d9345d5', data: {})
+    assert_equal 2, DiscoApp::Session.count
+    @shop.sessions.delete_all
+    assert_equal 1, DiscoApp::Session.count
+  end
+
+end

data/test/models/disco_app/shop_test.rb

@@ -0,0 +1,27 @@
+require 'test_helper'
+
+class DiscoApp::ShopTest < ActiveSupport::TestCase
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+  end
+
+  def teardown
+    @shop = nil
+  end
+
+  test 'shops can be extended via concerns' do
+    assert_equal 'Australia', @shop.country.name
+  end
+
+  test 'can fetch a list of all of a shops subscriptions' do
+    assert_equal 2, @shop.subscriptions.size
+  end
+
+  test 'can fetch a shops current subscription' do
+    assert_equal 1, @shop.subscriptions.active.size
+    assert @shop.current_subscription?
+    assert_equal disco_app_subscriptions(:current_widget_store_subscription), @shop.current_subscription
+  end
+
+end

data/test/models/disco_app/subscription_test.rb

@@ -0,0 +1,19 @@
+require 'test_helper'
+
+class DiscoApp::SubscriptionTest < ActiveSupport::TestCase
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+  end
+
+  def teardown
+    @shop = nil
+  end
+
+  test 'changing the amount on a subscription cancels its charge' do
+    assert @shop.current_subscription.active_charge?
+    @shop.current_subscription.update(amount: 1138)
+    assert_not @shop.current_subscription.active_charge?
+  end
+
+end

data/test/services/disco_app/charges_service_test.rb

@@ -0,0 +1,112 @@
+require 'test_helper'
+
+class DiscoApp::ChargesServiceTest < ActiveSupport::TestCase
+  include DiscoApp::Test::ShopifyAPI
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+    @subscription = disco_app_subscriptions(:current_widget_store_subscription)
+    @new_charge = disco_app_recurring_application_charges(:new_widget_store_subscription_recurring_charge)
+
+    @dev_shop = disco_app_shops(:widget_store_dev)
+    @dev_subscription = disco_app_subscriptions(:current_widget_store_dev_subscription)
+    @dev_new_charge = disco_app_application_charges(:new_widget_store_dev_subscription_one_time_charge)
+
+    Timecop.freeze
+  end
+
+  def teardown
+    @shop = nil
+    @subscription = nil
+    @new_charge = nil
+
+    @dev_shop = nil
+    @dev_subscription = nil
+    @dev_new_charge = nil
+
+    Timecop.return
+
+    WebMock.reset!
+  end
+
+  test 'creating a new charge for a recurring subscription is successful' do
+    res = { "recurring_application_charge": { "name": "Basic",
+                    "price": "9.99",
+                    "trial_days": 14,
+                    "return_url": /^https:\/\/test\.example\.com\/subscriptions\/304261385\/charges\/53297050(1|2)\/activate$/,
+                    "test": true 
+      } }
+    stub_request(:post, "#{@shop.admin_url}/recurring_application_charges.json")
+      .with(body: res
+    ).to_return(status: 201, body:api_fixture("widget_store/charges/create_recurring_application_charge_response").to_json)
+
+    new_charge = DiscoApp::ChargesService.create(@shop, @subscription)
+    assert_equal 654381179, new_charge.shopify_id
+    assert_equal 'https://apple.myshopify.com/admin/charges/654381179/confirm_recurring_application_charge?signature=BAhpBHsQASc%3D--b2e90c6e4e94fbae15a464c566a31a1c23e6bffa', new_charge.confirmation_url
+  end
+
+  test 'activating a pending recurring charge is not successful' do
+    stub_api_request(:get, "#{@shop.admin_url}/recurring_application_charges/654381179.json", 'widget_store/charges/get_pending_recurring_application_charge')
+    assert_not DiscoApp::ChargesService.activate(@shop, @new_charge)
+    assert @new_charge.pending?
+  end
+
+  test 'activating a declined recurring charge is not successful' do
+    stub_api_request(:get, "#{@shop.admin_url}/recurring_application_charges/654381179.json", 'widget_store/charges/get_declined_recurring_application_charge')
+    assert_not DiscoApp::ChargesService.activate(@shop, @new_charge)
+    assert @new_charge.declined?
+  end
+
+  test 'activating an accepted recurring charge is successful and cancels any existing recurring charges' do
+    stub_api_request(:get, "#{@shop.admin_url}/recurring_application_charges/654381179.json", 'widget_store/charges/get_accepted_recurring_application_charge')
+    stub_api_request(:post, "#{@shop.admin_url}/recurring_application_charges/654381179/activate.json", 'widget_store/charges/activate_recurring_application_charge')
+
+    old_charge = @subscription.active_charge
+    assert old_charge.active?
+
+    assert DiscoApp::ChargesService.activate(@shop, @new_charge)
+    assert @new_charge.active?
+
+    old_charge.reload
+    assert old_charge.cancelled?
+  end
+
+  test 'activating an accepted recurring charge cancels other recurring charges' do
+    stub_api_request(:get, "#{@shop.admin_url}/recurring_application_charges/654381179.json", 'widget_store/charges/get_accepted_recurring_application_charge')
+    stub_api_request(:post, "#{@shop.admin_url}/recurring_application_charges/654381179/activate.json", 'widget_store/charges/activate_recurring_application_charge')
+
+    assert DiscoApp::ChargesService.activate(@shop, @new_charge)
+    assert @new_charge.active?
+  end
+
+  test 'creating a new charge for a one-time subscription is successful' do
+    stub_api_request(:post, "#{@dev_shop.admin_url}/application_charges.json", 'widget_store/charges/create_application_charge')
+
+    new_charge = DiscoApp::ChargesService.create(@dev_shop, @dev_subscription)
+    assert_equal 1012637323, new_charge.shopify_id
+    assert_equal 'https://apple.myshopify.com/admin/charges/1012637323/confirm_application_charge?signature=BAhpBIueWzw%3D--0ea1abacaf9d6fd538b7e9a7023e9b71ce1c7e98', new_charge.confirmation_url
+  end
+
+  test 'activating a pending one-time charge is not successful' do
+    stub_api_request(:get, "#{@dev_shop.admin_url}/application_charges/1012637323.json", 'widget_store/charges/get_pending_application_charge')
+
+    assert_not DiscoApp::ChargesService.activate(@dev_shop, @dev_new_charge)
+    assert @dev_new_charge.pending?
+  end
+
+  test 'activating a declined one-time charge is not successful' do
+    stub_api_request(:get, "#{@dev_shop.admin_url}/application_charges/1012637323.json", 'widget_store/charges/get_declined_application_charge')
+
+    assert_not DiscoApp::ChargesService.activate(@dev_shop, @dev_new_charge)
+    assert @dev_new_charge.declined?
+  end
+
+  test 'activating an accepted one-time charge is successful' do
+    stub_api_request(:get, "#{@dev_shop.admin_url}/application_charges/1012637323.json", 'widget_store/charges/get_accepted_application_charge')
+    stub_api_request(:post, "#{@dev_shop.admin_url}/application_charges/1012637323/activate.json", 'widget_store/charges/activate_application_charge')
+
+    assert DiscoApp::ChargesService.activate(@dev_shop, @dev_new_charge)
+    assert @dev_new_charge.active?
+  end
+
+end

data/test/services/disco_app/subscription_service_test.rb

@@ -0,0 +1,60 @@
+require 'test_helper'
+
+class DiscoApp::SubscriptionServiceTest < ActiveSupport::TestCase
+  include ActiveJob::TestHelper
+
+  def setup
+    @shop = disco_app_shops(:widget_store)
+    @subscription = disco_app_subscriptions(:current_widget_store_subscription)
+    Timecop.freeze
+  end
+
+  def teardown
+    @shop = nil
+    @subscription = nil
+    Timecop.return
+  end
+
+  test 'subscribing to a new plan deactivates current subscription and swaps to new plan' do
+    DiscoApp::SubscriptionService.subscribe(@shop, disco_app_plans(:premium))
+    @subscription.reload
+    assert @subscription.cancelled?
+    assert_equal disco_app_plans(:premium), @shop.current_subscription.plan
+  end
+
+  test 'subscribing to a new plan enqueues subscription changed background job' do
+    assert_enqueued_with(job: DiscoApp::SubscriptionChangedJob) do
+      DiscoApp::SubscriptionService.subscribe(@shop, disco_app_plans(:premium))
+    end
+  end
+
+  test 'subscribing to a new plan works for a shop without a subscription' do
+    @subscription.destroy
+    DiscoApp::SubscriptionService.subscribe(@shop, disco_app_plans(:premium))
+    assert_equal disco_app_plans(:premium), @shop.current_subscription.plan
+  end
+
+  test 'new subscription for a plan without a trial period created correctly' do
+    new_subscription = DiscoApp::SubscriptionService.subscribe(@shop, disco_app_plans(:comped))
+    assert new_subscription.active?
+    assert_equal nil, new_subscription.trial_start_at
+    assert_equal nil, new_subscription.trial_end_at
+  end
+
+  test 'new subscription for a plan with a trial period created correctly' do
+    new_subscription = DiscoApp::SubscriptionService.subscribe(@shop, disco_app_plans(:premium))
+    assert new_subscription.trial?
+    assert_equal Time.now, new_subscription.trial_start_at
+    assert_equal 28.days.from_now, new_subscription.trial_end_at
+  end
+
+  test 'new subscription for a plan with a plan code created correctly' do
+    new_subscription = DiscoApp::SubscriptionService.subscribe(@shop, disco_app_plans(:premium), 'PODCAST')
+    assert new_subscription.trial?
+    assert_equal Time.now, new_subscription.trial_start_at
+    assert_equal 60.days.from_now, new_subscription.trial_end_at
+    assert_equal 60, new_subscription.trial_period_days
+    assert_equal 8999, new_subscription.amount
+  end
+
+end

data/test/support/test_file_fixtures.rb

@@ -0,0 +1,29 @@
+module DiscoApp::Test
+  module FileFixtures
+
+    # Return an XML fixture as an XML string.
+    def xml_fixture(path)
+      filename = File.join(File.dirname(File.dirname(__FILE__)), 'fixtures', 'xml', "#{path}.xml")
+      File.read(filename)
+    end
+
+    # Return a JSON fixture as an indifferent hash.
+    def json_fixture(path)
+      filename = File.join(File.dirname(File.dirname(__FILE__)), 'fixtures', 'json', "#{path}.json")
+      HashWithIndifferentAccess.new(ActiveSupport::JSON.decode(File.read(filename)))
+    end
+
+    # API fixtures are special-case JSON fixtures.
+    def api_fixture(path)
+      filename = File.join(File.dirname(File.dirname(__FILE__)), 'fixtures', 'api', "#{path}.json")
+      HashWithIndifferentAccess.new(ActiveSupport::JSON.decode(File.read(filename)))
+    end
+
+    # Webhook fixtures are special-case JSON fixtures.
+    def webhook_fixture(path)
+      filename = File.join(File.dirname(File.dirname(__FILE__)), 'fixtures', 'webhooks', "#{path}.json")
+      File.read(filename)
+    end
+
+  end
+end

data/test/support/test_shopify_api.rb

@@ -0,0 +1,16 @@
+module DiscoApp::Test
+  module ShopifyAPI
+
+    def stub_api_request(method, endpoint, fixture_name)
+      if method == :get
+        stub_request(method, endpoint)
+          .to_return(status: 200, body: api_fixture("#{fixture_name}_response").to_json)
+      elsif method == :post or method == :put
+        stub_request(method, endpoint)
+          .with(body: api_fixture("#{fixture_name}_request").to_json)
+          .to_return(status: 201, body: api_fixture("#{fixture_name}_response").to_json)
+      end
+    end
+
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,55 @@
+# Prevent warnings from showing up during testing.
+$VERBOSE=nil
+
+# Configure Rails Environment
+ENV['RAILS_ENV'] = 'test'
+ENV['DEFAULT_HOST'] = 'https://test.example.com'
+ENV['SHOPIFY_APP_NAME'] = 'Test Application'
+ENV['SHOPIFY_APP_API_KEY'] = 'f61b26d635309536c3c83c0adc3cb972'
+ENV['SHOPIFY_APP_SECRET'] = 'b607d1f8b992dccb017f9315f07af9c4'
+ENV['SHOPIFY_APP_REDIRECT_URI'] = 'https://test.example.com/shopify/auth/callback'
+ENV['SHOPIFY_APP_SCOPE'] = 'read_products'
+ENV['SHOPIFY_CHARGES_REAL'] = 'false'
+
+require File.expand_path("../../test/dummy/config/environment.rb",  __FILE__)
+ActiveRecord::Migrator.migrations_paths = [File.expand_path("../../test/dummy/db/migrate", __FILE__)]
+ActiveRecord::Migrator.migrations_paths << File.expand_path('../../db/migrate', __FILE__)
+require "rails/test_help"
+
+# Require our additional test support helpers.
+require 'support/test_file_fixtures'
+require 'support/test_shopify_api'
+
+# Require WebMock
+require 'webmock/minitest'
+
+# Filter out Minitest backtrace while allowing backtrace from other libraries
+# to be shown.
+Minitest.backtrace_filter = Minitest::BacktraceFilter.new
+
+# Load support files
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+
+# Load fixtures from the engine
+if ActiveSupport::TestCase.respond_to?(:fixture_path=)
+  ActiveSupport::TestCase.fixture_path = File.expand_path("../fixtures", __FILE__)
+  ActiveSupport::TestCase.fixtures :all
+end
+
+# Set up the base test class.
+class ActiveSupport::TestCase
+
+  # Include helper modules common to all tests.
+  include DiscoApp::Test::FileFixtures
+
+  def log_in_as(shop)
+    session[:shopify] = shop.id
+    session[:shopify_domain] = shop.shopify_domain
+  end
+
+  def log_out
+    session[:shopify] = nil
+    session[:shopify_domain] = nil
+  end
+
+end