feat(tool,nextcloud)!: Generate rich object parameters

Signed-off-by: provokateurin <kate@provokateurin.de>
nextcloud · Apr 27, 2024 · 2406cda · 2406cda
1 parent 70b76b0
commit 2406cda
Show file tree

Hide file tree

Showing 16 changed files with 707 additions and 468 deletions.
diff --git a/.cspell/nextcloud.txt b/.cspell/nextcloud.txt
@@ -1,3 +1,4 @@
+addressbook
 apirequest
 apppassword
 bigfilechunking

diff --git a/packages/neon/neon_talk/lib/src/widgets/message.dart b/packages/neon/neon_talk/lib/src/widgets/message.dart
@@ -117,40 +117,34 @@ InlineSpan buildRichObjectParameter({
   required TextStyle? textStyle,
   required bool isPreview,
 }) {
-  Widget child;
-
-  const mentionTypes = ['user', 'call', 'guest', 'user-group', 'group'];
-  if (mentionTypes.contains(parameter.type)) {
-    child = TalkRichObjectMention(
-      parameter: parameter,
-      textStyle: textStyle,
-    );
-  } else {
-    if (isPreview) {
-      child = Text(parameter.name);
-    } else {
-      switch (parameter.type) {
-        case 'file':
-          child = TalkRichObjectFile(
-            parameter: parameter,
-            textStyle: textStyle,
-          );
-        case 'deck-card':
-          child = TalkRichObjectDeckCard(
-            parameter: parameter,
-          );
-        default:
-          child = TalkRichObjectFallback(
-            parameter: parameter,
-            textStyle: textStyle,
-          );
-      }
-    }
-  }
-
   return WidgetSpan(
     alignment: PlaceholderAlignment.middle,
-    child: child,
+    child: switch (parameter.type) {
+      spreed.RichObjectParameter_Type.user ||
+      spreed.RichObjectParameter_Type.call ||
+      spreed.RichObjectParameter_Type.guest ||
+      spreed.RichObjectParameter_Type.userGroup ||
+      spreed.RichObjectParameter_Type.group =>
+        TalkRichObjectMention(
+          parameter: parameter,
+          textStyle: textStyle,
+        ),
+      _ => isPreview
+          ? Text(parameter.name)
+          : switch (parameter.type) {
+              spreed.RichObjectParameter_Type.file => TalkRichObjectFile(
+                  parameter: parameter,
+                  textStyle: textStyle,
+                ),
+              spreed.RichObjectParameter_Type.deckCard => TalkRichObjectDeckCard(
+                  parameter: parameter,
+                ),
+              _ => TalkRichObjectFallback(
+                  parameter: parameter,
+                  textStyle: textStyle,
+                ),
+            },
+    },
   );
 }
 

diff --git a/packages/neon/neon_talk/lib/src/widgets/rich_object/file.dart b/packages/neon/neon_talk/lib/src/widgets/rich_object/file.dart
@@ -27,7 +27,7 @@ class TalkRichObjectFile extends StatelessWidget {
   Widget build(BuildContext context) {
     Widget child;
 
-    if (parameter.previewAvailable == spreed.RichObjectParameter_PreviewAvailable.yes) {
+    if (parameter.previewAvailable == 'yes') {
       final maxHeight = MediaQuery.sizeOf(context).height / 2;
 
       var width = _parseDimension(parameter.width);

diff --git a/packages/neon/neon_talk/lib/src/widgets/rich_object/mention.dart b/packages/neon/neon_talk/lib/src/widgets/rich_object/mention.dart
@@ -27,7 +27,7 @@ class TalkRichObjectMention extends StatelessWidget {
     final bool highlight;
 
     switch (parameter.type) {
-      case 'user':
+      case spreed.RichObjectParameter_Type.user:
         final accountsBloc = NeonProvider.of<AccountsBloc>(context);
         final account = accountsBloc.activeAccount.value!;
 
@@ -36,7 +36,7 @@ class TalkRichObjectMention extends StatelessWidget {
           username: parameter.id,
           showStatus: false,
         );
-      case 'call':
+      case spreed.RichObjectParameter_Type.call:
         highlight = true;
         child = CircleAvatar(
           child: ClipOval(
@@ -45,13 +45,13 @@ class TalkRichObjectMention extends StatelessWidget {
             ),
           ),
         );
-      case 'guest':
+      case spreed.RichObjectParameter_Type.guest:
         // TODO: Add highlighting when the mention is about the current guest user.
         highlight = false;
         child = CircleAvatar(
           child: Icon(AdaptiveIcons.person),
         );
-      case 'user-group' || 'group':
+      case spreed.RichObjectParameter_Type.userGroup || spreed.RichObjectParameter_Type.group:
         final accountsBloc = NeonProvider.of<AccountsBloc>(context);
         final userDetailsBloc = accountsBloc.activeUserDetailsBloc;
         final groups = userDetailsBloc.userDetails.valueOrNull?.data?.groups ?? BuiltList();

diff --git a/packages/neon/neon_talk/test/message_test.dart b/packages/neon/neon_talk/test/message_test.dart
@@ -646,8 +646,14 @@ void main() {
     for (final isPreview in [true, false]) {
       group(isPreview ? 'As preview' : 'Complete', () {
         group('Mention', () {
-          for (final type in ['user', 'call', 'guest', 'user-group', 'group']) {
-            testWidgets(type, (tester) async {
+          for (final type in [
+            spreed.RichObjectParameter_Type.user,
+            spreed.RichObjectParameter_Type.call,
+            spreed.RichObjectParameter_Type.guest,
+            spreed.RichObjectParameter_Type.userGroup,
+            spreed.RichObjectParameter_Type.group,
+          ]) {
+            testWidgets(type.value, (tester) async {
               final userDetails = MockUserDetails();
               when(() => userDetails.groups).thenReturn(BuiltList());
 
@@ -699,7 +705,7 @@ void main() {
                 text: buildRichObjectParameter(
                   parameter: spreed.RichObjectParameter(
                     (b) => b
-                      ..type = 'file'
+                      ..type = spreed.RichObjectParameter_Type.file
                       ..id = ''
                       ..name = 'name',
                   ),
@@ -721,7 +727,7 @@ void main() {
                 text: buildRichObjectParameter(
                   parameter: spreed.RichObjectParameter(
                     (b) => b
-                      ..type = 'deck-card'
+                      ..type = spreed.RichObjectParameter_Type.deckCard
                       ..id = ''
                       ..name = 'name'
                       ..boardname = 'boardname'
@@ -745,7 +751,7 @@ void main() {
                 text: buildRichObjectParameter(
                   parameter: spreed.RichObjectParameter(
                     (b) => b
-                      ..type = 'unknown'
+                      ..type = spreed.RichObjectParameter_Type.addressbook
                       ..id = ''
                       ..name = 'name',
                   ),
@@ -786,7 +792,7 @@ void main() {
               BuiltMap({
                 type: spreed.RichObjectParameter(
                   (b) => b
-                    ..type = ''
+                    ..type = spreed.RichObjectParameter_Type.user
                     ..id = ''
                     ..name = '',
                 ),
@@ -806,7 +812,7 @@ void main() {
           BuiltMap({
             'file': spreed.RichObjectParameter(
               (b) => b
-                ..type = 'file'
+                ..type = spreed.RichObjectParameter_Type.file
                 ..id = ''
                 ..name = '',
             ),
@@ -828,13 +834,13 @@ void main() {
         BuiltMap({
           'actor1': spreed.RichObjectParameter(
             (b) => b
-              ..type = 'user'
+              ..type = spreed.RichObjectParameter_Type.user
               ..id = ''
               ..name = '',
           ),
           'actor2': spreed.RichObjectParameter(
             (b) => b
-              ..type = 'user'
+              ..type = spreed.RichObjectParameter_Type.user
               ..id = ''
               ..name = '',
           ),

diff --git a/packages/neon/neon_talk/test/rich_object_test.dart b/packages/neon/neon_talk/test/rich_object_test.dart
@@ -48,7 +48,7 @@ void main() {
         child: TalkRichObjectDeckCard(
           parameter: spreed.RichObjectParameter(
             (b) => b
-              ..type = ''
+              ..type = spreed.RichObjectParameter_Type.deckCard
               ..id = ''
               ..name = 'name'
               ..boardname = 'boardname'
@@ -74,7 +74,7 @@ void main() {
             child: TalkRichObjectMention(
               parameter: spreed.RichObjectParameter(
                 (b) => b
-                  ..type = 'user'
+                  ..type = spreed.RichObjectParameter_Type.user
                   ..id = 'username'
                   ..name = 'name',
               ),
@@ -97,7 +97,7 @@ void main() {
             child: TalkRichObjectMention(
               parameter: spreed.RichObjectParameter(
                 (b) => b
-                  ..type = 'user'
+                  ..type = spreed.RichObjectParameter_Type.user
                   ..id = 'other'
                   ..name = 'name',
               ),
@@ -122,7 +122,7 @@ void main() {
             child: TalkRichObjectMention(
               parameter: spreed.RichObjectParameter(
                 (b) => b
-                  ..type = 'call'
+                  ..type = spreed.RichObjectParameter_Type.call
                   ..id = ''
                   ..name = 'name'
                   ..iconUrl = '',
@@ -146,7 +146,7 @@ void main() {
           child: TalkRichObjectMention(
             parameter: spreed.RichObjectParameter(
               (b) => b
-                ..type = 'guest'
+                ..type = spreed.RichObjectParameter_Type.guest
                 ..id = ''
                 ..name = 'name',
             ),
@@ -162,8 +162,11 @@ void main() {
       );
     });
 
-    for (final type in ['user-group', 'group']) {
-      testWidgets(type, (tester) async {
+    for (final type in [
+      spreed.RichObjectParameter_Type.userGroup,
+      spreed.RichObjectParameter_Type.group,
+    ]) {
+      testWidgets(type.value, (tester) async {
         final userDetails = MockUserDetails();
         when(() => userDetails.groups).thenReturn(BuiltList(['group']));
 
@@ -192,7 +195,7 @@ void main() {
         expect(find.text('name'), findsOne);
         await expectLater(
           find.byType(TalkRichObjectMention),
-          matchesGoldenFile('goldens/rich_object_mention_${type}_highlight.png'),
+          matchesGoldenFile('goldens/rich_object_mention_${type.value}_highlight.png'),
         );
 
         await tester.pumpWidget(
@@ -215,7 +218,7 @@ void main() {
         expect(find.text('name'), findsOne);
         await expectLater(
           find.byType(TalkRichObjectMention),
-          matchesGoldenFile('goldens/rich_object_mention_${type}_other.png'),
+          matchesGoldenFile('goldens/rich_object_mention_${type.value}_other.png'),
         );
       });
     }
@@ -230,10 +233,10 @@ void main() {
             child: TalkRichObjectFile(
               parameter: spreed.RichObjectParameter(
                 (b) => b
-                  ..type = ''
+                  ..type = spreed.RichObjectParameter_Type.file
                   ..id = '0'
                   ..name = 'name'
-                  ..previewAvailable = spreed.RichObjectParameter_PreviewAvailable.yes
+                  ..previewAvailable = 'yes'
                   ..path = '',
               ),
               textStyle: null,
@@ -255,10 +258,10 @@ void main() {
           child: TalkRichObjectFile(
             parameter: spreed.RichObjectParameter(
               (b) => b
-                ..type = ''
+                ..type = spreed.RichObjectParameter_Type.file
                 ..id = '0'
                 ..name = 'name'
-                ..previewAvailable = spreed.RichObjectParameter_PreviewAvailable.no
+                ..previewAvailable = 'no'
                 ..path = '',
             ),
             textStyle: null,
@@ -281,7 +284,7 @@ void main() {
           child: TalkRichObjectFallback(
             parameter: spreed.RichObjectParameter(
               (b) => b
-                ..type = ''
+                ..type = spreed.RichObjectParameter_Type.addressbook
                 ..id = ''
                 ..name = 'name',
             ),
@@ -305,7 +308,7 @@ void main() {
             child: TalkRichObjectFallback(
               parameter: spreed.RichObjectParameter(
                 (b) => b
-                  ..type = ''
+                  ..type = spreed.RichObjectParameter_Type.addressbook
                   ..id = ''
                   ..name = 'name'
                   ..iconUrl = '',