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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A quick fix to our social object Post unit test #54

Merged
merged 3 commits into from
Dec 19, 2018

Conversation

hamiltonalex
Copy link
Contributor

No description provided.

@@ -56,7 +56,9 @@ public class Post: NSObject, BaseObjectDecodable {
public let likeCount: Int
public let links: [Link]?
public let longitude: Double
public let originalPostIdentifier: String?
public var originalPostIdentifier: String? {
return String(describing: originalPostIdentifierRaw)
Copy link
Contributor

Choose a reason for hiding this comment

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

This (and the other functions using String(describing:)) should check whether the raw value is nil first, otherwise the string returned is "nil"!

i.e.

guard let x = x else {
    return nil
}

return String(describing: x)

@michaeleustace michaeleustace merged commit 3342a6d into develop Dec 19, 2018
@michaeleustace michaeleustace deleted the qf/fixing-post-unitest branch December 19, 2018 12:16
hamiltonalex pushed a commit that referenced this pull request Feb 14, 2019
* Post object can now handle different types for original post identifier (#54)

* Added guest consent podspec

* Updated quark to work with scope changes.

* Swapped Brotli for GZIP compression.

* Swapped Brotli for GZIP compression. (#55)

* Version increment to add GZIP support and timing tool support (#56)

* DIGI-13285 load new environments dynamically from the documents folder (#57)

* DIGI-13285 load new environments dynamically from the documents folder

* additional null check to be cconsistent with the other code

* Private CASession header for core.

* Reflected changes for postbox.

* DIGI-13286, DIGI-13289, DIGI-13290 - Filter out specific values into the Session dictionary (#59)

* DIGI-13286, DIGI-13289, DIGI-13290 - Filter out specific values into the Session dictionary

* Remove obsolete vars

* Quick doc fix

* Postbox included in the podspec

* Updated SDK to remove support for CAFileObject. Updated Default p12 + contract combo

* Updated p12 + contract combo in Swift example

* Updated Documentation to clarify change to the delegate pattern. (#64)

* Updated Documentation

* Link fix

* Update copyright date

* Updated podspec
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.

3 participants