Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: FavIconTests/FavIconsTests.swift

Issue 29664569: Favicon: Issue 6245 - SwiftLinted project files (Closed)
Left Patch Set: Added sorted_imports ignore command to FavIconTests class Created Feb. 19, 2018, 10:32 a.m.
Right Patch Set: Removed the sorted_imports rule and reversed the import calls to be sorted alphabetically Created Feb. 19, 2018, 10:38 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « FavIcon/Utils.swift ('k') | FavIconTests/HTMLDocumentTests.swift » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 // 1 //
2 // FavIcon 2 // FavIcon
3 // Copyright © 2016 Leon Breedt 3 // Copyright © 2016 Leon Breedt
4 // 4 //
5 // Licensed under the Apache License, Version 2.0 (the "License"); 5 // Licensed under the Apache License, Version 2.0 (the "License");
6 // you may not use this file except in compliance with the License. 6 // you may not use this file except in compliance with the License.
7 // You may obtain a copy of the License at 7 // You may obtain a copy of the License at
8 // 8 //
9 // http://www.apache.org/licenses/LICENSE-2.0 9 // http://www.apache.org/licenses/LICENSE-2.0
10 // 10 //
11 // Unless required by applicable law or agreed to in writing, software 11 // Unless required by applicable law or agreed to in writing, software
12 // distributed under the License is distributed on an "AS IS" BASIS, 12 // distributed under the License is distributed on an "AS IS" BASIS,
13 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. 13 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
14 // See the License for the specific language governing permissions and 14 // See the License for the specific language governing permissions and
15 // limitations under the License. 15 // limitations under the License.
16 // 16 //
17 17
18 // swiftlint:disable sorted_imports 18 @testable import FavIcon
19
20 import XCTest 19 import XCTest
21 @testable import FavIcon
22 20
23 #if os(iOS) 21 #if os(iOS)
24 import UIKit 22 import UIKit
25 #elseif os(OSX) 23 #elseif os(OSX)
26 import Cocoa 24 import Cocoa
27 #endif 25 #endif
28 26
29 // swiftlint:disable function_body_length 27 // swiftlint:disable function_body_length
30 28
31 class FavIconTests: XCTestCase { 29 class FavIconTests: XCTestCase {
(...skipping 274 matching lines...) Expand 10 before | Expand all | Expand 10 after
306 private extension XCTestCase { 304 private extension XCTestCase {
307 func performWebRequest(name: String, timeout: TimeInterval = 50000.0, callba ck: (@escaping () -> Void) -> Void) { 305 func performWebRequest(name: String, timeout: TimeInterval = 50000.0, callba ck: (@escaping () -> Void) -> Void) {
308 FavIcon.urlSessionProvider = { URLSession.shared } 306 FavIcon.urlSessionProvider = { URLSession.shared }
309 let expectation = self.expectation(description: "web request - \(name)") 307 let expectation = self.expectation(description: "web request - \(name)")
310 callback(expectation.fulfill) 308 callback(expectation.fulfill)
311 waitForExpectations(timeout: timeout, handler: nil) 309 waitForExpectations(timeout: timeout, handler: nil)
312 } 310 }
313 } 311 }
314 312
315 // swiftlint:enable function_body_length 313 // swiftlint:enable function_body_length
LEFTRIGHT

Powered by Google App Engine
This is Rietveld