Skip to content

Commit c3bd80e

Browse files
committed
Remove some @available checks that are no long needed.
1 parent ce724ab commit c3bd80e

File tree

2 files changed

+124
-141
lines changed

2 files changed

+124
-141
lines changed

UnitTests/GTLRErrorObjectTest.m

Lines changed: 43 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -121,54 +121,50 @@ - (void)testObjectCoding {
121121

122122
// Make sure the underlying JSON support didn't get busted.
123123

124-
if (@available(macOS 10.13, iOS 11.0, tvOS 11.0, *)) {
125-
GTLRErrorObject *obj = [GTLRErrorObject object];
126-
obj.code = @123;
127-
obj.message = @"A message";
128-
129-
NSError *err;
130-
NSData *data = [NSKeyedArchiver archivedDataWithRootObject:obj
131-
requiringSecureCoding:YES
132-
error:&err];
133-
XCTAssertNil(err);
134-
XCTAssertTrue(data.length > 0);
135-
136-
NSSet<Class> *classes = [NSSet setWithArray:@[ [GTLRErrorObject class],
137-
[NSString class] ]];
138-
GTLRErrorObject *obj2 =
139-
[NSKeyedUnarchiver unarchivedObjectOfClasses:classes
140-
fromData:data
124+
GTLRErrorObject *obj = [GTLRErrorObject object];
125+
obj.code = @123;
126+
obj.message = @"A message";
127+
128+
NSError *err;
129+
NSData *data = [NSKeyedArchiver archivedDataWithRootObject:obj
130+
requiringSecureCoding:YES
131+
error:&err];
132+
XCTAssertNil(err);
133+
XCTAssertTrue(data.length > 0);
134+
135+
NSSet<Class> *classes = [NSSet setWithArray:@[ [GTLRErrorObject class],
136+
[NSString class] ]];
137+
GTLRErrorObject *obj2 =
138+
[NSKeyedUnarchiver unarchivedObjectOfClasses:classes
139+
fromData:data
140+
error:&err];
141+
XCTAssertNil(err);
142+
XCTAssertNotNil(obj2);
143+
XCTAssertNotEqual(obj, obj2); // Pointer compare
144+
XCTAssertEqualObjects(obj, obj2);
145+
146+
// Test with a foundation error.
147+
148+
err = [NSError errorWithDomain:@"my.domain"
149+
code:111
150+
userInfo:nil];
151+
obj = [GTLRErrorObject objectWithFoundationError:err];
152+
153+
err = nil;
154+
data = [NSKeyedArchiver archivedDataWithRootObject:obj
155+
requiringSecureCoding:YES
141156
error:&err];
142-
XCTAssertNil(err);
143-
XCTAssertNotNil(obj2);
144-
XCTAssertNotEqual(obj, obj2); // Pointer compare
145-
XCTAssertEqualObjects(obj, obj2);
146-
147-
// Test with a foundation error.
148-
149-
err = [NSError errorWithDomain:@"my.domain"
150-
code:111
151-
userInfo:nil];
152-
obj = [GTLRErrorObject objectWithFoundationError:err];
153-
154-
err = nil;
155-
data = [NSKeyedArchiver archivedDataWithRootObject:obj
156-
requiringSecureCoding:YES
157-
error:&err];
158-
XCTAssertNil(err);
159-
XCTAssertTrue(data.length > 0);
160-
161-
err = nil;
162-
obj2 = [NSKeyedUnarchiver unarchivedObjectOfClass:[GTLRErrorObject class]
163-
fromData:data
164-
error:&err];
165-
XCTAssertNil(err);
166-
XCTAssertNotNil(obj2);
167-
XCTAssertNotEqual(obj, obj2); // Pointer compare.
168-
XCTAssertEqualObjects(obj, obj2);
169-
} else {
170-
XCTFail("use a newer test device");
171-
}
157+
XCTAssertNil(err);
158+
XCTAssertTrue(data.length > 0);
159+
160+
err = nil;
161+
obj2 = [NSKeyedUnarchiver unarchivedObjectOfClass:[GTLRErrorObject class]
162+
fromData:data
163+
error:&err];
164+
XCTAssertNil(err);
165+
XCTAssertNotNil(obj2);
166+
XCTAssertNotEqual(obj, obj2); // Pointer compare.
167+
XCTAssertEqualObjects(obj, obj2);
172168
}
173169

174170
@end

UnitTests/GTLRObjectTest.m

Lines changed: 81 additions & 94 deletions
Original file line numberDiff line numberDiff line change
@@ -288,23 +288,18 @@ - (void)testSetBasicTypes {
288288
}
289289

290290
- (GTLRTestingObject *)objectFromRoundTripArchiveDearchiveWithObject:(GTLRTestingObject *)obj {
291-
if (@available(macOS 10.13, iOS 11.0, tvOS 11.0, *)) {
292-
NSError *err;
293-
NSData *data =
294-
[NSKeyedArchiver archivedDataWithRootObject:obj
295-
requiringSecureCoding:YES
296-
error:&err];
297-
XCTAssertNil(err);
298-
GTLRTestingObject *obj2 =
299-
[NSKeyedUnarchiver unarchivedObjectOfClass:[obj class]
300-
fromData:data
301-
error:&err];
302-
XCTAssertNil(err);
303-
return obj2;
304-
} else {
305-
XCTFail("use a newer test device");
306-
return nil;
307-
}
291+
NSError *err;
292+
NSData *data =
293+
[NSKeyedArchiver archivedDataWithRootObject:obj
294+
requiringSecureCoding:YES
295+
error:&err];
296+
XCTAssertNil(err);
297+
GTLRTestingObject *obj2 =
298+
[NSKeyedUnarchiver unarchivedObjectOfClass:[obj class]
299+
fromData:data
300+
error:&err];
301+
XCTAssertNil(err);
302+
return obj2;
308303
}
309304

310305
- (void)testSecureCodingEmptyObject {
@@ -1671,88 +1666,80 @@ - (void)testObjectCoding {
16711666
XCTAssertTrue([GTLRObject supportsSecureCoding]);
16721667
XCTAssertTrue([GTLRTestingObject supportsSecureCoding]);
16731668

1674-
if (@available(macOS 10.13, iOS 11.0, tvOS 11.0, *)) {
1675-
GTLRTestingObject *obj = [GTLRTestingObject object];
1676-
obj.aStr = @"a string";
1677-
obj.aNum = @123;
1678-
obj.aBool = @YES;
1679-
1680-
NSError *err = nil;
1681-
NSData *data =
1682-
[NSKeyedArchiver archivedDataWithRootObject:obj
1683-
requiringSecureCoding:YES
1684-
error:&err];
1685-
XCTAssertNil(err);
1686-
XCTAssertTrue(data.length > 0);
1687-
1688-
GTLRTestingObject *obj2 =
1689-
[NSKeyedUnarchiver unarchivedObjectOfClass:[GTLRTestingObject class]
1690-
fromData:data
1691-
error:&err];
1692-
XCTAssertNil(err);
1693-
XCTAssertNotNil(obj2);
1694-
XCTAssertNotEqual(obj, obj2); // Pointer compare
1695-
XCTAssertEqualObjects(obj, obj2);
1696-
} else {
1697-
XCTFail("use a newer test device");
1698-
}
1669+
GTLRTestingObject *obj = [GTLRTestingObject object];
1670+
obj.aStr = @"a string";
1671+
obj.aNum = @123;
1672+
obj.aBool = @YES;
1673+
1674+
NSError *err = nil;
1675+
NSData *data =
1676+
[NSKeyedArchiver archivedDataWithRootObject:obj
1677+
requiringSecureCoding:YES
1678+
error:&err];
1679+
XCTAssertNil(err);
1680+
XCTAssertTrue(data.length > 0);
1681+
1682+
GTLRTestingObject *obj2 =
1683+
[NSKeyedUnarchiver unarchivedObjectOfClass:[GTLRTestingObject class]
1684+
fromData:data
1685+
error:&err];
1686+
XCTAssertNil(err);
1687+
XCTAssertNotNil(obj2);
1688+
XCTAssertNotEqual(obj, obj2); // Pointer compare
1689+
XCTAssertEqualObjects(obj, obj2);
16991690
}
17001691

17011692
- (void)testBatchResultCoding {
17021693
XCTAssertTrue([GTLRBatchResult supportsSecureCoding]);
17031694

1704-
if (@available(macOS 10.13, iOS 11.0, tvOS 11.0, *)) {
1705-
GTLRTestingObject *obj = [GTLRTestingObject object];
1706-
obj.aStr = @"a string";
1707-
obj.aNum = @123;
1708-
obj.aBool = @YES;
1709-
1710-
GTLRErrorObject *err = [GTLRErrorObject object];
1711-
err.code = @101;
1712-
err.message = @"My Error";
1713-
1714-
NSDictionary *responseHeaders = @{
1715-
@"key1" : @{
1716-
@"X-Header1" : @"Value1",
1717-
@"X-Header2" : @"Value2",
1718-
},
1719-
@"key2" : @{
1720-
@"X-Header3" : @"Value4",
1721-
@"X-Header4" : @"Value3",
1722-
},
1723-
};
1724-
1725-
GTLRBatchResult *result = [GTLRBatchResult object];
1726-
result.successes = @{ @"key2" : obj };
1727-
result.failures = @{ @"key1" : err };
1728-
result.responseHeaders = responseHeaders;
1729-
1730-
NSError *error = nil;
1731-
NSData *data =
1732-
[NSKeyedArchiver archivedDataWithRootObject:result
1733-
requiringSecureCoding:YES
1734-
error:&error];
1735-
XCTAssertNil(error);
1736-
XCTAssertTrue(data.length > 0);
1737-
1738-
GTLRBatchResult *result2 =
1739-
[NSKeyedUnarchiver unarchivedObjectOfClass:[GTLRBatchResult class]
1740-
fromData:data
1741-
error:&error];
1742-
XCTAssertNil(error);
1743-
XCTAssertNotNil(result2);
1744-
XCTAssertNotEqual(result, result2); // Pointer compare.
1745-
XCTAssertNotEqual(result.successes, result2.successes); // Pointer compare.
1746-
XCTAssertNotEqual(result.failures, result2.failures); // Pointer compare.
1747-
XCTAssertNotEqual(result.responseHeaders, result2.responseHeaders); // Pointer compare.
1748-
XCTAssertNotEqual(result, result2); // Pointer compare.
1749-
XCTAssertEqualObjects(result, result2);
1750-
XCTAssertEqualObjects(result.successes, result2.successes);
1751-
XCTAssertEqualObjects(result.failures, result2.failures);
1752-
XCTAssertEqualObjects(result.responseHeaders, result2.responseHeaders);
1753-
} else {
1754-
XCTFail("use a newer test device");
1755-
}
1695+
GTLRTestingObject *obj = [GTLRTestingObject object];
1696+
obj.aStr = @"a string";
1697+
obj.aNum = @123;
1698+
obj.aBool = @YES;
1699+
1700+
GTLRErrorObject *err = [GTLRErrorObject object];
1701+
err.code = @101;
1702+
err.message = @"My Error";
1703+
1704+
NSDictionary *responseHeaders = @{
1705+
@"key1" : @{
1706+
@"X-Header1" : @"Value1",
1707+
@"X-Header2" : @"Value2",
1708+
},
1709+
@"key2" : @{
1710+
@"X-Header3" : @"Value4",
1711+
@"X-Header4" : @"Value3",
1712+
},
1713+
};
1714+
1715+
GTLRBatchResult *result = [GTLRBatchResult object];
1716+
result.successes = @{ @"key2" : obj };
1717+
result.failures = @{ @"key1" : err };
1718+
result.responseHeaders = responseHeaders;
1719+
1720+
NSError *error = nil;
1721+
NSData *data =
1722+
[NSKeyedArchiver archivedDataWithRootObject:result
1723+
requiringSecureCoding:YES
1724+
error:&error];
1725+
XCTAssertNil(error);
1726+
XCTAssertTrue(data.length > 0);
1727+
1728+
GTLRBatchResult *result2 =
1729+
[NSKeyedUnarchiver unarchivedObjectOfClass:[GTLRBatchResult class]
1730+
fromData:data
1731+
error:&error];
1732+
XCTAssertNil(error);
1733+
XCTAssertNotNil(result2);
1734+
XCTAssertNotEqual(result, result2); // Pointer compare.
1735+
XCTAssertNotEqual(result.successes, result2.successes); // Pointer compare.
1736+
XCTAssertNotEqual(result.failures, result2.failures); // Pointer compare.
1737+
XCTAssertNotEqual(result.responseHeaders, result2.responseHeaders); // Pointer compare.
1738+
XCTAssertNotEqual(result, result2); // Pointer compare.
1739+
XCTAssertEqualObjects(result, result2);
1740+
XCTAssertEqualObjects(result.successes, result2.successes);
1741+
XCTAssertEqualObjects(result.failures, result2.failures);
1742+
XCTAssertEqualObjects(result.responseHeaders, result2.responseHeaders);
17561743
}
17571744

17581745
#pragma mark NSObject description

0 commit comments

Comments
 (0)