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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

XcodeServer Credential tests #75

Merged
merged 1 commit into from
Jul 20, 2015
Merged

Conversation

cojoj
Copy link
Contributor

@cojoj cojoj commented Jul 20, 2015

I've written two more tests to XcodeServer which check credential property.

Nothing fancy, but... 馃槵

let pass = server.credential?.password

XCTAssertEqual(user!, "ICanCreateBots")
XCTAssertEqual(pass!, "superSecr3t")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about the host?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It only checks NSURLCredential so there's no checking for host. It's another test case responsibility.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha, I completely missed that it's the credential we're testing. Fair enough.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

host is actually already tested in XcodeServerConfigTests. Nevermind me then.

@buildasaur
Copy link
Collaborator

Result of Integration 1

Duration: 15 seconds
Result: -1error, failing state: trigger-error

@cojoj
Copy link
Contributor Author

cojoj commented Jul 20, 2015

@buildasaur Y U NO SUPPORT COCOAPODS? 馃槸

@buildasaur
Copy link
Collaborator

Result of Integration 3

Duration: 51 seconds
Result: Perfect build! All 29 tests passed. 馃憤
Test Coverage: 45%.

czechboy0 pushed a commit that referenced this pull request Jul 20, 2015
XcodeServer Credential tests
@czechboy0 czechboy0 merged commit 198e365 into buildasaurs:swift-2 Jul 20, 2015
@czechboy0
Copy link
Member

Thanks @cojoj!

@cojoj cojoj deleted the server-tests branch July 20, 2015 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants