Merge branch 'fix-classes-non-nullable' into 'master'
[Classes] Non nullable id and mxcurl See merge request famedly/famedlysdk!39
This commit is contained in:
commit
0316795a49
|
@ -66,13 +66,13 @@ class User {
|
||||||
MxContent get avatar_url => avatarUrl;
|
MxContent get avatar_url => avatarUrl;
|
||||||
|
|
||||||
User(
|
User(
|
||||||
this.id, {
|
String id, {
|
||||||
this.membership,
|
this.membership,
|
||||||
this.displayName,
|
this.displayName,
|
||||||
this.avatarUrl,
|
this.avatarUrl,
|
||||||
this.powerLevel,
|
this.powerLevel,
|
||||||
this.room,
|
this.room,
|
||||||
});
|
}) : this.id = id ?? "";
|
||||||
|
|
||||||
/// Returns the displayname or the local part of the Matrix ID if the user
|
/// Returns the displayname or the local part of the Matrix ID if the user
|
||||||
/// has no displayname.
|
/// has no displayname.
|
||||||
|
|
|
@ -29,7 +29,7 @@ class MxContent {
|
||||||
final String _mxc;
|
final String _mxc;
|
||||||
|
|
||||||
/// Insert a mxc:// uri here.
|
/// Insert a mxc:// uri here.
|
||||||
MxContent(this._mxc);
|
MxContent(String mxcUrl) : this._mxc = mxcUrl ?? "";
|
||||||
|
|
||||||
/// Returns the mxc uri.
|
/// Returns the mxc uri.
|
||||||
get mxc => _mxc;
|
get mxc => _mxc;
|
||||||
|
|
|
@ -43,5 +43,12 @@ void main() {
|
||||||
width: 50, height: 50, method: ThumbnailMethod.scale),
|
width: 50, height: 50, method: ThumbnailMethod.scale),
|
||||||
"${client.homeserver}/_matrix/media/r0/thumbnail/exampleserver.abc/abcdefghijklmn?width=50&height=50&method=scale");
|
"${client.homeserver}/_matrix/media/r0/thumbnail/exampleserver.abc/abcdefghijklmn?width=50&height=50&method=scale");
|
||||||
});
|
});
|
||||||
|
test("Not crashing if null", () async {
|
||||||
|
Client client = Client("testclient");
|
||||||
|
client.homeserver = "https://testserver.abc";
|
||||||
|
final MxContent content = MxContent(null);
|
||||||
|
expect(content.getDownloadLink(client),
|
||||||
|
"${client.homeserver}/_matrix/media/r0/download/");
|
||||||
|
});
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue