Skip to content

Commit

Permalink
Rename method, move to new Auth module
Browse files Browse the repository at this point in the history
  • Loading branch information
mkevinosullivan committed Aug 5, 2022
1 parent e009c91 commit 8d49a90
Show file tree
Hide file tree
Showing 4 changed files with 61 additions and 65 deletions.
26 changes: 26 additions & 0 deletions lib/shopify_api/auth.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
# typed: strict
# frozen_string_literal: true

module ShopifyAPI
module Auth
extend T::Sig

class << self
extend T::Sig

sig { params(host: T.nilable(String)).returns(String) }
def embedded_app_url(host)
unless Context.setup?
raise Errors::ContextNotSetupError, "ShopifyAPI::Context not setup, please call ShopifyAPI::Context.setup"
end

unless host
raise Errors::MissingRequiredArgumentError, "host argument is required"
end

decoded_host = Base64.decode64(host)
"https://#{decoded_host}/apps/#{Context.api_key}"
end
end
end
end
28 changes: 0 additions & 28 deletions lib/shopify_api/utils/app_url.rb

This file was deleted.

35 changes: 35 additions & 0 deletions test/auth_test.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
# typed: false
# frozen_string_literal: true

require_relative "./test_helper"

module ShopifyAPITest
class AuthTest < Test::Unit::TestCase
def setup
super
@host = "some-shopify-store.myshopify.com/admin"
@encoded_host = Base64.strict_encode64(@host)
end

def test_valid_host
assert_equal(
ShopifyAPI::Auth.embedded_app_url(@encoded_host),
"https://#{@host}/apps/#{ShopifyAPI::Context.api_key}"
)
end

def test_no_host
assert_raises(ShopifyAPI::Errors::MissingRequiredArgumentError) do
ShopifyAPI::Auth.embedded_app_url(nil)
end
end

def test_context_not_setup
modify_context(api_key: "", api_secret_key: "", host_name: "")

assert_raises(ShopifyAPI::Errors::ContextNotSetupError) do
ShopifyAPI::Auth.embedded_app_url(@encoded_host)
end
end
end
end
37 changes: 0 additions & 37 deletions test/utils/app_url_test.rb

This file was deleted.

0 comments on commit 8d49a90

Please sign in to comment.