Missing api or documentation for testability when using NStack

[nstack-io]

Missing api or documentation for testability when using NStack

[jakobmygind]

An alternative to supporting this could be an option for the generator to not generate the accessor tr/lo variable or to keep it internal instead of public to allow for more control over how Localizations is accessed when used in a module.

Edit: I just found out there's a -standalone flag for the generator which prevents the addition of the static tr/lo, which eases wrapping of Localizations in client code, however built in support would still be nice ☺️

[mariusc]

Hi Jakob; we haven't had problems with this yet, but we see how this can be an issue; we'll look into this soon 😊

[jakobmygind]

Hi @mariusc Thanks!

I solved it for now using the -standalone flag along with this code that enables control over the tr instance

import NStackSDK

internal var lo: Localizations {
    guard let manager = NStack.sharedInstance.localizationManager else {
        return Localizations()
    }
    return try! manager.localization()
}

public internal (set) var tr: Localizations = { lo }()

#if DEBUG
public func setTranslations(_ localizations: Localizations) {
    tr = localizations
}

public func loadTranslationsFromJSON(_ filename: String, in bundle: Bundle) -> Localizations {
    let path = bundle.path(forResource: filename, ofType: "json")!
    
    let data = try! String(contentsOfFile: path).data(using: .utf8)!
    let dict = try! JSONSerialization.jsonObject(with: data, options: .allowFragments) as! [String: Any]
    let translationsData = try! JSONSerialization.data(withJSONObject: dict["data"]!, options: .fragmentsAllowed)
    
    let result = try! JSONDecoder().decode(Localizations.self, from: translationsData)
    return result
}

#endif

and then in my Test class

 class override func setUp() {
        super.setUp()
        let staticLocalizations = Shared.loadTranslationsFromJSON("Localizations_da-DK", in: Bundle(for: self))
        Shared.setTranslations(staticLocalizations)
    }