Skip to content
This repository has been archived by the owner on Jun 25, 2021. It is now read-only.

Fixes #689 #690

Merged
merged 25 commits into from Sep 15, 2015
Merged

Fixes #689 #690

merged 25 commits into from Sep 15, 2015

Conversation

ghost
Copy link

@ghost ghost commented Sep 15, 2015

Review on Reviewable

Mabeebam and others added 25 commits September 11, 2015 11:35
Added unit tests for id
69dd9eb
Merge remote-tracking branch 'upstream/master'
36e96b9
Finished unit tests for id
cd4876d
Merge remote-tracking branch 'upstream/master'
388f362
Removed 'use' statements in test-section
918fd17
Added unit test for set_name
d6a2e25
Corrected identation
075e3e2
Added unit test for function is_relocated
7287705
Reordered tests
835fd0b
Added unit tests for data.rs
23ffdda
Merge remote-tracking branch 'upstream/master'
1666daf
Added unit test for direct_messages.rs
8350631
Extended unit test for testing serialization of ResponseError
e5a7188
Added unit test for from() for ResponseError
2be281f
Added unit test for description() for ResponseError
9b81332
Added unit test for cause() of ResponseError
ee6c11a
Added unit tests for InterfaceError
76cbba1
Added unit tests for RoutingError
ba6202f
Merge remote-tracking branch 'upstream/master'
c1b5aee
Removed trailing white-spaces and limited lines to 100 chars
a21d666
Removed trailing white-spaces
1afcbb6
Removed trailing white-spaces and limited lines to 100 chars
bdda277
Removed trailing white-spaces and limited lines to 100 chars
4579b82
Uncommented assert that will be fixed with issue #689, removed traili…
3091ece 
…ng white-spaces and limited lines to 100 chars
Fixes #689
ece5d27
@maidsafe-highfive
Copy link

r? @benjaminbollen

(maidsafe_highfive has picked a reviewer for you, use r? to override)

@benjaminbollen
Copy link

looks good to me

benjaminbollen pushed a commit that referenced this pull request Sep 15, 2015
Merge pull request #690 from brian-js/master
dffe646 
Fixes #689
@benjaminbollen benjaminbollen merged commit dffe646 into maidsafe:master Sep 15, 2015
@benjaminbollen benjaminbollen mentioned this pull request Sep 15, 2015
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@benjaminbollen benjaminbollen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@maidsafe-highfive @benjaminbollen @dirvine