From 9432986de1d25527b133715ab12a8dffc2c5f6b6 Mon Sep 17 00:00:00 2001 From: stuconnolly Date: Tue, 21 May 2013 21:57:57 +0000 Subject: PostgresKit: Add a common integration test case base. --- .../PostgresKit.xcodeproj/project.pbxproj | 6 ++ Frameworks/PostgresKit/Tests/PGDataTypeTests.h | 10 +-- Frameworks/PostgresKit/Tests/PGDataTypeTests.m | 41 +-------- .../Tests/PGPostgresIntegrationTestCase.h | 41 +++++++++ .../Tests/PGPostgresIntegrationTestCase.m | 96 ++++++++++++++++++++++ .../PostgresKit/Tests/PGPostgresResultTests.h | 5 +- .../PostgresKit/Tests/PGPostgresResultTests.m | 49 +---------- 7 files changed, 156 insertions(+), 92 deletions(-) create mode 100644 Frameworks/PostgresKit/Tests/PGPostgresIntegrationTestCase.h create mode 100644 Frameworks/PostgresKit/Tests/PGPostgresIntegrationTestCase.m (limited to 'Frameworks/PostgresKit') diff --git a/Frameworks/PostgresKit/PostgresKit.xcodeproj/project.pbxproj b/Frameworks/PostgresKit/PostgresKit.xcodeproj/project.pbxproj index d26fdee6..57c95355 100644 --- a/Frameworks/PostgresKit/PostgresKit.xcodeproj/project.pbxproj +++ b/Frameworks/PostgresKit/PostgresKit.xcodeproj/project.pbxproj @@ -47,6 +47,7 @@ 173D508F15BBD9BF0007F267 /* PGConstants.m in Sources */ = {isa = PBXBuildFile; fileRef = 173D508E15BBD9BF0007F267 /* PGConstants.m */; }; 173D513515BBE50D0007F267 /* PGPostgresConnectionDelegate.h in Headers */ = {isa = PBXBuildFile; fileRef = 173D513415BBE50D0007F267 /* PGPostgresConnectionDelegate.h */; settings = {ATTRIBUTES = (Public, ); }; }; 1763D486174ACA1200EA8D60 /* PGPostgresResultTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 1763D485174ACA1200EA8D60 /* PGPostgresResultTests.m */; }; + 1763D4F1174C21DE00EA8D60 /* PGPostgresIntegrationTestCase.m in Sources */ = {isa = PBXBuildFile; fileRef = 1763D4F0174C21DE00EA8D60 /* PGPostgresIntegrationTestCase.m */; }; 177AC5F015C50F6E00A3658D /* PGPostgresConnectionTypeHandling.h in Headers */ = {isa = PBXBuildFile; fileRef = 177AC5EE15C50F6E00A3658D /* PGPostgresConnectionTypeHandling.h */; }; 177AC5F115C50F6E00A3658D /* PGPostgresConnectionTypeHandling.m in Sources */ = {isa = PBXBuildFile; fileRef = 177AC5EF15C50F6E00A3658D /* PGPostgresConnectionTypeHandling.m */; }; 177AC66915C53CB000A3658D /* PGPostgresConnectionQueryExecution.h in Headers */ = {isa = PBXBuildFile; fileRef = 177AC66715C53CB000A3658D /* PGPostgresConnectionQueryExecution.h */; settings = {ATTRIBUTES = (Public, ); }; }; @@ -118,6 +119,8 @@ 173D513415BBE50D0007F267 /* PGPostgresConnectionDelegate.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PGPostgresConnectionDelegate.h; sourceTree = ""; }; 1763D484174ACA1200EA8D60 /* PGPostgresResultTests.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PGPostgresResultTests.h; sourceTree = ""; }; 1763D485174ACA1200EA8D60 /* PGPostgresResultTests.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = PGPostgresResultTests.m; sourceTree = ""; }; + 1763D4EF174C21DE00EA8D60 /* PGPostgresIntegrationTestCase.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PGPostgresIntegrationTestCase.h; sourceTree = ""; }; + 1763D4F0174C21DE00EA8D60 /* PGPostgresIntegrationTestCase.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = PGPostgresIntegrationTestCase.m; sourceTree = ""; }; 177AC5EE15C50F6E00A3658D /* PGPostgresConnectionTypeHandling.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PGPostgresConnectionTypeHandling.h; sourceTree = ""; }; 177AC5EF15C50F6E00A3658D /* PGPostgresConnectionTypeHandling.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = PGPostgresConnectionTypeHandling.m; sourceTree = ""; }; 177AC66715C53CB000A3658D /* PGPostgresConnectionQueryExecution.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PGPostgresConnectionQueryExecution.h; sourceTree = ""; }; @@ -210,6 +213,8 @@ 171D586F1612E9B900F84472 /* PGDataTypeTests.m */, 1763D484174ACA1200EA8D60 /* PGPostgresResultTests.h */, 1763D485174ACA1200EA8D60 /* PGPostgresResultTests.m */, + 1763D4EF174C21DE00EA8D60 /* PGPostgresIntegrationTestCase.h */, + 1763D4F0174C21DE00EA8D60 /* PGPostgresIntegrationTestCase.m */, ); path = Tests; sourceTree = ""; @@ -526,6 +531,7 @@ files = ( 171D58701612E9B900F84472 /* PGDataTypeTests.m in Sources */, 1763D486174ACA1200EA8D60 /* PGPostgresResultTests.m in Sources */, + 1763D4F1174C21DE00EA8D60 /* PGPostgresIntegrationTestCase.m in Sources */, ); runOnlyForDeploymentPostprocessing = 0; }; diff --git a/Frameworks/PostgresKit/Tests/PGDataTypeTests.h b/Frameworks/PostgresKit/Tests/PGDataTypeTests.h index 4d0debe1..bd1bd4a2 100644 --- a/Frameworks/PostgresKit/Tests/PGDataTypeTests.h +++ b/Frameworks/PostgresKit/Tests/PGDataTypeTests.h @@ -31,14 +31,14 @@ #import #import -@interface PGDataTypeTests : SenTestCase +#import "PGPostgresIntegrationTestCase.h" + +@interface PGDataTypeTests : PGPostgresIntegrationTestCase { id _result; id _expectedResult; - NSString *_field; - - PGPostgresConnection *_connection; + NSString *_field; } @property (readwrite, retain) id result; @@ -47,8 +47,6 @@ @property (readwrite, retain) NSString *field; -@property (readwrite, retain) PGPostgresConnection *connection; - - (id)initWithInvocation:(NSInvocation *)invocation connection:(PGPostgresConnection *)connection expectedResult:(id)result diff --git a/Frameworks/PostgresKit/Tests/PGDataTypeTests.m b/Frameworks/PostgresKit/Tests/PGDataTypeTests.m index 765404a3..60108c29 100644 --- a/Frameworks/PostgresKit/Tests/PGDataTypeTests.m +++ b/Frameworks/PostgresKit/Tests/PGDataTypeTests.m @@ -30,19 +30,8 @@ #import "PGDataTypeTests.h" -#import - -static NSString *PGTestDatabaseHost = @"localhost"; -static NSString *PGTestDatabaseUser = @"pgkit_test"; -static NSString *PGTestDatabaseName = @"pgkit_test"; -static NSString *PGTestDatabasePassword = @"pgkit"; - -static NSUInteger PGTestDatabasePort = 5432; - @interface PGDataTypeTests () -- (void)_establishConnection; - + (void)_addTestForField:(NSString *)field withExpectedResult:(id)result connection:(PGPostgresConnection *)connection @@ -54,7 +43,6 @@ static NSUInteger PGTestDatabasePort = 5432; @synthesize field = _field; @synthesize result = _result; -@synthesize connection = _connection; @synthesize expectedResult = _expectedResult; #pragma mark - @@ -111,9 +99,9 @@ static NSUInteger PGTestDatabasePort = 5432; - (void)setUp { - [self _establishConnection]; + [super setUp]; - PGPostgresResult *queryResult = [_connection executeWithFormat:@"SELECT \"%@_field\" FROM \"data_types\"", _field]; + PGPostgresResult *queryResult = [[self connection] executeWithFormat:@"SELECT \"%@_field\" FROM \"data_types\"", _field]; [self setResult:[[queryResult row] objectForKey:[NSString stringWithFormat:@"%@_field", _field]]]; } @@ -134,24 +122,6 @@ static NSUInteger PGTestDatabasePort = 5432; #pragma mark - #pragma mark Private API -- (void)_establishConnection -{ - [_connection setHost:PGTestDatabaseHost]; - [_connection setUser:PGTestDatabaseUser]; - [_connection setPort:PGTestDatabasePort]; - [_connection setDatabase:PGTestDatabaseName]; - [_connection setPassword:PGTestDatabasePassword]; - - if (![_connection connect]) { - STFail(@"Request to establish connection to local database failed."); - } - - do { - sleep(0.1); - } - while (![_connection isConnected]); -} - + (void)_addTestForField:(NSString *)field withExpectedResult:(id)result connection:(PGPostgresConnection *)connection @@ -170,14 +140,9 @@ static NSUInteger PGTestDatabasePort = 5432; #pragma mark - - (void)dealloc -{ - if (_connection && [_connection isConnected]) { - [_connection disconnect]; - } - +{ if (_result) [_result release], _result = nil; if (_field) [_field release], _field = nil; - if (_connection) [_connection release], _connection = nil; [super dealloc]; } diff --git a/Frameworks/PostgresKit/Tests/PGPostgresIntegrationTestCase.h b/Frameworks/PostgresKit/Tests/PGPostgresIntegrationTestCase.h new file mode 100644 index 00000000..feba5d43 --- /dev/null +++ b/Frameworks/PostgresKit/Tests/PGPostgresIntegrationTestCase.h @@ -0,0 +1,41 @@ +// +// $Id$ +// +// PGPostgresIntegrationTestCase.h +// PostgresKit +// +// Created by Stuart Connolly (stuconnolly.com) on May 21, 2013. +// Copyright (c) 2013 Stuart Connolly. All rights reserved. +// +// Permission is hereby granted, free of charge, to any person +// obtaining a copy of this software and associated documentation +// files (the "Software"), to deal in the Software without +// restriction, including without limitation the rights to use, +// copy, modify, merge, publish, distribute, sublicense, and/or sell +// copies of the Software, and to permit persons to whom the +// Software is furnished to do so, subject to the following +// conditions: +// +// The above copyright notice and this permission notice shall be +// included in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, +// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES +// OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND +// NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT +// HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, +// WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING +// FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR +// OTHER DEALINGS IN THE SOFTWARE. + +#import +#import + +@interface PGPostgresIntegrationTestCase : SenTestCase +{ + PGPostgresConnection *_connection; +} + +@property (readwrite, retain) PGPostgresConnection *connection; + +@end diff --git a/Frameworks/PostgresKit/Tests/PGPostgresIntegrationTestCase.m b/Frameworks/PostgresKit/Tests/PGPostgresIntegrationTestCase.m new file mode 100644 index 00000000..2e0dd2aa --- /dev/null +++ b/Frameworks/PostgresKit/Tests/PGPostgresIntegrationTestCase.m @@ -0,0 +1,96 @@ +// +// $Id$ +// +// PGPostgresIntegrationTestCase.m +// PostgresKit +// +// Created by Stuart Connolly (stuconnolly.com) on May 21, 2013. +// Copyright (c) 2013 Stuart Connolly. All rights reserved. +// +// Permission is hereby granted, free of charge, to any person +// obtaining a copy of this software and associated documentation +// files (the "Software"), to deal in the Software without +// restriction, including without limitation the rights to use, +// copy, modify, merge, publish, distribute, sublicense, and/or sell +// copies of the Software, and to permit persons to whom the +// Software is furnished to do so, subject to the following +// conditions: +// +// The above copyright notice and this permission notice shall be +// included in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, +// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES +// OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND +// NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT +// HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, +// WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING +// FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR +// OTHER DEALINGS IN THE SOFTWARE. + +#import "PGPostgresIntegrationTestCase.h" + +static NSString *PGTestDatabaseHost = @"localhost"; +static NSString *PGTestDatabaseUser = @"pgkit_test"; +static NSString *PGTestDatabaseName = @"pgkit_test"; +static NSString *PGTestDatabasePassword = @"pgkit"; + +static NSUInteger PGTestDatabasePort = 5432; + +@interface PGPostgresIntegrationTestCase () + +- (void)_establishConnection; + +@end + +@implementation PGPostgresIntegrationTestCase + +@synthesize connection = _connection; + +#pragma mark - +#pragma mark Setup & Teardown + +- (void)setUp +{ + _connection = [[PGPostgresConnection alloc] init]; + + [self _establishConnection]; +} + +#pragma mark - +#pragma mark Private API + +- (void)_establishConnection +{ + [_connection setHost:PGTestDatabaseHost]; + [_connection setUser:PGTestDatabaseUser]; + [_connection setPort:PGTestDatabasePort]; + [_connection setDatabase:PGTestDatabaseName]; + [_connection setPassword:PGTestDatabasePassword]; + + if (![_connection connect]) { + STFail(@"Request to establish connection to local database failed."); + + exit(1); + } + + do { + sleep(0.1); + } + while (![_connection isConnected]); +} + +#pragma mark - + +- (void)dealloc +{ + if (_connection && [_connection isConnected]) { + [_connection disconnect]; + } + + if (_connection) [_connection release], _connection = nil; + + [super dealloc]; +} + +@end diff --git a/Frameworks/PostgresKit/Tests/PGPostgresResultTests.h b/Frameworks/PostgresKit/Tests/PGPostgresResultTests.h index e519ebeb..f28ac361 100644 --- a/Frameworks/PostgresKit/Tests/PGPostgresResultTests.h +++ b/Frameworks/PostgresKit/Tests/PGPostgresResultTests.h @@ -31,10 +31,11 @@ #import #import -@interface PGPostgresResultTests : SenTestCase +#import "PGPostgresIntegrationTestCase.h" + +@interface PGPostgresResultTests : PGPostgresIntegrationTestCase { PGPostgresResult *_result; - PGPostgresConnection *_connection; } @end diff --git a/Frameworks/PostgresKit/Tests/PGPostgresResultTests.m b/Frameworks/PostgresKit/Tests/PGPostgresResultTests.m index 046b4ce6..5553cd75 100644 --- a/Frameworks/PostgresKit/Tests/PGPostgresResultTests.m +++ b/Frameworks/PostgresKit/Tests/PGPostgresResultTests.m @@ -30,19 +30,6 @@ #import "PGPostgresResultTests.h" -static NSString *PGTestDatabaseHost = @"localhost"; -static NSString *PGTestDatabaseUser = @"pgkit_test"; -static NSString *PGTestDatabaseName = @"pgkit_test"; -static NSString *PGTestDatabasePassword = @"pgkit"; - -static NSUInteger PGTestDatabasePort = 5432; - -@interface PGPostgresResultTests () - -- (void)_establishConnection; - -@end - @implementation PGPostgresResultTests #pragma mark - @@ -50,11 +37,9 @@ static NSUInteger PGTestDatabasePort = 5432; - (void)setUp { - _connection = [[PGPostgresConnection alloc] init]; + [super setUp]; - [self _establishConnection]; - - _result = [_connection execute:@"SELECT * FROM \"data_types\""]; + _result = [[self connection] execute:@"SELECT * FROM \"data_types\""]; } #pragma mark - @@ -80,39 +65,11 @@ static NSUInteger PGTestDatabasePort = 5432; "}"], nil); } -#pragma mark - -#pragma mark Private API - -- (void)_establishConnection -{ - [_connection setHost:PGTestDatabaseHost]; - [_connection setUser:PGTestDatabaseUser]; - [_connection setPort:PGTestDatabasePort]; - [_connection setDatabase:PGTestDatabaseName]; - [_connection setPassword:PGTestDatabasePassword]; - - if (![_connection connect]) { - STFail(@"Request to establish connection to local database failed."); - - exit(1); - } - - do { - sleep(0.1); - } - while (![_connection isConnected]); -} - #pragma mark - - (void)dealloc -{ - if (_connection && [_connection isConnected]) { - [_connection disconnect]; - } - +{ if (_result) [_result release], _result = nil; - if (_connection) [_connection release], _connection = nil; [super dealloc]; } -- cgit v1.2.3