Issue #425: Fix parsing AlternateMailboxCollection

[thomaswoeckinger]

Fixed typo

[azurecla]

Hi @thomaswoeckinger, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

[vboctor]

Thanks @thomaswoeckinger for your contribution. Please update the pull request to only include the functional fix without the unrelated formatting fixes. Please also include test cases.

[thomaswoeckinger]

It is formated with Google code format as described on the contribution page
Am 01.09.2015 05:40 schrieb "Victor Boctor" notifications@github.com:

Thanks @thomaswoeckinger https://github.com/thomaswoeckinger for your
contribution. Please update the pull request to only include the functional
fix without the unrelated formatting fixes. Please also include test cases.

—
Reply to this email directly or view it on GitHub
#431 (comment)
.

[serious6]

@thomaswoeckinger maybe you can just revert the changes in the license-header? The other ones may be no problem.

[thomaswoeckinger]

License header reverted/added

[serious6]

@thomaswoeckinger thanks for your contribution. Think we can get this in shortly.

[vboctor]

Add return tag value

[vboctor]

@thomaswoeckinger the PR looks good. I added a couple of minor comments.

[serious6]

@thomaswoeckinger maybe we can have this merged once you fixed the documentation issues. Maybe also squash your commits.

[thomaswoeckinger]

Once again; I can add documentation on the properties or the class.

The difference is that the OwnerSmtpAddress is the SMTP address of owner of the alternate mailbox, and the SmtpAddress belongs to the mailbox itself.

The SmtpAddress is only set if it is available for the given Type e.g. Delegate has an SmtpAddress set Archive not.

[codecov-io]

Current coverage is 10.43%

Merging #431 into master will increase coverage by +0.35% as of 0c45ae5

@@            master    #431   diff @@
======================================
  Files          550     550       
  Stmts        20419   20429    +10
  Branches      2620    2625     +5
  Methods          0       0       
======================================
+ Hit           2060    2131    +71
- Partial        148     153     +5
+ Missed       18211   18145    -66

Review entire Coverage Diff as of 0c45ae5

Powered by Codecov. Updated on successful CI builds.

[serious6]

plz dont format license header

[serious6]

please Squash your commits.

[thomaswoeckinger]

Documented and squashed

[serious6]

PR still contains several commits.

[thomaswoeckinger]

Squashed to a single commit

[serious6]

@thomaswoeckinger Thanks for your contribution