Bgerstle has uploaded a new change for review. https://gerrit.wikimedia.org/r/209529
Change subject: fix parsing test ...................................................................... fix parsing test Change-Id: Ib2e35990c8f0c55e938c0c5a93c99883521d1dd5 --- M Wikipedia.xcodeproj/project.pbxproj A WikipediaUnitTests/MWKArticle_ShareSnippetTests.m M WikipediaUnitTests/NSString+WMFHTMLParsingTests.m D WikipediaUnitTests/WMFShareSnippetTests.m 4 files changed, 52 insertions(+), 59 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/apps/ios/wikipedia refs/changes/29/209529/1 diff --git a/Wikipedia.xcodeproj/project.pbxproj b/Wikipedia.xcodeproj/project.pbxproj index 2fdbf1c..eaa3407 100644 --- a/Wikipedia.xcodeproj/project.pbxproj +++ b/Wikipedia.xcodeproj/project.pbxproj @@ -290,7 +290,7 @@ BCC185D81A9E5628005378F8 /* UILabel+WMFStyling.m in Sources */ = {isa = PBXBuildFile; fileRef = BCC185D71A9E5628005378F8 /* UILabel+WMFStyling.m */; }; BCC185E01A9EC836005378F8 /* UIButton+FrameUtils.m in Sources */ = {isa = PBXBuildFile; fileRef = BCC185DF1A9EC836005378F8 /* UIButton+FrameUtils.m */; }; BCC185E81A9FA498005378F8 /* UICollectionViewFlowLayout+AttributeUtils.m in Sources */ = {isa = PBXBuildFile; fileRef = BCC185E71A9FA498005378F8 /* UICollectionViewFlowLayout+AttributeUtils.m */; }; - BCC9B2681AFA615000FF9593 /* WMFShareSnippetTests.m in Sources */ = {isa = PBXBuildFile; fileRef = BCC9B2671AFA615000FF9593 /* WMFShareSnippetTests.m */; }; + BCC9B2681AFA615000FF9593 /* MWKArticle_ShareSnippetTests.m in Sources */ = {isa = PBXBuildFile; fileRef = BCC9B2671AFA615000FF9593 /* MWKArticle_ShareSnippetTests.m */; }; BCC9B26B1AFA61DB00FF9593 /* MWKArticle+ShareSnippet.m in Sources */ = {isa = PBXBuildFile; fileRef = BCC9B26A1AFA61DB00FF9593 /* MWKArticle+ShareSnippet.m */; }; BCCED2D01AE03BE20094EB7E /* MWKSectionListTests.m in Sources */ = {isa = PBXBuildFile; fileRef = BCCED2CF1AE03BE20094EB7E /* MWKSectionListTests.m */; }; BCDB75C41AB0E8300005593F /* WMFSubstringUtilsTests.m in Sources */ = {isa = PBXBuildFile; fileRef = BCDB75C31AB0E8300005593F /* WMFSubstringUtilsTests.m */; }; @@ -896,7 +896,7 @@ BCC185DF1A9EC836005378F8 /* UIButton+FrameUtils.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = "UIButton+FrameUtils.m"; sourceTree = "<group>"; }; BCC185E61A9FA498005378F8 /* UICollectionViewFlowLayout+AttributeUtils.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = "UICollectionViewFlowLayout+AttributeUtils.h"; sourceTree = "<group>"; }; BCC185E71A9FA498005378F8 /* UICollectionViewFlowLayout+AttributeUtils.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = "UICollectionViewFlowLayout+AttributeUtils.m"; sourceTree = "<group>"; }; - BCC9B2671AFA615000FF9593 /* WMFShareSnippetTests.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = WMFShareSnippetTests.m; sourceTree = "<group>"; }; + BCC9B2671AFA615000FF9593 /* MWKArticle_ShareSnippetTests.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = MWKArticle_ShareSnippetTests.m; sourceTree = "<group>"; }; BCC9B2691AFA61DB00FF9593 /* MWKArticle+ShareSnippet.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = "MWKArticle+ShareSnippet.h"; sourceTree = "<group>"; }; BCC9B26A1AFA61DB00FF9593 /* MWKArticle+ShareSnippet.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = "MWKArticle+ShareSnippet.m"; sourceTree = "<group>"; }; BCCED2CF1AE03BE20094EB7E /* MWKSectionListTests.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = MWKSectionListTests.m; sourceTree = "<group>"; }; @@ -2069,7 +2069,7 @@ 0EBC56951AD5B22800E82CDD /* BITHockeyManagerWMFExtensionsTests.m */, BCCED2CF1AE03BE20094EB7E /* MWKSectionListTests.m */, BC49B3631AEECFD8009F55BE /* ArticleLoadingTests.m */, - BCC9B2671AFA615000FF9593 /* WMFShareSnippetTests.m */, + BCC9B2671AFA615000FF9593 /* MWKArticle_ShareSnippetTests.m */, BC92A7721AFA88D3003C4212 /* MWKSection+WMFSharingTests.m */, ); path = WikipediaUnitTests; @@ -2920,7 +2920,7 @@ 0EBC56961AD5B22800E82CDD /* BITHockeyManagerWMFExtensionsTests.m in Sources */, BC0FED6A1AAA0268002488D7 /* MWKDataStorePathTests.m in Sources */, BC0FED761AAA026C002488D7 /* NSString+WMFHTMLParsingTests.m in Sources */, - BCC9B2681AFA615000FF9593 /* WMFShareSnippetTests.m in Sources */, + BCC9B2681AFA615000FF9593 /* MWKArticle_ShareSnippetTests.m in Sources */, BCA676571AC05FE200A16160 /* XCTestCase+WMFBundleConvenience.m in Sources */, BC0FED671AAA0268002488D7 /* MWKTitleTests.m in Sources */, BC0FED631AAA0263002488D7 /* MWKTestCase.m in Sources */, diff --git a/WikipediaUnitTests/MWKArticle_ShareSnippetTests.m b/WikipediaUnitTests/MWKArticle_ShareSnippetTests.m new file mode 100644 index 0000000..59635da --- /dev/null +++ b/WikipediaUnitTests/MWKArticle_ShareSnippetTests.m @@ -0,0 +1,48 @@ +// +// WMFShareSnippetTests.m +// Wikipedia +// +// Created by Brian Gerstle on 5/6/15. +// Copyright (c) 2015 Wikimedia Foundation. All rights reserved. +// + +#import <UIKit/UIKit.h> +#import <XCTest/XCTest.h> +#import "WMFTestFixtureUtilities.h" +#import "MWKTitle.h" +#import "MWKSite.h" +#import "MWKArticle+ShareSnippet.h" + +#define HC_SHORTHAND 1 +#import <OCHamcrest/OCHamcrest.h> + +@interface MWKArticle_ShareSnippetTests : XCTestCase +@property MWKArticle* article; +@end + +@implementation MWKArticle_ShareSnippetTests + +- (void)setUp { + [super setUp]; + NSDictionary* obamaMobileViewJSON = [[self wmf_bundle] wmf_jsonFromContentsOfFile:@"Obama"]; + MWKTitle* dummyTitle = + [MWKTitle titleWithString:@"foo" site:[MWKSite siteWithDomain:@"wikipedia.org" language:@"en"]]; + self.article = [[MWKArticle alloc] initWithTitle:dummyTitle dataStore:nil dict:obamaMobileViewJSON[@"mobileview"]]; +} + +- (void)tearDown { + // Put teardown code here. This method is called after the invocation of each test method in the class. + [super tearDown]; +} + +- (void)testExpectedSnippetForObamaArticle { + assertThat(self.article.shareSnippet, is(@"Barack Hussein Obama II is the 44th and current President of the United States, and the first African American to hold the office. Born in Honolulu, Hawaii, Obama is a graduate of Columbia University and Harvard Law School, where he served as president of the Harvard Law Review. He was a community organizer in Chicago before earning his law degree. He worked as a civil rights attorney and taught constitutional law at the University of Chicago Law School from 1992 to 2004. He served three terms representing the 13th District in the Illinois Senate from 1997 to 2004, running unsuccessfully for the United States House of Representatives in 2000.")); +} + +- (void)testPerformanceExample { + [self measureBlock:^{ + [self.article shareSnippet]; + }]; +} + +@end diff --git a/WikipediaUnitTests/NSString+WMFHTMLParsingTests.m b/WikipediaUnitTests/NSString+WMFHTMLParsingTests.m index 2f54237..d411c69 100644 --- a/WikipediaUnitTests/NSString+WMFHTMLParsingTests.m +++ b/WikipediaUnitTests/NSString+WMFHTMLParsingTests.m @@ -21,13 +21,6 @@ @implementation NSString_WMFHTMLParsingTests -//- (void)testHillaryRodhamClintonHTML { -// NSString* hillaryHTML = -// [[NSString alloc] initWithData:[[self wmf_bundle] wmf_dataFromContentsOfFile:@"HillaryRodhamClinton" ofType:@"html"] -// encoding:NSUTF8StringEncoding]; -// assertThat(hillaryHTML.wmf_shareSnippetFromHTML, is(@"Hillary Diane Rodham Clinton is a former United States Secretary of State in the administration of President Barack Obama from 2009 to 2013; a former United States Senator representing New York from 2001 to 2009; and, as the wife of President Bill Clinton, was First Lady of the United State")); -//} - - (void)testSnippetFromTextWithCitaiton { assertThat([@"March 2011.[9][10] It was the first spacecraft to orbit Mercury.[7]" wmf_shareSnippetFromText], is(@"March 2011. It was the first spacecraft to orbit Mercury.")); diff --git a/WikipediaUnitTests/WMFShareSnippetTests.m b/WikipediaUnitTests/WMFShareSnippetTests.m deleted file mode 100644 index dea11a1..0000000 --- a/WikipediaUnitTests/WMFShareSnippetTests.m +++ /dev/null @@ -1,48 +0,0 @@ -// -// WMFShareSnippetTests.m -// Wikipedia -// -// Created by Brian Gerstle on 5/6/15. -// Copyright (c) 2015 Wikimedia Foundation. All rights reserved. -// - -#import <UIKit/UIKit.h> -#import <XCTest/XCTest.h> -#import "WMFTestFixtureUtilities.h" -#import "MWKTitle.h" -#import "MWKSite.h" -#import "MWKArticle+ShareSnippet.h" - -#define HC_SHORTHAND 1 -#import <OCHamcrest/OCHamcrest.h> - -@interface WMFShareSnippetTests : XCTestCase -@property MWKArticle* article; -@end - -@implementation WMFShareSnippetTests - -- (void)setUp { - [super setUp]; - NSDictionary* obamaMobileViewJSON = [[self wmf_bundle] wmf_jsonFromContentsOfFile:@"Obama"]; - MWKTitle* dummyTitle = - [MWKTitle titleWithString:@"foo" site:[MWKSite siteWithDomain:@"wikipedia.org" language:@"en"]]; - self.article = [[MWKArticle alloc] initWithTitle:dummyTitle dataStore:nil dict:obamaMobileViewJSON[@"mobileview"]]; -} - -- (void)tearDown { - // Put teardown code here. This method is called after the invocation of each test method in the class. - [super tearDown]; -} - -- (void)testExpectedSnippetForObamaArticle { - assertThat(self.article.shareSnippet, is(@"Obama was born on August 4, 1961, at Kapiʻolani Maternity & Gynecological Hospital in Honolulu, Hawaii, and would become the first President to have been born in Hawaii. His mother, Stanley Ann Dunham, was born in Wichita, Kansas, and was of mostly English ancestry. His fa")); -} - -- (void)testPerformanceExample { - [self measureBlock:^{ - [self.article shareSnippet]; - }]; -} - -@end -- To view, visit https://gerrit.wikimedia.org/r/209529 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib2e35990c8f0c55e938c0c5a93c99883521d1dd5 Gerrit-PatchSet: 1 Gerrit-Project: apps/ios/wikipedia Gerrit-Branch: master Gerrit-Owner: Bgerstle <bgers...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits