Skip to content

Commit 56cb936

Browse files
committed
Use HTTPS instead of HTTP in test.com URL in unit tests
I noticed a runtime warning in the Wormholy library: > App Transport Security has blocked a cleartext HTTP connection > totest.comsince [sic] it is insecure. Use HTTPS instead or add this domain > to Exception Domains in your Info.plist.
1 parent dcb836d commit 56cb936

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

Networking/NetworkingTests/Remote/JetpackConnectionRemoteTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import XCTest
33

44
final class JetpackConnectionRemoteTests: XCTestCase {
55

6-
private let siteURL = "http://test.com"
6+
private let siteURL = "https://test.com"
77

88
/// Dummy Network Wrapper
99
///

Networking/NetworkingTests/Requests/WordPressOrgRequestTests.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import XCTest
33

44
final class WordPressOrgRequestTests: XCTestCase {
55

6-
private let baseURL = "http://test.com"
6+
private let baseURL = "https://test.com"
77
private let path = "/test/request"
88

99
func test_request_url_is_correct() throws {
@@ -14,7 +14,7 @@ final class WordPressOrgRequestTests: XCTestCase {
1414
let url = try XCTUnwrap(request.asURLRequest().url)
1515

1616
// Then
17-
let expectedURL = "http://test.com/wp-json/test/request"
17+
let expectedURL = "https://test.com/wp-json/test/request"
1818
assertEqual(url.absoluteString, expectedURL)
1919
}
2020

WooCommerce/WooCommerceTests/Authentication/AuthenticationManagerTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import Yosemite
77
/// Test cases for `AuthenticationManager`.
88
final class AuthenticationManagerTests: XCTestCase {
99

10-
let testSiteURL = "http://test.com"
10+
let testSiteURL = "https://test.com"
1111

1212
/// We do not allow automatic WPCOM account sign-up if the user entered an email that is not
1313
/// registered in WordPress.com. This configuration is set up in

Yosemite/YosemiteTests/Stores/JetpackConnectionStoreTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import XCTest
44

55
final class JetpackConnectionStoreTests: XCTestCase {
66

7-
private let siteURL = "http://test.com"
7+
private let siteURL = "https://test.com"
88

99
/// Mock Dispatcher
1010
///

0 commit comments

Comments
 (0)