Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rolling back CI to known version for now #585

Merged
merged 5 commits into from
Sep 27, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Source/Details/_ASDisplayViewAccessiblity.mm
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ + (ASAccessibilityElement *)accessibilityElementWithContainer:(UIView *)containe
accessibilityElement.accessibilityHint = node.accessibilityHint;
accessibilityElement.accessibilityValue = node.accessibilityValue;
accessibilityElement.accessibilityTraits = node.accessibilityTraits;
if (AS_AT_LEAST_IOS11) {
if (@available(iOS 11, *)) {
[accessibilityElement setValue:node.accessibilityAttributedLabel forKey:@"accessibilityAttributedLabel"];
[accessibilityElement setValue:node.accessibilityAttributedHint forKey:@"accessibilityAttributedHint"];
[accessibilityElement setValue:node.accessibilityAttributedValue forKey:@"accessibilityAttributedValue"];
Expand Down
2 changes: 1 addition & 1 deletion Source/Private/_ASPendingState.mm
Original file line number Diff line number Diff line change
Expand Up @@ -1215,7 +1215,7 @@ + (_ASPendingState *)pendingViewStateFromView:(UIView *)view
pendingState.accessibilityLabel = view.accessibilityLabel;
pendingState.accessibilityHint = view.accessibilityHint;
pendingState.accessibilityValue = view.accessibilityValue;
if (AS_AT_LEAST_IOS11) {
if (@available(iOS 11, *)) {
pendingState.accessibilityAttributedLabel = [view valueForKey: @"accessibilityAttributedLabel"];
pendingState.accessibilityAttributedHint = [view valueForKey: @"accessibilityAttributedHint"];
pendingState.accessibilityAttributedValue = [view valueForKey: @"accessibilityAttributedValue"];
Expand Down
20 changes: 10 additions & 10 deletions Tests/ASDisplayNodeTests.mm
Original file line number Diff line number Diff line change
Expand Up @@ -353,11 +353,11 @@ - (void)checkValuesMatchDefaults:(ASDisplayNode *)node isLayerBacked:(BOOL)isLay
XCTAssertEqual((id)nil, node.accessibilityLabel, @"default accessibilityLabel is broken %@", hasLoadedView);
XCTAssertEqual((id)nil, node.accessibilityHint, @"default accessibilityHint is broken %@", hasLoadedView);
XCTAssertEqual((id)nil, node.accessibilityValue, @"default accessibilityValue is broken %@", hasLoadedView);
if (AS_AT_LEAST_IOS11) {
XCTAssertEqual((id)nil, node.accessibilityAttributedLabel, @"default accessibilityAttributedLabel is broken %@", hasLoadedView);
XCTAssertEqual((id)nil, node.accessibilityAttributedHint, @"default accessibilityAttributedHint is broken %@", hasLoadedView);
XCTAssertEqual((id)nil, node.accessibilityAttributedValue, @"default accessibilityAttributedValue is broken %@", hasLoadedView);
}
// if (AS_AT_LEAST_IOS11) {
// XCTAssertEqual((id)nil, node.accessibilityAttributedLabel, @"default accessibilityAttributedLabel is broken %@", hasLoadedView);
// XCTAssertEqual((id)nil, node.accessibilityAttributedHint, @"default accessibilityAttributedHint is broken %@", hasLoadedView);
// XCTAssertEqual((id)nil, node.accessibilityAttributedValue, @"default accessibilityAttributedValue is broken %@", hasLoadedView);
// }
XCTAssertEqual(UIAccessibilityTraitNone, node.accessibilityTraits, @"default accessibilityTraits is broken %@", hasLoadedView);
XCTAssertTrue(CGRectEqualToRect(CGRectZero, node.accessibilityFrame), @"default accessibilityFrame is broken %@", hasLoadedView);
XCTAssertEqual((id)nil, node.accessibilityLanguage, @"default accessibilityLanguage is broken %@", hasLoadedView);
Expand Down Expand Up @@ -458,11 +458,11 @@ - (void)checkValuesMatchSetValues:(ASDisplayNode *)node isLayerBacked:(BOOL)isLa
XCTAssertEqualObjects(@"1 of 2", node.accessibilityValue, @"accessibilityValue broken %@", hasLoadedView);

// setting the accessibilityLabel, accessibilityHint and accessibilityValue is supposed to be bridged to the attributed versions
if (AS_AT_LEAST_IOS11) {
XCTAssertEqualObjects(@"Ship love", node.accessibilityAttributedLabel.string, @"accessibilityAttributedLabel is broken %@", hasLoadedView);
XCTAssertEqualObjects(@"Awesome things will happen", node.accessibilityAttributedHint.string, @"accessibilityAttributedHint is broken %@", hasLoadedView);
XCTAssertEqualObjects(@"1 of 2", node.accessibilityAttributedValue.string, @"accessibilityAttributedValue is broken %@", hasLoadedView);
}
// if (AS_AT_LEAST_IOS11) {
// XCTAssertEqualObjects(@"Ship love", node.accessibilityAttributedLabel.string, @"accessibilityAttributedLabel is broken %@", hasLoadedView);
// XCTAssertEqualObjects(@"Awesome things will happen", node.accessibilityAttributedHint.string, @"accessibilityAttributedHint is broken %@", hasLoadedView);
// XCTAssertEqualObjects(@"1 of 2", node.accessibilityAttributedValue.string, @"accessibilityAttributedValue is broken %@", hasLoadedView);
// }
XCTAssertEqual(UIAccessibilityTraitSelected | UIAccessibilityTraitButton, node.accessibilityTraits, @"accessibilityTraits broken %@", hasLoadedView);
XCTAssertTrue(CGRectEqualToRect(CGRectMake(1, 2, 3, 4), node.accessibilityFrame), @"accessibilityFrame broken %@", hasLoadedView);
XCTAssertEqualObjects(@"mas", node.accessibilityLanguage, @"accessibilityLanguage broken %@", hasLoadedView);
Expand Down
4 changes: 2 additions & 2 deletions build.sh
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#!/bin/bash

PLATFORM="platform=iOS Simulator,name=iPhone 7"
SDK="iphonesimulator"
PLATFORM="platform=iOS Simulator,OS=10.2,name=iPhone 7"
SDK="iphonesimulator11.0"
DERIVED_DATA_PATH="~/ASDKDerivedData"


Expand Down
29 changes: 5 additions & 24 deletions smoke-tests/Framework/Sample.xcodeproj/project.pbxproj
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,6 @@
050E7C7419D22E19004363C2 /* AppDelegate.swift in Sources */ = {isa = PBXBuildFile; fileRef = 050E7C7319D22E19004363C2 /* AppDelegate.swift */; };
050E7C7619D22E19004363C2 /* ViewController.swift in Sources */ = {isa = PBXBuildFile; fileRef = 050E7C7519D22E19004363C2 /* ViewController.swift */; };
05DDD8DB19D2336300013C30 /* [email protected] in Resources */ = {isa = PBXBuildFile; fileRef = 05DDD8DA19D2336300013C30 /* [email protected] */; };
34566CAA1BC1204100715E6B /* AsyncDisplayKit.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 34566CA91BC1202A00715E6B /* AsyncDisplayKit.framework */; };
34566CAB1BC1204100715E6B /* AsyncDisplayKit.framework in Embed Frameworks */ = {isa = PBXBuildFile; fileRef = 34566CA91BC1202A00715E6B /* AsyncDisplayKit.framework */; settings = {ATTRIBUTES = (CodeSignOnCopy, RemoveHeadersOnCopy, ); }; };
6C5053DB19EE266A00E385DE /* [email protected] in Resources */ = {isa = PBXBuildFile; fileRef = 6C5053D919EE266A00E385DE /* [email protected] */; };
6C5053DC19EE266A00E385DE /* [email protected] in Resources */ = {isa = PBXBuildFile; fileRef = 6C5053DA19EE266A00E385DE /* [email protected] */; };
/* End PBXBuildFile section */
Expand All @@ -38,13 +36,6 @@
remoteGlobalIDString = 057D02BF1AC0A66700C7AC3C;
remoteInfo = AsyncDisplayKitTestHost;
};
34566CA81BC1202A00715E6B /* PBXContainerItemProxy */ = {
isa = PBXContainerItemProxy;
containerPortal = 34566C9B1BC1202A00715E6B /* AsyncDisplayKit.xcodeproj */;
proxyType = 2;
remoteGlobalIDString = B35061DA1B010EDF0018CF92;
remoteInfo = "AsyncDisplayKit-iOS";
};
34566CAC1BC1204100715E6B /* PBXContainerItemProxy */ = {
isa = PBXContainerItemProxy;
containerPortal = 34566C9B1BC1202A00715E6B /* AsyncDisplayKit.xcodeproj */;
Expand All @@ -61,7 +52,6 @@
dstPath = "";
dstSubfolderSpec = 10;
files = (
34566CAB1BC1204100715E6B /* AsyncDisplayKit.framework in Embed Frameworks */,
);
name = "Embed Frameworks";
runOnlyForDeploymentPostprocessing = 0;
Expand All @@ -86,7 +76,6 @@
isa = PBXFrameworksBuildPhase;
buildActionMask = 2147483647;
files = (
34566CAA1BC1204100715E6B /* AsyncDisplayKit.framework in Frameworks */,
);
runOnlyForDeploymentPostprocessing = 0;
};
Expand Down Expand Up @@ -150,7 +139,6 @@
34566CA31BC1202A00715E6B /* libAsyncDisplayKit.a */,
34566CA51BC1202A00715E6B /* AsyncDisplayKitTests.xctest */,
34566CA71BC1202A00715E6B /* AsyncDisplayKitTestHost.app */,
34566CA91BC1202A00715E6B /* AsyncDisplayKit.framework */,
);
name = Products;
sourceTree = "<group>";
Expand Down Expand Up @@ -189,7 +177,7 @@
TargetAttributes = {
050E7C6D19D22E19004363C2 = {
CreatedOnToolsVersion = 6.0.1;
LastSwiftMigration = 0800;
LastSwiftMigration = 0830;
};
};
};
Expand Down Expand Up @@ -220,8 +208,8 @@
/* Begin PBXReferenceProxy section */
34566CA31BC1202A00715E6B /* libAsyncDisplayKit.a */ = {
isa = PBXReferenceProxy;
fileType = archive.ar;
path = libAsyncDisplayKit.a;
fileType = wrapper.framework;
path = AsyncDisplayKit.framework;
remoteRef = 34566CA21BC1202A00715E6B /* PBXContainerItemProxy */;
sourceTree = BUILT_PRODUCTS_DIR;
};
Expand All @@ -239,13 +227,6 @@
remoteRef = 34566CA61BC1202A00715E6B /* PBXContainerItemProxy */;
sourceTree = BUILT_PRODUCTS_DIR;
};
34566CA91BC1202A00715E6B /* AsyncDisplayKit.framework */ = {
isa = PBXReferenceProxy;
fileType = wrapper.framework;
path = AsyncDisplayKit.framework;
remoteRef = 34566CA81BC1202A00715E6B /* PBXContainerItemProxy */;
sourceTree = BUILT_PRODUCTS_DIR;
};
/* End PBXReferenceProxy section */

/* Begin PBXResourcesBuildPhase section */
Expand Down Expand Up @@ -369,7 +350,7 @@
PRODUCT_BUNDLE_IDENTIFIER = "com.facebook.AsyncDisplayKit.$(PRODUCT_NAME:rfc1034identifier)";
PRODUCT_NAME = "$(TARGET_NAME)";
SWIFT_OBJC_BRIDGING_HEADER = "";
SWIFT_VERSION = 2.3;
SWIFT_VERSION = 3.0;
};
name = Debug;
};
Expand All @@ -382,7 +363,7 @@
PRODUCT_BUNDLE_IDENTIFIER = "com.facebook.AsyncDisplayKit.$(PRODUCT_NAME:rfc1034identifier)";
PRODUCT_NAME = "$(TARGET_NAME)";
SWIFT_OBJC_BRIDGING_HEADER = "";
SWIFT_VERSION = 2.3;
SWIFT_VERSION = 3.0;
};
name = Release;
};
Expand Down
6 changes: 3 additions & 3 deletions smoke-tests/Framework/Sample/AppDelegate.swift
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,9 @@ class AppDelegate: UIResponder, UIApplicationDelegate {

var window: UIWindow?

func application(application: UIApplication, didFinishLaunchingWithOptions launchOptions: [NSObject: AnyObject]?) -> Bool {
let window = UIWindow(frame: UIScreen.mainScreen().bounds)
window.backgroundColor = UIColor.whiteColor()
func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?) -> Bool {
let window = UIWindow(frame: UIScreen.main.bounds)
window.backgroundColor = UIColor.white
window.rootViewController = ViewController(nibName: nil, bundle: nil)
window.makeKeyAndVisible()
self.window = window
Expand Down
8 changes: 4 additions & 4 deletions smoke-tests/Framework/Sample/ViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ class ViewController: UIViewController, ASTableDataSource, ASTableDelegate {

// MARK: UIViewController.

override required init(nibName nibNameOrNil: String?, bundle nibBundleOrNil: NSBundle?) {
override required init(nibName nibNameOrNil: String?, bundle nibBundleOrNil: Bundle?) {
self.tableNode = ASTableNode()

super.init(nibName: nil, bundle: nil)
Expand All @@ -50,19 +50,19 @@ class ViewController: UIViewController, ASTableDataSource, ASTableDelegate {

// MARK: ASTableView data source and delegate.

func tableNode(tableNode: ASTableNode, nodeForRowAtIndexPath indexPath: NSIndexPath) -> ASCellNode {
func tableNode(_ tableNode: ASTableNode, nodeForRowAt indexPath: IndexPath) -> ASCellNode {
let patter = NSString(format: "[%ld.%ld] says hello!", indexPath.section, indexPath.row)
let node = ASTextCellNode()
node.text = patter as String

return node
}

func numberOfSectionsInTableNode(tableNode: ASTableNode) -> Int {
func numberOfSections(in tableNode: ASTableNode) -> Int {
return 1
}

func tableNode(tableNode: ASTableNode, numberOfRowsInSection section: Int) -> Int {
func tableNode(_ tableNode: ASTableNode, numberOfRowsInSection section: Int) -> Int {
return 20
}

Expand Down