Skip to content

Commit 696fa23

Browse files
committed
Rename site creation error JSON name to mention domain, and update SiteRemote unit tests.
1 parent 80974fb commit 696fa23

File tree

3 files changed

+15
-6
lines changed

3 files changed

+15
-6
lines changed

Networking/Networking.xcodeproj/project.pbxproj

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636
025CA2C6238F4F3500B05C81 /* ProductShippingClassRemoteTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 025CA2C5238F4F3500B05C81 /* ProductShippingClassRemoteTests.swift */; };
3737
025CA2C8238F4FF400B05C81 /* product-shipping-classes-load-all.json in Resources */ = {isa = PBXBuildFile; fileRef = 025CA2C7238F4FF400B05C81 /* product-shipping-classes-load-all.json */; };
3838
02616F8C292132800095BC00 /* SiteRemoteTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 02616F8B292132800095BC00 /* SiteRemoteTests.swift */; };
39-
02616F8F2921336C0095BC00 /* site-creation-error.json in Resources */ = {isa = PBXBuildFile; fileRef = 02616F8D2921336C0095BC00 /* site-creation-error.json */; };
39+
02616F8F2921336C0095BC00 /* site-creation-domain-error.json in Resources */ = {isa = PBXBuildFile; fileRef = 02616F8D2921336C0095BC00 /* site-creation-domain-error.json */; };
4040
02616F902921336C0095BC00 /* site-creation-success.json in Resources */ = {isa = PBXBuildFile; fileRef = 02616F8E2921336C0095BC00 /* site-creation-success.json */; };
4141
0261F5A928D4641500B7AC72 /* products-sku-search.json in Resources */ = {isa = PBXBuildFile; fileRef = 0261F5A828D4641500B7AC72 /* products-sku-search.json */; };
4242
02698CF624C17FC1005337C4 /* product-alternative-types.json in Resources */ = {isa = PBXBuildFile; fileRef = 02698CF524C17FC1005337C4 /* product-alternative-types.json */; };
@@ -785,7 +785,7 @@
785785
025CA2C5238F4F3500B05C81 /* ProductShippingClassRemoteTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ProductShippingClassRemoteTests.swift; sourceTree = "<group>"; };
786786
025CA2C7238F4FF400B05C81 /* product-shipping-classes-load-all.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = "product-shipping-classes-load-all.json"; sourceTree = "<group>"; };
787787
02616F8B292132800095BC00 /* SiteRemoteTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SiteRemoteTests.swift; sourceTree = "<group>"; };
788-
02616F8D2921336C0095BC00 /* site-creation-error.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = "site-creation-error.json"; sourceTree = "<group>"; };
788+
02616F8D2921336C0095BC00 /* site-creation-domain-error.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = "site-creation-domain-error.json"; sourceTree = "<group>"; };
789789
02616F8E2921336C0095BC00 /* site-creation-success.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = "site-creation-success.json"; sourceTree = "<group>"; };
790790
0261F5A828D4641500B7AC72 /* products-sku-search.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = "products-sku-search.json"; sourceTree = "<group>"; };
791791
02698CF524C17FC1005337C4 /* product-alternative-types.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = "product-alternative-types.json"; sourceTree = "<group>"; };
@@ -2181,7 +2181,7 @@
21812181
2670C3FD270F4E6A002FE931 /* sites-malformed.json */,
21822182
7426CA1221AF34A3004E9FFC /* site-api.json */,
21832183
74AB5B4E21AF3F0D00859C12 /* site-api-no-woo.json */,
2184-
02616F8D2921336C0095BC00 /* site-creation-error.json */,
2184+
02616F8D2921336C0095BC00 /* site-creation-domain-error.json */,
21852185
02616F8E2921336C0095BC00 /* site-creation-success.json */,
21862186
CE50346621B5DCBE007573C6 /* site-plan.json */,
21872187
453305EC2459E1AA00264E50 /* site-post.json */,
@@ -2900,7 +2900,7 @@
29002900
028CB717290223CB00331C09 /* create-account-error-password.json in Resources */,
29012901
D800DA0E25EFEC21001E13CE /* wcpay-connection-token.json in Resources */,
29022902
74159628224D63CE003C21CF /* settings-product-alt.json in Resources */,
2903-
02616F8F2921336C0095BC00 /* site-creation-error.json in Resources */,
2903+
02616F8F2921336C0095BC00 /* site-creation-domain-error.json in Resources */,
29042904
45AB8B2024AB3E1F00B5B36E /* product-tags-empty.json in Resources */,
29052905
0359EA2127AAE58C0048DE2D /* wcpay-charge-card-present.json in Resources */,
29062906
451274A625276C82009911FF /* product-variation.json in Resources */,

Networking/NetworkingTests/Remote/SiteRemoteTests.swift

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,18 @@ final class SiteRemoteTests: XCTestCase {
3636
siteSlug: "wapuu.store"))
3737
}
3838

39-
func test_createSite_returns_failure_on_error() async throws {
39+
func test_createSite_returns_invalidDomain_error_when_domain_is_empty() async throws {
40+
// When
41+
let result = await remote.createSite(name: "Wapuu swags", domain: "")
42+
43+
// Then
44+
let error = try XCTUnwrap(result.failure as? SiteCreationError)
45+
XCTAssertEqual(error, .invalidDomain)
46+
}
47+
48+
func test_createSite_returns_DotcomError_failure_on_domain_error() async throws {
4049
// Given
41-
network.simulateResponse(requestUrlSuffix: "sites/new", filename: "site-creation-error")
50+
network.simulateResponse(requestUrlSuffix: "sites/new", filename: "site-creation-domain-error")
4251

4352
// When
4453
let result = await remote.createSite(name: "Wapuu swags", domain: "wapuu.store")

0 commit comments

Comments
 (0)